Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Interpret all M3U paths as relative and absolute #673

Open
4 tasks done
JHXmas opened this issue Jan 14, 2024 · 3 comments
Open
4 tasks done

Interpret all M3U paths as relative and absolute #673

JHXmas opened this issue Jan 14, 2024 · 3 comments
Assignees
Labels
bug Something isn't working music Related to music loading non-criticial Will fix later

Comments

@JHXmas
Copy link

JHXmas commented Jan 14, 2024

Describe the Bug/Crash

  1. Press plus to make new playlist
  2. Select imported playlist
  3. Choose m3u (uploaded below)
  4. Met with error "Unable to import a playlist from this file."

Screenshot

THPS3.zip

Describe the intended behavior

Should import as a new playlist, can confirm these files worked in Poweramp at least.

What android version do you use?

Android 13

What device model do you use?

Asus Zenfone 8

Bug report

bugreport-WW_I006D-TKQ1.220807.001-2024-01-14-04-59-09.zip

Duplicates

@JHXmas JHXmas added the bug Something isn't working label Jan 14, 2024
@OxygenCobalt
Copy link
Owner

The files seemingly all consist of absolute paths, like:

/The Nextmen/Amongst the Madness/1-01 Amongst the Madness.m4a

Do you have folders at the root of your internal storage that are like this? If not, then your M3U file is improperly indicating absolute paths as relative paths.

@JHXmas

@JHXmas
Copy link
Author

JHXmas commented Jan 15, 2024

Nah, they're all sitting in their own folder. Definitely a me problem then, thanks for looking.

@OxygenCobalt
Copy link
Owner

At the same time though, given that Poweramp seems to be really leniant with this, I'm going to assume I need to be more tolerant of this error. I'll make it so I'm searching by both the absolute and relative path when working with playlists, regardless of the true path composition.

@OxygenCobalt OxygenCobalt changed the title Unable to import a playlist from file Try relative and absolute paths no matter what when reading M3U files Jan 15, 2024
@OxygenCobalt OxygenCobalt changed the title Try relative and absolute paths no matter what when reading M3U files Try interpreting paths as relative and absolute when reading M3U files Jan 15, 2024
@OxygenCobalt OxygenCobalt added the music Related to music loading label Jan 15, 2024
@OxygenCobalt OxygenCobalt changed the title Try interpreting paths as relative and absolute when reading M3U files Interpret all M3U paths as relative and absolute Jan 25, 2024
@OxygenCobalt OxygenCobalt added the non-criticial Will fix later label Mar 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working music Related to music loading non-criticial Will fix later
Projects
Status: v4.0.0
Development

No branches or pull requests

2 participants