Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Using this module to present different themes based on a query key.
Comment | File | Size | Author |
---|---|---|---|
#12 | switch_page_theme-query-key.patch | 4.07 KB | StijnStroobants |
#3 | 2984636-query-key-1.patch | 4.44 KB | john.money |
#2 | 2018-07-09_1133.png | 263.24 KB | john.money |
Comments
Comment #2
john.money CreditAttribution: john.money at Acquia commentedPatch attached.
Comment #3
john.money CreditAttribution: john.money at Acquia commentedFixed null logic for theme_key.
Comment #4
kumkum29 CreditAttribution: kumkum29 commentedVery useful patch !!
Do you think include this patch/code in the next version ? (question to maintainers)
Thanks.
Comment #5
kumkum29 CreditAttribution: kumkum29 commentedFor each configuration, an interesting evolution for this patch is to choose the name of the query, i.e.: ?section=..., ?chapter=...
Comment #6
ressa CreditAttribution: ressa at Ardea commentedThis would be a great feature, thanks for creating the patch @john.money.
Comment #8
deepali_agarwal CreditAttribution: deepali_agarwal commentedThank You @john.money for the patch.
@kumkum29: I think, choosing the name of the query, i.e.: ?section=..., ?chapter=... is not required as you can always pass ?theme_key=... and make things work. Besides adding the name of the query will only complicate the module. Reach out to me, if you have any scenario where you cannot pass ?theme_key= and the name of the query is a must, we can discuss and then open a new issue for same.
Comment #9
deepali_agarwal CreditAttribution: deepali_agarwal commentedComment #10
ressa CreditAttribution: ressa at Ardea commentedThanks for committing the patch @deepali_agarwal.
I agree with your thoughts about having only one query name. I do think that changing it to something shorter might be worthwhile, from
?theme_key=
to perhaps?tkey=
, or even?tk=
. What do you think? Or perhaps it could be configurable from the GUI?Comment #12
StijnStroobantsAs I don't want to use a development-release, here is a patch that applies against the current 8.x-1.0 release
But I would suggest to create a new release asap, as the current one is not updated for a few years.