#1308326: PHP strict warning when using panels in Views which was committed and is in 6.x-3.11 fixes the compatibility with Views with this:

- function options_submit($form, &$form_state) {
+ function options_submit(&$form, &$form_state) {

Unfortunately, the fix in Views itself does the opposite in this RTBC issue: #2411093: PHP strict warnings: Declaration of ... should be compatible with ...

So if someone has patched their views for the PHP compatibility and then updates Panels, the PHP compatibility in here breaks it again.

I'll be attaching a patch that needs to be applied to remove the & once that patch is committed (or for anyone who is using it).

Comments

Michelle created an issue. See original summary.

michelle’s picture

Status: Active » Needs review
StatusFileSize
new715 bytes

Here's the patch.

damienmckenna’s picture

Status: Needs review » Reviewed & tested by the community
Issue tags: +PHP 5.4
Parent issue: » #2639478: Plan for Panels 6.x-3.12 release

Another one X-) RTBC.

japerry’s picture

hrm, maybe this will get committed down the line.. but most likely not. Leaving it as the last issue for d6.

  • japerry committed a45e2d5 on 6.x-3.x authored by Michelle
    Issue #2654582 by Michelle: Keep compatibility with Views patched for...
japerry’s picture

Status: Reviewed & tested by the community » Fixed

Actually, fixed in head. We'll roll a release when views is updated.

michelle’s picture

Thanks!

Status: Fixed » Closed (fixed)

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

osopolar’s picture