Review: Needs Fixing
> It is performed as expected when server-branch is compatible.

OK, sorry, forgot about that one.

> And for demo user no record exist in `res.widget.user`, so displays all 
> widgets including `openerp favorites` and does not display ADD and Close 
> button.

No, that's not good, if the user has no widget then the user has no widget 
period end of the story.

Logic is the following, very simple:

1. widgets in res.widget.user with user_id is null are displayed for current 
user and don't have delete button

2. widgets in res.widget.user with user_id=uid are displayed for current user 
and have delete button

3. user can always add new widget to bar, even when he has none displayed

> Can't understand this comment.

The part about demo user (if no widget for user in res.widget.user, you just 
read everything in res.widget) is not good
-- 
https://code.launchpad.net/~openerp-dev/openobject-client-web/widget-per-user/+merge/41128
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-client-web/widget-per-user.

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to