Closed (won't fix)
Project:
Panelizer (obsolete)
Version:
7.x-3.x-dev
Component:
Code
Priority:
Normal
Category:
Support request
Assigned:
Unassigned
Reporter:
Created:
29 Apr 2013 at 12:53 UTC
Updated:
15 Jan 2015 at 03:56 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
rlmumfordHere's another patch, made directly against head. The last one failed to apply for some reason.
Comment #2
merlinofchaos commentedNo, these are CTools plugins. CTools automatically registers plugin classes via hook_registry_alter(). There is no need to put these files in the .info.
Comment #3
merlinofchaos commentedAlso, without applying your patch:
So if those don't appear in your registry, something else is wrong.
Comment #3.0
merlinofchaos commentedUpdated issue summary.
Comment #4
andrewbelcher commentedSorry to re-open this - we are still having an issue with this.
I think when building the registry it attempts to add our class before it adds the ones provided by Panelizer and then throws the error as the class being extended isn't yet registered. I'm guessing this is due to our modules beginning with c and f, which means it gets processed before Panelizer?
Is there a solution to that other than messing with module weights?
Comment #5
mglamanandrewbelcher, a solution is to use hook_implements_alter() to ensure your module runs hook_ctools_plugin_directory() last.
Unfortunately there isn't much of a way to work around this (modules and alphabetical sorting) unless you modify the system weight or implementation alters.
Putting back to the original won't fix status, as defining the classes in the .info would go against the ctools plugin implementation, stated in #2
Somewhat relevant issue on topic of module naming weight - #2183937: Error on install of Panopoly-based install profile: "Column 'module' cannot be null" when profile starts with "e"