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

Create a new release #587

Merged
merged 4 commits into from
Nov 16, 2024
Merged

Create a new release #587

merged 4 commits into from
Nov 16, 2024

Conversation

johman10
Copy link
Owner

This PR releases:

  1. Fixing the file selector
  2. Filtering torrents with non-latin characters
  3. Upgrade of dependencies

johman10 and others added 4 commits November 15, 2024 17:17
This PR aims to fix the issue described in
#575.

Sorting the file list before creating the folder structure could cause
the file indices to be incorrect, leading to problems such as not
downloading the correct file or folder.

I tested it with a torrent where I encountered this problem and it seems
to work properly with this patch.

---------

Co-authored-by: Johan van Eck <johveck@gmail.com>
for #129 

Some test cases:

```
> regex = /[ \[\]\.·_-]/g

> str1='[这是一部中文电影.Movie.With.Chinese.Title.2024.1080p.WEB-DL.HEVC.10bit.DTS5.1.2Audios-Release]'
'[这是一部中文电影.Movie.With.Chinese.Title.2024.1080p.WEB-DL.HEVC.10bit.DTS5.1.2Audios-Release]'
> str2='English.Movie.2024.1080p.WEB-DL.HEVC.10bit.DTS5.1.2Audios-Release]'
'English.Movie.2024.1080p.WEB-DL.HEVC.10bit.DTS5.1.2Audios-Release]'

> search1 = '中文'
'中文'
> search2 = 'English Movie'
'English Movie'

> str1.toLowerCase().replace(regex, '')
'这是一部中文电影moviewithchinesetitle20241080pwebdlhevc10bitdts512audiosrelease'
> str2.toLowerCase().replace(regex, '')
'englishmovie20241080pwebdlhevc10bitdts512audiosrelease'
> search1.toLowerCase().replace(regex, ' ')
'中文'
> search2.toLowerCase().replace(regex, ' ')
'english movie'
> search1.toLowerCase().replace(regex, '')
'中文'
> search2.toLowerCase().replace(regex, '')
'englishmovie'
> str1.toLowerCase().replace(regex, '').includes(search1.toLowerCase().replace(regex,''))
true
> str2.toLowerCase().replace(regex, '').includes(search2.toLowerCase().replace(regex,''))
true
```

Also tested locally in my transmission docker that points to the dev
version of UI and it's working.

Cannot figure out how to get CORS working to just use `npm start` :/

Co-authored-by: Ivan Xiao <me@ivanxiao.com>
@johman10 johman10 merged commit 16bb341 into master Nov 16, 2024
2 checks passed
@johman10 johman10 deleted the release branch November 16, 2024 12:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants