Created attachment 117295 [details] The wallpaper shows up anyway This item is marked as ghns_exclude: https://store.kde.org/p/1082769 But it shows up anyway. See attachment. Happens when using KDE Frameworks 5.53, 5.54, or git master.
Could you check and make sure this is still marked as ghns-excluded for you? Just looked here, and it seems not to be excluded for me...
As a test (because the item is already excluded by other means) i just tried to mark https://share.krita.org/p/1207357/ as excluded, and was informed by the site that it was successfully excluded. I just went back to check again, and it looks like that was not entirely true. Looks like we've probably got a bit of a server issue here...
So, the plot thickens. We are now at a point where i can confirm that there is indeed a problem and that items are not being excluded, even when the ocs data says it should be. Taking and poking, shall report when i work out what's going on.
Right, found the problem. The OCS server is exporting the exclusion tag as "ghns_excluded" rather than the expected "ghns_exclude", and so it is entirely ignored. The server team is aware of this, and i'll close this as fixed once it has, in fact, been fixed :)
So, turns out that this is a deeper issue after all. Simply put, the problem is that there are semantic issues, which given this is meta data becomes very important to get fixed. So, what will happen next is that i whip up a patch to fix the client's expectations. The result will be that older versions of Frameworks will not have items excluded, but future metadata will be more consistent (in short, the metadata should not be action verbs - for a specific example, (ghns_)excluded rather than (ghns_)exclude).
Related review: https://phabricator.kde.org/D18038
This was fixed in https://phabricator.kde.org/R304:301fe73569ba9dbd1f80398fd616d8755accda15 Fix semantics for ghns_exclude After much discussion on the topic, it came to light that the specific grammar used for the default tag in KNSCore was incorrect, as it was an action verb (rather than an adjective the way the metadata tags are supposed to be). While this does mean that previous versions of Frameworks will not have the filtering functionality by default, it is otherwise non-invasive and causes no side effects apart from the server-suggested filtering working. BUG: 402888 Differential Revision: https://phabricator.kde.org/D18038