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.
That bug was is not new (#1286410: Access denied - You are not authorized to access this page.) and finally I thought I should fix it.
The problem is that Display Suite doesn't check whether a layout is present and the view mode isn't overridden. I also fixed the typo ("overriden" instead of "overridden").
Comment | File | Size | Author |
---|---|---|---|
#12 | ds-access_deniced-1988900-12.patch | 982 bytes | yannickoo |
#11 | ds-access_deniced-1988900-11.patch | 972 bytes | yannickoo |
#9 | ds-access_deniced-1988900-9.patch | 472 bytes | yannickoo |
#8 | ds-access_deniced-1988900-8.patch | 472 bytes | yannickoo |
#6 | ds-access_deniced-1988900-6.patch | 662 bytes | yannickoo |
Comments
Comment #1
yannickooComment #2
yannickooThe patch should work for 8.x-2.x but I created an extra patch for it :)
Comment #3
swentel CreditAttribution: swentel commentedthis looks freaky because you now have this:
AND OR AND
without any parentheses, I assume we want to group this no ?
Comment #4
yannickooOh sorry, will add this, hold on :D
Comment #5
yannickooComment #6
yannickooComment #7
aspilicious CreditAttribution: aspilicious commentedI don't think your fix is correct:
First you set the variable to TRUE or FALSE and afterwards you check if it equals NULL....
I have the same problem but it is only related to the default view mode afaict.
When accessing the default view_mode the machine name is not appended to the manage display path.
Comment #8
yannickooI will create a follow-up issue for the typo but I figured out that the check for the
$layout
isn't necessary.Comment #9
yannickooComment #10
aspilicious CreditAttribution: aspilicious commentedHmmm this could work. I let this to Swentel to verify if it's ok.
If we don't need the $layout checkout we should remove
Comment #11
yannickooThat is true!
Comment #12
yannickooComment #13
swentel CreditAttribution: swentel commentedSo yeah, that problably makes sense indeed, will test this some more, going to write for a test for it as well to make sure this stays ok in the future.
Comment #14
swentel CreditAttribution: swentel commentedIronically enough I hit this bug during my demo in Portland, but could not reproduce anymore locally. Anyway committed because the fix looks sane, I've changed the typo's as well in both branches.
Comment #15.0
(not verified) CreditAttribution: commentedLinked issue