Document that the StyleHandler is final now (#126)
authorTim Düsterhus <duesterhus@woltlab.com>
Thu, 4 Feb 2021 13:03:39 +0000 (14:03 +0100)
committerGitHub <noreply@github.com>
Thu, 4 Feb 2021 13:03:39 +0000 (14:03 +0100)
Resolves #125

Co-authored-by: Matthias Schmidt <gravatronics@live.com>
docs/migration/wsc53/php.md

index 9626c59e24ffbdc6d7e343b40ffdf6a053c26fb4..98bb62d7d0d99ffb06f81e95fedbf40ae70a29d7 100644 (file)
@@ -80,3 +80,11 @@ If you implemented a custom search engine and relied on this method, you can inl
 You should take the time to verify the rewritten queries against the manual of the search engine to make sure it cannot generate malformed queries or security issues.
 
 See [WoltLab/WCF#3815](https://github.com/WoltLab/WCF/issues/3815) for details.
+
+## Styles
+
+The `StyleCompiler` class is marked `final` now.
+The internal SCSS compiler object being stored in the `$compiler` property was a design issue that leaked compiler state across multiple compiled styles, possibly causing misgenerated stylesheets.
+As the removal of the `$compiler` property effectively broke compatibility within the `StyleCompiler` and as the `StyleCompiler` never was meant to be extended, it was marked final.
+
+See [WoltLab/WCF#3929](https://github.com/WoltLab/WCF/pull/3929) for details.