Skip to content

#1213: Resolve @see and @link to Markdown link#1314

Merged
ashnazg merged 2 commits into
phpDocumentor:developfrom
mvriel:feature/1213
Jun 6, 2014
Merged

#1213: Resolve @see and @link to Markdown link#1314
ashnazg merged 2 commits into
phpDocumentor:developfrom
mvriel:feature/1213

Conversation

@mvriel

@mvriel mvriel commented Jun 6, 2014

Copy link
Copy Markdown
Member

In descriptions it is allowed to use the @see and @link tags; these tags
provide a link to a URL or other element in the project. In this commit
I have added a compiler pass that visits every description and replaces
the @see and @link tag with a Markdown-formatted link to that URL
or href.

I have chosen for a compiler pass because the transformer is the only
part that really knows the URLs for Descriptors and has a list of all
elements that are documented in the current project.

In descriptions it is allowed to use the @see and @link tags; these tags
provide a link to a URL or other element in the project. In this commit
I have added a compiler pass that visits every description and replaces
the `@see` and `@link` tag with a Markdown-formatted link to that URL
or href.

I have chosen for a compiler pass because the transformer is the only
part that really knows the URLs for Descriptors and has a list of all
elements that are documented in the current project.
@mvriel mvriel added this to the 2.6 milestone Jun 6, 2014
@mvriel

mvriel commented Jun 6, 2014

Copy link
Copy Markdown
Member Author

@ashnazg can you review this?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this var_dump() left in intentionally?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is no var_dump, that was just your imagination ;)

Removed it :S

@ashnazg

ashnazg commented Jun 6, 2014

Copy link
Copy Markdown
Member

What var_dump?

@mvriel

mvriel commented Jun 6, 2014

Copy link
Copy Markdown
Member Author

LOL! 👍

ashnazg added a commit that referenced this pull request Jun 6, 2014
@ashnazg ashnazg merged commit 698a82e into phpDocumentor:develop Jun 6, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants