aboutsummaryrefslogtreecommitdiff
path: root/launcher/modplatform
AgeCommit message (Collapse)Author
2022-11-10Merge pull request #397 from flowln/windows_mod_updater_fixes_maybeSefa Eyeoglu
Fixes https://github.com/PrismLauncher/PrismLauncher/issues/226
2022-11-07fix: use cross-platform toStdString in FlameHasherflow
Almost the same issue from toml++ :p Signed-off-by: flow <flowlnlnln@gmail.com>
2022-11-04Merge pull request #366 from TheLastRar/MSVC-BuildSefa Eyeoglu
2022-11-04fix: separate types of std::string in Packwizflow
Signed-off-by: flow <flowlnlnln@gmail.com>
2022-11-04Fix: Resolve ambiguous overload errorsTheLastRar
Clang-cl fails to select the correct function and instead errors Signed-off-by: TheLastRar <TheLastRar@users.noreply.github.com>
2022-11-03Fix: Explicitly specify Version.h pathTheLastRar
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-01fix: don't use forward-declared Ptr types in meta/flow
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-01clang_format and code cleanupRachel Powers
Signed-off-by: Rachel Powers <508861+Ryex@users.noreply.github.com>
2022-11-01chore(remove FS::copyFile):Rachel Powers
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-01ensure FS::copyFile is marked for removalRachel Powers
Signed-off-by: Rachel Powers <508861+Ryex@users.noreply.github.com>
2022-11-01code quality cleanupRachel Powers
Signed-off-by: Rachel Powers <508861+Ryex@users.noreply.github.com>
2022-11-01minor clean up and add some docsRachel Powers
Signed-off-by: Rachel Powers <508861+Ryex@users.noreply.github.com>
2022-11-01copy found mods to instance (FTB and Flame)Rachel Powers
Signed-off-by: Rachel Powers <508861+Ryex@users.noreply.github.com>
2022-11-01watch filesystem, compute and match hashesRachel Powers
Signed-off-by: Rachel Powers <508861+Ryex@users.noreply.github.com>
2022-11-01send blocked mod info to dialog & prototype UIRachel Powers
Signed-off-by: Rachel Powers <508861+Ryex@users.noreply.github.com>
2022-10-28fix: only allow workarounds for blocked mods from MR when 100% safeflow
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-28fix: avoid segfault for unexpected API reponseSefa Eyeoglu
Signed-off-by: Sefa Eyeoglu <contact@scrumplex.net>
2022-10-24fix: retry mod search job after aborting itflow
This way, we don't get stuck with an aborted job in our way! :o Signed-off-by: flow <flowlnlnln@gmail.com>
2022-10-22Merge pull request #224 from jamierocks/atl-abort-close-optional-mods-dialogSefa Eyeoglu
2022-10-22fix: fix segfault when resolving Flame resourcesSefa Eyeoglu
Signed-off-by: Sefa Eyeoglu <contact@scrumplex.net>
2022-10-22ATLauncher: Abort install if optional mods dialog is closedJamie Mansfield
This matches the behaviour of ATLauncher.
2022-10-20fix: memory leak when finishing blocked mods jobflow
Signed-off-by: flow <flowlnlnln@gmail.com>
2022-10-14fix: correct ftb legacy tooflow
Signed-off-by: flow <flowlnlnln@gmail.com>
2022-10-13fix: issues with aborts (again)flow
i hate it Signed-off-by: flow <flowlnlnln@gmail.com>
2022-10-13refactor+fix: Make FTB install task similar to other install tasksflow
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-09-30Merge pull request #1162 from Trial97/feature/Replace-toml-librarySefa Eyeoglu
2022-09-26fix: Fixed memory leaktimoreo
Signed-off-by: timoreo <contact@timoreo.fr>
2022-09-26huge nit: added const refs, everywheretimoreo
Signed-off-by: timoreo <contact@timoreo.fr>
2022-09-24Replaced tomlc99 with tomlplusplusTrial97
Signed-off-by: Trial97 <alexandru.tripon97@gmail.com>
2022-09-20feat: add dialog to ask whether to chaneg instance's nameflow
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-20fix: show warning in case there's no old index when updatingflow
Signed-off-by: flow <flowlnlnln@gmail.com>
2022-09-20refactor: setAbortStatus -> setAbortableflow
Signed-off-by: flow <flowlnlnln@gmail.com>
2022-09-20fix: move file deletion to the end of the instance updateflow
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-20fix: some abort-related issuesflow
Signed-off-by: flow <flowlnlnln@gmail.com>
2022-09-20feat(ui): improve info dialog before updating an instanceflow
Adds a 'Cancel' option, and add a note about doing a backup before updating. Signed-off-by: flow <flowlnlnln@gmail.com>
2022-09-20fix: hook up setAbortStatus in instance import tasksflow
Signed-off-by: flow <flowlnlnln@gmail.com>
2022-09-20fix: simplify abort handling and add missing emitsflow
Signed-off-by: flow <flowlnlnln@gmail.com>
2022-09-20fix: correctly set managed pack fields in CF packflow
Signed-off-by: flow <flowlnlnln@gmail.com>
2022-09-20feat: add override awareness to CF modpack updatingflow
Signed-off-by: flow <flowlnlnln@gmail.com>
2022-09-20fix: correctly set all managed pack fields in Modrinth packflow
Signed-off-by: flow <flowlnlnln@gmail.com>
2022-09-20feat: add override handling in modrinth pack updateflow
Signed-off-by: flow <flowlnlnln@gmail.com>
2022-09-20feat: add override helper functionsflow
These help us keep track of relevant metadata information about overrides, so that we know what they are when we update a pack. Signed-off-by: flow <flowlnlnln@gmail.com>
2022-09-20refactor: change the way instance names are handledflow
While working on pack updating, instance naming always gets in the way, since we need both way of respecting the user's name choice, and a standarized way of getting the original pack name / version. This tries to circunvent such problems by abstracting away the naming schema into it's own struct, holding both the original name / version, and the user-defined name, so that everyone can be happy and world peace can be achieved! (at least that's what i'd hope :c). Signed-off-by: flow <flowlnlnln@gmail.com>
2022-09-20feat: add curseforge modpack updatingflow
Signed-off-by: flow <flowlnlnln@gmail.com>
2022-09-20refactor: move creation of CF file download task to a separate functionflow
Signed-off-by: flow <flowlnlnln@gmail.com>
2022-09-20refactor: move flame modpack import to separate fileflow
Signed-off-by: flow <flowlnlnln@gmail.com>
2022-09-20refactor: add `throw_on_blocked` arg to Flame file parseflow
Signed-off-by: flow <flowlnlnln@gmail.com>
2022-09-20feat: add 'getFiles' by fileIds route in Flame APIflow
Signed-off-by: flow <flowlnlnln@gmail.com>
2022-09-20feat: add early modrinth pack updatingflow
Still some FIXMEs and TODOs to consider, but the general thing is here! Signed-off-by: flow <flowlnlnln@gmail.com>
2022-09-20refactor: move modrinth modpack import to separate fileflow
Signed-off-by: flow <flowlnlnln@gmail.com>