Re: [PATCH 07/14] Add days and cities filters

[Chris Lumens <clumens@xxxxxxxxxx>]

> @@ -100,3 +115,36 @@ class DatetimeSpoke(NormalSpoke):
>  
>      def add_to_store(self, store, item):
>          store.append([item])
> +
> +    def existing_date(self, model, itr, user_data=None):
> +        day = model[itr][0]

You should be defensive here and consider the possibility that itr is
none (like in the case where the user ctrl-clicked on the current
selection and unselected everything, among others).

> +    def city_in_region(self, model, itr, user_data=None):
> +        city = model[itr][0]
> +        regions_model = self._regionCombo.get_model()
> +        regions_iter = self._regionCombo.get_active_iter()
> +        region = regions_model[regions_iter][0]

Same here with itr and regions_itr.

- Chris

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list