Item13360: Undefined variables when uninstall ext.

foswiki · Apr 8, 2015 · d2ceb73 · d2ceb73
1 parent 4ae8ec5
commit d2ceb73
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/core/lib/Foswiki/Configure/Package.pm b/core/lib/Foswiki/Configure/Package.pm
@@ -1306,7 +1306,7 @@ sub uninstall {
     $reporter->WARN(
         "Don't forget to save your configuration to complete removal of "
           . join( ', ', @plugins ) )
-      if ( scalar( keys %{ $reporter->changes() } ) );
+      if ( $reporter->changes() && scalar( keys %{ $reporter->changes() } ) );
 
     $reporter->NOTE( "> Removal "
           . ( $this->option('SIMULATE') ? 'simulated' : 'finished' ) );

diff --git a/core/tools/extender.pl b/core/tools/extender.pl
@@ -265,7 +265,7 @@ sub _uninstall {
 
         my ( $ok, $plugins ) = $thispkg->uninstall($reporter);
 
-        if ( $ok && scalar @$plugins && !$simulate ) {
+        if ( $ok && defined $plugins && scalar @$plugins && !$simulate ) {
             $reporter->NOTE(
                 "> Don't forget to disable uninstalled plugins:
 "