Hi, AGStore defines the following func: -(NSArray*)filter:(id)filterObject; Right now, there is NO implementation, so I'd vote to remove it.... Otherwise we will have this function around in future releases... If (later) we want some filtering (e.g. when more complex stores are around), we can define a proper API - instead of having some 'guess-ware' around... FWIW, I created this ticket: https://issues.jboss.org/browse/AEROGEAR-938 -M -- Matthias Wessendorf blog: http://matthiaswessendorf.wordpress.com/ sessions: http://www.slideshare.net/mwessendorf twitter: http://twitter.com/mwessendorf _______________________________________________ aerogear-dev mailing list aerogear-dev(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/aerogear-dev

+1 (I was thinking on using an NSPredicate to match what is offered in Android, but let's revisit this with a proper API) Thanks, Christos On Feb 22, 2013, at 5:49 PM, Summers Pittman <supittma(a)redhat.com&gt; wrote: > +1. I have some implementations in Android, but they aren't great yet. (One throws an exception if you search for nested objects for instance). > > > On 02/22/2013 09:17 AM, Matthias Wessendorf wrote: >> Hi, >> >> AGStore defines the following func: >> >> -(NSArray*) filter:(id)filterObject; >> >> Right now, there is NO implementation, so I'd vote to remove it.... Otherwise we will have this function around in future releases... >> If (later) we want some filtering (e.g. when more complex stores are around), we can define a proper API - instead of having some 'guess-ware' around... >> >> >> FWIW, I created this ticket: >> >> https://issues.jboss.org/browse/AEROGEAR-938 >> >> -M >> >> >> -- >> Matthias Wessendorf >> >> blog: http://matthiaswessendorf.wordpress.com/ >> sessions: http://www.slideshare.net/mwessendorf >> twitter: http://twitter.com/mwessendorf >> >> >> _______________________________________________ >> aerogear-dev mailing list >> aerogear-dev(a)lists.jboss.org >> https://lists.jboss.org/mailman/listinfo/aerogear-dev > > _______________________________________________ > aerogear-dev mailing list > aerogear-dev(a)lists.jboss.org > https://lists.jboss.org/mailman/listinfo/aerogear-dev _______________________________________________ aerogear-dev mailing list aerogear-dev(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/aerogear-dev

I like the comment from Christos, on iOS; Using a NSPredicate as the 'filter argument' makes sense for the overal iOS platform, and it should smoothly work with our current impls -M On Fri, Feb 22, 2013 at 6:25 PM, Kris Borchers <kris(a)redhat.com&gt; wrote: JS has implemented a pretty generic and rather useful filter method : https://github.com/aerogear/aerogear-js/blob/master/src/data-manager/adap... It is currently used in the JS TODO app. I am hesitant to remove it as it is useful and is one of the main useful features of DataManager for 1.0 other than it being a base for future development. At the same time, I am leery of leaving it in if iOS and Android don't have something similar since this feature was not thoroughly discussed. I would hate to leave this in, only to have to deprecate it in a later release for something that we decide works better for all libs. Other input would be appreciated. On Feb 22, 2013, at 10:08 AM, Christos Vasilakis <cvasilak(a)gmail.com&gt; wrote: > +1 > > (I was thinking on using an NSPredicate to match what is offered in Android, but let's revisit this with a proper API) > > Thanks, > Christos > > On Feb 22, 2013, at 5:49 PM, Summers Pittman <supittma(a)redhat.com&gt; wrote: > >> +1. I have some implementations in Android, but they aren't great yet. (One throws an exception if you search for nested objects for instance). >> >> >> On 02/22/2013 09:17 AM, Matthias Wessendorf wrote: >>> Hi, >>> >>> AGStore defines the following func: >>> >>> -(NSArray*) filter:(id)filterObject; >>> >>> Right now, there is NO implementation, so I'd vote to remove it.... Otherwise we will have this function around in future releases... >>> If (later) we want some filtering (e.g. when more complex stores are around), we can define a proper API - instead of having some 'guess-ware' around... >>> >>> >>> FWIW, I created this ticket: >>> >>> https://issues.jboss.org/browse/AEROGEAR-938 >>> >>> -M >>> >>> >>> -- >>> Matthias Wessendorf >>> >>> blog: http://matthiaswessendorf.wordpress.com/ >>> sessions: http://www.slideshare.net/mwessendorf >>> twitter: http://twitter.com/mwessendorf >>> >>> >>> _______________________________________________ >>> aerogear-dev mailing list >>> aerogear-dev(a)lists.jboss.org >>> https://lists.jboss.org/mailman/listinfo/aerogear-dev >> >> _______________________________________________ >> aerogear-dev mailing list >> aerogear-dev(a)lists.jboss.org >> https://lists.jboss.org/mailman/listinfo/aerogear-dev > > _______________________________________________ > aerogear-dev mailing list > aerogear-dev(a)lists.jboss.org > https://lists.jboss.org/mailman/listinfo/aerogear-dev _______________________________________________ aerogear-dev mailing list aerogear-dev(a)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(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/aerogear-dev

At the same time, I am leery of leaving it in if iOS and Android don't have something similar since this feature was not thoroughly discussed. I would hate to leave this in, only to have to deprecate it in a later release for something that we decide works better for all libs.

On Fri, Feb 22, 2013 at 11:25:33AM -0600, Kris Borchers wrote: > At the same time, I am leery of leaving it in if iOS and Android don't > have something similar since this feature was not thoroughly > discussed. I would hate to leave this in, only to have to deprecate it > in a later release for something that we decide works better for all > libs. How do you feel about node.js stability indication on APIs? Perhaps we could borrow some inspiration there (and this filter API would be experimental then)

-- qmx _______________________________________________ aerogear-dev mailing list aerogear-dev(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/aerogear-dev

On Mon, Feb 25, 2013 at 2:15 PM, Kris Borchers <kris(a)redhat.com&gt; wrote: On Feb 24, 2013, at 1:07 AM, Douglas Campos <qmx(a)qmx.me&gt; wrote: > On Fri, Feb 22, 2013 at 11:25:33AM -0600, Kris Borchers wrote: >> At the same time, I am leery of leaving it in if iOS and Android don't >> have something similar since this feature was not thoroughly >> discussed. I would hate to leave this in, only to have to deprecate it >> in a later release for something that we decide works better for all >> libs. > How do you feel about node.js stability indication on APIs? Perhaps we > could borrow some inspiration there (and this filter API would be > experimental then) So I like this idea in theory. I have a couple of concerns. We're talking about a specific method here which is a pretty detailed place to mark something experimental. At the same time, I wouldn't want to mark all of DataManager as experimental just because of one method. I agree, just that particular method :) 2) Related to #1, we would then have to go through and give a stability rating to every method of every piece of AeroGear and maintain that for every new feature, method, etc. 3) We would need to document our index somewhere similar to http://nodejs.org/api/documentation.html#documentation_stability_index that would be nice, but having a doc on that particular function/method could be fine, for now…

> > -- > qmx > _______________________________________________ > aerogear-dev mailing list > aerogear-dev(a)lists.jboss.org > https://lists.jboss.org/mailman/listinfo/aerogear-dev _______________________________________________ aerogear-dev mailing list aerogear-dev(a)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(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/aerogear-dev