Skip to content

Commit 17e2b4b

Browse files
committed
PHPStan fixes
1 parent 8056655 commit 17e2b4b

File tree

1 file changed

+15
-2
lines changed

1 file changed

+15
-2
lines changed

src/Config_Command.php

Lines changed: 15 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -676,6 +676,9 @@ public function set( $args, $assoc_args ) {
676676
*/
677677
$type = Utils\get_flag_value( $assoc_args, 'type' );
678678

679+
/**
680+
* @var array{raw: bool, anchor?: string, separator?: string, placement?: 'after'|'before', add: bool} $options
681+
*/
679682
$options = $this->parse_config_transformer_options( $assoc_args, [ 'add' => true ] );
680683

681684
$adding = false;
@@ -882,6 +885,9 @@ public function update( $args, $assoc_args ) {
882885
*/
883886
$type = Utils\get_flag_value( $assoc_args, 'type', 'all' );
884887

888+
/**
889+
* @var array{raw: bool, anchor?: string, separator?: string, placement?: 'after'|'before', add: bool} $options
890+
*/
885891
$options = $this->parse_config_transformer_options( $assoc_args );
886892

887893
// update command always adds if not exists, so we set the 'add' option to true
@@ -1349,7 +1355,7 @@ private function config_file_not_found_error( $wp_config_file_name ) {
13491355
*
13501356
* @param string $separator Separator string to parse.
13511357
*
1352-
* @return mixed Parsed separator string.
1358+
* @return string Parsed separator string.
13531359
*/
13541360
private function parse_separator( $separator ) {
13551361
$separator = str_replace(
@@ -1370,11 +1376,17 @@ private function parse_separator( $separator ) {
13701376
* @param array $assoc_args Associative arguments from the command.
13711377
* @param array $defaults Default values for the options.
13721378
*
1373-
* @return array<string, bool> Parsed options array.
1379+
* @return array{raw: bool, anchor?: string, separator?: string, placement?: 'after'|'before'} Parsed options array.
13741380
*/
13751381
private function parse_config_transformer_options( $assoc_args, $defaults = [] ) {
1382+
/**
1383+
* @var array{raw: bool, anchor?: string, separator?: string, placement?: 'after'|'before'} $options
1384+
*/
13761385
$options = [];
13771386

1387+
/**
1388+
* @var array{raw: bool, anchor?: string, separator?: string, placement?: 'after'|'before'} $option_flags
1389+
*/
13781390
$option_flags = array_merge(
13791391
[
13801392
'raw' => false,
@@ -1390,6 +1402,7 @@ private function parse_config_transformer_options( $assoc_args, $defaults = [] )
13901402
if ( null !== $option_value ) {
13911403
$options[ $option ] = $option_value;
13921404
if ( 'separator' === $option ) {
1405+
// @phpstan-ignore offsetAccess.notFound
13931406
$options['separator'] = $this->parse_separator( $options['separator'] );
13941407
}
13951408
}

0 commit comments

Comments
 (0)