you mean 'outlet_type'? it is a PSI-specific property and the outlet_type
filter will only be useful for them, but it will still work for other
domains when 'all outlets types' is selected.


On Tue, May 21, 2013 at 12:55 PM, Cory Zue <notifications@github.com> wrote:

> In corehq/apps/reports/commtrack/psi_prototype.py:
>
> > @@ -64,7 +78,10 @@ def types_for_sel(sel):
> >                  return [sel]
> >          active_outlet_types = reduce(lambda a, b: a.union(b), (types_for_sel(sel) for sel in selected), set())
> >
> > -        return lambda outlet_type: ('_all' in selected) or (outlet_type in active_outlet_types)
> > +        def match_filter(loc):
> > +            outlet_type = loc.dynamic_properties().get('outlet_type')
>
> is this right? would have expected the 'outlet' bit to not be hard-coded.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/dimagi/commcare-hq/pull/321/files#r4321202>
> .
>