Restore documentation on variable arguments in Drupal\Core\Database\Query\AlterableInterface::hasAllTags() and hasAnyTag() in Drupal 8.8

The Needs Review Queue Bot → tested this issue. It either no longer applies to Drupal core, or fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".

Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.

Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.

Attached patch against Drupal 10.1.x

Patch #9 is not applied for Drupal 10.1.x so Inter-diff file is not added.

I just converted this to an MR.

The Needs Review Queue Bot → tested this issue.

While you are making the above changes, we recommend that you convert this patch to a merge request → . Merge requests are preferred over patches. Be sure to hide the old patch files as well. (Converting an issue to a merge request without other contributions to the issue will not receive credit.)

Seems pretty straight forward.

nod_ → closed merge request !8994

nod_ → reopened merge request !8994

This doesn't seem enough, we need a variable name. In PHP you need to do

function test(...$params) {}

for this to work, so doc needs to be @param string …$params, we can't have only the ….

@nod_, thanks. I converted that without much though, sorry.

Believe feedback has been addressed on this one. Based on the D11 release know we search for drupal:12.0.0 so should be good there.

I checked in on this and noticed that the links in the @todo were incorrect. I updated them and rebased.

@quietone: It is for me RTBC with or without the nitpicks.

believe feedback has been addressed on this one.

nod_ → closed merge request !8994

Committed 80adfea and pushed to 11.x. Thanks!

Maybe there is a small misconception in the deprecation ignore file? See comment inline

longwave → closed merge request !11802

Thank you @mondrake.

Automatically closed - issue fixed for 2 weeks with no activity.