Show error when parameter is missing in wp option patch (#371)

Co-authored-by: Bunty <[email protected]>

Author: BhargavBhandari90

features/option-pluck-patch.feature

@@ -299,3 +299,27 @@ Feature: Option commands have pluck and patch.
       """
       1
       """
+
+  @patch
+  Scenario: When we don't pass all necessary argumants.
+    Given a WP install
+    And an input.json file:
+      """
+      {
+        "foo": "bar"
+      }
+      """
+    And I run `wp option update option_name --format=json < input.json`
+
+    When I try `wp option patch update option_name foo`
+    And STDERR should contain:
+      """
+      Please provide value to update.
+      """
+    And the return code should be 1
+
+    When I run `wp option patch update option_name foo 0`
+    And STDOUT should be:
+      """
+      Success: Updated 'option_name' option.
+      """

src/Option_Command.php

@@ -567,9 +567,20 @@ function( $key ) {
 			$stdin_value = EntityUtils::has_stdin()
 				? trim( WP_CLI::get_value_from_arg_or_stdin( $args, -1 ) )
 				: null;
-			$patch_value = ! empty( $stdin_value )
-				? WP_CLI::read_value( $stdin_value, $assoc_args )
-				: WP_CLI::read_value( array_pop( $key_path ), $assoc_args );
+
+			if ( ! empty( $stdin_value ) ) {
+				$patch_value = WP_CLI::read_value( $stdin_value, $assoc_args );
+			} else {
+				if ( count( $key_path ) > 1 ) {
+					$patch_value = WP_CLI::read_value( array_pop( $key_path ), $assoc_args );
+				} else {
+					$patch_value = null;
+				}
+			}
+
+			if ( null === $patch_value ) {
+				WP_CLI::error( 'Please provide value to update.' );
+			}
 		}
 
 		/* Need to make a copy of $current_value here as it is modified by reference */