Comments (13)
the expected behaviour should be to check for permission when you try to use any action but display all the registered actions. (see code here ).
If you can confirm it is a bug and you send a PR, I will be happy to merge.
from django-adminactions.
@olivierdalang any updates on this issue ?
from django-adminactions.
Hi,
Sorry I gave no news, but I tested right now on a fresh install, and it seems indeed that I get
Sorry you do not have rights to execute this action (testapp.adminactions_export_people)
for any user except super-user, and no matter if the permission is assigned directly to the user or to one of his groups.
I'll definitely give it a shot as soon as I have a bit more time, so I'd suggest to keep the issue open (unless you can't reproduce the bug)
from django-adminactions.
did you grant 'adminactions_[ACTION]_[MODEL]' to your users? in thjs case testapp.adminactions_export_people
from django-adminactions.
yes (from the groups/users admin module)
2015-11-26 13:03 GMT+00:00 saxix [email protected]:
did you grant 'adminactions_[ACTION]_[MODEL]' to your users?
—
Reply to this email directly or view it on GitHub
#79 (comment)
.
from django-adminactions.
this is strange, just added a test for that, can you try the develop branch ? I'm close to release 1.0 but don't want to package with this error.
from django-adminactions.
I don't have any adminactions_[ACTION]_[MODEL]
permissions in the user administration (/admin/auth/user/2/
). Is that a symptom? I also have the permission problem and I can't even assign the permission in the admin panel. Checking if the permission exists in DB.
from django-adminactions.
All adminactions
permissions are listed in user.get_all_permissions()
but not in Permission.objects.all()
. Any way to set the permission? As I can't get it through django.contrib.auth.models.Permission
.
from django-adminactions.
permissions are created by create_extra_permission handler, it should be invoked when syncdb/migrate is executed. e8297f8 add new test to check Permission
table.
from django-adminactions.
any updates on this issue ? I'm not able to reproduce it and the related test works as expected.
from django-adminactions.
Hi,
Sorry for the lack of feedback. I'm testing again right now with 1.0, it works fine on a clean install, but still doesn't work on my site... I tried dropping all permissions, but I still get Sorry you do not have rights to execute this action
when trying to export entries while logged as an user in a group that has the export permission.
Any idea where the bug could come from ?
from django-adminactions.
@olivierdalang realy do not get the issue where come from. I added a test for it. can you share some other info (ie. where did you add the action (Modeladmin or site), are you able to invoke pdb to investigte the issue ?
from django-adminactions.
Ok I found the cause, it's because my modules have capital letters. Here's a PR to fix it :
#89
(there are some conflicts, I'll rebase this afternoon)
from django-adminactions.
Related Issues (20)
- Fix Documentation about how to use custom MassUpdateForm? HOT 1
- raising validation error when trying to merge instances with ArrayField
- mass_update_fields doesn't work HOT 1
- Configuration of default CSV export settings HOT 1
- Allow callables from the ModelAdmin HOT 1
- Mass update many-to-many field
- Can defaults for the export to CSV/XLS be set via Django config?
- Merge Selected action throws an exception
- create_extra_permissions crashes on occasion
- Charts are not accurate as I have only two entries in database model and it show that you have 9%-9% division in pie chart
- ForeignKey' object has no attribute 'rel' HOT 1
- Let the OperationManager walk the MRO of fields
- Make the OperationManager more convenient to add a function
- Merge action signals HOT 1
- TypeError: ExportMixin.get_export_form() takes 1 positional argument but 3 were given HOT 1
- Mass Update: Type Error HOT 1
- `TemplateSyntaxError 'extra_urls' is not a registered tag library. Must be one of: ...` HOT 2
- Create fixture for a whole app HOT 1
- adminactions.min.css cannot be found
- Merge not working with polymorphic models
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from django-adminactions.