Do not write an updated style file in StyleEditor::setVariables()
authorTim Düsterhus <duesterhus@woltlab.com>
Fri, 31 Mar 2023 10:05:08 +0000 (12:05 +0200)
committerTim Düsterhus <duesterhus@woltlab.com>
Fri, 31 Mar 2023 10:05:08 +0000 (12:05 +0200)
commit74679f0c31a097d27d9abb1ba2fae1486b1e9447
treed5742b27290a33e7427d531238bb629114cb4aa8
parent5a75686fe56109b8fb5abdf41e80105dacf5ca4e
Do not write an updated style file in StyleEditor::setVariables()

If the style variables have already been loaded, which happens during style
import where the existing and to-be-updated style is currently taken from the
cache, the style will be rewritten with the old variables, effectively
resulting in a noop, making it a useless operation.

However if the existing variables are broken, e.g. because they contain a
syntax error, updating a broken style with a fixed one during package
installation will be impossible (without manually clearing the cache), because
the package installation aborts during compilation of the old SCSS, despite the
new SCSS already been written.

Fixes #5333
wcfsetup/install/files/lib/data/style/StyleEditor.class.php