If you create a panels template with layout settings, there's no way to validate those settings.

includes/display_edit.inc has a function panels_edit_display_settings_form_validate which looks for a validate function in the template and calls it, but panels_edit_display_settings_form_validate doesn't appear to ever get called.

The attached patch fixes it by adding validate functions to the wrapped forms which in turn call panels_edit_display_settings_form_validate, though I'm not sure if this is the right approach.

CommentFileSizeAuthor
validate_layout_settings.patch1.26 KBdlerman2
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Letharion’s picture

Status: Active » Needs review

Patch still applies.

merlinofchaos’s picture

Status: Needs review » Fixed

Does this mean that the layout settings form actually still works?

Status: Fixed » Closed (fixed)

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

  • Commit 0cab672 on 7.x-3.x, 7.x-3.x-i18n, 8.x-3.x by merlinofchaos:
    Issue #1234616 by dlerman2: No way to validate layout settings on...