On Wed, Mar 2, 2016 at 12:02 PM, Christophe Fergeau <cfergeau@xxxxxxxxxx> 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@xxxxxxxxxx > https://www.redhat.com/mailman/listinfo/virt-tools-list _______________________________________________ virt-tools-list mailing list virt-tools-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/virt-tools-list
