You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
BaseResource.dict() about which I am less sure it is a good idea to change its behaviour;
That latter method seem to be dead code: never called anywhere as far as grep can tell, but then it is also part of the class public AP, so removing it may not be the best idea. But if we agree the omission of dict attributes is a bug, then it should be fixed.
Any clue ?
Regards, Nicolas.
The text was updated successfully, but these errors were encountered:
By the way should that ._keys() method be a @classmethod makes to make introspection at the class level as much as at the instance level no ? Changing that would not break the API).
The method BaseResource._key() does not include the attributes decalred in
BaseResource._dicts
.Is that intentional or not ? Chaiging this would impact the following methods:
BaseResource._bootstrap()
which seems for the best;BaseResource.dict()
about which I am less sure it is a good idea to change its behaviour;That latter method seem to be dead code: never called anywhere as far as grep can tell, but then it is also part of the class public AP, so removing it may not be the best idea. But if we agree the omission of dict attributes is a bug, then it should be fixed.
Any clue ?
Regards, Nicolas.
The text was updated successfully, but these errors were encountered: