Skip to content

Remove validation-state from rendered link #4109

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

reflexxion
Copy link
Contributor

The LinkTag adds an attribute named validation-state if the target is not published.
After publishing the target page and not editing the link again the resulting link contains the invalid attribute.

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Fixed tickets fixes #issuenum
Related issues/PRs #issuenum
License MIT
Documentation PR sulu/sulu-docs#prnum

What's in this PR?

The LinkTag adds an attribute named validation-state if the target is not published.
After publishing the target page and not editing the link again the resulting link contains the invalid attribute.

Why?

Invalid attribute is rendered

@reflexxion reflexxion changed the base branch from master to develop August 15, 2018 11:26
@reflexxion reflexxion changed the base branch from develop to master August 15, 2018 11:27
@reflexxion reflexxion force-pushed the bugfix/link-tag-validation-state-attribute branch from acb7310 to 70b257e Compare August 15, 2018 11:38
@alexander-schranz alexander-schranz added the Bug Error or unexpected behavior of already existing functionality label Aug 16, 2018
@alexander-schranz alexander-schranz added this to the Release 1.6 milestone Aug 16, 2018
@danrot
Copy link
Contributor

danrot commented Aug 17, 2018

Code looks good to me, but can you add a test to the LinkTagTest? Not that we are reintroducing that error 😃

@wachterjohannes Are there any more properties we should add to this list?

The LinkTag adds an attribute named `validation-state` if the target is not published.
After publishing the target page and not editing the link again the resulting link contains the invalid attribute.
@reflexxion reflexxion force-pushed the bugfix/link-tag-validation-state-attribute branch from 70b257e to faa639d Compare August 17, 2018 09:13
@reflexxion
Copy link
Contributor Author

Hi @danrot,
I added it to the test.

@wachterjohannes wachterjohannes merged commit 03cdb89 into sulu:master Aug 21, 2018
@wachterjohannes
Copy link
Member

@reflexxion thanks for this contribution

@reflexxion reflexxion deleted the bugfix/link-tag-validation-state-attribute branch August 21, 2018 06:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Error or unexpected behavior of already existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants