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

            Bug ID: 480247
           Summary: Missing emojis where name includes " E"
    Classification: Plasma
           Product: plasmashell
           Version: 5.27.10
          Platform: Fedora RPMs
                OS: Linux
            Status: REPORTED
          Severity: minor
          Priority: NOR
         Component: Emoji Selector
          Assignee: plasma-b...@kde.org
          Reporter: m...@treier.xyz
  Target Milestone: 1.0

Created attachment 165171
  --> https://bugs.kde.org/attachment.cgi?id=165171&action=edit
Screen recording example of missing emojis

SUMMARY
Any emoji whose name includes the string " E" (space followed by uppercase E)
is missing from the Emoji Selector. For example, all flag emojis whose names
start with an uppercase E are missing. There could be other examples that I've
missed.

STEPS TO REPRODUCE
1. Open Emoji Selector
2. Search for any emoji whose name includes " E" (e.g., "flag: European Union")

OBSERVED RESULT
The emoji cannot be found in the program.

EXPECTED RESULT
The emoji should show up in the program.

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: Fedora Linux 39
KDE Plasma Version: 5.27.10
KDE Frameworks Version: 5.113.0
Qt Version: 5.15.11

ADDITIONAL INFORMATION
The emoji data is generated with the script emojier/data/update-emoji-data.py
(see
https://invent.kde.org/plasma/plasma-desktop/-/blob/v5.27.10/emojier/data/update-emoji-data.py?ref_type=tags#L60).
 

On line 60, a part of the emoji metadata, which includes the name, is split
using the delimiter " E". This should split the string into two, but it fails
to consider that emojis can have " E" in the name, like with "flag: European
Union", in which case the code could and does split in the middle of the emojis
name. Line 61 is then evaluated as false, which causes the emoji to be ignored
entirely and, therefore, is missing in the output files.

Initial independent testing seems to confirm this to be the culprit. I modified
line 60 to only split on the first occurrence (`split(b" E") => split(b" E",
1)`), and replacing /usr/share/plasma/emoji/* with the newly generated files
got the missing emojis to show up.

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

Reply via email to