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.
If you create a page with an alias of "test", and set a block to show up only on "test", then go to "TeSt", the same page will appear, but the block sees it as a different page and will not appear.
This _might_ have security implications, although I don't see how.
Comment | File | Size | Author |
---|---|---|---|
#5 | drupal_block_visibility_case_702010.patch | 1.31 KB | jastraat |
#4 | drupal_block_visibility_case_702010.patch | 1.31 KB | jastraat |
Comments
Comment #1
mr.baileysDuplicate of #368093: Block visibility settings are case sensitive ?
Comment #2
adamfeldman CreditAttribution: adamfeldman commentedThis is not fixed as of 6.19. I have a URL of type "aBcd", which shouldn't be a problem if the entire URL string is being converted to lowercase.
Comment #3
manos_ws CreditAttribution: manos_ws commentedThis is not fixed for 6.20 , in the issue #368093: Block visibility settings are case sensitive ? there is a patch on post 47 and they say its committed. I guess its only committed for drupal 7. When I review the code of block.module in 6.20 and the patch code is not there.
Manos
http://websynergy.gr/
Comment #4
jastraat CreditAttribution: jastraat commentedConfirmed this is still a problem with 6.30.
Re-rolled the patch from #368093 and tested the new patch with the latest version of 6.x
It fixes the issue. This does seem to be a bug since paths should be case insensitive. Any chance we could get it applied?
Comment #5
jastraat CreditAttribution: jastraat commentedRemoving trailing spaces
Comment #6
mr.baileysTo be honest, I don't think this will ever make it into D6. For one, committing this to D6 might break existing sites (blocks that were not showing up on some pages because of the bug could suddenly start appearing after this fix. We should be very careful changing existing behavior). In this case it seems wiser to just leave it as is for D6 and mark this as a won't fix issue...
Comment #7
jastraat CreditAttribution: jastraat commentedThis was fixed in Drupal 7 however - so it would prepare clients who will eventually be upgrading to the new case-insensitive behavior.
I understand if it doesn't make it in, but it does seem to be a bug and was reported by one of our clients as such.