PERF: Fix performance regression in SvgSprite.settings_icons (#34377)

This commit reverts the changes made to `SvgSprite.settings_icons` in
19af83d39e
which resulted in significantly slower boot times on multisite clusters
since `SvgSprite.settings_icons` is called during application preload.
This commit is contained in:
Alan Guo Xiang Tan
2025-08-18 11:32:54 +02:00
committed by GitHub
parent dbf408025b
commit 3f5e8926af
2 changed files with 19 additions and 7 deletions
+3 -7
View File
@@ -489,13 +489,9 @@ License - https://fontawesome.com/license/free (Icons: CC BY 4.0, Fonts: SIL OFL
# includes svg_icon_subset and any settings containing _icon (incl. plugin settings)
site_setting_icons = []
SiteSetting
.all_settings(include_locale_setting: false)
.select do |setting|
site_setting_icons |= setting[:value].split("|") if setting[:setting].to_s.include?(
"_icon",
) && String === setting[:value]
end
SiteSetting.settings_hash.each do |key, value|
site_setting_icons |= value.split("|") if key.to_s.include?("_icon") && String === value
end
site_setting_icons
end