The big hook definition for plugins should be changing slightly in the upcoming release of ctools. The attached patch is a backwards compatible solution to continue using this style definition going forward.

It may be a larger issue but the way they're currently being included with the include and the second function might be eased by using the preferred per plugin includes method. In that case, the attached patch would not me needed.

Related issue: #711664: Make global plugins definition hooks optional

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

zzolo’s picture

I have tested this patch for 2.x and current stable ctools: seems fine.
Testing with ctools HEAD: I get an unrelated ctools error.

tmcw’s picture

Issues with patches are the best :). Should we expand this to cover layer types, etc?

neclimdul’s picture

Oh, if there's a plugin type I missed, probably. Sorry, I actually wrote this patch from the hip based grep and the the ctools api and didn't test it since I'm not terribly familiar with the inner workings of openlayers.

I don't believe the other issue has been committed so to /really/ give it a good test you probably want to apply that patch to ctools HEAD as well.

tmcw’s picture

Status: Active » Fixed

Status: Fixed » Closed (fixed)

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