Over in #2822837: Replace @expectedException @expectedExceptionMessage with $this->setExpectedException

we are trying to write a Pharborist script that transforms core's source code.

{@inheritdoc} is valid but when Pharborist's parser encounters the invalid form @{inheritdoc} this exception is thrown.

Uncaught InvalidArgumentException: Invalid tag_line detected: @{inheritdoc} in /MyProject/vendor/phpdocumentor/reflection-docblock/src/phpDocumentor/Reflection/DocBlock/Tag.php:122

I am going to do a search and replace and fix core.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

martin107 created an issue. See original summary.

martin107’s picture

Assigned: martin107 » Unassigned
Issue summary: View changes
FileSize
3.5 KB

9 changes.

martin107’s picture

Status: Active » Needs review
Issue tags: +Quickfix

I think this is a quick fix.

tstoeckler’s picture

Status: Needs review » Reviewed & tested by the community

Looks good to me, thanks!

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed 47fcaa0 and pushed to 8.3.x. Thanks!
Committed 081971f and pushed to 8.2.x. Thanks!

Since this is a docs fix I cherry-picked back to 8.2.x and fixed the conflicts.

  • alexpott committed 47fcaa0 on 8.3.x
    Issue #2824727 by martin107: Malformed inheritdoc prevents Pharborist...

Status: Fixed » Closed (fixed)

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