summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* [pre-commit.ci] Pre-commit autoupdate. (#2421)HEADmasterpre-commit-ci[bot]13 days1-1/+1
| | | | | | updates: - [github.com/pre-commit/mirrors-clang-format: v22.1.2 → v22.1.5](https://github.com/pre-commit/mirrors-clang-format/compare/v22.1.2...v22.1.5) Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* Update VCPKG registries to use 7z 26.01. (#2413)Mikaël Capelle2026-06-221-2/+2
|
* Send local saves to recycle bin when switching from global to local saves ↵Mikaël Capelle2026-06-061-1/+1
| | | | instead of deleting them directly. (#2401)
* Update about dialog (#2400)Jonathan Feenstra2026-05-231-868/+885
|
* Translation file updatesJeremy Rimpo2026-05-211-815/+1016
|
* Restore tutorialsJeremy Rimpo2026-05-172-11/+2
|
* Improve code for reading text files line-by-lineJonathan Feenstra2026-05-176-53/+65
|
* Add app.manifest to set longPathAwareJeremy Rimpo2026-05-171-0/+8
|
* Add setting to show notifications when downloads complete or fail (#2338)Jonathan Feenstra2026-05-1513-5/+97
|
* Use new Nexus Tools location for MO2Jeremy Rimpo2026-05-154-64/+42
|
* Install manager metadata updatesJonathan Feenstra2026-05-142-28/+47
| | | | * Fix missing author and uploader when installing mods * Refactor doInstall parameters into a struct
* Migrating to OAuth Authentication (#2374)Jeremy Rimpo2026-05-1232-1387/+1919
| | | | Co-authored-by: aglowinthefield <146008217+aglowinthefield@users.noreply.github.com> Co-authored-by: Jonathan Feenstra <26406078+JonathanFeenstra@users.noreply.github.com>
* Use filesystem paths with BSATK input / output (#2391)Jeremy Rimpo2026-05-072-10/+10
|
* Stable DownloadId refactor (#2375)Al2026-05-077-557/+792
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Encapsulate the downloads directory watcher in DirWatcherManager QFileSystemWatcher suppression currently relies on public static start/end methods and a static counter. Seven call sites pair them raw, one of them outside the class. Any exception between a pair permanently disables the watcher, and the static counter implies a singleton DownloadManager. A new DirWatcherManager owns the watcher, the counter (now an instance member), and the filtering. The only way to suspend is an RAII Guard obtained via a scopedGuard() factory. All raw pairs migrate to guards. A TODO flags the existing processEvents() in the dtor as a known reentrancy hazard worth replacing later. * Replace aboutToUpdate/update(int) with ModelResetGuard Replace the fragile two-signal protocol with a refcounted RAII ModelResetGuard. Split update(int) into aboutToResetModel/modelReset (guard only) and rowChanged(int); notifyRowChanged() is suppressed while a reset is active. Fixes "beginResetModel without endResetModel" warnings from three sites in downloadFinished/removeDownload that were pairing reset with a row update. removePending only opens a guard when an actual match is removed. * Centralize row notifications in setState and fix missed emits setState emits notifyRowChanged itself, uses indexByInfo (-1 when untracked), and re-looks up the row at each use so reply->abort() and plugin callbacks that re-enter and erase info don't produce stale signals. Remove the trailing emit loop from createMetaFile and the now-redundant notifyRowChanged calls scattered after setState. Add the two missing emits in restoreDownload (after m_Hidden) and metaDataChanged (after rename). Guard downloadFinished with a top-level DirWatcherGuard to prevent filesystem events from its writes racing with model updates. * Fix comma operator in addNXMDownload pending-dedup check The game-name comparison result was discarded by the comma operator, so the dedup only matched modId/fileId across all games. * Fix lost finished() signal on fast downloads Hoist the file-exists prompt out of startDownload so setup is straight-line. Connect finished() last and dispatch manually if the reply already finished. * Fix memory leak in DownloadInfo::createFromMeta Move the allocation past the early-return checks so path-mismatch and hidden-skip paths no longer leak a fresh DownloadInfo. * Sanitize suffix path in getDownloadFileName The collision-avoidance branch was using the raw baseName, so invalid characters sanitized out of the initial path leaked into the suffixed one. * [pre-commit.ci] Auto fixes from pre-commit.com hooks. * Remove unused alphabetical translation vector m_AlphabeticalTranslation was written but never read; drop it along with refreshAlphabeticalTranslation, ByName, and the LessThanWrapper helper. * Address PR feedback: fix redundant check and move refresh outside try catch. * Coalesce the removeDownload reset with the following refreshList Moves the ModelResetGuard out of the try-catch so it also wraps the refreshList() call below. Without this, one reset fires when the guard destructs at the end of the try block and another fires from refreshList's own guard, producing two resets where one is sufficient. * Guard the .meta creation in openMetaFile against the directory watcher openMetaFile creates the .meta file via QSettings when one does not exist; the disk write fires directoryChanged and triggers a spurious refreshList. Wrap it in a DirWatcherManager::Guard like the other meta-file editing paths. * Extract getValidGameShortName method in download manager (#2380) * Add stable download id index and PendingDownload struct Replace the (game, mod, file) tuple backing m_PendingDownloads with a named struct, and add m_ByID as an O(1) m_DownloadID-to-info index kept in sync with every m_ActiveDownloads mutation. Encapsulate the id counter behind DownloadInfo::newDownloadID(), the only supported way to consume from s_NextDownloadID. Infrastructure only; external behaviour is unchanged. * Return stable ids from the plugin-facing download API startDownloadURLs / startDownloadNexusFile / addNXMDownload now reserve and return m_DownloadID instead of a stale index. Plugin callbacks fire with m_DownloadID; downloadPath looks up via m_ByID. nxmDownloadURLsAvailable threads the reserved id into the materializing DownloadInfo, and Nexus API failures wake waiting plugins via notifyPendingDownloadFailed. Incidental: startDownload now returns bool and frees newDownload on output-open failure; createMetaFile is deferred past that check so failed starts no longer leave an orphan .meta. * Split downloadFinished into onReplyFinished slot and finishDownload The old dual-use downloadFinished(int = 0) took either an explicit index or relied on sender() when called as a slot. Split into a sender-resolved slot and an id-based direct call, removing the ambiguous index-zero path. * Introduce DownloadID alias and row/id accessors Add a DownloadID type alias for the stable per-download handle and two public accessors (downloadIDAtRow, rowForDownloadID) so callers can translate between the view's row vocabulary and the model's id vocabulary without reaching into the manager's internals. DownloadList now embeds the DownloadID in QModelIndex::internalId() so any code holding an index can identify the download directly. * Convert cancel/pause/resume action methods to take DownloadID The four methods (cancel, pause, resume, resumeDownloadInt) now accept a DownloadID, resolve through m_ByID, and no longer care about row positions. Internal callers iterate DownloadInfo* or look up via id; DownloadsTab translates row -> id at the connect boundary so the view's int-shaped signals keep working unchanged. Also switches the remaining unsigned int signatures that refer to the download id (finishDownload, downloadInfoByID, PendingDownload::reservedID, m_ByID, newDownloadID, s_NextDownloadID) to the DownloadID alias. Drive-by fix: finishDownload's retry branch could read info->m_Tries after info had been deleted in the CANCELED/retries-exhausted branch above; now re-resolves via m_ByID.value(id) before touching any fields. * [pre-commit.ci] Auto fixes from pre-commit.com hooks. * fix warnings about unused variables and size_t types * cleanup dead code * avoid calling processEvents when releasing the DirWatcherGuard * [pre-commit.ci] Auto fixes from pre-commit.com hooks. * use QEventLoop instead of manual ProcessEvents * don't call processEvents in download started and defer handling finish state in event loop * Cleanup pending download in case of failure. * Add missing notifyPendingDownloadFailed if user cancels * [pre-commit.ci] Auto fixes from pre-commit.com hooks. * Refactor pending download failure handling and cover rename failures * fix rebase bug, addNXMDownload not returning the correct type --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Jonathan Feenstra <26406078+JonathanFeenstra@users.noreply.github.com>
* Don't apply LOOT-sorted load order until user clicks "Apply" (#2382)Jonathan Feenstra2026-05-066-255/+372
| | | | * Add sorted plugin list to LOOT dialog (markdown and button) * Prefix plugins with checkboxes to show whether they're enabled
* Defer second nxmhandler call (#2390)Jeremy Rimpo2026-05-063-18/+27
| | | | - Calls run in parallel leading to extraneous setup dialogs - This only calls the nxm schema registration after modl is done
* Download command: Add game instance check (#2388)Jeremy Rimpo2026-05-043-7/+52
| | | | | * Add game instance check - Should be ignored if not passed to command - Functions much like NXM game check
* Improve mod update check accuracy (#2385)Al2026-05-035-109/+225
| | | | | | | | | | | | | | | | | | | | | * Index API response into lookup maps * extract new findLatestActiveSuccessor method from update check function * extract method to check if a file is active * use new isActiveFileStatus in modInfoRegular * fix bug when merging during mod install * remember ordering of installed nexus file ids * refactor update check logic to prioritize Nexus file IDs over filenames * refactor nxmUpdatesAvailable to simplify update checking logic * refactor update check logic to find Nexus file IDs by filename and streamline successor retrieval * refactor update checking to streamline version resolution and improve successor retrieval
* Extended MODL / direct download handling (#2384)Jeremy Rimpo2026-05-0311-59/+162
| | | | | | | * Extended MODL / direct download handling - name, modname, version, and source options added to download command - nxmhandler init adds schemas and MODL entry with default launch args - Add MODL register button to general tab - On window display, call meta function to trigger both registrations
* Handle Nexus collections links (#2383)Jeremy Rimpo2026-05-021-0/+8
| | | | - Pops up dialog when the NXM link is a collection - Collection link data available, though still unsupported
* Change IconDelegate::paintIcons to only execute when iconWidth > 0 (#2362)Mick2026-04-271-24/+24
|
* Make command-line arguments -i "" launch the portable instance (#2341)Jonathan Feenstra2026-04-271-5/+19
|
* fix crash when CWD is not set to app directory (#2379)Al2026-04-261-0/+4
|
* Disabling tutorials (#2366)Jeremy Rimpo2026-04-172-4/+11
|
* Add more contributors to the about dialog (#2369)Jonathan Feenstra2026-04-171-8/+163
|
* Starfield: Updated blueprint / blueprint prefix support (#2368)Jeremy Rimpo2026-04-165-15/+156
| | | | | | | | | | * Add blueprint handling with blueprintships * Blueprint changes - Add tooltips - Add warnings - Handle blueprint prefixes properly * Make sure we're assigning the property
* Update libloot dll name in sanity check (#2370)Jeremy Rimpo2026-04-161-1/+1
|
* Removing references to openssl (#2367)Jeremy Rimpo2026-04-164-7/+0
|
* Update vcpkg targets (#2365)Jeremy Rimpo2026-04-151-4/+4
|
* fix header guard typos (#2364)Mick2026-04-153-6/+6
|
* [pre-commit.ci] Pre-commit autoupdate. (#2357)pre-commit-ci[bot]2026-04-097-20/+20
| | | | | | | | | | | | * [pre-commit.ci] Pre-commit autoupdate. updates: - [github.com/pre-commit/mirrors-clang-format: v21.1.8 → v22.1.2](https://github.com/pre-commit/mirrors-clang-format/compare/v21.1.8...v22.1.2) * [pre-commit.ci] Auto fixes from pre-commit.com hooks. --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* Remove "Categories: " tooltips when there are no categories (#2339)Jonathan Feenstra2026-02-182-0/+8
|
* Add instance manager to plugin API (#2335)Jonathan Feenstra2026-02-0811-37/+206
|
* Add executables list to plugin API (#2327)Jonathan Feenstra2026-01-316-6/+132
|
* Change IOrganizer::profile return type to a shared_ptr (#2322)Jonathan Feenstra2026-01-1111-21/+24
|
* Add instanceName and profiles methods to plugin API (#2321)Jonathan Feenstra2026-01-116-2/+51
|
* [pre-commit.ci] Pre-commit autoupdate. (#2320)pre-commit-ci[bot]2026-01-051-1/+1
| | | | | | updates: - [github.com/pre-commit/mirrors-clang-format: v21.1.2 → v21.1.8](https://github.com/pre-commit/mirrors-clang-format/compare/v21.1.2...v21.1.8) Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* Add executable setting to minimize MO2 to the system tray while running (#2313)Jonathan Feenstra2026-01-038-12/+195
|
* Add missing Qt dependencies in CI. (#2317)Mikaël Capelle2026-01-031-1/+1
|
* [pre-commit.ci] Pre-commit autoupdate. (#2295)pre-commit-ci[bot]2025-10-071-2/+2
| | | | | | | updates: - [github.com/pre-commit/pre-commit-hooks: v5.0.0 → v6.0.0](https://github.com/pre-commit/pre-commit-hooks/compare/v5.0.0...v6.0.0) - [github.com/pre-commit/mirrors-clang-format: v20.1.7 → v21.1.2](https://github.com/pre-commit/mirrors-clang-format/compare/v20.1.7...v21.1.2) Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* fix(qt): remove obsolete call to AA_EnableHighDpiScaling (#2283)itch2025-09-151-1/+0
|
* Fix an issue with implementation of CombinedModDataContent. (#2278)Mikaël Capelle2025-09-032-16/+18
|
* Add author and uploader columns to mod list (#2269)Jonathan Feenstra2025-08-1821-35/+336
|
* Allow non-cache build in CI when Azure variables are not available. (#2270)Mikaël Capelle2025-08-171-1/+1
|
* [pre-commit.ci] Pre-commit autoupdate. (#2258)pre-commit-ci[bot]2025-07-081-1/+1
| | | | | | updates: - [github.com/pre-commit/mirrors-clang-format: v19.1.5 → v20.1.7](https://github.com/pre-commit/mirrors-clang-format/compare/v19.1.5...v20.1.7) Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* Add version of MO2 plugin loaded in logs. (#2252)Mikaël Capelle2025-06-062-14/+15
|
* Fix registry for 2.5.3-beta.2. (#2250)Mikaël Capelle2025-05-291-1/+1
|
* Update registry for 2.5.3-beta.2. (#2249)Mikaël Capelle2025-05-292-1/+6
|
* Move to VCPKG (#2068)Mikaël Capelle2025-05-2964-8010/+1425
| | | | | | | | | * Remove SConscript related files. * Force-load translations from uibase and gamebryo/creation. * Bring githubpp here and add a standalone preset. * Switch VersionInfo -> Version for ModOrganizer2. (#2063) * Add pre-commit hook. * Use 7zip build from VCPKG registry. * Use archive.dll from the bin folder instead of dlls.
* Oblivion Remastered Meta PR (#2241)Jeremy Rimpo2025-05-2314-78/+280
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Allow for mod directory maps - Set main data files based on game - Mapped mod directories to VFS - Update overwrite setup * Skip if mod contains no 'data' dir * More mod directory compatibility fixes * Workaround for Obl:Rem save location - SLocalSavePath does nothing yet MO2 wants to use it to override the default save location - This only applies to BGS games anyway, we should move this logic * First pass for overwrite mod directory support * More overwrite move / delete restrictions * Fix issue with moving directories that are not required * Formatting pass * More modDataDirectory updates * Formatting --------- Co-authored-by: Mikaël Capelle <capelle.mikael@gmail.com>