Bug 465629 - Skip empty and comment lines when loading m3u8 playlist
Summary: Skip empty and comment lines when loading m3u8 playlist
Status: RESOLVED FIXED
Alias: None
Product: Elisa
Classification: Applications
Component: general (show other bugs)
Version: 22.12.2
Platform: Fedora RPMs Linux
: NOR wishlist
Target Milestone: ---
Assignee: Matthieu Gallien
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-02-12 17:58 UTC by Toby
Modified: 2024-04-17 10:57 UTC (History)
2 users (show)

See Also:
Latest Commit: https://invent.kde.org/multimedia/elisa/-/commit/0a4901a87c7161a0c4ba342d1a71212325f2e13c
Version Fixed In: 23.04
Sentry Crash Report:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Toby 2023-02-12 17:58:44 UTC 
SUMMARY
Does not skip empty and comment lines in the m3u8 file when loading a playlist


STEPS TO REPRODUCE
1. Create a playlist in Elisa, save it to an m3u8 file
2. Open the file in a text editor, add some empty lines and comment lines (lines start with #), save
3. Load the file back, now many invalid entries appear in the playlist entry

OBSERVED RESULT
Empty lines and comment lines becomes invalid entries.

EXPECTED RESULT
They should be ignored

SOFTWARE/OS VERSIONS
Operating System: Fedora Linux 37
KDE Plasma Version: 5.26.5
KDE Frameworks Version: 5.102.0
Qt Version: 5.15.8
Kernel Version: 6.1.10-200.fc37.x86_64 (64-bit)
Graphics Platform: X11
Comment 1 Jack Hill 2024-04-16 11:39:05 UTC 
Please can you provide an example file?
Comment 2 Toby 2024-04-16 22:20:04 UTC 
(In reply to Jack Hill from comment #1)
> Please can you provide an example file?

Just tested again and seems like it has been fixed in the latest release.
Comment 3 Jack Hill 2024-04-17 10:57:45 UTC 
Ah, yeah. It was implemented here https://invent.kde.org/multimedia/elisa/-/commit/0a4901a87c7161a0c4ba342d1a71212325f2e13c.