-
Notifications
You must be signed in to change notification settings - Fork 122
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
encoder error #160
Comments
Please verify if the problem still remains with phpDox 0.7.0. In case it does, please reopen / add a comment to this issue. |
New try with current version
same result...
|
I added a testcase for this issue based on the error message and the byte sequence libxml complains about. If I run a xmllint check on an xml file with that sequence i get the exact same error, which is correct and expected:
I fail to reproduce your problem with phpDox though as the problematic bytes should and get replaced:
Can you you verify if said testcase works or fails on your system? Would it be possible to get a (potentially stripped down) version of the file causing the crash on your system? |
Here it is:
|
Thank you for verifying and the version details.
If that does not crash as it did with your codebase, I'd really like to have a (stripped down) version of the file that makes it crash. |
Too bad - it doesn't crash. It's a big project where it failes. How can I find which file might be corrupt?
|
I was having the same problem but with another byte sequence. I was able to locate the files with
It was a file with some Swedish characters in it. |
@mattriverm Swedish charecters are part of UTF-8 last I checked thus shouldn't be a problem. Would it be possible to get the/a file which crashes phpDox (or a stripped down version)? @guckykv Any updates? Does the for mentioned tip with |
Among others, it was this one |
@mattriverm The file you mentioned is a JavaScript file which gets ignored by phpDox. It's thus very unlikely to cause any issues let alone a crash. I just cloned the repository and at least it's current state doesn't cause any issues on my system. Can you point me to a version/commit where it crashes? |
@theseer I've tried the find, but got 'fgrep: illegal byte sequence' while searching for the sequence phpdox complains about (0xA0 0x7B 0x40 0x69). But today, I've got more info from phpdox:
With this starting point I've found the illegal character:
The space in front of "{@" looks like a space (0xA0 - non breaking space), but wasn't one. After replacing it with a real space, phpdox can parse the whole project! But in jenkins environment I've got a new error: See issue #166 Thanks! |
Thank you for the feedback. |
Just to give you a heads up: I finally managed to reproduce this issue and am working on a fix. |
I am facing the same problem but with a bit different codes:
|
My problem was solved by fixing comment from
To
|
Tried latest version - installed via composer "theseer/phpdox": "*"
The text was updated successfully, but these errors were encountered: