lncorrect PHPDoc tags for stubs (e.g. acf_get_field_types, acf_get_setting)

[php-stubs]

lncorrect PHPDoc tags for stubs (e.g. acf_get_field_types, acf_get_setting)

[szepeviktor]

...after a bit of thinking. Please try to talk to Elliot. Maybe he accepts a patch.

Last time I talked to him 🤣 he told me that applying my patch would fix all docblocks and he prefers fixing source code and docblocks at the same time, so he did not apply it.

[szepeviktor]

Yes. This is not a selling point of ACF Pro, so these tags are slowly improving.
Please talk to Elliot as this is an automatically generated repo.

All I have is this.

acf-pro-stubs/generate.sh

Lines 27 to 43 in 3f02988

	# Remove determine_locale function
	sed -i -e 's#^function determine_locale()$#//&#' "$FILE"
	# - Make docblocks start with "/**"
	# - Fix type and variable name order for @param
	# - Remove remaining parentheses for @param
	# - Fix type and variable name order for @return
	# - Remove remaining parentheses for @return
	# - Fix "void"
	find . -type f -name "$FILE" -exec sed \
	-e 's#^\(\s*/\*\)$#\1*#' \
	-e 's#^\(\s*\*\s*@param\s\+\)\(\$\S\+\)\s\+(\(\S\+\))\(.*\)$#\1\3 \2\4#' \
	-e 's#^\(\s*\*\s*@param\s\+\)(\(\S\+\))\(.*\)$#\1\2\3#' \
	-e 's#^\(\s*\*\s*@return\s\+\)\(\$\S\+\)\s\+(\(\S\+\))\(.*\)$#\1\3 \2\4#' \
	-e 's#^\(\s*\*\s*@return\s\+\)(\(\S\+\))\(.*\)$#\1\2\3#' \
	-e 's#n/a#void#i' \
	-i "{}" ";"

Related #8 and many more.

What do you suggest?

[mundschenk-at]

I don't really have a suggestion, I just noted that paulthewalton/acf-stubs was deprecated and it was advised to install php-stubs/acf-pro-stubs instead, but doing so resulted in invalid findings with PHPStan.

[szepeviktor]

Yes, Paul did it with his hands, line by line.

I would be very happy if you figure out a mechanism to fix the docblock of daily used functions.

[szepeviktor]

Yes, Paul did it with his hands, line by line.

No! My mistake. Paul also used giacocorsiglia/stubs-generator 3 years ago.

[szepeviktor]

@mundschenk-at All right. Please provide me that one file (includes/fields.php) with fixed types in a secret gist and I figure out something.

[szepeviktor]

@mundschenk-at Did a series of releases but still no improvement on "your" front.

Many docblocks have improved in v6.0.7...v6.1.0 but not in these reported functions.

[crstauf]

Using acf()->fields->get_field_type() was recommended (#8 (comment)), but that throws an undefined property error.

[szepeviktor]

You could download https://patch-diff.githubusercontent.com/raw/php-stubs/acf-pro-stubs/pull/8.patch and apply it as a patch.

[crstauf]

Ended up using composer-patches:~1.0 to apply a patch (on composer install/update) to fix the return type of acf_get_field_type().

Generated the patch with this command:

git diff --no-index vendor/php-stubs/acf-pro-stubs/acf-pro-stubs.php.old vendor/php-stubs/acf-pro-stubs/acf-pro-stubs.php > dev/patches/acf-pro-stubs.patch

I tried using symplify/vendor-patches, but couldn't get the patch to apply.