- Issue created by @super_romeo
- Status changed to Closed: works as designed
5 months ago 7:04pm 7 August 2024 - πΊπΈUnited States jcontreras
! is to exclude the previous wild card pattern.
For what you want to do, make sure you have the "Hide for the listed pages" checked then:
/foo/*
!/foo/bar - π·πΈSerbia super_romeo Belgrade
Thank you for answer.
But I wanna keep "Show for the listed pages" mode, because I know exactly only paths to show.
My code:/order/* !/order/*/* /order/*/complete
Is it possible?
- π·πΈSerbia super_romeo Belgrade
This case is covered by Condition Path β module (see "Example 2").
- Status changed to Active
5 months ago 3:49pm 8 August 2024 - πΊπΈUnited States jcontreras
I see what you mean. I think your point is very valid. I will follow up with a new update soon to allow linear exclusion.
- π·πΈSerbia super_romeo Belgrade
Thank you very much. I'm looking forward to it.
- Status changed to Needs review
5 months ago 5:32pm 8 August 2024 - πΊπΈUnited States jcontreras
@super_romeo - I created a new major version for this as there was a lot to rewrite to accomplish this and fixed some other issues.
Please try it and let me know if this worked for what you intended. - π·πΈSerbia super_romeo Belgrade
I installed new version. And my Site branding block is disappeared. And contextual menu is corrupted.
Please see pics. - πΊπΈUnited States jcontreras
You are correct, I refined the access adjustment to the block for viewing only as it was altering some admin functions as well.
Please update to the latest 2.2.x-dev and let me know if it works this time.
( you may have manually delete the module's folder from the contrib modules on your setup to force to update.) - π·πΈSerbia super_romeo Belgrade
Thank you for update.
1. "Site branding" block is still disappeared.
2. "Site branding" block config fixed.
3. Cases:
a)!/order/*/* /order/*/complete
/order/1/foo - hidden (OK)
/order/1/complete - visible (OK)b)
/order/*/complete !/order/*/*
/order/1/foo - hidden (OK)
/order/1/complete - hidden (wrong)I believe the path order should not depend on the path order.
- πΊπΈUnited States jcontreras
There is no other way it has to be in order.
So both examples are not correct, first, you set the allowed or not-allowed setting as you need, The exclusion is to override any previous path patterns.
Your examples should be:
A)
configuration is set to: [x] Hide on pages :
/order/*/*
!/order/*/complete
B)
configuration is set to: [x] Show on pages:
/order/*/*
<-- This will display your block on paths with this pattern
!/order/*/complete
<-- and this will exclude this specific path pattern, overwriting previous patternBut based on your original logic I think what you want is this:
configuration is set to: [x] Show on pages:
/order/*
<-- will display on ay page after order/
!/order/*/*
<-- exception will hide on any page after order/*/
/order/*/complete
<-- This will take over the exception allowing the block to display on /order/*/complete page - π·πΈSerbia super_romeo Belgrade
Thank you. Got it.
Now all good except "Site branding block is still disappeared". - π·πΈSerbia super_romeo Belgrade
"Hide for the listed pages"
/trains/order/*/timetable /trains/order/*/timetable2 /trains/order/*/passenger /trains/order/*/payment
And block is hidden on
/order/modification/3/aa7e50fddf30b0a5384b4b44c916f5c3/passenger
.I think wildcard
/trains/order/*/passenger
wrongly applied to this path. - π·πΈSerbia super_romeo Belgrade
Can I hope to have this problem fixed in the future? So that I can plan.
- π·πΈSerbia super_romeo Belgrade
Hello @jcontreras,
please check.Tests as a bonus :)
- πΊπΈUnited States jcontreras
Hi @super_romeo - Sorry It's been a crazy week last week and will continue on to the next week. I am working on a large project with crazy deadline. I took a quick glance at you changes, and it looks really good.
Since you did a lot of changes, I'll have to take closer look, pull it all and run some tests and if all good I'll do the merge.
I'll see if I can carb some time tomorrow as of today I still got a lot to do. Thank you so much for your time on this.
- πΊπΈUnited States jcontreras
I moved this to 2.2.0-alpha2 - please review.