Small bug on MS3 - ejGridAdvanced.mipage

[woody]

Greetings MS3 development team.

I spotted this little bug while testing the current release of MS3 (MiServer 3.0).

1. Edit the grid
2. Change value in a cell
3. Click SAVE icon (or click away to another cell)
4. SYNTAX ERROR

FYI.

Cheers,

//W

[Brian|Dyalog]

Hi Woody!

This was an interesting issue because it looked like the code expected the data to have been already parsed as JSON and converted to APL form, but, instead we were just getting a character vector of the raw JSON. I think this may have been a result of the work we did a while back to rationalize event handling across all types of elements (HTML, jQuery widgets, Syncfusion widgets, and other widget libraries).

The interesting problem is how to distinguish data that should be interpreted as JSON versus something that someone might, for instance, type into a text input field. Is '"Test"' JSON or just a character vector with double quotes? So, I've added some additional intelligence when we know we're generating JSON data in the client that the server can leverage to translate things properly.

I've made the change and pushed it to the master MiServer branch on GitHub.

/Brian

[gil]

I haven't looked at the innards of MS3, but wouldn't the http header define the content type?
When would you need to guess the type?

[Brian|Dyalog]

Hi Woody,

jQuery's ajax() function, which we use for handlers and callbacks, defaults to a MIME type of 'application/x-www-form-urlencoded; charset=UTF-8'. We could potentially set other MIME types (like 'application/json'), but that would require some recoding of the server side of event and callback handling.

/Brian