Encoding issue when requesting teacher list about th-admin HOT 3 CLOSED

This is one of those things that doesn't (currently) break with the mock server, but which does (judging by this issue) break with the real server .

I propose I do 3 things:

1. Fix this exact issue
2. Look for and fix any similar issues
3. Change the demo server so that it breaks if parameters aren't serialized - i.e. so that I don't create similar issues in the future

Sound good?

from th-admin.

Fully agree!
You can also check out the post submissions for /admin/service/teacher-process method as guidance - that one works perfect!

On 3 Jul 2013, at 02:22, Ryan Randall [email protected] wrote:

This is one of those things that doesn't (currently) break with the mock server, but which does (judging by this issue) break with the real server .

I propose I do 3 things:

1. Fix this exact issue
2. Look for and fix any similar issues
3. Change the demo server so that it breaks if parameters aren't serialized - i.e. so that I don't create similar issues in the future

Sound good?

—
Reply to this email directly or view it on GitHub.

from th-admin.

(for future reference)

Research:

R1) http://stackoverflow.com/questions/12190166/angularjs-any-way-for-http-post-to-send-request-parameters-instead-of-json
R2) angular/angular.js#738
R3) http://stackoverflow.com/questions/1131630/the-param-inverse-function-in-javascript-jquery

Action:

A1) Implemented as in R1 (i.e. globally), and made to work with all $http requests (this covers proposals 1 and 2). Did the same for the API viewer. Since this is global, proposal 3 is no longer necessary.
A2) Implemented R3 in serverServices.js. Will keep an eye on R2 and may substitute that in later.

from th-admin.