We could though technically the extra ',' is allowed by the JSON spec and there's a reasonable chance you'll get JSON with these artifacts coming in from 3rd parties on the RESTful API. What are the library options on the PHP side to have a more tolerant parser.
For comments we can certainly adopt a convetion to use comments on newlines only within Shindig if that helps. On Wed, Jul 23, 2008 at 8:32 AM, Ropu <[EMAIL PROTECTED]> wrote: > yeap, thats an issue of the parser. > > we had the same problem with config/oauth.json > > comments can be removed programatically, but extra ',' no. > > can we standardize that with the JAVA guys? > > ropu > > On Wed, Jul 23, 2008 at 12:01 PM, Gonzalo Aune (JIRA) <[EMAIL PROTECTED]> > wrote: > > > > > [ > > > https://issues.apache.org/jira/browse/SHINDIG-469?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel > ] > > > > Gonzalo Aune updated SHINDIG-469: > > --------------------------------- > > > > Attachment: canonicaldb-fix.patch > > > > This is the patch fixing the db.json, it seems like it doesnt accept > > comments, and there are a couple of errors in some lines with an extra > "," > > like: > > > > "body" : "what a color!", > > > > and > > > > "unstructuredAddress" : "1 Edu St.", > > > > The patch fix these lines and removes the comments, without these > changes, > > PHP json function doesnt decode the string. Is there any possibilities to > > commit this and add the comments in an README file? > > > > G.- > > > > > Error parsing canonicaldb.json in PHP > > > ------------------------------------- > > > > > > Key: SHINDIG-469 > > > URL: https://issues.apache.org/jira/browse/SHINDIG-469 > > > Project: Shindig > > > Issue Type: Bug > > > Components: RESTful API (Java), RESTful API (PHP) > > > Reporter: Gonzalo Aune > > > Attachments: canonicaldb-fix.patch > > > > > > > > > I had a couple of errors trying to decode the json db in PHP, it simply > > dont parse. > > > > -- > > This message is automatically generated by JIRA. > > - > > You can reply to this email to add a comment to the issue online. > > > > > > > -- > .-. --- .--. ..- > R o p u >
