Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

CMS_PLACEHOLDER_CONF limit does not work in frontend #1308

Closed
Schwankenson opened this Issue · 1 comment

4 participants

@Schwankenson

The example from the docs:

'right-column': {
    "plugins": ['TeaserPlugin', 'LinkPlugin'],
    "extra_context": {"width":280},
    'name':gettext("Right Column"),
    'limits': {
        'global': 2,
        'TeaserPlugin': 1,
        'LinkPlugin': 1,
    },
},

Does not work in the frontend. I can paste as many plugins as I want to.

I think, it would even be a good thing, to hide the add Button for Plugins, if the placeholders limit has reached.

@mkoistinen
Owner

In my project’s 2.3.5 install, CMS_PLACEHOLDER_CONF does limit the user to only the plugin's specified and the number, but that the user can use the "Move to placeholder" menu to move plugins into places that they shouldn't be allowed to. Also, one can create arbitrary plugins and move them into a placeholder in excess of the destination placeholder’s limits.

So, in summary, Its the "Move to placeholder" function on the front end that doesn't respect the CMS_PLACEHOLDER_CONF configuration.

In the backend, the user can drag-and-drop essentially any plugin into any placeholder, despite any CMS_PLACEHOLDER_CONF settings.

@digi604 digi604 added the frontend label
@FinalAngel FinalAngel was assigned by digi604
@FinalAngel FinalAngel removed their assignment
@digi604 digi604 was assigned by FinalAngel
@digi604 digi604 removed their assignment
@FinalAngel FinalAngel was assigned by digi604
@FinalAngel FinalAngel referenced this issue from a commit in FinalAngel/django-cms
@FinalAngel FinalAngel fixes issue #1308 8c4eda4
@FinalAngel FinalAngel referenced this issue from a commit in FinalAngel/django-cms
@FinalAngel FinalAngel fixes issue #1308 748e1f6
@FinalAngel FinalAngel referenced this issue
Merged

Minor fixes #2766

@digi604 digi604 closed this in #2766
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.