Once this patch lands:
http://drupal.org/node/904416#comment-6541140
to have Context query string conditions, it would be super handy to update the metatag_context module to allow for the query string configuration directly on the metatags by path page (admin/config/search/metatags/context) rather than having to go to the Context configuration to add the query string conditions configuration.
I did apply the patch and was able to configure the meta tags for paths with query strings (e.g. mypage?page=1) via the Context configuration which is useful, for example, if you don't want to have duplicate meta tags on your pager pages.
Comment | File | Size | Author |
---|---|---|---|
#6 | metatag_context_add_query_string_condition-1997786-6.patch | 3.87 KB | jmsosso |
| |||
#5 | metatag_context_add_query_string_condition-1997786-5.patch | 3.8 KB | jmsosso |
| |||
#4 | metatag_context_add_query_string_condition-1997786-4.patch | 3.65 KB | jmsosso |
|
Comments
Comment #0.0
Kristen PolUpdated issue summary.
Comment #0.1
Kristen PolUpdated issue summary.
Comment #1
Chris Gillis CreditAttribution: 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 CreditAttribution: jmsosso at TUI commentedHere is the patch!
Also changed the context dependency to 3.2 because that is the version where #904416 is commited.
Comment #5
jmsosso CreditAttribution: jmsosso at TUI 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 CreditAttribution: jmsosso at TUI 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 CreditAttribution: jmsosso at TUI 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?