Skip to content

Conversation

diosmosis
Copy link
Member

@diosmosis diosmosis commented Feb 10, 2022

Description:

As title. I thought of skipping the UMD files, but I'm not sure why the test was originally added, and it looked related to security. Since   could be easily needed in the future it also doesn't seem acceptable to just never use it.

I found the place where this happens in the vue compiler and found a way to patch it.

Review

@diosmosis diosmosis added c: Tests & QA For issues related to automated tests or making it easier to QA & test issues. Needs Review PRs that need a code review labels Feb 10, 2022
@diosmosis diosmosis added this to the 4.8.0 milestone Feb 10, 2022
@peterhashair
Copy link
Contributor

@diosmosis it still failed https://app.travis-ci.com/github/matomo-org/matomo/jobs/559193206. I guess just replace<p>&nbsp;</p> to <br/> maybe

@diosmosis
Copy link
Member Author

@peterhashair that will fix it now, but it means using &nbsp; in vue templates is not allowed which seems like it could become a problem.

@diosmosis
Copy link
Member Author

I can find one simple way to fix this but it the result still requires remembering and explaining something, so I don't think its much better than having to explain that   cannot be used directly in templates. Will close this.

@diosmosis diosmosis closed this Feb 10, 2022
@diosmosis diosmosis deleted the vue-ignore-space-check-vue-umd branch February 10, 2022 22:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c: Tests & QA For issues related to automated tests or making it easier to QA & test issues. Needs Review PRs that need a code review
Development

Successfully merging this pull request may close these issues.

2 participants