Since "Preprocess functions are not merged when a module registers a theme hook for a theme-provided template" is fixed.

the patch created in Invalid argument supplied for foreach() in ds_preprocess_ds_layout() line 366 of ds.module can be rewritten.

CommentFileSizeAuthor
#2 3004581-2.patch1.13 KBMartijn de Wit
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Martijn de Wit created an issue. See original summary.

Martijn de Wit’s picture

Martijn de Wit
Primary language Dutch
Location 🇳🇱 The Netherlands
CreditAttribution: Martijn de Wit at Ablerz commented
FileSize
3004581-2.patch1.13 KB

created a patch to remove legacy code-lines and bump core supported version.

Martijn de Wit’s picture

Martijn de Wit
Primary language Dutch
Location 🇳🇱 The Netherlands
CreditAttribution: Martijn de Wit at Ablerz commented
Status: Needs work » Needs review
Martijn de Wit’s picture

Martijn de Wit
Primary language Dutch
Location 🇳🇱 The Netherlands
CreditAttribution: Martijn de Wit at Ablerz commented
Title: Follow up for Invalid argument supplied for foreach() in ds_preprocess_ds_layout() line 366 of ds.module » Follow up for "Invalid argument supplied for foreach() in ds_preprocess_ds_layout() line 366 of ds.module"

swentel’s picture

swentel CreditAttribution: swentel at eps & kaas for Dropsolid commented
Status: Needs review » Fixed

hurray for removing obsolete code :)
committed and pushed, thanks!

Status: Fixed » Closed (fixed)

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