zas: 
Hi, I'm a bit confused by your recommendations [here](https://github.com/metabrainz/picard/pull/2545#pullrequestreview-2441960574)
> Declaring attributes as "protected" (prefixing them with underscore) is good, but then they aren't meant to be used outside the class, apart its subclasses.
> For _genres and _folksonomy_tags, please check carefully, and use the getter method if not in the class itself or a subclass.

genres and folksonomy_tags are both attributes of MetadataItem class and which is inherited by Album class
and in Track, Album is an attribute
So since we are modifying genres and folksonomy_tags in Track, so I would need to change it to do it so through getters and setters yeah?

But then in your suggested changes, you removed the `_` from genres and folksonomy_tags, making both of these public(?)

So should genres and tags be public or protected (thus only be accessed via getters and setters)?

(also sorry for the delay)