Feature Advanced Options #246
Draft
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Still to add output
queueSizeLogG is default on 0.
Issue on line 95 with duplication
Output does not.
Close to done just don't have time to finish
Aligned variable declarations for consistency in Arma3Profile and BasicCfg. Removed unused properties and redundant event handling from ServerCfg. Ensured AdvancedOptions and its content are properly initialized in ServerProfile constructors.
Corrected inconsistent indentation for assignment of AdvancedOptionsContent in the ServerProfile class constructor to improve code readability.
Corrected typos in comments and property change notifications.
Corrects the assignment to set AdvancedOptionsContent instead of overwriting ServerCfgContent with advanced options in ServerProfile.cs.
Reorganized the FASTER.csproj file for improved readability and maintainability by properly indenting and grouping properties. No functional changes were made. Minor whitespace adjustments were also made in the CodeQL workflow YAML.
Changed target framework from net9.0-windows to net8.0-windows in both FASTER and FASTERTests projects. Updated version to 1.9.7.2 in project and version files to reflect the framework change.
Added <EnableWindowsTargeting>true</EnableWindowsTargeting> to make Actions happy even though it isn't needed for net8 :shurg:
Added <EnableWindowsTargeting>true</EnableWindowsTargeting> to both FASTER - Backup.csproj and FASTERTests.csproj to explicitly enable Windows-specific APIs and features during build and development 🤷
Corrected the path to BytexDigital.Steam.csproj in FASTER.sln to remove a duplicate folder segment.
This reverts commit decc343.
Changed the checkout step in the CodeQL workflow to use 'submodules: recursive' and added a token for authentication. This ensures all nested submodules are checked out and authenticated properly.
Introduced a debugging step to list all files before checking out the repository in the CodeQL analysis workflow. This helps diagnose issues related to file structure or missing files during CI runs. (unlikely to keep in)
Removed the debugging file listing step and added an explicit step to initialize and update submodules using git. This ensures submodules are properly set up before running CodeQL analysis.
The SDK version specified in global.json has been changed from 9.0.0 to 8.0.0, likely to ensure compatibility with projects or tooling that require .NET 8.0.0.
Hotfix/AdvancedOptions
…und" This reverts commit 9847124.
This was
linked to
issues
Jul 25, 2025
Corrects the casing of the backing field for AdvancedOptionsContent to follow C# conventions and resolves property getter/setter logic.
Corrected the setter for AdvancedOptionsContent to assign the value to the backing field instead of recursively calling the property, preventing a stack overflow.
Eliminated a redundant assignment to ServerCfg.AdvancedOptionsContent in the ProcessProfile method of ServerProfile.cs.
Updated project and version XML files to reflect new release version 1.9.7.3.
Introduced a new variable 'advanced' to store the path for the advanced server configuration file in the SaveProfile method of ProfileViewModel.
Changed AdvancedOptions from ServerCfg to AdvancedOptions type in ServerProfile. Updated related property assignments and command line generation to use the new advanced options file. This improves type safety and ensures advanced options are handled separately from server configuration.
Moved the AdvancedOptions class from ServerCfg.cs to its own file AdvancedOptions.cs for better separation of concerns and maintainability. Updated ServerCfg to use the new AdvancedOptions property and adjusted related logic to reference the new class.
Replaced hard-coded string literals in RaisePropertyChanged calls with the nameof operator.
Ensures compile-time checking of property names
Prevents runtime bugs if properties are renamed
Makes the code more maintainable and refactor-friendly
- Replaced hard-coded paths with local variables - Added checks for missing mod directories and handled failures - Used Distinct() to avoid duplicate mod key processing - Improved error reporting with more precise file names - Updated ClearModKeys to skip ignored keys by file name instead of partial string match - General cleanup for better readability and reliability
Collaborator
Author
|
Dev No output to check if it is working but i think i can embed it into "Server.Cfg" if i get smart enough. If it works that is like 50% good for me Edited File same release |
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: