Improvement of class pyroSAR._dev_config.Storage

[ibaris]

I found a way to improve our Storage class. With this we could allow only particular keys. this could help not to mess around with this class (look at __setattr__)

class Storage(dict):
     def __getattr__(self, name):
        try:
            return self[name]
        except KeyError:
            raise AttributeError(name)

    __delattr__ = dict.__delitem__

    KEYLIST = ['KEY1', 'KEY2', 'KEY3']

    def __repr__(self):
        if self.keys():
            m = max(map(len, list(self.keys()))) + 1
            return '\n'.join([k.rjust(m) + ': ' + repr(v)
                              for k, v in sorted(self.items())])
        else:
            return self.__class__.__name__ + "()"

    def __dir__(self):
        return list(self.keys())

    def __setattr__(self, key, value):
        if key not in Storage.KEYLIST:
            raise KeyError("{} is not a legal key of this Storage".format(repr(key)))
        dict.__setitem__(self, key, value)

What do you think?

[johntruckenbrodt]

The Storage class is no longer used and was removed from the code. See #305.