82699cc297
feat: allow skipping the update confirmation dialog
...
Signed-off-by: flow <flowlnlnln@gmail.com >
2022-12-06 17:00:32 -03:00
08d008a5aa
refactor: abstract away update confirmation dialog
...
... so that we can avoid code duplication.
Signed-off-by: flow <flowlnlnln@gmail.com >
2022-12-06 17:00:29 -03:00
cba2608c1c
feat: add logic for the modrinth instance modpack page
...
Signed-off-by: flow <flowlnlnln@gmail.com >
2022-12-06 16:54:54 -03:00
c8c3fe1023
chore: add license headers
...
Signed-off-by: Sefa Eyeoglu <contact@scrumplex.net >
2022-12-06 13:18:50 +01:00
85bddbed86
chore: reformat code
...
Signed-off-by: Sefa Eyeoglu <contact@scrumplex.net >
2022-12-06 13:16:27 +01:00
07359865c6
fix: don't allocate BlockedsModsDialog
...
Fixes temporary memory leak!
Signed-off-by: Sefa Eyeoglu <contact@scrumplex.net >
2022-12-05 21:39:26 +01:00
386fb28a35
fix(FileResolvingTask): Fail instead of crash when there's API errors
...
Also remove non-used variable (rip).
Signed-off-by: flow <flowlnlnln@gmail.com >
2022-12-01 15:33:35 -03:00
9af1b00df5
feat: add list of watched folders
...
move explanation text into dialog class and it's own label
Signed-off-by: Rachel Powers <508861+Ryex@users.noreply.github.com >
2022-11-11 18:05:19 -07:00
3f6cc17818
properly handle a currently running hashing task
...
Signed-off-by: Rachel Powers <508861+Ryex@users.noreply.github.com >
2022-11-11 15:45:40 -07:00
5909af9878
drag&drop + add folder to watch
...
Signed-off-by: Rachel Powers <508861+Ryex@users.noreply.github.com >
2022-11-11 14:17:33 -07:00
64576f4c4c
Merge pull request #304 from Ryex/move-downloads
...
Fixes https://github.com/PrismLauncher/PrismLauncher/issues/222
2022-11-11 12:48:53 -08:00
2f10fa8b61
add some extra debug logs for CF blocked mods
...
Signed-off-by: Rachel Powers <508861+Ryex@users.noreply.github.com >
2022-11-11 05:41:32 -07:00
392452d422
Merge pull request #397 from flowln/windows_mod_updater_fixes_maybe
...
Fixes https://github.com/PrismLauncher/PrismLauncher/issues/226
2022-11-10 14:34:15 +01:00
589d160515
fix: use cross-platform toStdString in FlameHasher
...
Almost the same issue from toml++ :p
Signed-off-by: flow <flowlnlnln@gmail.com >
2022-11-07 14:04:48 -03:00
5e9b26dbef
Merge pull request #366 from TheLastRar/MSVC-Build
2022-11-04 23:49:54 +01:00
d35c2db41e
fix: separate types of std::string in Packwiz
...
Signed-off-by: flow <flowlnlnln@gmail.com >
2022-11-04 16:53:25 -03:00
e29259cd55
Fix: Resolve ambiguous overload errors
...
Clang-cl fails to select the correct function and instead errors
Signed-off-by: TheLastRar <TheLastRar@users.noreply.github.com >
2022-11-04 18:11:39 +00:00
c993209b76
Fix: Explicitly specify Version.h path
...
MSVC uses a different search mechanism that ends up picking the mete Version.h
Signed-off-by: TheLastRar <TheLastRar@users.noreply.github.com >
2022-11-03 21:35:27 +00:00
7956e6f04e
fix: don't use forward-declared Ptr types in meta/
...
This would cause ODR violations when those headers were included in
other places that also included stuff like "Version.h" (note the
"meta/Version.h"), which can cause problems, especially in LTO.
Signed-off-by: flow <flowlnlnln@gmail.com >
2022-11-01 19:48:26 -03:00
209a1650e4
clang_format and code cleanup
...
Signed-off-by: Rachel Powers <508861+Ryex@users.noreply.github.com >
2022-11-01 07:06:36 -07:00
6010ce0dc5
chore(remove FS::copyFile):
...
Now that #333 is merged and FS::copy works on non directory copyFile can be removed.
Signed-off-by: Rachel Powers <508861+Ryex@users.noreply.github.com >
2022-11-01 04:28:57 -07:00
a7a331a26e
ensure FS::copyFile is marked for removal
...
Signed-off-by: Rachel Powers <508861+Ryex@users.noreply.github.com >
2022-11-01 04:24:11 -07:00
fda2c116be
code quality cleanup
...
Signed-off-by: Rachel Powers <508861+Ryex@users.noreply.github.com >
2022-11-01 04:24:11 -07:00
e9d4793b1e
minor clean up and add some docs
...
Signed-off-by: Rachel Powers <508861+Ryex@users.noreply.github.com >
2022-11-01 04:24:11 -07:00
13c7efa058
copy found mods to instance (FTB and Flame)
...
Signed-off-by: Rachel Powers <508861+Ryex@users.noreply.github.com >
2022-11-01 04:24:11 -07:00
1598d65824
watch filesystem, compute and match hashes
...
Signed-off-by: Rachel Powers <508861+Ryex@users.noreply.github.com >
2022-11-01 04:24:11 -07:00
028e086960
send blocked mod info to dialog & prototype UI
...
Signed-off-by: Rachel Powers <508861+Ryex@users.noreply.github.com >
2022-11-01 04:24:11 -07:00
e6e92f2b0e
fix: only allow workarounds for blocked mods from MR when 100% safe
...
If a version on Modrinth has more than a single mod loader associated,
it means that it's possible we might get the wrong file for download,
since individual files don't really have this kind of metadata in the
API response.
So, in such cases, it's best to let the user take care of it instead.
Signed-off-by: flow <flowlnlnln@gmail.com >
2022-10-28 13:31:21 -03:00
93894f62ff
fix: avoid segfault for unexpected API reponse
...
Signed-off-by: Sefa Eyeoglu <contact@scrumplex.net >
2022-10-28 00:08:15 +02:00
b638a6ae95
fix: retry mod search job after aborting it
...
This way, we don't get stuck with an aborted job in our way! :o
Signed-off-by: flow <flowlnlnln@gmail.com >
2022-10-24 09:09:24 -03:00
24e4db676a
Merge pull request #224 from jamierocks/atl-abort-close-optional-mods-dialog
2022-10-22 15:31:53 +02:00
970e4b020e
fix: fix segfault when resolving Flame resources
...
Signed-off-by: Sefa Eyeoglu <contact@scrumplex.net >
2022-10-22 14:11:51 +02:00
a6b13487f0
ATLauncher: Abort install if optional mods dialog is closed
...
This matches the behaviour of ATLauncher.
2022-10-22 12:35:34 +01:00
01b90809e8
fix: memory leak when finishing blocked mods job
...
Signed-off-by: flow <flowlnlnln@gmail.com >
2022-10-20 16:55:22 -03:00
c90a88b6b6
fix: correct ftb legacy too
...
Signed-off-by: flow <flowlnlnln@gmail.com >
2022-10-14 12:18:06 -03:00
dfa220ef02
fix: issues with aborts (again)
...
i hate it
Signed-off-by: flow <flowlnlnln@gmail.com >
2022-10-13 15:10:35 -03:00
83654a193e
refactor+fix: Make FTB install task similar to other install tasks
...
In particular, this changes the order so that the instance gets created
before downloading the mods (like other install tasks), and the mod download
directly puts the files in the staging folder (like the others), instead
of that weird makeCached and copy stuff.
This fixes some issues with modpack downloads from FTB, like creating an
instance with no mods in it.
Signed-off-by: flow <flowlnlnln@gmail.com >
2022-10-13 13:27:52 -03:00
0dca9cb6b8
Merge pull request #1162 from Trial97/feature/Replace-toml-library
2022-09-30 19:44:45 +02:00
dd6f670dec
fix: Fixed memory leak
...
Signed-off-by: timoreo <contact@timoreo.fr >
2022-09-26 11:50:55 +02:00
9ff364b0d3
huge nit: added const refs, everywhere
...
Signed-off-by: timoreo <contact@timoreo.fr >
2022-09-26 11:50:31 +02:00
600c49f7f0
Replaced tomlc99 with tomlplusplus
...
Signed-off-by: Trial97 <alexandru.tripon97@gmail.com >
2022-09-24 00:10:27 +03:00
06019f01e3
feat: add dialog to ask whether to chaneg instance's name
...
This prevents custom names from being lost when updating, by only
changing the name if the old instance name constains the old version,
so that we can update it if the user whishes to.
Signed-off-by: flow <flowlnlnln@gmail.com >
2022-09-20 18:36:10 -03:00
ddde885084
fix: show warning in case there's no old index when updating
...
Signed-off-by: flow <flowlnlnln@gmail.com >
2022-09-20 18:36:10 -03:00
be8c6f218c
refactor: setAbortStatus -> setAbortable
...
Signed-off-by: flow <flowlnlnln@gmail.com >
2022-09-20 18:36:10 -03:00
d2fdbec41d
fix: move file deletion to the end of the instance update
...
This makes it harder for problems in the updating process to affect the
current instance. Network issues, for instance, will no longer put the
instance in an invalid state.
Still, a possible improvement to this would be passing that logic to
InstanceStaging instead, to be handled with the instance commiting
directly. However, as it is now, the code would become very spaguetti-y,
and given that the override operation in the commiting could also put
the instance into an invalid state, it seems to me that, in order to
fully error-proof this, we would need to do a copy operation on the
whole instance, in order to modify the copy, and only in the end
override everything an once with a rename. That also has the possibility
of corrupting the instance if done without super care, however, so I
think we may need to instead create an automatic backup system, with an
undo command of sorts, or something like that. This doesn't seem very
trivial though, so it'll probably need to wait until another PR. In the
meantime, the user is advised to always backup their instances before
doing this kind of action, as always.
What a long commit message o.O
Signed-off-by: flow <flowlnlnln@gmail.com >
2022-09-20 18:36:09 -03:00
2dd372600c
fix: some abort-related issues
...
Signed-off-by: flow <flowlnlnln@gmail.com >
2022-09-20 18:36:09 -03:00
eda6cf11ef
feat(ui): improve info dialog before updating an instance
...
Adds a 'Cancel' option, and add a note about doing a backup before
updating.
Signed-off-by: flow <flowlnlnln@gmail.com >
2022-09-20 18:36:09 -03:00
87002fb8f8
fix: hook up setAbortStatus in instance import tasks
...
Signed-off-by: flow <flowlnlnln@gmail.com >
2022-09-20 18:36:09 -03:00
6541570969
fix: simplify abort handling and add missing emits
...
Signed-off-by: flow <flowlnlnln@gmail.com >
2022-09-20 18:36:09 -03:00
4b0ceea894
fix: correctly set managed pack fields in CF pack
...
Signed-off-by: flow <flowlnlnln@gmail.com >
2022-09-20 18:36:09 -03:00
