Git Product home page Git Product logo

Comments (11)

dajotim937 avatar dajotim937 commented on July 22, 2024 1

I could try to try a different node package, since I have tried with different applications and some of them partially open the file.

From what I saw, Honeyview and 7zip both can see 28 normal images+1 corrupted.
I tested with cbz I sent to you and tried recreate situation from #194 (comment) and so far there is no app freeze and\or error messages. For situation from comment there is usually long\infinite loading, sometimes it shows that it can't unzip file.

from opencomic.

ollm avatar ollm commented on July 22, 2024

Can you send me the CBZ file that is causing the bug? [email protected]

from opencomic.

dajotim937 avatar dajotim937 commented on July 22, 2024

Done.

from opencomic.

ollm avatar ollm commented on July 22, 2024

I have fixed the freezing problem when opening corrupt cbz files 05eefb3, but I have not been able to partially read the contents of the file that you have sent me, at least with unzipper, I could try to try a different node package, since I have tried with different applications and some of them partially open the file.

Build with fix: https://mega.nz/file/zXglnC7R#YnHNrUcUOLOAbTSUBPHwo9GwRjt-vB88cL3yJ6GsXiY

If you can check if the following error has also been fixed #194 (comment)

from opencomic.

ollm avatar ollm commented on July 22, 2024

Thanks for mentioning that it works in 7zip.

I have changed the default extraction for zip files to 7z d07feac and now all 28 images are extracted +1 corrupted, it also slightly increases performance in extracion zip files in general.

Build: https://mega.nz/file/rSQjxDZC#KhQjUtQHk6KpZc7xvndzShe9uCsSDx4cEsj218HFYlU

from opencomic.

ollm avatar ollm commented on July 22, 2024

Something I forgot to mention is that you may have to delete the cache, since OpenComic will use the previously generated cache when reading the file.

from opencomic.

dajotim937 avatar dajotim937 commented on July 22, 2024

I usually just create new folder with testN, where N = 1,2,3.... It do the trick with cache.
With cbz file, yeah it shows 28+1 images. And there is error notification about wrong CRC, however I think it should shows every time when corrupted cbz is opened, not only first time.

from opencomic.

dajotim937 avatar dajotim937 commented on July 22, 2024

While there is notification when you open file, there is also notification every time when you open folder with cbz files, when one cbz is corrupted. The issue is that the message is Error when unzipping: unknown error. when you didn't open corrupted file yet and Error when unzipping: CRC failed : **.jpg when you opened file once(which means there is cache of file). And when you open folder with cbz files you can't understand which file is corrupted, but notification is showed just when you opened folder, not file.

2024-03-29_19-57-14.mp4

from opencomic.

ollm avatar ollm commented on July 22, 2024

I've improved a little how this is handled, note that once a compressed file has been extracted, the next times it will be accessed from the cache/tmp, currently I save the last error message in the cache to continue showing it, but it does not always work as expected.

Extract corrupt file show first error message: b95de2c
Separate compressed error messages of different files 3d77bb3

Build: https://mega.nz/file/nbQUlDIJ#iVhuBj0ES_H4VUCR0Kl36l980bmSBkYmKKmNpt0anpA

from opencomic.

dajotim937 avatar dajotim937 commented on July 22, 2024

but it does not always work as expected.

Current situation for me: error is not shown anymore if I open folder. However, I have one corrupted cbz, but error is shown for every cbz in folder (UPD: first time it has been shown presumably for every cbz, after clearing cache and tmp it's shown for some unrelated uncorrupted cbz, but isn't for others), and for corrupted cbz (which contains corrupted image) error is shown 2 times.

from opencomic.

ollm avatar ollm commented on July 22, 2024

I'm going to reopen this, since I've only tested it with the corrupt file and not alongside other files.

from opencomic.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.