rednir / OsuSkinMixer

Create a skin mix in a couple clicks.
MIT License
124 stars 3 forks source link

skin.ini for copy of skins are incorrect if name option contains brackets #135

Closed MF-JEWM closed 2 months ago

MF-JEWM commented 2 months ago

very_helpful_logs.txt

Provide details about the bug below:

When modifying a skin, you are given the option to make a copy of it. The skin.ini file that is generated for the copy bugs and turns any option containing brackets into a new section, leading to certain settings not getting properly read by the game when using those skin copies.

Original skin.ini Copied skin.ini
original skin.ini copied skin.ini

Notice that when the skin with the name option of - Seoul v11 - [Test] is copied it it creates a new section in the skin.ini called [Test], or whatever bracketed text is there, which leads to the settings not being processed, since it expects them to be under the [General] section

rednir commented 2 months ago

Fixed in the latest release, launch osu! skin mixer to update