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

File name template $num adds [ ] in the number of any playlists #377

Closed
4 of 5 tasks
OMSmolina opened this issue Sep 16, 2023 · 5 comments · Fixed by #402
Closed
4 of 5 tasks

File name template $num adds [ ] in the number of any playlists #377

OMSmolina opened this issue Sep 16, 2023 · 5 comments · Fixed by #402
Labels
bug Something isn't working

Comments

@OMSmolina
Copy link
Contributor

Version

1.10.3

Platform

Windows 10

Steps to reproduce

With any youtube playlist $num adds [ ]

For example: https://www.youtube.com/playlist?list=PL3eBqMwkHPomNoHnm9cjf5CUrvYAapB3m

Application Settings:

File name template: $num - $title

or

File name template: S01E$num - $title

Details

Expected:

  • 01 - Title Name of the video
    or
  • S01E01 - Title Name of the video

Actual:

  • [1] - Title Name of the video
    or
  • S01E[01] - Title Name of the video

Checklist

  • I have looked through existing issues to make sure that this bug has not been reported before
  • I have provided a descriptive title for this issue
  • I have made sure that that this bug is reproducible on the latest version of the application
  • I have provided all the information needed to reproduce this bug as efficiently as possible
  • I have sponsored this project
@OMSmolina OMSmolina added the bug Something isn't working label Sep 16, 2023
@Tyrrrz
Copy link
Owner

Tyrrrz commented Sep 16, 2023

This is not really a bug as it's just how $num has historically come to be. The file name templates are pretty inflexible, overall.

@OMSmolina
Copy link
Contributor Author

Oh, I didn't know, I just found this program and when I saw that the parameter was there, it said $num for the number, I expected a number not [00]. I will have to do a fork or look for another program since Jellyfin or Plex do not detect the number with [ ] sorry for the inconvenience, thank you.

@Tyrrrz
Copy link
Owner

Tyrrrz commented Sep 16, 2023

Yeah, you're right. I don't remember exactly why square brackets are there, but I remember it was intended. It's not the best design, admittedly.

I think you should be able to strip the brackets in all files using a simple shell script or using some batch rename tool.

@OMSmolina
Copy link
Contributor Author

The issue was that I didn't want to use other tools. After looking at several tools, in the end I decided to compile a version without the square brackets since with these it was no longer a number, it took 2 seconds and it works perfectly with all the combinations.

Whether you decide to change it or not, you can close the issue, thank you.

@Tyrrrz Tyrrrz closed this as completed in f09d743 Nov 7, 2023
@Tyrrrz
Copy link
Owner

Tyrrrz commented Nov 7, 2023

I added an undocumented $numc token that doesn't include []

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants