Replace shutil with patool

[olavstornes]

Replace shutil with patool

[alan-cugler]

I am adding the following two imports to main.py

import patoolib
from pathlib import Path

I am also adding patool to the requirments.txt file as a dependency.

[alan-cugler]

I am trying to make this improvement myself, but currently after successfully adding the new mod in with patools, ModBuddy doesnt seem to update its mods list in the json files and therefore refresh the GUI with the new mod row.

[OlavStornes]

Thanks for the issue. I'm glad to hear that this project is used in a game i'm patiently waiting for further development on.

Regarding patool, i'm noticing that the last release is back in 2016. As long as the package doesn't seem to create any issues with newer versions of python (>=3.9), I'd be thrilled to extend this functionality.

In the example you provided, does the folder actually extracts to the correct place? My suspicion is that patooolib throws FileNoutFoundError, which would prevent calling self.add_mod() in this case (else-clauses in a try-block only fires if there's no handled/unhandled exceptions).

[alan-cugler]

You are correct about FileNotFoundError, I have reworked the above pseudo code to a working improvement. I just haven't cleaned it up yet to do a PR. After talking with another developer I went with a code solution that remove the try clause and has the target_directory created just before the patoolib extraction.

The main reason i decided this was good enough is because FileNotFoundError shouldn't be the error to populate if the directory is made just before instantiation. There are a other reasons to have a try/except there for the archive extraction, but I wanted to focus on the improvement.

[alan-cugler]

If you feel like assigning this to me, I expect to have a PR here at some point in the week.

[OlavStornes]

Glad to hear that. Feel free to provide a solution when ever you have the time, and I'll be happy to look over it.

Regarding removal of the try clause: I'm hesitant to remove that in this case, however it isn't there because of a missing file (judging by your draft which is using FileNotFoundError), but because of potential errors with the archive, such as something unexpected errors regarding the archive itself.
For instance, Shutil.ReadError can occur when an archive is unable to extract for an unspecified reason, such as corrupt data (even though my implementation heavily hints that it is due to an unsupported archive).

[alan-cugler]

That was my inference. With patoolib taking over for shutil I knew that particular pain should not populate. Though the corrupted archive is a real risk.