Fragjacker / DoW-Mod-Manager

The original repository for the DoW Mod Manager application, which was made for the Dawn of War:tm: series.

MIT License

33 stars 7 forks source link

Merging Campaign experimental into the Public version #37

Open Fragjacker opened 2 years ago

Fragjacker commented 2 years ago

Description

This request adds the experimental features of the DLL injector functionality developed by @AndO3131 into the public branch.

TODO

[x] Check the Pull locally and see if no functionality has been overwritten.
[x] Make sure the app will update properly.
[x] Add some kind of safety mechanism in case the DLL file does not locally exist on the recipients machine.
- ~~Either do an auto downloader functionality. (harder solution)~~ OR
- [x] Simply deactivate the checkbox if the file does not exist. (easiest solution)

AndO3131 commented 2 years ago

I haven't checked update functionality, but I think other functions work properly.

AndO3131 commented 2 years ago

Regarding FogRemover.cs I only added two "int" keywords. Otherwise this file didn't compile. I highly doubt that could be the cause of anything.

About DLL injection messing with memory, that's another story. I honestly don't know. Newer had any issues with fog using injection (which I use almost all the time).

AndO3131 commented 2 years ago

One behaviour has been solved: mod manager should no longer reset settings on mods. It was caused by additional "."(dot) in unification campaign addon folder and module file, so unrelated to mod manager code.

Fragjacker commented 1 year ago

Alright now there's another pull-request up for updating the Mod Manager. Now there's the question which request should come first either way one would need to get updated anyways. Is this request still solid and ready for production?

AndO3131 commented 1 year ago

I didn't make any changes since august in this branch. I've compiled mod manager and corresponding UNI_EXT.DLL today and it loads correctly.