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

It should not list excluded fields as read only or unique #102

Open
atleta opened this issue Jan 4, 2022 · 0 comments
Open

It should not list excluded fields as read only or unique #102

atleta opened this issue Jan 4, 2022 · 0 comments

Comments

@atleta
Copy link

atleta commented Jan 4, 2022

I don't want read only fields to be displayed (in my case it's just cruft that confuses the user). However, listing them in massadmin_exclude doesn't work, because it's only checked for fields that are not read only or unique.

The order of the checks should be reversed in the template (admin/includes/mass_fieldset.html) so that the first (outermost) if statement verifies if the field is in massadmin_exclude. If not then it can go on to the current logic. This way it's still possible to display the read only and unique fields as such but they can also be hidden by adding them tomassadmin_exclude`.

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

1 participant