LibModel.items() confusion

The docs describe LibModel.items() to

Iterate over (key, value) pairs that this object contains. Computed fields are not included.

This is the case for Item.items(), but Album.items() gives all the Items associated with the Album.

LibModel.items() is actually defined in Model.items(), and overwritten in the Album class.

So I’m wondering what the intended behavior is 😃

I know I have used Album.items() as it currently is in the past (and it seems to me a sensibly named function). Just now I tried to use it in the sense defined in the docs and ran into this problem (although I want computed fields so not using it any more). In the docs sense I guess the items() mimics dict.items(), but does create some confusion with Items 😄

[govynnus]

Yep I think just documenting the conflict would be good. Doesn’t seem like it’s been too much of an issue in the past.

[wisp3rwind]

So yes, it’s not a great situation, but it’s not clear what we should do instead. Perhaps rename Album.items() to something else someday?

While unfortunate; this seems like a change that would break many third-party plugins. In my opinion, we should avoid renaming .items() (unless there’s a stronger reason than being somewhat confusing). Maybe extending the Album.items() docstring to point this out would help to alleviate the issue?