Fix code review issues: add default type and break statement

Copilot · swissspidy · Copilot · commit 537ddf803c85 · 2025-11-02T15:24:32.000Z
Co-authored-by: swissspidy &lt;841956+swissspidy@users.noreply.github.com&gt;
diff --git a/src/Config_Command.php b/src/Config_Command.php
@@ -924,7 +924,7 @@ public function update( $args, $assoc_args ) {
 		/**
 		 * @var string $type
 		 */
-		$type = Utils\get_flag_value( $assoc_args, 'type' );
+		$type = Utils\get_flag_value( $assoc_args, 'type', 'all' );
 
 		$options = [];
 
@@ -977,6 +977,7 @@ public function update( $args, $assoc_args ) {
 					if ( ! $config_transformer->exists( $type, $name ) ) {
 						$adding = true;
 					}
+					break;
 			}
 
 			$config_transformer->update( $type, $name, $value, $options );

Original file line number	Diff line number	Diff line change
`@@ -924,7 +924,7 @@ public function update( $args, $assoc_args ) {`
`924`	`924`	`/**`
`925`	`925`	`* @var string $type`
`926`	`926`	`*/`
`927`		`- $type = Utils\get_flag_value( $assoc_args, 'type' );`
	`927`	`+ $type = Utils\get_flag_value( $assoc_args, 'type', 'all' );`
`928`	`928`
`929`	`929`	`$options = [];`
`930`	`930`
`@@ -977,6 +977,7 @@ public function update( $args, $assoc_args ) {`
`977`	`977`	`if ( ! $config_transformer->exists( $type, $name ) ) {`
`978`	`978`	`$adding = true;`
`979`	`979`	`}`
	`980`	`+ break;`
`980`	`981`	`}`
`981`	`982`
`982`	`983`	`$config_transformer->update( $type, $name, $value, $options );`