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

Restrict model class name charset #32

Open
Lythimus opened this issue Nov 13, 2013 · 0 comments
Open

Restrict model class name charset #32

Lythimus opened this issue Nov 13, 2013 · 0 comments

Comments

@Lythimus
Copy link

Using HL7 XSD schema https://dl.dropboxusercontent.com/u/8709992/HL7.xsd, I found that elements with periods in their name attributes were converted into classes with periods in their names, which is illegal syntax based on the PHP standard. I recommend name attributes be processed and substituted with legal characters as this is probably a general issue, not restricted to periods.

I do not receive an error or warning when generating the PHP classes from the XSD file. Rather PHP throws an error when trying to load the files.

This library seems good, but unfortunately due to lack of documentation and minor hangups, I'm going to parse my XML files manually. Not to dissuade you from working on the tool. But I've found most people have a low threshold for incorporating these sorts of libraries and will quickly fall back to one-off hacks due to time constraints.

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

No branches or pull requests

1 participant