Item13560: Clarify messages a bit more.

foswiki · Aug 27, 2015 · e54f9df · e54f9df
1 parent 2804659
commit e54f9df
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 3 deletions.
diff --git a/core/lib/Foswiki/Configure/Checkers/ExtensionsRepositories.pm b/core/lib/Foswiki/Configure/Checkers/ExtensionsRepositories.pm
@@ -64,7 +64,7 @@ sub check_current_value {
     foreach my $ext ( Foswiki::Configure::Load::specChanged() ) {
         my $specfile = ( $ext eq 'the core' ) ? 'Foswiki.spec' : 'Config.spec';
         $reporter->WARN(
-"The $specfile for $ext is more recent than the latest configuration. 'save of extension settings' is required."
+"The $specfile for $ext is more recent than the latest configuration. You should run 'import extension settings' and then save the configuration."
         );
     }
 }

diff --git a/core/lib/Foswiki/Configure/Wizards/Plugins.pm b/core/lib/Foswiki/Configure/Wizards/Plugins.pm
@@ -149,9 +149,19 @@ sub import {
 
     if ($changes) {
         $reporter->WARN("Configuration changes are required.");
-        $reporter->NOTE(
-            "You should save your settings after running this wizard.");
         $reporter->hint( "require_save", 1 );
+        if ( defined $Foswiki::cfg{Engine}
+            && $Foswiki::cfg{Engine} !~ /(CLI|Legacy)$/ )
+        {
+            $reporter->WARN(
+"You should save the configuration, and reload =$Foswiki::cfg{ScriptUrlPath}/configure= to verify any new settings."
+            );
+        }
+        else {
+            $reporter->WARN(
+"If you did not include the -save option, you should rerun this wizard, specifying -save."
+            );
+        }
     }
     else {
         $reporter->NOTE("No changes to the configuration needed.");