-
Notifications
You must be signed in to change notification settings - Fork 93
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
XML completion based on internal DTD #251
Labels
completion
This issue or enhancement is related to completion support
DTD
enhancement
New feature or request
Milestone
Comments
angelozerr
added
enhancement
New feature or request
completion
This issue or enhancement is related to completion support
DTD
labels
Dec 4, 2018
angelozerr
added a commit
that referenced
this issue
Dec 5, 2018
angelozerr
added
the
needs tests
This issue is probably fixed but needs regression tests
label
Dec 5, 2018
angelozerr
added a commit
that referenced
this issue
Dec 5, 2018
@fbricon it should work now. I have written tests. You can play with this XML:
<?xml version = "1.0"?>
<!DOCTYPE Folks [
<!ELEMENT Folks (Person*)>
<!ELEMENT Person (Name,Email?)>
<!ATTLIST Person Pin ID #REQUIRED>
<!ATTLIST Person Friend IDREF #IMPLIED>
<!ATTLIST Person Likes IDREFS #IMPLIED>
<!ELEMENT Name (#PCDATA)>
<!ELEMENT Email (#PCDATA)>
]>
<Folks>
| // here completion should provide Person element
</Folks>
<?xml version = "1.0"?>
<!DOCTYPE Folks [
<!ELEMENT Folks (Person*)>
<!ELEMENT Person (Name,Email?)>
<!ATTLIST Person Pin ID #REQUIRED>
<!ATTLIST Person Friend IDREF #IMPLIED>
<!ATTLIST Person Likes IDREFS #IMPLIED>
<!ELEMENT Name (#PCDATA)>
<!ELEMENT Email (#PCDATA)>
]>
<Folks>
<Person | // here completion should provide Pin , Friend, Likes attributes
</Folks> Please note that completion works only if DTD must is not malformed (like external DTD) Please close the issue if it works for you. |
@fbricon could you play with feature and close it, if you think it's good for you. Thanks! |
angelozerr
removed
in progress
needs tests
This issue is probably fixed but needs regression tests
labels
Dec 7, 2018
@fbricon please reopen it if you find bugs. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
completion
This issue or enhancement is related to completion support
DTD
enhancement
New feature or request
No description provided.
The text was updated successfully, but these errors were encountered: