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.

Sign up for GitHub

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

Jump to bottom

Set Doctype for xml file without schemas #61

Closed
apupier opened this issue Oct 13, 2017 · 3 comments
Closed

Set Doctype for xml file without schemas #61

apupier opened this issue Oct 13, 2017 · 3 comments
Labels
bug
Milestone
4.9.0

Comments

@apupier
Copy link
Contributor

apupier commented Oct 13, 2017

it avoids to have warnings in Eclipse
apupier added a commit to apupier/wildfly-camel-examples that referenced this issue Oct 13, 2017
@apupier
wildfly-extras#61 - set DocType xml for file without schema
cd22ba5 
Signed-off-by: Aurélien Pupier <apupier@redhat.com>
@apupier
Copy link
Contributor Author

apupier commented Oct 13, 2017

I fixed several warnings with this PR #63
but there are still several files reporting errors:
"No grammar constraints (DTD or XML Schema) referenced in the document."

image

they contain a schemalocation without a real location, is it a fake schemalocation? Do we really need it?

@jamesnetherton
Copy link
Contributor

Probably not needed. Think I took the XML structure from another Fuse / Camel quickstart.

If the tests pass with it removed, I'm fine with removing it.

@jamesnetherton jamesnetherton added this to the 4.9.0 milestone Oct 16, 2017
jamesnetherton pushed a commit to apupier/wildfly-camel-examples that referenced this issue Oct 16, 2017
@apupier @jamesnetherton
[resolves wildfly-extras#61] set DocType for file without schema
1b066b3
@jamesnetherton
Copy link
Contributor

Thanks for pointing this out. I fixed up the erroneous schema locations in the other files.

I also corrected the !DOCTYPE declaration. Think it needs to be <!DOCTYPE root_element_name> instead of <!DOCTYPE xml>.

tdiesler pushed a commit that referenced this issue Oct 18, 2017
@apupier @tdiesler
[resolves #61] set DocType for file without schema
6325e14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
4.9.0
Development

No branches or pull requests
2 participants
@apupier @jamesnetherton