__getitem__ subclassing issue for new 2.2.0 about addict

[mewwts]

Happy new year @yhhsteven,

If you only want to use addict as an attribute dict perhaps you should instead make a class like

class D:
    def __init__(self, x):
        self.attrs = addict.Dict()
        self.x = x

and then use it like D(1).attrs.y?

Thanks for using addict!

[yhhsteven]

Hi, any update?

[mewwts]

Hey Steven,

Please see this discussion to see if it helps resolve your issue #101. If it doesn't please ping me again!

[MisterVladimir]

Hi @yhhsteven,

The difference between addict 2.2.0 and 2.1.3 is that passing a missing key to __getitem__ 2.2.0 returns an empty instance of your derived class instead of an empty addict.Dict. What do you think of replacing if out == {} with if len(out) == 0? A more elegant solution would be to re-implement the __missing__(self, name) method of PLAYER to return getattr(PLAYERDB(self.pid), name). What do you think?

[yhhsteven]

@MisterVladimir @mewwts Thanks for your contributing! I like the idea changed in 2.2.0. I'm using the missing style right now. However, I ran into a problem when implement addict in this way, I'd like know if there is a better way to change my code:

class D(addict.Dict):
    def __init__( self, x ):
        super(D, self).__init__()
        self.x = x

This intends to design a custom attribute x while only utilizing addict as an attribute dict (not the recursive feature). This gives error TypeError: __init__() got an unexpected keyword argument '__parent' when trying to access the missing attribute like D(1).y.

I wonder if the addict's missing should add an AttributeError instead, for a better exception handling, or should I change my code to more correct way when subclassing addict?

Thank you and happy new year!