This repository was archived by the owner on Dec 5, 2018. It is now read-only.
Éviter de masquer les AttributeError sur __acl__ #810
Comments
tonthon
added a commit
that referenced
this issue
Nov 8, 2018
Fix #810 : Cesse d'utiliser property pour définir __acl__
tonthon
pushed a commit
to tonthon/autonomie
that referenced
this issue
Nov 9, 2018
Le passage d'acl sous forme de property vers des acls sour forme de callable PR CroissanceCommune#811 (bug CroissanceCommune#810) a introduit des erreurs dans le cas des acls "forwardées" a.__acl__ = lambda self: "acl de a" b.__acl__ = lambda self: a.__acl__ b.__acl__() est une lambda alors qu'auparavant b.__acl__ renvoyait "acl de a"
tonthon
pushed a commit
to tonthon/autonomie
that referenced
this issue
Nov 9, 2018
Le passage d'acl sous forme de property vers des acls sour forme de callable PR CroissanceCommune#811 (bug CroissanceCommune#810) a introduit des erreurs dans le cas des acls "forwardées" a.__acl__ = lambda self: "acl de a" b.__acl__ = lambda self: a.__acl__ b.__acl__() est une lambda alors qu'auparavant b.__acl__ renvoyait "acl de a"
sbodrero
pushed a commit
to sbodrero/autonomie
that referenced
this issue
Nov 10, 2018
…acl__ Cela complexifiait le debug lorsque le code d'une ACL déclenchait une AttributeError car Pyramid ignorait cette dernière silencieusement. Je ne vois pas d'inconvénient à ce nouveau fonctionnement. Ref Pylons/pyramid#735 Ref CroissanceCommune/autonomie_base#2
sbodrero
pushed a commit
to sbodrero/autonomie
that referenced
this issue
Nov 10, 2018
Le passage d'acl sous forme de property vers des acls sour forme de callable PR CroissanceCommune#811 (bug CroissanceCommune#810) a introduit des erreurs dans le cas des acls "forwardées" a.__acl__ = lambda self: "acl de a" b.__acl__ = lambda self: a.__acl__ b.__acl__() est une lambda alors qu'auparavant b.__acl__ renvoyait "acl de a"
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
En définissant
__acl__comme callable plutôt qu'en property.cf Pylons/pyramid@aae62a0
The text was updated successfully, but these errors were encountered: