Bug 402640

Summary: Two search entries after install
Product: [Applications] Discover Reporter: Peter Eszlari <peter.eszlari>
Component: Flatpak BackendAssignee: Aleix Pol <aleixpol>
Status: RESOLVED FIXED    
Severity: normal CC: jgrulich, nate
Priority: NOR    
Version First Reported In: 5.14.4   
Target Milestone: ---   
Platform: Neon   
OS: Linux   
Latest Commit: https://commits.kde.org/discover/d41e1ce6f304e7125f88c8b4dfd1d2f80e18ed38 Version Fixed In:
Sentry Crash Report:

Description Peter Eszlari 2018-12-28 11:02:49 UTC 
1. search for "avidemux"
2. find one entry
3. install avidemux flatpak
4. search for "avidemux" again
5. find two entries
Comment 1 Nate Graham 2018-12-28 20:58:25 UTC 
Cannot reproduce with git master. Are the the entries identical? Or do they have different text for the "Source" line at the bottom of their app pages?
Comment 2 Peter Eszlari 2019-01-04 15:10:55 UTC 
> Cannot reproduce with git master.

I tested this on both, Neon User and Dev Edition.

> Are the the entries identical?

No. One entry only shows categories, version, size, source - nothing more, not even screenshots. The other entry is complete.

> Or do they have different text for the "Source" line at the bottom of their app pages?

The source is the same: flathub (Flatpak)
Comment 3 Nate Graham 2019-01-05 04:39:00 UTC 
Thanks Peter.

Aleix, do you know if is this a bug in Flathub or Appstream?
Comment 4 Aleix Pol 2019-01-07 16:41:45 UTC 
Do you still get it after closing Discover and opening it again?
Comment 5 Peter Eszlari 2019-01-07 18:53:39 UTC 
> Do you still get it after closing Discover and opening it again?

Yes.

Maybe it's a flatpak bug and the flatpak version (1.0.1) in Neon is too old ?
Comment 6 Aleix Pol 2019-01-08 00:20:00 UTC 
It's probably an issue in Discover then, will have to look into it.
Comment 7 Aleix Pol 2019-01-08 16:52:58 UTC 
Git commit d41e1ce6f304e7125f88c8b4dfd1d2f80e18ed38 by Aleix Pol.
Committed on 08/01/2019 at 16:50.
Pushed by apol into branch 'master'.

flatpak: Don't add .desktop to installed apps

We're already testing X and X.desktop, so it's more correct this way.

M  +1    -3    libdiscover/backends/FlatpakBackend/FlatpakBackend.cpp

https://commits.kde.org/discover/d41e1ce6f304e7125f88c8b4dfd1d2f80e18ed38