On Thu, Jan 22, 2009 at 2:40 PM, <[email protected]> wrote: > Author: lindner > Date: Thu Jan 22 14:40:18 2009 > New Revision: 736836 > > URL: http://svn.apache.org/viewvc?rev=736836&view=rev > Log: > SHINDIG-879 | Patch from Jordan Zimmerman | Add missing fields declaration > > Modified: > incubator/shindig/trunk/features/opensocial-reference/datarequest.js > > Modified: incubator/shindig/trunk/features/opensocial-reference/datarequest.js > URL: > http://svn.apache.org/viewvc/incubator/shindig/trunk/features/opensocial-reference/datarequest.js?rev=736836&r1=736835&r2=736836&view=diff > ============================================================================== > --- incubator/shindig/trunk/features/opensocial-reference/datarequest.js > (original) > +++ incubator/shindig/trunk/features/opensocial-reference/datarequest.js Thu > Jan 22 14:40:18 2009 > @@ -489,6 +489,8 @@ > opt_params) { > opt_params = opt_params || {}; > > + var fields = opensocial.DataRequest.PeopleRequestFields; // THIS LINE IS > MISSING
Paul, I don't think "// THIS LINE IS MISSING" really needed to be included... -- Adam > + > this.addDefaultParam(opt_params, fields.FIRST, 0); > this.addDefaultParam(opt_params, fields.MAX, 20); > > > >
