In plugins/display_renderers/panels_renderer_editor.class.php this signature:

function panels_edit_configure_access_settings_form(&$form_state) {

should be changed to

function panels_edit_configure_access_settings_form($form, &$form_state) {

otherwise the form will not be displayed. Patch follows in a sec …

CommentFileSizeAuthor
#2 panels_edit_configure_access_settings_form-signature-1264404-2.patch1.04 KBPisco
#1 panels_edit_configure_access_settings_form-signature-1264404-1.patch666 bytesPisco
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Pisco’s picture

Pisco CreditAttribution: Pisco commented
Status: Active » Needs review
FileSize
panels_edit_configure_access_settings_form-signature-1264404-1.patch666 bytes

Here you go, for your testing and reviewing pleasure! :-)

Pisco’s picture

Pisco CreditAttribution: Pisco commented
FileSize
panels_edit_configure_access_settings_form-signature-1264404-2.patch1.04 KB

This time with a nicer patch.

das-peter’s picture

das-peter CreditAttribution: das-peter commented
Status: Needs review » Reviewed & tested by the community

Just checked this out. Makes absolutely sense to me.
Thus I mark this as RTBC.

merlinofchaos’s picture

merlinofchaos CreditAttribution: merlinofchaos commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed! Thanks for the patch!

Status: Fixed » Closed (fixed)

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

  • Commit e88496a on 7.x-3.x, 7.x-3.x-i18n, 8.x-3.x by merlinofchaos: 
    Issue #1264404 by Pisco: Wrong signature on pane edit access settings...