← Back to team overview

schooltool-developers team mailing list archive

Re: Cando person filter widget bug

 

2010/10/26 Alan Elkner <aelkner@xxxxxxxxx>:
> Hey Guys,
>
> I found a bug in cando's newskin where it overrides the person filter
> widget with its own widget, and it has a bug getting the group
> container.  It uses IGroupContainer(ISchoolToolApplication, {}) with
> the None part clearly missing.  This went unnoticed because (and I'm
> guessing here) the pre-schoolyears adapter to get the group container
> didn't even look at the context.  I suppose this bug came about after
> we added schoolyears and had to change all of those adapters.  The
> Virginia folks didn't notice that the widget wasn't working because
> they don't enroll by group.  I think they do it via import.
>
> Anyway, I submitted a branch for merging,
> lp:~aelkner/cando/filter_widget_bug.  We'll need an egg and package as
> soon as possible.

I have merged this fix and released it as cando 2010.10. The deb will
be built shortly (for Lucid).



References