https://bugs.kde.org/show_bug.cgi?id=523849

            Bug ID: 523849
           Summary: Regression in ServiceRunner wrongly deduplicating
                    distinct desktop entries
    Classification: Plasma
           Product: krunner
      Version First 6.7.3
       Reported In:
          Platform: Manjaro
                OS: Linux
            Status: REPORTED
          Severity: major
          Priority: NOR
         Component: general
          Assignee: [email protected]
          Reporter: [email protected]
                CC: [email protected], [email protected]
  Target Milestone: ---

DESCRIPTION

If you create custom entries in the Application launcher using the KDE Menu
Editor that have the same Exec but different Path, only one will appear when
typing, even though they should be distinct.
This makes it fully unusable for some valid use cases.

STEPS TO REPRODUCE
1. in KDE Menu Editor, create two entries with the same Exec (command line),
but give them different Path (working directory).
2. In the Application Launcher, type keywords for one, then the other.

OBSERVED RESULT

Both should appear when typing their respective name.

EXPECTED RESULT

One of the two will never appear, even when fully typing the correct name.

SOFTWARE/OS VERSIONS
Operating System: Manjaro Linux
KDE Plasma Version: 6.7.3
KDE Frameworks Version: 6.28.0
Qt Version: 6.11.1

ADDITIONAL INFORMATION

I'm not familiar with KDE sources, but the problem seems to be that the dedup
is only made based on Exec (seen() and hasSeen() is servicerunner.cpp), even
though Path can be different. Here are some commits that seems to be related to
the problem:

The bug was likely introduced in this optimization commit from 2023 (which also
adds redundant 'inline' to already inline functions):
19b93f2c11c34ac179c641472d41821a34931677

Which seems to revert behavior from this 2017 commit:
7b1dc9a4bb039ff5ff2a5b71a2c6bc687ad1db72

I suggest to either revert the 2023 commit, or add a Path criterion for dedup.

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to