When a skin is defined in *.skinr.inc (which is auto-loaded for select hooks), and include a custom 'form callback' function, the includefile won't be loaded when displaying the skin settings form (e.g. /admin/structure/skinr/edit/block/system__user-menu/configure ) resulting in the form element not being created.

We'll need to add custom loading of *.skinr.inc file when a skin with the a 'form callback' is defined in the *.skinr.inc file.

CommentFileSizeAuthor
#2 patch_commit_7f112bd27bdf.patch3.92 KBmoonray
#1 patch_commit_0b3a8b6ab045.patch2.93 KBmoonray
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

moonray’s picture

moonray CreditAttribution: moonray commented
Status: Active » Needs review
FileSize
patch_commit_0b3a8b6ab045.patch2.93 KB

And here's a patch.

moonray’s picture

moonray CreditAttribution: moonray commented
FileSize
patch_commit_7f112bd27bdf.patch3.92 KB

Needed to tweak some test to account for new 'include file' for all $module.skinr.inc, even when they're normally auto-loaded.

moonray’s picture

moonray CreditAttribution: moonray commented
Status: Needs review » Fixed

Committed. This fixed testing issues.

Status: Fixed » Closed (fixed)

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