Do not process the pageLogo in StyleAction::updateVariables() if it is unchanged
authorTim Düsterhus <duesterhus@woltlab.com>
Wed, 26 May 2021 14:48:00 +0000 (16:48 +0200)
committerTim Düsterhus <duesterhus@woltlab.com>
Wed, 26 May 2021 14:48:00 +0000 (16:48 +0200)
commitc72c06dc7bb21c5098613ab8fafc8596c8f6ce41
treeab4a2b332d4a26eddf8ac9fb82105d460ed9aaf7
parentef420eedd0192941ca371c6f9622b87b687d7a0a
Do not process the pageLogo in StyleAction::updateVariables() if it is unchanged

This change ensures that the filename of the logo remains the same, unless the
file actually is modified.

A fix for this issue was initially attempted in
cc17f302902fb62c78b4411dadc3243913f18c74. However that commit was much more
complex and it introduced additional bugs, because it no longer passed an
explicit `null` for removed files, thus breaking the update of the related
database columns. It was subsequently reverted in
dd7c98858d406f8ca9824770ae8a68edde451493.
wcfsetup/install/files/lib/data/style/StyleAction.class.php