On Wed, Mar 2, 2016 at 12:02 PM, Christophe Fergeau <cferg...@redhat.com> wrote: > On Wed, Mar 02, 2016 at 08:52:28AM +0100, Christophe Fergeau wrote: >> oVirt storage domains can be in various states (inactive, in >> maintainance, ...). We only want to show the ISOs it contains in the >> foreign menu when the storage domain is actually active, not in the >> other states. >> >> https://bugzilla.redhat.com/show_bug.cgi?id=1310450 >> --- >> Changes since v1: >> - added new condition to an existing if() block > > Fwiw, I prefer the v1 way which I find much easier to read.
Same here. > > Christophe > > _______________________________________________ > virt-tools-list mailing list > virt-tools-list@redhat.com > https://www.redhat.com/mailman/listinfo/virt-tools-list _______________________________________________ virt-tools-list mailing list virt-tools-list@redhat.com https://www.redhat.com/mailman/listinfo/virt-tools-list