The admin setting form should be inside src/Form folder as per the drupal standard.

Please write a patch to move LinkIconSettingsForm.php from /src to /src/Form

CommentFileSizeAuthor
#2 psr-3025991-2.patch886 bytesmahtab_alam

Comments

mahtab_alam created an issue. See original summary.

mahtab_alam’s picture

mahtab_alam commented
StatusFileSize
new psr-3025991-2.patch886 bytes
mahtab_alam’s picture

mahtab_alam commented
Status: Active » Needs review
gausarts’s picture

gausarts commented
Status: Needs review » Needs work

Thanks! We must also move the file LinkIconSettingsForm into src/Form for the autoload to work.

gausarts’s picture

gausarts commented
Status: Needs work » Fixed

Truly sorry for delay. Please ignore my previous comment.

Committed. Thank you for contribution!

  • gausarts committed 95c6b44 on 8.x-1.x 
    - Added hook_update for the careless update due to moving file #3025991.

Status: Fixed » Closed (fixed)

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