Skip to content

Django Toolbar XML fix #793

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

Merged
merged 1 commit into from
Dec 28, 2015
Merged

Django Toolbar XML fix #793

merged 1 commit into from
Dec 28, 2015

Conversation

edelbluth
Copy link
Contributor

The SQL Panel produces errors with XHTML documents, because of the href attribute in some links without a content. As result, the SQL Panel stays empty and an error occurs on the console.

This little fix sets the attributes to an empty content, which is valid XHTML.

The SQL Panel produces errors with XHTML documents, because of the `href` attribute in some links without a content. As result, the SQL Panel stays empty and an error occurs on the console.

This sets the attributes to an empty content, which is valid XHTML.
aaugustin added a commit that referenced this pull request Dec 28, 2015
@aaugustin aaugustin merged commit 8ccf09d into django-commons:master Dec 28, 2015
@edelbluth edelbluth deleted the XML_Fix branch December 28, 2015 15:38
ryneeverett pushed a commit to ryneeverett/django-debug-toolbar that referenced this pull request Oct 2, 2016
ryneeverett pushed a commit to ryneeverett/django-debug-toolbar that referenced this pull request Oct 2, 2016
Like in Pull Request django-commons#793, the Timing Panel produces an error when used in XHTML documents. This time, the entity   is undefined for XHTML documents.

It was replaced by &django-commons#160; – which is the numeric pendant. Now the panel works for all kind of documents.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants