Only execute conditions/reactions if they are actually used by at least one context on a given site.

Some condition checks (in particular, the menu active trail check) can be very expensive on certain sites. Given that context already caches a map of conditions to contexts, it should be straightforward to check that contexts actually use a given condition before executing its condition.

A solution for this should be pushed into both 2.x and 3.x.

Files: 
CommentFileSizeAuthor
#5 context-811424.patch859 bytesSteven Jones

Comments

yhahn’s picture

Assigned: yhahn » Steven Jones

I've committed a fix for this here: http://drupal.org/project/cvs/64885

@Steven Jones I'll let you decide whether a similar solution should go into 2.x.

Steven Jones’s picture

Issue tags: +Context-2.1

Adding to context 2.x roadmap

Steven Jones’s picture

Status: Active » Closed (won't fix)

No more features for the 2.x branch I'm afraid.

Steven Jones’s picture

Status: Closed (won't fix) » Needs work

Actually, I might fix this.

Steven Jones’s picture

Status: Needs work » Reviewed & tested by the community
FileSize
859 bytes

Patch attached.

Steven Jones’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)
Issue tags: -Context-2.1

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