From 9bf0c0cdb738ca59edd2bec43df2fa0935af3d67 Mon Sep 17 00:00:00 2001 From: sunnavy Date: Mon, 30 Dec 2024 15:32:12 -0500 Subject: [PATCH 1/2] Refresh system attributes so new logo can show up right after submission We initialize a new RT::System object with current user to update UserLogo attribute in 9d9e524089 to track who makes the change, thus we need to explicitly refresh attributes of RT->System. --- share/html/Admin/Tools/Theme.html | 1 + 1 file changed, 1 insertion(+) diff --git a/share/html/Admin/Tools/Theme.html b/share/html/Admin/Tools/Theme.html index d075c6114cd..261bcbeef8d 100644 --- a/share/html/Admin/Tools/Theme.html +++ b/share/html/Admin/Tools/Theme.html @@ -382,6 +382,7 @@

<&|/l&>Customize the RT theme

colors => $colors, }, ); + RT->System->Attributes->RedoSearch; push @results, loc("Unable to set UserLogo: [_1]", $msg) unless $id; From 30355a8f862e8ebe50f0276c8889f3048cbd0ec3 Mon Sep 17 00:00:00 2001 From: sunnavy Date: Mon, 30 Dec 2024 16:30:14 -0500 Subject: [PATCH 2/2] Revert "Drop unused submit trigger in lifecycle UI" This reverts commit a6c12e6bbc252546883fc93cacf747f13af7579a. Turns out that we need this trigger for layout-only changes. This commit also tweaks code to trigger "ExportAsConfiguration()" on form submit, instead of the original imprecise version that was on ".submit" click. --- share/static/js/lifecycleui-editor.js | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/share/static/js/lifecycleui-editor.js b/share/static/js/lifecycleui-editor.js index c61412dfce8..ed172d1926f 100644 --- a/share/static/js/lifecycleui-editor.js +++ b/share/static/js/lifecycleui-editor.js @@ -242,6 +242,10 @@ jQuery(function () { self.ToggleSimulation(); return true; }); + + jQuery('form[name=ModifyLifecycle]').submit(function(e) { + self.ExportAsConfiguration(); + }); } RenderNode() {