[aerogear-dev] [Doc] AerogearAndroidPipes101

Apostolos Emmanouilidis aemmanou at redhat.com
Mon Jun 3 04:28:58 EDT 2013


On Mon, 2013-06-03 at 10:09 +0200, Matthias Wessendorf wrote:
> 
> 
> 
> 
> 
> On Mon, Jun 3, 2013 at 10:01 AM, Apostolos Emmanouilidis
> <aemmanou at redhat.com> wrote:
> 
>         On Mon, 2013-06-03 at 09:54 +0200, Matthias Wessendorf wrote: 
>         
>         > 
>         > 
>         > 
>         > On Mon, Jun 3, 2013 at 9:48 AM, Sebastien Blanc
>         > <scm.blanc at gmail.com> wrote:
>         > 
>         >         
>         >         
>         >         
>         >         On Sat, Jun 1, 2013 at 3:56 PM, Apostolos
>         >         Emmanouilidis <aemmanou at redhat.com> wrote: 
>         >         
>         >                 
>         >                 On Sat, 2013-06-01 at 11:45 +0200, Matthias
>         >                 Wessendorf wrote: 
>         >                 
>         >                 > Hi,
>         >                 > 
>         >                 > 
>         >                 > reading [1], I have two questions:
>         >                 > 
>         >                 > 
>         >                 > * the "http://www.server.com/car" URL,
>         >                 > shouldn't it be
>         >                 > "http://www.server.com/cars" (plural)? 
>         >                 > ==> and the later mentioned HTTP methods,
>         >                 > e.g. "GET:/car" ==> "GET:/cars"?
>         >                 
>         >                 
>         >                 
>         >                 I agree that it should be plural 
>         >                 
>         >                 
>         >                 
>         >                 > 
>         >                 > 
>         >                 > * Do we really promote the direct access
>         >                 > to the RestAdapter, instead of via the
>         >                 > Pipeline ?
>         >                 > ==> Pipe<Car> pipe = new
>         >                 > RestAdapter<pipe>(Car.class, new
>         >                 > URL("http://www.server.com/"));
>         >                 > 
>         >                 
>         >                 
>         >                 
>         >                 Except that, maybe this line should change
>         >                 to Pipe<Car> pipe = new
>         >                 RestAdapter<Car>(Car.class, new
>         >                 URL("http://www.server.com/car")); in order
>         >                 to match the URLs of the described web
>         >                 service. 
>         >                 
>         >         
>         >         
>         >         
>         >         
>         >         
>         >         I don't know the android details implementation but
>         >         if we define the URL like and we call a read after
>         >         that like pipe.read() won't we end up with an url
>         >         like that  http://www.server.com/car/car ?   
>         > 
>         > 
>         > 
>         > yeah, that statement was wrong. I was more concerned,
>         > promoting "impl details (RestAdapter) instead of using
>         > "public" API: pipeline.pipe(.....);
>         
>         
>         
>         
>         
>         I think that when using the RestAdapter constructor, the
>         Entity name is not added to the URL. I'm pretty sure at least
>         for the pipe.remove method.
> 
> 
> 
> 
> The pipe should be created by using Pipeline.pipe()   - not dircetly
> accessing the RestAdapter (impl details)


I was referring to the current documentation as is. I agree that
Pipeline.pipe should be used instead of direct accessing the
RestAdapter. Daniel Passos informed me about that in a relevant JIRA. 

>  
>         
>         
>         
>         > 
>         > 
>         > 
>         > 
>         > -Matthias
>         > 
>         > 
>         > 
>         > 
>         > 
>         > 
>         >  
>         > 
>         >                 
>         >                 
>         >                 > 
>         >                 > Thanks!
>         >                 > Matthias
>         >                 > 
>         >                 > 
>         >                 > 
>         >                 > 
>         >                 > [1] aerogear.org/docs/guides/aerogear-android/AerogearAndroidPipes101/
>         >                 > 
>         >                 > 
>         >                 > 
>         >                 > _______________________________________________
>         >                 > aerogear-dev mailing list
>         >                 > aerogear-dev at lists.jboss.org
>         >                 > https://lists.jboss.org/mailman/listinfo/aerogear-dev
>         >                 
>         >                 
>         >                 
>         >                 
>         >                 _______________________________________________
>         >                 aerogear-dev mailing list
>         >                 aerogear-dev at lists.jboss.org
>         >                 https://lists.jboss.org/mailman/listinfo/aerogear-dev 
>         >         
>         >         
>         >         
>         >         
>         >         _______________________________________________
>         >         aerogear-dev mailing list
>         >         aerogear-dev at lists.jboss.org
>         >         https://lists.jboss.org/mailman/listinfo/aerogear-dev 
>         > 
>         > 
>         > 
>         > 
>         > 
>         > -- 
>         > Matthias Wessendorf 
>         > 
>         > blog: http://matthiaswessendorf.wordpress.com/
>         > sessions: http://www.slideshare.net/mwessendorf
>         > twitter: http://twitter.com/mwessendorf
>         > 
>         > _______________________________________________
>         > aerogear-dev mailing list
>         > aerogear-dev at lists.jboss.org
>         > https://lists.jboss.org/mailman/listinfo/aerogear-dev
>         
>         
>         
>         
>         
>         _______________________________________________
>         aerogear-dev mailing list
>         aerogear-dev at lists.jboss.org
>         https://lists.jboss.org/mailman/listinfo/aerogear-dev
> 
> 
> 
> 
> 
> 
> 
> -- 
> Matthias Wessendorf 
> 
> blog: http://matthiaswessendorf.wordpress.com/
> sessions: http://www.slideshare.net/mwessendorf
> twitter: http://twitter.com/mwessendorf
> 
> _______________________________________________
> aerogear-dev mailing list
> aerogear-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/aerogear-dev


-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/aerogear-dev/attachments/20130603/a3c6ad17/attachment-0001.html 


More information about the aerogear-dev mailing list