Skip to content
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

HTML syntax error - "ampersanded" ampersand #16286

Closed
supervisitor opened this issue Aug 11, 2020 · 5 comments
Closed

HTML syntax error - "ampersanded" ampersand #16286

supervisitor opened this issue Aug 11, 2020 · 5 comments
Labels
answered For when a question was asked and we referred to forum or answered it.

Comments

@supervisitor
Copy link

<noscript><p><img src="//piwik-server/piwik/matomo.php?idsite=1&rec=1" style="border:0;" alt="" /></p></noscript>

in line 68, you should delete character 89 to 92 (amp;), this line should include ...matomo.php?idsite=1&rec=1... and not contain a ampersanded ampersand ...matomo.php?idsite=1&rec=1...

@tsteur
Copy link
Member

tsteur commented Aug 11, 2020

@supervisitor thanks for creating the issue. Do you have an actual issue with the tracker generator? the double escaped & might be expected there.

@tsteur tsteur added the Waiting for user feedback Indicates the Matomo team is waiting for feedback from the author or other users. label Aug 11, 2020
@supervisitor
Copy link
Author

supervisitor commented Aug 12, 2020

I think twice when I saw the result & in the generated tracker code.
Since then, I have been considering " and " on the same line... yes, I know the difference an reason for this &, but somewhere I suspect a nesting error.

I will double check tomorrow and come back with a more detailed report.

@tsteur
Copy link
Member

tsteur commented Aug 13, 2020

Thanks. I just checked the regular output and there things look good see eg https://demo.matomo.org/index.php?module=API&format=json&method=SitesManager.getImageTrackingCode&idSite=62&period=day&date=yesterday and embedding eg

    <img src="//example.matomo/matomo.php?idsite=1&amp;rec=0" style="border:0" alt="" />

worked for me. It's only that in the tests it is encoded like this.

@supervisitor
Copy link
Author

Sorry, I'm not one to just carelessly submit an issue report. Please close it, this error happens only in a very special version and only installed here. 😳
Apologize again for the trouble I have caused.

@Findus23 Findus23 added answered For when a question was asked and we referred to forum or answered it. and removed Waiting for user feedback Indicates the Matomo team is waiting for feedback from the author or other users. labels Aug 15, 2020
@Findus23
Copy link
Member

No problem 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
answered For when a question was asked and we referred to forum or answered it.
Projects
None yet
Development

No branches or pull requests

3 participants