Needs work
Project:
Metatag
Version:
7.x-1.x-dev
Component:
Context integration
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
18 May 2013 at 05:15 UTC
Updated:
27 May 2019 at 17:49 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #0.0
kristen polUpdated issue summary.
Comment #0.1
kristen polUpdated issue summary.
Comment #1
Chris Gillis commented#904416: Allow path contexts to handle query string has landed. Kristen, do you think you could describe the approach you took?
Comment #2
damienmckennaComment #3
damienmckennaThis would also need to define a minimum version requirement for the Context module in the metatag_context.info file.
Comment #4
jmsosso commentedHere is the patch!
Also changed the context dependency to 3.2 because that is the version where #904416 is commited.
Comment #5
jmsosso commentedPatch updated because the option Require all conditions must be checked if the user adds any query string condition. If not, query strings are ignored because context module match the path and stops evaluating conditions.
Comment #6
jmsosso commentedSorry, me again :)
I forgot to remove the query_string setting if the user leave the field empty.
Comment #7
damienmckennaThanks jmsosso, I appreciate the work.
I'd prefer to not have the requirement on a specific version of Context for this, so could you please undo the change to the info file? Thanks.
Comment #8
jmsosso commentedBut if context isn't at least in 3.2 the query_string input will be ignored. This can be confusing for the users... are you sure?