63 Commits (45035413e9abf7f083474bbd04851df7dfc10968)

Author	SHA1	Message	Date
AnyOldName3	bb3c22e4a5	Add and register SettingValue stream operators	9 months ago
AnyOldName3	a06ab94a20	Canonicalise resolved representation of data directories	10 months ago
AnyOldName3	243b5b6666	Hopefully convince the old MSVC version on GitLab CI to work ... The old code was legal, and the things it did worked in other places, so should have worked here, too. Hopefully just rearranging stuff convinces what I assume to be a compiler bug to not happen.	10 months ago
AnyOldName3	a130ca57a4	Track source of settings ... This one's a biggie. The basic idea's that GameSettings should know: * what the interpreted value of a setting is, so it can actually be used. * what the original value the user put in their config was, so it can be put back when the config's saved. * which path it's processing the openmw.cfg from so relative paths can be resolved correctly. * whether a setting's a user setting that can be modified, or from one of the other openmw.cfg files that can't necessarily be modified. This had fairly wide-reaching implications. The first is that paths are resolved properly in cases where they previously wouldn't have been. Without this commit, if the launcher saw a relative path in an openmw.cfg, it'd be resolved relative to the process' working directory (which we always set to the binary directory for reasons I won't get into). That's not what the engine does, so is bad. It's also not something a user's likely to suspect. This mess is no longer a problem as paths are resolved correctly when they're loaded instead of on demand when they're used by whatever uses them. Another problem was that if paths used slugs like ?userconfig? would be written back to openmw.cfg with the slugs replaced, which defeats the object of using the slugs. This is also fixed. Tracking which settings are user settings and which are in a non-editable openmw.cfg allows the launcher to grey out rows so they can't be edited (which is sensible as they can't be edited on-disk) while still being aware of content files that are provided by non-user data directories etc. This is done in a pretty straightforward way for the data directories and fallback-archives, as those bits of UI are basic, but it's more complicated for content files as that uses a nmodel/view approach and has a lot more moving parts. Thankfully, I'd already implemented that when dealing with builtin.omwscripts, so it just needed wiring up. One more thing of note is that I made the SettingValue struct storable as a QVariant so it could be attached to the UI widgets as userdata, and then I could just grab the original representation and use it instead of needing any complicated mapping from display value to on-disk value.	10 months ago
AnyOldName3	b8cb757ca4	Oopsie	10 months ago
AnyOldName3	9e1334cc09	Resync composing and path openmw.cfg settings with options.cpp	10 months ago
AnyOldName3	f476301670	There's no such thing as the global data directory ... That's what resources/vfs is for.	10 months ago
AnyOldName3	7d28788aee	data-local is already unquoted when it's read	10 months ago
AnyOldName3	dbdecfe94b	Use approved safety comment for path escaping explanation ... I thought I'd got this one already	10 months ago
AnyOldName3	90966ecc47	Handle replace= lines properly in the launcher	10 months ago
Andrei Kortunov	87c9f395f1	Move local variables in components	12 months ago
elsid	f6fce5ee15	Cleanup includes	2 years ago
AnyOldName3	106dbba086	Restore and clarify comments damaged by !2971	2 years ago
Bret Curtis	0db31207dc	remove remaining boost::filesystem cruft	2 years ago
Petr Mikheev	0769e3daf0	Fix #7294 (launcher creates new contentlist everytime)	2 years ago
elsid	9815f930d9	Setup launcher configuration manager and logging before initializing UI	2 years ago
elsid	cf75363290	Typed launcher settings ... QMultiMap is not clear about what settings exist and it's not efficient way to access them after they are loaded.	2 years ago
elsid	dd89403df0	Move ensureUtf8Encoding to named namespace ... To follow https://github.com/isocpp/CppCoreGuidelines/blob/master/CppCoreGuidelines.md#sf21-dont-use-an-unnamed-anonymous-namespace-in-a-header Add QtGlobal include to define QT_VERSION and QT_VERSION_CHECK macroses before they're used.	2 years ago
Andrei Kortunov	ee9ab8d393	Use STL-style iterators instead of Java-style ones	2 years ago
Andrei Kortunov	bfcbc2350d	Handle UTF-8 in Qt streams in the Qt6-compatible way	2 years ago
Andrei Kortunov	307a60e87c	Migrate from QRegExp to more modern QRegularExpression	2 years ago
elsid	843753da14	Remove unused includes	2 years ago
clang-format-bot	ddb0522bbf	Apply clang-format to code base	2 years ago
Project579	ca14fc00dc	Added dedicated functions for conversions between QString and std::filesystem::path.	2 years ago
Project579	c226b35f1f	Fix some remaining encoding errors due to std::filesystem transition.	2 years ago
Project579	a13709c510	Replace implicit convertions from std::filesystem::path to std::string with correctly converting functions.	2 years ago
Project579	e5c417c968	Make sure all paths are passed as std::filesystem::path instead of std::string where possible.	2 years ago
Project579	35fe214588	Updated components/misc/timeconvert.hpp to fix the Android build.	2 years ago
Project579	4bb07282c9	Replace all remaining occurrences of boost::filesystem with std::filesystem.	2 years ago
Andrei Kortunov	e3ad30a517	Do not copy data when it is not needed	2 years ago
Evil Eye	c081b8cfa9	Don't load content entries from global and local configs	3 years ago
Petr Mikheev	c7ab67c2c1	Allow relative paths in openmw.cfg; support --replace=config.	3 years ago
fredzio	b88d32ff5b	Add 3 tabs in the "Data Files" page ... 1 with the data directories 2 with the BSA archives 3 with the content selector When user select a directory to be added, first we walk the directory hierarchy to make a list of all potential data= entries. If we find none, the selected directory is added. If more than one data directory is found, user is presented with a directory list to check which one(s) are to be added. Directories containing one or more content file are marked with an icon. data= and fallback-archive= lines are handled like content= lines: - they are part of the profile in launcher.cfg, prefixed by the profile name - they are updated in openmw.cfg when profile is selected / created Directories can be moved in the list by drag and drop or by buttons. Insertion is possible anywhere in the list. Global data path and data local are shown but are greyed out, as they are always included. No attempt is made to ensure that the user choice are valid (dependencies, overwrite of content). After a profile is loaded, any added content is highlighted in green.	3 years ago
Bret Curtis	d1fb854521	move most of the files from esm to esm3, keep common code in esm; this is make space for a future with esm4 ... esm typo esm typo	3 years ago
Andrei Kortunov	14cf0ce1dc	Implement instanced groundcover	4 years ago
fredzio	dbdd397716	Remove deadcode.	4 years ago
psi29a	957a1425d1	Merge branch 'cleanup_1' into 'master' ... Cleanup 1 See merge request OpenMW/openmw!365	4 years ago
AnyOldName3	538314b03a	Make path settings have path type	4 years ago
Bret Curtis	e51ca542d4	components/config cleanup	4 years ago
Andrei Kortunov	487bfed672	Use QMultiMap instead of QMap	5 years ago
elsid	045ceeac11	Replace foreach macro by for-loop	5 years ago
Andrei Kortunov	71227ffc04	Handle possible invalid iterator issue during settings parsing	6 years ago
Capostrophic	be542507f8	Make hasMaster() behavior consistent with isGameFile() (fixes #3618)	7 years ago
Marcin Baszczewski	46b015d3d9	Fix assert with empty setting value ... Local value could be empty, so it's important to check it size before read first char.	7 years ago
AnyOldName3	49dbb4a9ca	Add a third copy of a comment where I felt clarification was missing	7 years ago
AnyOldName3	7329e6a9ef	Load 'data=...' lines correctly when starting the wizard or launcher, and save them correctly when exiting the launcher.	7 years ago
AnyOldName3	28ff677337	Save 'data=...' lines correctly when exiting the wizard	7 years ago
scrawl	177a6f4a68	Launcher: ensure to clear previous settings when reloading settings	9 years ago
cc9cii	6b28955f31	Fix attempt to erase using a const_iterator.	10 years ago
cc9cii	36aa4aa9bd	If a removed 'content=' item has comments, keep them in config file for later use.	10 years ago