feature request #4644
task #6992: Refactor and modernize REST web service API
task #4009: consider using jackson as json library
json lib incorrectly serializes values containig the characters [, ], {, }
0%
Description
for example in the cichorieae Hedypnois Mill. has the nomenclaturalMicroReference @[606]@, which is serialized to json as an array:
"nomenclaturalMicroReference": [ 606 ],
https://test.e-taxonomy.eu/cdmserver/cichorieae/name/4da14cc4-fb0b-43a2-a909-82535363f14e.json
this causes the a php warning to appear in the portal, like:
Warning: urlencode() expects parameter 1 to be string, array given in cdm_ws_getNomenclaturalReference() (line 1500 of /var/www/drupal-7.32/sites/_dataportal-production/modules/cdm_dataportal/cdm_api/cdm_api.module).
This is a general problem in json-lib, an should be solved in a general way, e.g. by upgrading to a newer version?
TODO
- remove cdm-dataportal|ecb8526 (svn: r28125) avoiding the red error box as long #4644 is not fixed
Related issues
History
#1 Updated by Andreas Kohlbecker about 6 years ago
- Target version deleted (
cdmlib - Next Major Release)
#2 Updated by Andreas Müller about 6 years ago
- Target version set to cdmlib - Next Major Release
#3 Updated by Andreas Kohlbecker over 3 years ago
- Description updated (diff)
- Private changed from Yes to No
#4 Updated by Andreas Kohlbecker over 2 years ago
- Duplicated by bug #7973: Pages not shown in portal if no digit? added
#5 Updated by Andreas Kohlbecker 5 months ago
- Related to task #9265: Adapt dataportal to new protologue handling added
#6 Updated by Andreas Kohlbecker 5 months ago
- Description updated (diff)
#7 Updated by Andreas Kohlbecker 5 months ago
- Description updated (diff)
#8 Updated by Andreas Kohlbecker 5 months ago
- Status changed from New to Rejected
- Target version deleted (
cdmlib - Old Next Major Release)
this is no longer relevant since we no longer need to get the microreference separately aa06c1f1a2
The hack mentioned above is removed already.
#9 Updated by Andreas Kohlbecker 5 months ago
- Status changed from Rejected to New
- Priority changed from New to Priority14
- Target version set to Release 5.19
the above mentioned commit had to be reverted, so this issue is open again
#10 Updated by Andreas Kohlbecker 3 months ago
- Target version changed from Release 5.19 to Release 5.21
#11 Updated by Andreas Müller about 1 month ago
- Target version changed from Release 5.21 to Release 5.22