plugins_installer.rb in killbill-kpm-0.0.4

- old
+ new

@@ -20,11 +20,11 @@
       @initialized = true
     end
 
     # If an earlier version of the plugin is installed, Kill Bill will only start the latest one (see org.killbill.billing.osgi.FileInstall)
     def install(plugin_key, specified_artifact_id, specified_version=nil, specified_group_id=nil, specified_packaging=nil, specified_classifier=nil, specified_type=nil, force_download=false)
-      @logger.info("Instructed to install artifact_id=#{specified_artifact_id} version=#{specified_version} group_id=#{specified_group_id} packaging=#{specified_packaging} classifier=#{specified_classifier} type=#{specified_type} force_download=#{force_download}")
+      @logger.info("Instructed to install plugin_key=#{plugin_key} artifact_id=#{specified_artifact_id} version=#{specified_version} group_id=#{specified_group_id} packaging=#{specified_packaging} classifier=#{specified_classifier} type=#{specified_type} force_download=#{force_download}")
       info = @installer.install_plugin(plugin_key, specified_group_id, specified_artifact_id, specified_packaging, specified_classifier, specified_version, @bundles_dir, specified_type, force_download, @glob_config[:kpm][:verify_sha1])
       if info.nil?
         @logger.warn("Error during installation of plugin #{specified_artifact_id}")
       else
         path = info[:bundle_dir] || info[:dir_name]
@@ -67,13 +67,14 @@
     private
 
     def notify_fs_change(plugin_key, path, state)
       return if path.nil?
 
-      # Plugin name should be the directory name (path is something like /var/tmp/bundles/plugins/ruby/killbill-stripe/2.0.0)
+      # Plugin name should be the directory name (path is something like /var/tmp/bundles/plugins/ruby/killbill-stripe/2.0.0 or /var/tmp/bundles/plugins/ruby)
       fs_info = path.to_s.split('/')
       plugin_type = fs_info[-3].upcase
+      plugin_type = fs_info[-1].upcase unless %w(JAVA RUBY).include?(plugin_type)
 
       unless %w(JAVA RUBY).include?(plugin_type)
         @logger.warn("Invalid plugin type #{plugin_type} (path #{path}): Kill Bill won't be notified of new state #{state}")
         return
       end
@@ -83,10 +84,10 @@
         return
       end
 
       plugin_name = fs_info[-2]
       plugin_version = fs_info[-1]
-      @logger.info("Notifying Kill Bill: state=#{state} plugin_name=#{plugin_name} plugin_version=#{plugin_version} plugin_type=#{plugin_type}")
+      @logger.info("Notifying Kill Bill: state=#{state} plugin_key=#{plugin_key} plugin_name=#{plugin_name} plugin_version=#{plugin_version} plugin_type=#{plugin_type}")
       @kb_apis.plugins_info_api.notify_of_state_changed(state, plugin_key, plugin_name, plugin_version, plugin_type)
     end
 
     def configure!(config_file)
       @glob_config = {}