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.

Sign up for GitHub

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

Jump to bottom

fix(archive): unable to preview #7843

Merged
merged 7 commits into from
Jan 27, 2025
Merged

fix(archive): unable to preview #7843

merged 7 commits into from
Jan 27, 2025
+287 −73

Conversation

j2rong4cn
Copy link
Contributor

@j2rong4cn j2rong4cn commented Jan 19, 2025 •
edited
Loading

@KirCute 我只能到这了，接力
最新成果：
{F7BF0F02-BD4E-49BB-90F3-CCA751461837}

@j2rong4cn
Copy link
Contributor Author

j2rong4cn commented Jan 19, 2025 •
edited
Loading

测试下载zip压缩包里的文件。配合 下面PR的功能 可以提速！！！

下载客户端的下载速度是解压速度，流量监控显示的才是真实下载速度

{428F1C44-F293-4B5E-9207-30A1657F61F5}

{2FDB6785-04A9-49C6-A5C5-A665A65B37B4}

@j2rong4cn
Copy link
Contributor Author

@KirCute 现在zip支持返回tree了，但没排序，好像只适合前端排序，毕竟不是每个文件夹都会被浏览，你可以在前端写个排序吗？

@KirCute
Copy link
Contributor

KirCute commented Jan 20, 2025 •
edited
Loading

@KirCute 现在zip支持返回tree了，但没排序，好像只适合前端排序，毕竟不是每个文件夹都会被浏览，你可以在前端写个排序吗？

前端已经实现排序了，不过默认状态下应该是不排序的，点列表的表头就能排序

@j2rong4cn
Copy link
Contributor Author

j2rong4cn commented Jan 20, 2025 •
edited
Loading

@KirCute 现在zip支持返回tree了，但没排序，好像只适合前端排序，毕竟不是每个文件夹都会被浏览，你可以在前端写个排序吗？

前端已经实现排序了，不过默认状态下应该是不排序的，点列表的表头就能排序

请求/api/fs/archive/meta的返回中有tree，没有请求/api/fs/archive/list，就会这样，少了置顶文件夹
image

@KirCute
Copy link
Contributor

KirCute commented Jan 20, 2025

@KirCute 现在zip支持返回tree了，但没排序，好像只适合前端排序，毕竟不是每个文件夹都会被浏览，你可以在前端写个排序吗？

前端已经实现排序了，不过默认状态下应该是不排序的，点列表的表头就能排序

请求/api/fs/archive/meta的返回中有tree，没有请求/api/fs/archive/list，就会这样，少了置顶文件夹 image

调用/list的结果看起来像是排过序是因为archive.go#L134处做了op.List会做的一些排序操作，在op.GetArchiveMeta里对每级目录也做类似的操作就能实现排序和提取文件夹

@j2rong4cn
Copy link
Contributor Author

在op.GetArchiveMeta里对每级目录也做类似的操作就能实现排序和提取文件夹

得考虑压缩包的文件数量。如果/api/fs/archive/meta返回中有tree 就在进入文件夹的时候排序，，这样比较合适

@j2rong4cn
Copy link
Contributor Author

在op.GetArchiveMeta里对每级目录也做类似的操作就能实现排序和提取文件夹

这么多文件夹和文件，排序耗时不得巨久
{FD2C2367-10AB-4160-B5D9-45C3C593334E}

@KirCute
Copy link
Contributor

KirCute commented Jan 20, 2025

Front-end part AlistGo/alist-web#236

@KirCute
Copy link
Contributor

KirCute commented Jan 20, 2025

记得合并j2rong4cn#2

@j2rong4cn
Copy link
Contributor Author

记得合并j2rong4cn#2

ok，辛苦了

@j2rong4cn
Copy link
Contributor Author

Front-end part AlistGo/alist-web#236

刷新按钮是不是忘记了？

@KirCute
Copy link
Contributor

KirCute commented Jan 20, 2025

刷新按钮是不是忘记了？

写后端的时候模仿前几个接口加了刷新，写到前端的时候发现这样一个刷新按钮不好加，要不算了，目录里的内容可能会变，有刷新的需求，压缩文件里的内容一般变不了，感觉刷新按钮的需求不大

@j2rong4cn
Copy link
Contributor Author

刷新按钮是不是忘记了？

写后端的时候模仿前几个接口加了刷新，写到前端的时候发现这样一个刷新按钮不好加，要不算了，目录里的内容可能会变，有刷新的需求，压缩文件里的内容一般变不了，感觉刷新按钮的需求不大

我是看到后端刷新选项的代码，而前端没有就以为忘了呢，😂

@j2rong4cn
Copy link
Contributor Author

image

在这个浏览压缩包界面 浏览器点刷新之后右键菜单会有bug，会固定到左下角，得返回到文件夹重新打开才正常

@KirCute
Copy link
Contributor

KirCute commented Jan 20, 2025

在这个浏览压缩包界面 浏览器点刷新之后右键菜单会有bug，会固定到左下角，得返回到文件夹重新打开才正常

图片有问题看不到

@j2rong4cn
Copy link
Contributor Author

在这个浏览压缩包界面 浏览器点刷新之后右键菜单会有bug，会固定到左下角，得返回到文件夹重新打开才正常

图片有问题看不到

因为那图是之前的，我拿来举例。
image

@KirCute
Copy link
Contributor

KirCute commented Jan 20, 2025

因为那图是之前的，我拿来举例。

行，有空看看

@j2rong4cn j2rong4cn changed the title fix(archive): unrecognition zip fix(archive): unrecognition archive Jan 22, 2025
@j2rong4cn
Copy link
Contributor Author

@KirCute 由云盘并发限制引起的问题基本解决了，预览压缩包结构基本没问题，但解压7z，rar时不是很理想，有时CPU占用异常的高，得等mholt/archives包 更新优化

@j2rong4cn j2rong4cn changed the title fix(archive): unrecognition archive fix(archive): unable to preview Jan 22, 2025
@KirCute
Copy link
Contributor

KirCute commented Jan 22, 2025

@KirCute 由云盘并发限制引起的问题基本解决了，预览压缩包结构基本没问题，但解压7z，rar时不是很理想，有时CPU占用异常的高，得等mholt/archives包 更新优化

可以，强的，辛苦

@KirCute
Copy link
Contributor

KirCute commented Jan 22, 2025

image

在这个浏览压缩包界面 浏览器点刷新之后右键菜单会有bug，会固定到左下角，得返回到文件夹重新打开才正常

已修

@xhofe xhofe merged commit 5c5d837 into AlistGo:main Jan 27, 2025
6 checks passed
@j2rong4cn j2rong4cn deleted the fix branch January 28, 2025 04:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xhofe xhofe xhofe approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@j2rong4cn @KirCute @xhofe