This patch moves the PURL processor classes outside the main .module file and loads them dynamically as needed.

CommentFileSizeAuthor
purl_includes.patch22.27 KByhahn

Comments

socialnicheguru’s picture

what is the result of this patch?

jmiccolis’s picture

Priority: Normal » Critical
Status: Needs review » Needs work

yhahn and I have discussed this, and the above patch won't got in as is, in favor of a ctools based approach. This will be a milestone that we need to hit before the 1.0 stable release.

socialnicheguru’s picture

Status: Fixed » Needs work

deleted comment (wrong issue)

jmiccolis’s picture

Status: Needs work » Fixed

I've just committed a first pass at changing class inclusion to use plugin.inc from ctools. There maybe some small api changes still here as I clean things up, but nothing major. Setting to fixed.

See http://drupal.org/cvs?commit=291782

Status: Needs work » Closed (fixed)

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