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

Invalid value for @language error #48

Open
warpr opened this issue Nov 2, 2013 · 3 comments
Open

Invalid value for @language error #48

warpr opened this issue Nov 2, 2013 · 3 comments

Comments

@warpr
Copy link

warpr commented Nov 2, 2013

While playing with the Hydra Console I came across this issue, which looks like a bug in your JSON-LD processor:

PHP Fatal error:  Uncaught exception 'ML\JsonLD\Exception\SyntaxException'
with message 'Invalid value for @language detected (must be a string) (near 
{"@language":{"__iri":"@language","__value":"ja"},"@value":
{"__iri":"@value","__value":"リズム天国ゴールド"}}).'

The original document does have just a string "ja" as value for @language, so it seems the processor is complaining about an expansion it performed itself.

I've uploaded a testcase which shows the problem using your HydraConsole proxy:

http://www.markus-lanthaler.com/hydra/console/proxy.php?debug=true&url=https://frob.nl/tmp/jsonld-language-bug/gmb.jsonld

@lanthaler
Copy link
Owner

The Hydra Console uses a modified processor in order to render the tooltips for the JSON-LD response. It expands the document while at the same time keeping the original structure. You can find that version of the processor in the “expansion-debug” branch. I haven’t updated it for a while now. I planned to refactor my processor quite a bit which should then simplify such things. How important is this to you? If it’s a deal breaker I’ll try to fix it now otherwise I’ll probably concentrate on other things first.

@warpr
Copy link
Author

warpr commented Nov 2, 2013

I'm just experimenting / prototyping, none of what I'm working on is in production -- so there's no hurry.

@lanthaler
Copy link
Owner

OK, great. Let me know when that changes :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants