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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> @@ -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


[Index of Archives]     [Kickstart]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]
  Powered by Linux