Problem/Motivation

The last comma in AddToAnySettingsForm (create method) is not needed and throws an error since the last update.
Error reported

Steps to reproduce

Upgrade the module to its latest version and clear drupal site cache

Proposed resolution

Removing the last comma in AddToAnySettingsForm (create method) of the last update

Remaining tasks

Make changes to the code

User interface changes

N/A

API changes

N/A

Data model changes

N/A

CommentFileSizeAuthor
#4 3364406-remove-last-comma.patch829 bytesdcimorra
#3 error-image.png21.84 KBdcimorra

Issue fork addtoany-3364406

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

luismigg created an issue. See original summary.

luismigg opened merge request !16

dcimorra’s picture

dcimorra
Primary language Spanish
Location 🇪🇸 Spain
CreditAttribution: dcimorra at Hiberus commented
Issue summary: View changes
Status: Active » Needs review
FileSize
error-image.png21.84 KB
dcimorra’s picture

dcimorra
Primary language Spanish
Location 🇪🇸 Spain
CreditAttribution: dcimorra at Hiberus commented
FileSize
3364406-remove-last-comma.patch829 bytes

  • micropat committed 94771bab on 8.x-1.x authored by luismigg 
    Issue #3364406 by luismigg, dcimorra: The last comma in...

micropat’s picture

micropat CreditAttribution: micropat commented
Priority: Major » Normal
Status: Needs review » Fixed

Thanks for the patch — committed to the dev branches and will be in the next releases.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.