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

sorttokens does not work #1613

Closed
2 tasks done
AnselmD opened this issue Jul 23, 2023 · 2 comments · Fixed by #1617
Closed
2 tasks done

sorttokens does not work #1613

AnselmD opened this issue Jul 23, 2023 · 2 comments · Fixed by #1617
Labels
bug component: settings GUI settings, advancedsettings.xml
Milestone

Comments

@AnselmD
Copy link

AnselmD commented Jul 23, 2023

Describe the bug
Ignore articles when sorting ("The") is checked
Having those movies - only examples for demonstration:
abc
def
ghi
The abc
The def
The ghi

In MediaElch it is showed like this:
abc
def
ghi
The abc
The def
The ghi

I would expect the following output:
abc
The abc
def
The def
ghi
The ghi

or
abc
abc, The
def
def, The
ghi
ghi, The

To Reproduce
Steps to reproduce the behavior:
see Describe the bug

Expected behavior
see Describe the bug

Screenshots

MediaElch Version:

  • 2.10.2 stable

Operating System:

  • Windows

Additional context
Should it also influence the rename behavior?

@bugwelle
Copy link
Collaborator

bugwelle commented Jul 23, 2023

Thanks for reporting. I was able to reproduce this with the latest advacedsettings.xml example. It appears that I had a Copy&Paste error...

I assume you use an advancedsettings.xml file, do you?

Workaround:
From your advancedsettings.xml, remove the <exclude> tag and its contents

It will be fixed in the next MediaElch version.

Regards,
Andre

@AnselmD
Copy link
Author

AnselmD commented Jul 24, 2023

Workaround:
From your advancedsettings.xml, remove the <exclude> tag and its contents

Thank you, this works

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug component: settings GUI settings, advancedsettings.xml
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants