1. Warning: Invalid argument supplied for foreach()
2. hook_uninstall missing()

CommentFileSizeAuthor
#2 disable_enable_all_assign_block-3116880-2.patch1.21 KBjoy29
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

joy29 created an issue. See original summary.

joy29’s picture

joy29 CreditAttribution: joy29 at Srijan | A Material+ Company for Srijan | A Material+ Company commented
FileSize
disable_enable_all_assign_block-3116880-2.patch1.21 KB

patch for above issue.

joy29’s picture

joy29 CreditAttribution: joy29 at Srijan | A Material+ Company for Srijan | A Material+ Company commented
Assigned: joy29 » Unassigned
Status: Active » Needs review

  • rohit-drupal committed 9fe5c00 on 8.x-1.x authored by joy29 
    Issue #3116880 by joy29: Warning message showing Invaild foreach() and...
rohit-rajput-sahab’s picture

rohit-rajput-sahab CreditAttribution: rohit-rajput-sahab commented
Status: Needs review » Fixed

Hi @Joy29,

Thank you so much for checking this module and code.
I have applied your patch.

https://git.drupalcode.org/project/disable_enable_all_assign_block/commi...

rohit-rajput-sahab’s picture

rohit-rajput-sahab CreditAttribution: rohit-rajput-sahab commented

Hi @Joy29,

After applying your patch. PAReview showing warning. I resolved these warnings and errors. See to open the below URL.

https://pareview.sh/pareview/https-git.drupal.org-project-disable_enable...

joy29’s picture

joy29 CreditAttribution: joy29 at Srijan | A Material+ Company for Srijan | A Material+ Company commented

Maybe for different issue, I saw warning for, default_value in form. So I fix that

rohit-rajput-sahab’s picture

rohit-rajput-sahab CreditAttribution: rohit-rajput-sahab commented
Status: Fixed » Closed (fixed)