8000 feat(archive): support deprioritize previewing by KirCute · Pull Request #7984 · AlistGo/alist · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

feat(archive): support deprioritize previewing #7984

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

Merged
merged 1 commit into from
Feb 16, 2025

Conversation

KirCute
Copy link
Contributor
@KirCute KirCute commented Feb 12, 2025

支持打开压缩文件时优先展示下载而不是预览,在设置->预览中设置。

Front-end part: AlistGo/alist-web#252

@Clouddark75
Copy link

Is there a chance you can implement that when selecting multiple compressed files, they can be extracted instead of doing so one by one?

@KirCute
Copy link
Contributor Author
KirCute commented Feb 13, 2025

Is there a chance you can implement that when selecting multiple compressed files, they can be extracted instead of doing so one by one?

During development, I initially intended to allow such an operation, but I found it difficult to handle situations where some compressed files have passwords while others do not, or where each compressed file has a different password. So finally, I gave up.

@Clouddark75
Copy link

Is there a chance you can implement that when selecting multiple compressed files, they can be extracted instead of doing so one by one?

During development, I initially intended to allow such an operation, but I found it difficult to handle situations where some compressed files have passwords while others do not, or where each compressed file has a different password. So finally, I gave up.

I believe having support for multiple passwords is a bit too much (even the majority of compressed files managers have a hard time with it); you will need to implement something like a database or handle them by separating the passwords with a new line (space) each, like how offline downloads handle multiple links or magnets.

My use of Alist is mostly for backups and simply supporting multiple files with no password, or having all of them with the same password will make things easier for handling them (because I believe nobody uploads their backups with no security to someone else's cloud storage).
I'm already grateful that you added the uncompressed functionality. So if you can implement it like that, it will be awesome.

And sorry for the late reply, for some reason github.com reply are directly send to spam section for some reason...
Cheers!

@xhofe xhofe merged commit 30d8c20 into AlistGo:main Feb 16, 2025
12 checks passed
@KirCute KirCute deleted the feat/archive-preview branch February 16, 2025 06:59
@KirCute
Copy link
Contributor Author
KirCute commented Feb 16, 2025

I believe having support for multiple passwords is a bit too much (even the majority of compressed files managers have a hard time with it); you will need to implement something like a database or handle them by separating the passwords with a new line (space) each, like how offline downloads handle multiple links or magnets.

My use of Alist is mostly for backups and simply supporting multiple files with no password, or having all of them with the same password will make things easier for handling them (because I believe nobody uploads their backups with no security to someone else's cloud storage). I'm already grateful that you added the uncompressed functionality. So if you can implement it like that, it will be awesome.

And sorry for the late reply, for some reason github.com reply are directly send to spam section for some reason... Cheers!

I will try my best, but I have been quite busy recently because these days mark the end of the Chinese New Year holiday. I may try to implement it afterward.

long2005a1 added a commit to long2005a1/Long-Cloud that referenced this pull request Feb 17, 2025
@Clouddark75
Copy link

I believe having support for multiple passwords is a bit too much (even the majority of compressed files managers have a hard time with it); you will need to implement something like a database or handle them by separating the passwords with a new line (space) each, like how offline downloads handle multiple links or magnets.
My use of Alist is mostly for backups and simply supporting multiple files with no password, or having all of them with the same password will make things easier for handling them (because I believe nobody uploads their backups with no security to someone else's cloud storage). I'm already grateful that you added the uncompressed functionality. So if you can implement it like that, it will be awesome.
And sorry for the late reply, for some reason github.com reply are directly send to spam section for some reason... Cheers!

I will try my best, but I have been quite busy recently because these days mark the end of the Chinese New Year holiday. I may try to implement it afterward.

Thanks for thinking about it and giving it a chance!

What do you think about this one?
https://github.com/bodgit/sevenzip
It supports Lzma, multiple part files and more type of compression.

@KirCute
Copy link
Contributor Author
KirCute commented Feb 18, 2025

Thanks for thinking about it and giving it a chance!

What do you think about this one? https://github.com/bodgit/sevenzip It supports Lzma, multiple part files and more type of compression.

Thank you for the recommendation. I will look into it later. However, the lack of support for split compressed files is largely not because existing libraries are incapable of handling them, but rather because AList requires some architectural changes to simultaneously select multiple files and pass them to the libraries. And this might involve a certain amount of work.

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
0