When running a cache clear via the drupal-extension for behat, we noticed that steps would fail due to the PanelizerEntityDefault class not being found.

Simple fix to add the entity plugins to the .info. Patch following.

CommentFileSizeAuthor
#5 panelizer-n2288683-5.patch456 bytesdamienmckenna
#3 2288683-resolve-fatal-3.patch448 bytesdstol
#1 2288683-resolve-fatal-1.patch691 bytesdstol

Comments

dstol’s picture

dstol
Primary language English
commented
Status: Active » Needs review
StatusFileSize
new 2288683-resolve-fatal-1.patch691 bytes
damienmckenna’s picture

damienmckenna
Location TN, USA
commented

Did you test it with *just* PanelizerEntityDefault.class.php?

dstol’s picture

dstol
Primary language English
commented
StatusFileSize
new 2288683-resolve-fatal-3.patch448 bytes

Should have done some due diligence there. Tested with this patch, without fatals.

Status: Needs review » Needs work

The last submitted patch, 3: 2288683-resolve-fatal-3.patch, failed testing.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs work » Needs review
StatusFileSize
new panelizer-n2288683-5.patch456 bytes

Weird that it didn't like your patch?

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

@dstol: Any updates? Thanks.

dstol’s picture

dstol
Primary language English
commented

We've been running this patch in production for a little over a month. Haven't noticed any regressions and certainly haven't had any fatals since we patched. Seems like it's probably RTBC, but I'll leave it up to you.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs review » Fixed

Sounds good. Committed!

  • DamienMcKenna committed ff3b834 on 7.x-3.x authored by dstol 
    Issue #2288683 by dstol: Ensure PanelizerEntityDefault class is always...

Status: Fixed » Closed (fixed)

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

denix’s picture

denix commented

Dear all,
unfortunately the error is still there. I could reproduce it simply having "file_entity" installed in /all/modules/contrib and "panelizer" in /default/modules/contrib.
Moving "panelizer" in /all/modules/contrib fixed the issue.
Thanks

dasginganinja’s picture

dasginganinja
Primary language English
Location Bethlehem, PA
commented

I can also echo the results from denix in my installation, albeit this is a multi-site installation.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

I highly recommend always having your contrib modules installed in sites/all instead of a per-hostname directory.

denix’s picture

denix commented

Hi @DamienMcKenna, although the recommendation is a good one, unfortunately it is not always possible i.e. a subsite use a module that you do not want to have installed for other entities of your farm. But yes, you are right.

joseph.olstad’s picture

joseph.olstad
Primary language French
commented

I just hit this , can we please get a tagged release that contains this fix?