Skip to content

Commit 673e961

Browse files
authored
Merge pull request #1777 from IrAlfred/enh-user-feedback-for-saved-searches
fix(other): improve user feedback for saved searches to include settings save reminder
2 parents 9127d6c + 4a747cf commit 673e961

File tree

1 file changed

+24
-4
lines changed

1 file changed

+24
-4
lines changed

modules/saved_searches/modules.php

Lines changed: 24 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,12 @@ public function process() {
4949
$this->user_config->set('saved_searches', $searches->dump());
5050
$this->session->set('user_data', $this->user_config->dump());
5151
$this->out('updated_search', true);
52-
Hm_Msgs::add('Saved search updated', 'info');
52+
if (isPageConfigured('save')) {
53+
Hm_Msgs::add("Saved search updated. To preserve your searches after logout, please go to <a class='alert-link' href='/?page=save'>Save Settings</a>.", 'info');
54+
}
55+
else {
56+
Hm_Msgs::add('Saved search updated', 'info');
57+
}
5358
}
5459
else {
5560
Hm_Msgs::add('Unable to update the search parameters', 'danger');
@@ -111,7 +116,12 @@ public function process() {
111116
$this->user_config->set('saved_searches', $searches->dump());
112117
$this->session->set('user_data', $this->user_config->dump());
113118
$this->out('saved_search', true);
114-
Hm_Msgs::add('Search saved', 'success');
119+
if (isPageConfigured('save')) {
120+
Hm_Msgs::add("Search saved. To preserve your searches after logout, please go to <a class='alert-link' href='/?page=save'>Save Settings</a>.", 'success');
121+
}
122+
else {
123+
Hm_Msgs::add('Search saved', 'success');
124+
}
115125
}
116126
else {
117127
Hm_Msgs::add('You already have a search by that name', 'warning');
@@ -145,7 +155,12 @@ public function process() {
145155
$this->user_config->set('saved_searches', $searches->dump());
146156
$this->session->set('user_data', $this->user_config->dump());
147157
$this->out('saved_advanced_search', true);
148-
Hm_Msgs::add('Advanced search saved', 'success');
158+
if (isPageConfigured('save')) {
159+
Hm_Msgs::add("Advanced search saved. To preserve your searches after logout, please go to <a class='alert-link' href='/?page=save'>Save Settings</a>.", 'success');
160+
}
161+
else {
162+
Hm_Msgs::add('Advanced search saved', 'success');
163+
}
149164
}
150165
else {
151166
Hm_Msgs::add('Failed to save advanced search - name already exists', 'danger');
@@ -194,7 +209,12 @@ public function process() {
194209
$this->user_config->set('saved_searches', $searches->dump());
195210
$this->session->set('user_data', $this->user_config->dump());
196211
$this->out('updated_advanced_search', true);
197-
Hm_Msgs::add('Advanced search updated', 'info');
212+
if (isPageConfigured('save')) {
213+
Hm_Msgs::add("Advanced search updated. To preserve your searches after logout, please go to <a class='alert-link' href='/?page=save'>Save Settings</a>.", 'info');
214+
}
215+
else {
216+
Hm_Msgs::add('Advanced search updated', 'info');
217+
}
198218
}
199219
else {
200220
Hm_Msgs::add('Unable to update the advanced search parameters', 'danger');

0 commit comments

Comments
 (0)