Feature to update old plugin version

Author: tungleduyxyz

app/controllers/kpm/plugins_controller.rb

@@ -10,9 +10,21 @@ def index
       @plugins = {}
 
       @installed_plugin_keys = Set.new
+      # Map of installed plugin_key => highest installed version found across nodes
+      @installed_plugins = {}
       (nodes_info || []).each do |node_info|
         (node_info.plugins_info || []).each do |plugin_info|
-          @installed_plugin_keys.add(plugin_info.plugin_key) unless plugin_info.plugin_key.nil?
+          next if plugin_info.plugin_key.nil?
+
+          @installed_plugin_keys.add(plugin_info.plugin_key)
+          next if plugin_info.version.nil?
+
+          current = @installed_plugins[plugin_info.plugin_key]
+          begin
+            @installed_plugins[plugin_info.plugin_key] = plugin_info.version if current.nil? || Gem::Version.new(plugin_info.version) > Gem::Version.new(current)
+          rescue ArgumentError
+            @installed_plugins[plugin_info.plugin_key] ||= plugin_info.version
+          end
         end
       end
 

app/views/kpm/plugins/_plugins_table.html.erb

@@ -13,11 +13,16 @@
           <td><%= name %></td>
           <td><%= version %></td>
           <td>
-            <% if @installed_plugin_keys.include?(name) %>
-              <span class="label label-success">Installed</span>
-            <% else %>
-              <%= link_to 'Install', plugin_install_path(:plugin_key => name), :method => :post, :title => 'Install' %>
-            <% end %>
+              <% if @installed_plugin_keys.include?(name) %>
+                <% installed_version = @installed_plugins && @installed_plugins[name] %>
+                <% if installed_version.present? && version.present? && Gem::Version.new(installed_version) < Gem::Version.new(version) %>
+                  <%= link_to 'Update', plugin_install_path(:plugin_key => name), :method => :post, :title => 'Update' %>
+                <% else %>
+                  <span class="label label-success">Installed</span>
+                <% end %>
+              <% else %>
+                <%= link_to 'Install', plugin_install_path(:plugin_key => name), :method => :post, :title => 'Install' %>
+              <% end %>
           </td>
         </tr>
     <% end %>