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

Subfolder support for file exporter #494

Closed
0x1f415 opened this issue Feb 12, 2024 · 3 comments
Closed

Subfolder support for file exporter #494

0x1f415 opened this issue Feb 12, 2024 · 3 comments
Labels
enhancement New feature or request

Comments

@0x1f415
Copy link

0x1f415 commented Feb 12, 2024

Is your feature request related to a problem? Please describe.
since I abandoned my old ticket #444 and decided to use the file exporter as an alternative, I've noticed that the file exporter strips slashes from the title template meaning the external folder needs to be a flat folder structure.

my default title template is:

{internalName}/{year}-{month}/[{date}] {internalName} - {title}

to attempt to create a show-like structure to be hosted by Jellyfin.

you can see slashes in the export menu that seem to imply that this would work:

image

but the exported file ends up in the root directory, taking on the entire title stripped of slashes:

image

also in case it wasn't clear I intend to use the auto exporter normally rather than manually exporting each file, but am using the manual export for this example - assuming they follow the same code path

possibly related: #472

Describe the solution you'd like
file exporter allows the user to use slashes to define a folder hierarchy of exported files.

Describe alternatives you've considered
this is the alternative to pointing jellyfin directly at the vods directory. that approach allowed jellyfin to read series/season/episode info from the folder hierarchy but stopped working eventually as the server would crash from attempting to read over 100 vods from its storage directory, as well as the UI becoming cluttered.

Additional context
S1.7.2.1/C2.4.2, running in docker on unraid

@0x1f415 0x1f415 added the enhancement New feature or request label Feb 12, 2024
@MrBrax
Copy link
Owner

MrBrax commented Apr 15, 2024

wonder if it's sane at all to just remove the sanitization checks, it's for security but on the other hand i'm not sure what it would even prevent

MrBrax added a commit that referenced this issue Apr 16, 2024
@MrBrax
Copy link
Owner

MrBrax commented Apr 16, 2024

the default directory config option is now a template field and should be parsed in the suitable exporters

@MrBrax MrBrax closed this as completed in 638e363 Jun 8, 2024
@0x1f415
Copy link
Author

0x1f415 commented Jun 19, 2024

sorry I just got around to testing this but the new problem is that the export errors saying the path does not exist, for my use case it needs to create the target folders as it exports and can't expect them to already exist.

I mean maybe I could create one flat folder for every streamer like #472 but I'd like to avoid doing that, month-based "season" folders keeps things better organized and I don't want to have to manually create a new folder every time I add a new streamer.

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants