Tryton Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(19)

Issue 68321002: trytond: Remove check access to trigger_write_get_eligibles (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
1 week, 4 days ago by ced
Modified:
2 hours, 11 minutes ago
Reviewers:
pokoli, rietveld-bot, reviewbot
Visibility:
Public.

Description

The triggers returned need to be read without any check access. issue7764

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -0 lines) Patch
M trytond/model/modelstorage.py View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 5
ced
1 week, 4 days ago (2018-10-04 17:03:35 UTC) #1
reviewbot
https://codereview.tryton.org/68321002/diff/1/trytond/model/modelstorage.py#newcode240 trytond/model/modelstorage.py:240: E501 line too long (82 > 79 characters) https://codereview.tryton.org/68321002/diff/1/trytond/model/modelstorage.py#newcode752 trytond/model/modelstorage.py:752: E501 line too ...
1 week, 4 days ago (2018-10-04 17:04:11 UTC) #2
pokoli
LGTM
1 week ago (2018-10-08 09:55:20 UTC) #3
rietveld-bot_tryton.org
New changeset 10a4e81a84f9 by Cédric Krier in branch 'default': Remove check access to trigger_write_get_eligibles https://hg.tryton.org/trytond/rev/10a4e81a84f9
2 hours, 12 minutes ago (2018-10-15 20:50:10 UTC) #4
rietveld-bot_tryton.org
2 hours, 11 minutes ago (2018-10-15 20:51:51 UTC) #5
New changeset ed5491c52565 by Cédric Krier in branch 'default':
Remove check access to trigger_write_get_eligibles
https://hg.tryton.org/tryton-env/rev/ed5491c52565
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 0147766