Patch (to be ported)
Project:
Context error
Version:
6.x-1.x-dev
Component:
User interface
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
4 Aug 2011 at 04:13 UTC
Updated:
24 Dec 2011 at 10:31 UTC
Jump to comment: Most recent file
Comments
Comment #1
George Bills commentedIt also looks like "~error_page" (when error_page is the name of the context) doesn't work in a "context" condition. Other "~context" specifiers work fine.
Comment #2
George Bills commentedI wrote a patch to reverse the logic here. It's pretty hacky and it removes the ability to say "only show on 404 error pages", but it seems to work. Attached.
Comment #3
George Bills commentedIgnore the last - here's a much nicer patch.
It lets the user pick between "An error page", "403 error pages only", "404 error pages only" or "not an error page".
One problem is that the selection is done with checkboxes, but the logic only makes sense if one and only one value is picked. The selection should really be done using a select dropdown, but I didn't know how to do that inside the Context API.
Comment #4
Jānis Bebrītis commentedcan we get this into release, please?
Comment #5
snufkin commented@PersonalPronoun Thanks very much for the patch! Could you take care that coding standards (http://drupal.org/coding-standards) are kept in the code and reroll the patch please?
Comment #6
szantog commentedThe refactored patch against today's update for d7.
Comment #7
snufkin commentedCommitted, thanks! Now if someone wants to backport it to D6 feel free, I am not going to work on that branch anymore.