Use write mode by default to check create and delete of resources

It is less astonishing for the user and it allows to customize easily the
check to perform per model.

issue7717
review54371002
This commit is contained in:
Cédric Krier
2018-10-04 13:41:50 +02:00
parent 48d894b08c
commit 9c5bdf2451
2 changed files with 10 additions and 1 deletions
+2
View File
@@ -1,3 +1,5 @@
* Use write mode by default to check create and delete of resources
Version 5.0.0 - 2018-10-01
* Bug fixes (see mercurial logs for details)
* Allow non translatable reports
+8 -1
View File
@@ -69,7 +69,14 @@ class ResourceMixin(ModelSQL, ModelView):
if record.resource:
model_names.add(str(record.resource).split(',')[0])
for model_name in model_names:
ModelAccess.check(model_name, mode=mode)
checks = cls._convert_check_access(model_name, mode)
for model, check_mode in checks:
ModelAccess.check(model, mode=check_mode)
@classmethod
def _convert_check_access(cls, model, mode):
return [
(model, {'create': 'write', 'delete': 'write'}.get(mode, mode))]
@classmethod
def read(cls, ids, fields_names=None):