Bug 465629

Summary: Skip empty and comment lines when loading m3u8 playlist
Product: [Applications] Elisa Reporter: Toby <tianer2820>
Component: generalAssignee: Matthieu Gallien <matthieu_gallien>
Status: RESOLVED FIXED    
Severity: wishlist CC: jackhill3103, nate
Priority: NOR    
Version First Reported In: 22.12.2   
Target Milestone: ---   
Platform: Fedora RPMs   
OS: Linux   
Latest Commit: https://invent.kde.org/multimedia/elisa/-/commit/0a4901a87c7161a0c4ba342d1a71212325f2e13c Version Fixed/Implemented In: 23.04
Sentry Crash Report:

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.