-
-
Notifications
You must be signed in to change notification settings - Fork 699
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
ExtractArchive: problems deleting multipart archives #1603
Comments
Also deep Extraction doesnt work. |
can someone please fix this! |
i also have this problem..... |
No time to fix it now, sorry. |
The files are not going to be deleted and it also trys to extract the last part with sevenzip after it´s already extracted. So everything what is downloaded gets destroyed after extraction!!!
|
Can somebody have a look on it? |
Sorry referencing was not intented. |
@wastez How would you exactly supposed to handle multipart archives? |
@GammaC0de |
I also just downloaded from one Hoster. (ShareOnline) |
@GammaC0de Its not a problem that files would come from different hosts. All files, for example part1 to part4, get downloaded from the same host. Problem (for me) occurs after the successful extraction with unrar. Only one part (which one is "random", probably depending on the link order in the package) is deleted. In the past all parts (in this case 1 to 4) got deleted. Additionally 7zip kicks in and tries to extract the not deleted parts of the multipart archive, which obviously fails, because one part is missing. |
@wastez are you using |
@GammaC0de I would say so. The extracted files were fine, but due to the incomplete deletion the package fails. If all files are deleted, the package should finish without errors. But I am not 100% sure about this last part. Maybe there are additional bugs. |
No it Extracts when the Package finished download. 7zip should not extract after unrar because if somebody dont delete the files it extract it two times. |
pyload/module/plugins/internal/UnRar.py Line 180 in 950250a
I think "is" has to be "==" /edit: was changed in commit 94d017c, stating Hotfixes(2) |
@TodsDeath i think you are right, please change this and post the results |
seems to work, no more errors, made pr #1689 |
👍 |
so can everyone confirm we can close this issue? |
@GammaC0de |
^^that would be great. |
which issue# are you referring to? |
this one. |
This was an encrypted multipart archive with 4 parts (*.part1.rar, ..., *.part4.rar). As you can see, the package finishes and ExtractArchive gets started. After guessing the right password and successful extraction, only part1 gets deleted (lines 79/80).
7zip is then trying to extract the remaining parts, which understandably fails.
The text was updated successfully, but these errors were encountered: