Local trailer from the "trailers" folder is not used.

[sl1288]

With an directory structure like this:
\Film (2000)\Film (2000).mkv
\Film (2000)\trailers\Film (2000).mkv
\Film (2000)\Extras\Honest Trailer.mkv

The File "\Film (2000)\Extras\Honest Trailer.mkv" is played as a local trailer.

With an directory structure like this:
\Film (2000)\Film (2000).mkv
\Film (2000)\trailers\Film (2000).mkv

It does play an youtube trailer.

This was working in an older version, see issue #1639

[stale]

This issue has gone 120 days without comment. To avoid abandoned issues, it will be closed in 21 days if there are no new comments.
If you're the original submitter of this issue, please comment confirming if this issue still affects you in the latest release or nightlies, or close the issue if it has been fixed. If you're another user also affected by this bug, please comment confirming so. Either action will remove the stale label.
This bot exists to prevent issues from becoming stale and forgotten. Jellyfin is always moving forward, and bugs are often fixed as side effects of other changes. We therefore ask that bug report authors remain vigilant about their issues to ensure they are closed if fixed, or re-confirmed - perhaps with fresh logs or reproduction examples - regularly. If you have any questions you can reach us on Matrix or Social Media.

[sl1288]

Tested version: a69397d

Trailer playback does work with this folder structure now:
\Film (2000)\Film (2000).mkv
\Film (2000)\trailers\Film (2000).mkv

But with this structure "\Film (2000)\Extras\Honest Trailer.mkv" is still played, instead of "\Film (2000)\trailers\Film (2000).mkv"
\Film (2000)\Film (2000).mkv
\Film (2000)\trailers\Film (2000).mkv
\Film (2000)\Extras\Honest Trailer.mkv

[stale]

This issue has gone 120 days without comment. To avoid abandoned issues, it will be closed in 21 days if there are no new comments.
If you're the original submitter of this issue, please comment confirming if this issue still affects you in the latest release or nightlies, or close the issue if it has been fixed. If you're another user also affected by this bug, please comment confirming so. Either action will remove the stale label.
This bot exists to prevent issues from becoming stale and forgotten. Jellyfin is always moving forward, and bugs are often fixed as side effects of other changes. We therefore ask that bug report authors remain vigilant about their issues to ensure they are closed if fixed, or re-confirmed - perhaps with fresh logs or reproduction examples - regularly. If you have any questions you can reach us on Matrix or Social Media.

[cvium]

Caused by this line

jellyfin/Emby.Server.Implementations/Library/LibraryManager.cs

Line 2779 in 8045a48

}).OrderBy(i => i.Path);

extras > trailers.

Whether the existence of a trailers folder means it should ignore extras is up for debate.

[taylor-schneider]

any fixes?

[crobibero]

#7028

Well, this PR fixed reading the folders, but did nothing about the ordering.

[hasezoey]

i dont know the current state of this issue, but i would also be in favor of having the folder trailers take priority over extras