Aalhad Saraf
b7467941a1
Handle leading whitespace in JSON body
Issue:
When sending JSON in the body of a POST
request from an HTML form - for creating
a new record, a new record was being created,
but with null as the value for all the fields,
instead of the values supplied.
Cause:
The template text in the textarea field, in the
HTML form had some leading whitespace. On looking
through the source, json_decode is being called
only if the first character of the $data variable
is a '{' or a '['.
JSON Specification RFC4627
https://tools.ietf.org/html/rfc4627#section -2
says that insignificant
whitespace is allowed before or after any of the
six structural characters - '{','[',']','}',':',','
where whitespace is defined as:
ws = *(
%x20 / ; Space
%x09 / ; Horizontal tab
%x0A / ; Line feed or New line
%x0D ; Carriage return
)
Fix:
trim the above characters from the beginning and ending
of the received data before checking that the first
character is a '[' or '{'
7 years ago
Gabriel Caruso
d500ba331e
Use PHPUnit\Framework\TestCase instead of PHPUnit_Framework_TestCase
7 years ago
Maurits van der Schee
9bbc5bbfeb
Update README.md
7 years ago
Maurits van der Schee
0a1c6ea6d6
Merge branch 'master' of github.com:mevdschee/php-crud-api
7 years ago
Maurits van der Schee
a534923d90
Add test case for Escaped JSON Issue #276
7 years ago
Maurits van der Schee
74bc8c2c43
Merge pull request #308 from xhorntail/master
Update api.php to fix API documentation
7 years ago
xhorntail
bbbfbb40f4
Update api.php
7 years ago
Maurits van der Schee
088df6db09
Merge pull request #296 from jpsider/Update_Core_Delete
Updated the Delete command in the core.php file
7 years ago
jpsider
cd6962e9e4
Updated the Delete command in the core.php file
7 years ago
Maurits van der Schee
dcb2088be3
Add batch build and run
7 years ago
Maurits van der Schee
febcabce43
add debian 7
7 years ago
Maurits van der Schee
8a7e81e00f
add debian 8
7 years ago
Maurits van der Schee
608dc3eba6
Merge branch 'master' of github.com:mevdschee/php-crud-api
7 years ago
Maurits van der Schee
2e73e1a48e
Add debian 9
7 years ago
Maurits van der Schee
479ce82aed
Update README.md
7 years ago
Maurits van der Schee
e87b65b7d3
Update README.md
7 years ago
Maurits van der Schee
bde6329581
Update README.md
7 years ago
Maurits van der Schee
8651f83376
Update README.md
7 years ago
Maurits van der Schee
47c09cdd6f
Added centos7 to docker
7 years ago
Maurits van der Schee
d5ee2625a7
Allow custom command in docker run
7 years ago
Maurits van der Schee
19efa9a0c3
Remove infinite sleep
7 years ago
Maurits van der Schee
dd34981cdd
small improvement
7 years ago
Maurits van der Schee
37c360f27f
Add ubuntu docker images for tests
7 years ago
Maurits van der Schee
f3183e24e0
Fix tests
7 years ago
Maurits van der Schee
c5e0074341
Make variable replacement possible
7 years ago
Maurits van der Schee
25afd1c633
improve docs
8 years ago
Maurits van der Schee
1bd7750d8c
improve docs
8 years ago
Maurits van der Schee
0eede2255f
Merge branch 'master' of github.com:mevdschee/php-crud-api
8 years ago
Maurits van der Schee
5fc993d967
improve docs
8 years ago
Maurits van der Schee
64b8f1ea77
Update README.md
8 years ago
Maurits van der Schee
690ec3641f
Merge pull request #248 from karllhughes/master
Removing DBs as services
8 years ago
Karl Hughes
e866b469b6
Updating travis
8 years ago
Karl Hughes
baab9a5849
Updating sudo
8 years ago
Karl Hughes
a158976e67
Updating sudo
8 years ago
Karl Hughes
29e0dd364f
Updating sudo
8 years ago
Maurits van der Schee
73e2d88e6e
bugfix for postgis detection
8 years ago
Maurits van der Schee
13d56e9d48
bugfix for postgis detection
8 years ago
Maurits van der Schee
dff574ac7d
bugfix
8 years ago
Maurits van der Schee
1b31cd102b
Improve error message
8 years ago
Maurits van der Schee
857a0407d7
Codestyle fixes (automatically)
8 years ago
Maurits van der Schee
2a2e49eb6c
PSR-4 comliant tests
8 years ago
Maurits van der Schee
211801a4d3
PSR-4 comliant tests
8 years ago
Maurits van der Schee
cca56e7859
Prepare for PSR compliant namespacing
8 years ago
Maurits van der Schee
9b141c53c5
Add mysql tests
8 years ago
Maurits van der Schee
fb82dfbd5d
bugfix
8 years ago
Maurits van der Schee
77b1c06084
Remove reflection
8 years ago
Maurits van der Schee
ee1e2e3383
Remove reflection
8 years ago
Maurits van der Schee
9b28263521
Cleanup after merge of #244
8 years ago
Maurits van der Schee
4a517b9fa4
Merge pull request #244 from karllhughes/master
Making mysql and postgres test suites adaptable
8 years ago
Maurits van der Schee
052934796a
Merge branch 'master' into master
8 years ago