Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

KeyError occurs when CMS toolbar is visible in Django CMS 3.11.8 #214

Open
jdgoettsch opened this issue Sep 11, 2024 · 1 comment · May be fixed by #215
Open

KeyError occurs when CMS toolbar is visible in Django CMS 3.11.8 #214

jdgoettsch opened this issue Sep 11, 2024 · 1 comment · May be fixed by #215
Labels
type: bug

Comments

@jdgoettsch
Copy link

jdgoettsch commented Sep 11, 2024 •
edited
Loading

Description

PageToolbarMeta.populate uses incorrect permission name to check if a user has change permission on the current page, causing a KeyError to be raised with django-cms >= 3.11.8.

Steps to reproduce

  1. Install djangocms-page-meta with django-cms 3.11.8.
  2. Log into your website and visit any page with the toolbar visible.

Versions

Python 3.10.14
Django 4.2.16
django-cms. 3.11.8

Expected behaviour

Page loads without crashing.

Actual behaviour

KeyError occurs and page crashes.

Additional information

In djangocms_page_meta/cms_toolbars.py on line 39, "change" should instead be "change_page".
@jdgoettsch jdgoettsch linked a pull request Sep 11, 2024 that will close this issue
Open
3 tasks
@tschale
Copy link

tschale commented Oct 25, 2024 •
edited
Loading

Thank you for the fix! What's the state of it? Is there a timeline for its release?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type: bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Bug/#214-Fix misnamed 'change_page' permission jdgoettsch/djangocms-page-meta
2 participants
@tschale @jdgoettsch