<table><tr><td style="">ngraham added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D25100">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
color: #6B748C;
font-style: italic;
margin: 4px 0 12px 0;
padding: 8px 12px;
background-color: #F8F9FC;">
<div style="font-style: normal;
padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D25100#592375" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: line-through;">D25100#592375</a>, <a href="https://phabricator.kde.org/p/mak/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@mak</a> wrote:</div>
<div style="margin: 0;
padding: 0;
border: 0;
color: rgb(107, 116, 140);"><p><a href="https://phabricator.kde.org/p/ngraham/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@ngraham</a> Did this have AppStream metadata before? Probably the distribution data still lists this as component, while the new file lists it as addon. The distro data is preferred, so that's why this shows up as app. You need your distro to ship this as update, or set <tt style="background: #ebebeb; font-size: 13px;">PreferLocalMetainfoData=true</tt> in <tt style="background: #ebebeb; font-size: 13px;">/etc/appstream.conf</tt> (that should override the distro-provided data with local one, which is nice for development but not usually desired).</p>
<p>The output of <tt style="background: #ebebeb; font-size: 13px;">appstreamcli dump org.kde.haenau</tt> (or <tt style="background: #ebebeb; font-size: 13px;">appstreamcli get org.kde.haenau --details</tt> is also always helpful to investigate issues like this. On my machine (Debian testing) the component is incorrectly marked as "generic component" instead of "addon" due to its metadata (although I would argue that Discover should hide generic components as well, as those may be things like shared libraries and developer tools as well).<br />
Changing it to addon is reflected in the metadata. but to be a valid addon component, it needs to <tt style="background: #ebebeb; font-size: 13px;"><extends/></tt> something.</p></div>
</blockquote>
<p>I'm afraid I didn't understand much of that, but here's the command output you requested:</p>
<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">appstreamcli dump org.kde.haenau
<?xml version="1.0" encoding="utf-8"?>
<component>
<id>org.kde.haenau</id>
<name>Haenau</name>
<summary>Haenau wallpaper</summary>
<project_license>LGPL-2.1</project_license>
<developer_name>Marco Martin <mart@kde.org></developer_name>
<pkgname>plasma5-addons</pkgname>
<url type="homepage">http://kde.org/</url>
<url type="donation">https://www.kde.org/donate.php?app=org.kde.haenau</url>
<icon type="stock">preferences-desktop-wallpaper</icon>
</component></pre></div>
<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">appstreamcli get org.kde.haenau --details
Identifier: org.kde.haenau [generic]
Name: Haenau
Summary: Haenau wallpaper
Package: plasma5-addons
Homepage: http://kde.org/
Icon: preferences-desktop-wallpaper
Developer: Marco Martin <mart@kde.org>
License: LGPL-2.1</pre></div>
<p>Was this patch correct? Is there something we should change here?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R114 Plasma Addons</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D25100">https://phabricator.kde.org/D25100</a></div></div><br /><div><strong>To: </strong>ngraham, apol, mak, Plasma<br /><strong>Cc: </strong>davidre, davidedmundson, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>