Tryton - Issues

 

Issue7005

Title Remove rule on user
Priority feature Status resolved
Superseder Nosy List ced, reviewbot, roundup-bot
Type feature request Components trytond
Assigned To ced Keywords review
Reviews 43821003
View: 43821003

Created on 2017-12-10.18:23:08 by ced, last changed by roundup-bot.

Messages
New changeset 43fa4d1e063a by C├ędric Krier in branch 'default':
Remove rules on user
http://hg.tryton.org/trytond/rev/43fa4d1e063a
review43821003 updated at https://codereview.tryton.org/43821003/#ps1
msg37228 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2017-12-10.18:27:58
Here is review43821003
I could not write a migration script because it would require to create a group for each rule linked to a user. I do not think there are many usage of this removed feature in production so it could be managed manually for the case where it was used. I will write a migration steps that will have to be done before the migration to set the user rules to a group.
msg37227 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2017-12-10.18:23:08
The group should be the only object that is used to assign rights to the user.
As group has already a link to rule, the user should not have one.
History
Date User Action Args
2017-12-20 19:21:02roundup-botsetstatus: testing -> resolved
nosy: + roundup-bot
messages: + msg37407
2017-12-10 18:39:32reviewbotsetnosy: + reviewbot
messages: + msg37230
2017-12-10 18:27:59cedsetstatus: in-progress -> testing
reviews: 43821003
messages: + msg37228
keyword: + review
2017-12-10 18:23:08cedcreate

Showing 10 items. Show all history (warning: this could be VERY long)