Revision f20684bf
Renommage de _grouptype en grouptype (#992).
Rend public l'attribute "grouptype" sur les Group.
Ceci évite une mauvaise pratique (accès à un attribut protégé depuis
d'autres classes) qui était utilisée jusqu'ici.
Change-Id: I0dda99d53756c11553d9e4d35818726f6e42db96
Refs: #992.
Reviewed-on: https://vigilo-dev.si.c-s.fr/review/1072
Tested-by: Build system <qa@vigilo-dev.si.c-s.fr>
Reviewed-by: Thomas BURGUIERE <thomas.burguiere@c-s.fr>
| vigigraph/controllers/rpc.py | ||
|---|---|---|
| 526 | 526 |
if not is_manager: |
| 527 | 527 |
direct_access = False |
| 528 | 528 |
user = get_current_user() |
| 529 |
# pylint: disable-msg=W0212 |
|
| 530 |
# W0212: Access to a protected member _grouptype of a client class |
|
| 531 | 529 |
|
| 532 | 530 |
# On calcule la distance de ce groupe par rapport aux groupes |
| 533 | 531 |
# sur lesquels l'utilisateur a explicitement les permissions. |
| ... | ... | |
| 553 | 551 |
(USER_GROUP_TABLE, USER_GROUP_TABLE.c.idgroup == \ |
| 554 | 552 |
UserGroup.idgroup), |
| 555 | 553 |
).filter(USER_GROUP_TABLE.c.username == user.user_name |
| 556 |
).filter(Group._grouptype == u'supitemgroup'
|
|
| 554 |
).filter(Group.grouptype == u'supitemgroup' |
|
| 557 | 555 |
).filter(GroupHierarchy.idchild == parent_id |
| 558 | 556 |
).order_by(GroupHierarchy.hops.desc()).scalar() |
| 559 | 557 |
|
| ... | ... | |
| 570 | 568 |
(USER_GROUP_TABLE, USER_GROUP_TABLE.c.idgroup == \ |
| 571 | 569 |
UserGroup.idgroup), |
| 572 | 570 |
).filter(USER_GROUP_TABLE.c.username == user.user_name |
| 573 |
).filter(Group._grouptype == u'supitemgroup'
|
|
| 571 |
).filter(Group.grouptype == u'supitemgroup' |
|
| 574 | 572 |
).filter(GroupHierarchy.idparent == parent_id |
| 575 | 573 |
).order_by(GroupHierarchy.hops.desc()).scalar() |
| 576 | 574 |
if distance is not None: |
Also available in: Unified diff