|
[
Permlink
| « Hide
]
Ludovic Dubost added a comment - 22/Dec/06 14:37
Moving to unassigned according to new policy
Ludovic, what happens if the user doesn't have the programming rights and he saves a document which needs it? Is it going to fail with an error?
Thanks No, the document looses the programming rights, which means the code in it will not be executed. This causes a page not to function anymore.
The normal way for preventing this is to protect all pages that need programming right (protect = only admins can edit), since such pages provide important functionality for the wiki (the registration page used to require programming rights in an older version, and that meant that whenever somebody edited the registration form, no users could register anymore). However, this has to be implemented. It seems to me that this can only be done by searching for functions in the API that require programming rights. This is bad, since the API changes quite often, and there are many functions that will be searched.
Searching for APIs is both time consuming and doesn't scale. A better solution discussed with Marta and Sergiu is change the architecture/design for checking programming rights. For example, a centralized module could register the different API requiring those rights and calls would go through it and be checked in a generic manner.
Thus, moving this to post 1.0 I propose a slightly different way to manage this, patch & screenshots attached
Commiting it now, shout if you think it's a bad idea
 Reopened since a more elegant solution can be implemented in 1.4M1
JV tells me that you're doing it sergiu so assigning it to you, remove if this is not correct...
Yes, I have some ideas, but it's not a top priority for me now. The velocity uberspectors need to be implemented first.
This doesn't look like a bug.
Doesn't seem important enough to be addressed during the course of the 1.9 roadmap. Ludovic, please detail your use case if you really think this issue needs to be addressed in 1.9.
This is hardly a showstopper, it looks like a nice-to-have but not an urgent thing. |
||||||||||||||||||||||||||||||||||||||||||||||||
 |
|
Improvement
Reopened
Minor
Show warning if document requires a programming right and the editor does not have it
