Tryton - Issues

 

Issue7172

Title Unique email constraint
Priority feature Status resolved
Superseder Nosy List ced, reviewbot, roundup-bot
Type feature request Components web_user
Assigned To ced Keywords review
Reviews 40141002
View: 40141002

Created on 2018-02-27.16:48:07 by ced, last changed by roundup-bot.

Messages
New changeset a1fdcc07c8ab by C├ędric Krier in branch 'default':
Require email only for active user
http://hg.tryton.org/modules/web_user/rev/a1fdcc07c8ab
review40141002 updated at https://codereview.tryton.org/40141002/#ps1
msg38647 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2018-02-27.16:48:07
There is a unique constraint on the email of web user but indeed this constraint is only useful for active user.
On the long term, user could return after being inactivated (ex: issue7169). In this case, we can not be sure that it is still the same user.
So I think we should only require the email for active user and so let the unique constraint allow multiple NULL.
History
Date User Action Args
2018-03-19 16:20:18roundup-botsetstatus: testing -> resolved
nosy: + roundup-bot
messages: + msg39095
2018-02-27 17:45:23reviewbotsetnosy: + reviewbot
messages: + msg38651
2018-02-27 17:44:11cedsetstatus: in-progress -> testing
reviews: 40141002
keyword: + review
2018-02-27 16:51:24cedlinkissue7169 superseder
2018-02-27 16:48:07cedcreate

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