I mean to say that we should follow whatever our preexisting pattern is for
all metadata.


On Wed, Feb 12, 2014 at 1:16 PM, wluyima <notifications@github.com> wrote:

> In
> omod-1.10/src/main/java/org/openmrs/module/webservices/rest/web/v1_0/resource/openmrs1_10/OrderFrequencyResource1_10.java:
>
> > +    }
> > +
> > +    /**
> > +     * @see org.openmrs.module.webservices.rest.web.resource.impl.DelegatingCrudResource#doGetAll(org.openmrs.module.webservices.rest.web.RequestContext)
> > +     */
> > +    @Override
> > +    protected NeedsPaging<OrderFrequency> doGetAll(RequestContext context) {
> > +        return new NeedsPaging<OrderFrequency>(Context.getOrderService().getOrderFrequencies(), context);
> > +    }
> > +
> > +    /**
> > +     * @see org.openmrs.module.webservices.rest.web.resource.impl.DelegatingCrudResource#doSearch(org.openmrs.module.webservices.rest.web.RequestContext)
> > +     */
> > +    @Override
> > +    protected NeedsPaging<OrderFrequency> doSearch(RequestContext context) {
> > +        throw new ResourceDoesNotSupportOperationException();
>
> I agree with adding getOrderFrequencies, an a little against having
> getAllOrderFrequencies, because it would mean we end up with
> getOrderFrequencies(includeRetired) too
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/openmrs/openmrs-module-webservices.rest/pull/85/files#r9686299>
> .
>