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.

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

Manipulation of backend config per request (e. g. with event) #2023

Closed
stollr opened this issue Jan 24, 2018 · 2 comments
Closed

Manipulation of backend config per request (e. g. with event) #2023

stollr opened this issue Jan 24, 2018 · 2 comments

Comments

@stollr
Copy link
Contributor

stollr commented Jan 24, 2018

The backend config is fetched from the ConfigManager's getBackendConfig method. It returns the compiled configuration.

This is fine, but there is no possibility to manipulate it on a per request basis. Currently EasyAdminBundle lacks built-in security methods. So we have to implement it on our own.

In our case, we need to hide menu items and actions dependend on the user. This cannot be achieved with configuration passes.
Currently we do this in the template, which is really bad of course.

My proposal is to add an event to the getBackendConfig (e.g. after calling processConfig()), so that one can hook into this process and manipulate the config once per request.

What do you think? If liked I'd create a PR

@javiereguiluz
Copy link
Collaborator

I'm closing this issue because we're starting a new phase in the history of this bundle (see #2059). We've moved it into a new GitHub organization and we need to start from scratch: no past issues, no pending pull requests, etc.

I understand if you are angry or disappointed by this, but we really need to "reset" everything in order to reignite the development of this bundle.

@stollr
Copy link
Contributor Author

stollr commented Feb 26, 2018

@javiereguiluz no problem ;-) I'm looking forward to see this bundle getting even better.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants