[ 
https://issues.apache.org/jira/browse/OLINGO-811?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15005787#comment-15005787
 ] 

Michael Bolz edited comment on OLINGO-811 at 11/16/15 1:59 PM:
---------------------------------------------------------------

Hi [~riccardomariani],

I made some minor fixes, added an additional test and merged your contribution 
into the feature branch (see [this 
commit|https://git1-us-west.apache.org/repos/asf?p=olingo-odata4.git;a=commit;h=8bd685b6ffc72a3c62a8cbdfa5e5568d7cb4094e]).

There is still the issue with an error for 
{{$expand=NavPropertyETTwoPrimMany/$count}} (sample url for local TecSvc: 
[/ESAllPrim/?$expand=NavPropertyETTwoPrimMany/$count=true|http://localhost:8080/test-svc/odata.svc/ESAllPrim/?$expand=NavPropertyETTwoPrimMany/$count]).
But I think this is based on no support for {{/$count}} within the URI parser.
Furthermore there are failing integration tests which should be checked (e.g. 
{{org.apache.olingo.fit.base.EntityCreateTestITCase}}).

Best Regards,
Michael


was (Author: mirbo):
Hi [~riccardomariani],

I made some minor fixes, added an additional test and merged your contribution 
into the feature branch (see [this 
commit|https://git1-us-west.apache.org/repos/asf?p=olingo-odata4.git;a=commit;h=8bd685b6ffc72a3c62a8cbdfa5e5568d7cb4094e].

There is still the issue with the {{The URI is malformed.}} error for 
{{$expand=NavPropertyETTwoPrimMany/$count=true}} (sample url for local TecSvc: 
[/ESAllPrim/?$expand=NavPropertyETTwoPrimMany/$count=true|http://localhost:8080/test-svc/odata.svc/ESAllPrim/?$expand=NavPropertyETTwoPrimMany/$count=true]).
But I think this is based on no support for {{/$cound}} within the URI parser.
Furthermore there are failing integration tests which should be checked (e.g. 
{{org.apache.olingo.fit.base.EntityCreateTestITCase}}).

Best Regards,
Michael

> Use of $count for expanded items does not work
> ----------------------------------------------
>
>                 Key: OLINGO-811
>                 URL: https://issues.apache.org/jira/browse/OLINGO-811
>             Project: Olingo
>          Issue Type: Bug
>          Components: odata4-server
>    Affects Versions: (Java) V4 4.0.0
>            Reporter: Michael Bolz
>            Assignee: Michael Bolz
>         Attachments: patch-Nov-09-2015.diff
>
>
> The {{$count}} query option within the {{$expand}} clause does not work 
> correctly (see in 
> [Specification|http://docs.oasis-open.org/odata/odata/v4.0/errata02/os/complete/part1-protocol/odata-v4.0-errata02-os-part1-protocol-complete.html#_The_$inlinecount_System]
> Example URL (for *TecSvc*):
> http://localhost:8080/odata.svc/ESAllPrim/?$expand=NavPropertyETTwoPrimMany($count=true)&$format=json&$count=true
> And example for OData-TripPin Service:
> [http://services.odata.org/V4/(S(kb3m31znaqv0xd4oqq0esxfv))/TripPinServiceRW/People?$expand=Trips($count=true)]



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to