You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The GET ..../info print out is wrong for the return field urlPart, missing first semicolon between ID section and first filter section.
Setup
There is a standard resource, say BookResource, which we can call GET on using restler.
How to trigger the bug
Do an HTTP call to retrieve by ID ("-" does not trigger this bug), and with at least one segment filter
GET /books/1;publisher=2;id__ne=3/info
Look at the return body:
{
... "urlPart": "1id <>=3;publisher=2",
...
}
See now that there is a missing semicolon ";" between the ID and the first segment filter id <>=... After the second one, all semicolons are included as they should
This is annoying of course as it makes it harder to read this debug info.
Where the problem comes from
In restdsl.queries.ServiceQuery.toUrlPart() the code only adds the first semicolon if the ID section is the "-" match all. Otherwise it forgets to add the first semicolon.
Possible solution
Add a first semicolon in all cases when criteria is not null.
The text was updated successfully, but these errors were encountered:
The
GET ..../info
print out is wrong for the return fieldurlPart
, missing first semicolon between ID section and first filter section.Setup
There is a standard resource, say
BookResource
, which we can call GET on using restler.How to trigger the bug
Do an HTTP call to retrieve by ID ("-" does not trigger this bug), and with at least one segment filter
Look at the return body:
See now that there is a missing semicolon ";" between the ID and the first segment filter
id <>=..
. After the second one, all semicolons are included as they shouldThis is annoying of course as it makes it harder to read this debug info.
Where the problem comes from
In restdsl.queries.ServiceQuery.toUrlPart() the code only adds the first semicolon if the ID section is the "-" match all. Otherwise it forgets to add the first semicolon.
Possible solution
Add a first semicolon in all cases when criteria is not null.
The text was updated successfully, but these errors were encountered: