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

Support M3U paths starting with /storage/ #693

Open
4 tasks done
paradoxical95 opened this issue Jan 24, 2024 · 15 comments
Open
4 tasks done

Support M3U paths starting with /storage/ #693

paradoxical95 opened this issue Jan 24, 2024 · 15 comments
Assignees
Labels
bug Something isn't working critical Need to get this fixed right now music Related to music loading

Comments

@paradoxical95
Copy link

Describe the Bug/Crash

I have over a 1000 songs and currently use BlackPlayer. Exported all my playlists from it. Tried importing those and it says "Unable to import playlist"
Is there some backwards compatibility that you need to add or is BlackPlayer messing around in the m3u file? I'd love to daily drive Auxio but I cannot without my playlists.

Describe the intended behavior

Should simply import playlist from the exported M3U file.

What android version do you use?

Android 11

What device model do you use?

RealMe Narzo 50A (Android 11)

Bug report

Duplicates

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

Can I have a sample playlist file? @paradoxical95

@paradoxical95
Copy link
Author

Yeah sure. Here you go.
FewPlaylists.zip

@OxygenCobalt
Copy link
Owner

OxygenCobalt commented Jan 25, 2024

Okay, so Black Player just outputs the absolute android path (with /storage/emulated/0) with their M3U files. Auxio expects absolute paths that are at least relative to their volume, since it's somewhat difficult to work with volume info and it's not cross-compatible with PCs. Evidently I'll need to handle and support this.

Try seeing if Black Player supports exporting with relative M3U paths. Auxio should support those fine @paradoxical95.

@paradoxical95
Copy link
Author

paradoxical95 commented Jan 25, 2024

I checked and currently BlackPlayer doesn't support relative paths.
Can you tell me what I must edit manually? If I can edit those files so that I can make them usable in Auxio? And will you be pushing this in next update?
2Playlists.zip

Edit - I tried an app called "Music Speed Changer" which, for a change, found and exported my playlists in .m3u8 instead of .m3u but that also failed to load from Auxio. Attaching the Zip file above. Has one m3u and one m3u8

@OxygenCobalt
Copy link
Owner

That's weird. Relative paths are usually supported by most players.

You would need to edit the files and remove all instances of /storage/B8E5-FB87 or /storage/emulated/0 in the paths. Then Auxio should be able to understand it as an absolute path.

Currently, I cannot work on this fix since a sponsor has prioritized another feature. I will get to it as soon I am done with that.

m3u and m3u8 will not change anything. Those file extensions just signal what the character encoding is.

@OxygenCobalt OxygenCobalt changed the title Cannot import .m3u Playlist file Auxio does not support M3U paths starting with /storage/ Jan 25, 2024
@OxygenCobalt OxygenCobalt changed the title Auxio does not support M3U paths starting with /storage/ Support M3U paths starting with /storage/ Jan 25, 2024
@OxygenCobalt OxygenCobalt added the music Related to music loading label Jan 25, 2024
@paradoxical95
Copy link
Author

Okay. I see. No worries mate, take your time. The app is superb otherwise.
By removing the instances of storage, you mean this right? I should edit all my playlist to look more like this right?
(starting with /MusicMine/... as that's my Music Folder in my SD Card)
image

@OxygenCobalt
Copy link
Owner

Yeah, that works.

@paradoxical95
Copy link
Author

Okay so I tried that and it didn't work. Attaching my fixed files. I removed the starting portion as I mentioned before and tried re-importing but it failed again. IDK how to fix this now.
Playlists_Fixed.zip

@OxygenCobalt
Copy link
Owner

OxygenCobalt commented Jan 25, 2024

Are the playlists in the same folder as your sd card @paradoxical95? The absolute paths are still treated as relative to volume in Auxio.

@paradoxical95
Copy link
Author

Yep, that helped. I made a folder and moved it to the root of my SD card instead of Phone storage and it worked. ! Thanks

Just a suggestion @OxygenCobalt , after importing it asks me to Name the playlist, but instead I suggest it should name whatever the name of the .m3u file was and let the user rename it if need be? Is that possible?

@OxygenCobalt
Copy link
Owner

Just a suggestion @OxygenCobalt , after importing it asks me to Name the playlist, but instead I suggest it should name whatever the name of the .m3u file was and let the user rename it if need be? Is that possible?

Eh, I don't like the idea of using the M3U filename, it just looks strange.

@chocmake
Copy link

chocmake commented Jan 29, 2024

Eh, I don't like the idea of using the M3U filename, it just looks strange.

fwiw when I save M3U/M3U8 playlists I name the file like I would the playlist in-player (whether desktop or mobile), which means it's the desired title as-is (eg: Video Game OST Selections.m3u8). Perhaps you've encountered some kind of auto-generated playlist filenames leading to a negative association though and prefer normalizing their appearance by a prompt.

@paradoxical95
Copy link
Author

Eh, I don't like the idea of using the M3U filename, it just looks strange.

fwiw when I save M3U/M3U8 playlists I name the file like I would the playlist in-player (whether desktop or mobile), which means it's the desired title as-is (eg: Video Game OST Selections.m3u8). Perhaps you've encountered some kind of auto-generated playlist filenames leading to a negative association though and prefer normalizing their appearance by a prompt.

IKR? I do that too but then it became redundant to me. Like my playlist is named "EDMs". Why are you asking me to name it when it's literally named EDMs 🤣

@OxygenCobalt
Copy link
Owner

Okay. While I could autofill with the file name in the playlist name dialog, the issue is that I don't want someone to have to search for a playlist (possibly within a big list) just to rename it. There's a lot of friction in that process compared to always asking to rename the playlist first.

@KraXen72
Copy link

KraXen72 commented Feb 7, 2024

You could have the playlist filename as a placeholder text, and if the name field is submitted empty, it uses the placeholder, otherwise, it uses the value

@OxygenCobalt OxygenCobalt added the critical Need to get this fixed right now 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 critical Need to get this fixed right now music Related to music loading
Projects
Status: v4.0.0
Development

No branches or pull requests

4 participants