From 6ca2263ce47bec67f6b4aebeab5b72109463d98b Mon Sep 17 00:00:00 2001 From: Tim Smith Date: Wed, 30 May 2018 20:37:17 -0700 Subject: Avoid some unnecessary complexity We already know the plugin name so there's no need to use find in order to return the plugin name. Signed-off-by: Tim Smith --- lib/chef/knife.rb | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'lib/chef/knife.rb') diff --git a/lib/chef/knife.rb b/lib/chef/knife.rb index 146f7101be..79fdaafd9d 100644 --- a/lib/chef/knife.rb +++ b/lib/chef/knife.rb @@ -271,9 +271,9 @@ class Chef if category_commands = guess_category(args) list_commands(category_commands) - elsif missing_plugin = ( OFFICIAL_PLUGINS.find { |plugin| plugin == args[0] } ) - ui.info("You can install the plugin with `(sudo) gem install knife-#{missing_plugin}`") - ui.info("Use `chef gem install knife-#{missing_plugin}` instead if using ChefDK") + elsif OFFICIAL_PLUGINS.include?(args[0]) # command was an uninstalled official chef knife plugin + ui.info("You can install the plugin with `(sudo) gem install knife-#{args[0]}`") + ui.info("Use `chef gem install knife-#{args[0]}` instead if using ChefDK") else list_commands end -- cgit v1.2.1