Community
Participate
Working Groups
Build Identifier: The RIO providers were not passing the ChangeRequestRdfXmlTests due to the result rdfs:members occupying an rdf:Description section with an about referring to the query itself, not the query base URL Please assign to Michael Fiedler Reproducible: Always
Created attachment 205955 [details] Update buildResponseResource to put query results in own element with correct about I wrote all this code and have the rights to contribute it to Eclipse under the eclipse.org web site terms of use. Commits for this bug make changes to the buiildResponseResource for all 3 RIOs. - put query result set in its own rdf:Description element with the proper about - add an oslc:totalCount to the query description section. The CM RdfXML query tests pass now.
Getting null at end of URI for query URI. For example when I use the queryBase URL without any oslc.where parameters such as: GET http://localhost:8080/rio-cm/changerequest I get a response that looks like: <rdf:Description rdf:about="http://localhost:8080/rio-cm/changerequest?null"> Notice the extra trailing "?null". This is one of the rare cases where the query URI == the container URI, so all predicates fit under 1 subject URI.
The base RIO code was appending the ?null, so will fix that. But, the proposed patch still was not considering the queryBase case correctly, so will fix that as well. In my jetty test env, I am also seeing the case where the first GET on the queryBase as a url parameter that we need to ignore...i.e. make sure there is really an OSLC query parameter. e.g. http://localhost:8080/rio-cm/changerequest?url=http%3A%2F%2Flocalhost%3A8080%2Frio-cm%2Fchangerequest
Created attachment 205994 [details] Additional updates to query parm handling 1) Don't add ?null to the queryUri when no parms exist 2) Put query predicates under the baseQuery subject when baseQuery used 3) handle case when all parameters are non-OSLC parameters
Committed http://git.eclipse.org/c/lyo/org.eclipse.lyo.rio.git/commit/?id=1090e036e9cd839651127763d08a197029e38806 and http://git.eclipse.org/c/lyo/org.eclipse.lyo.rio.git/commit/?id=cbe6129f53c06c080f1aa36f6b699a1d70e4a02f
Bulk update: Closing out Lyo bugs fixed long ago. Feel free to reopen if you feel your bug was closed in error.
