> @@ -417,28 +421,16 @@ class CustomPartitioningSpoke(NormalSpoke): > > # Make sure there's something displayed on the RHS. Just default to > # the first mountpoint in the page. > + # FIXME: the current page appears to be the default/empty/create page, > + # even if you've obviously gone into one of the roots to remove > + # a device > page = self._accordion.currentPage() > - if not page or not page._members: > - return > - > - self._populate_right_side(page._members[0]) > + if getattr(page, "_members", []): > + self._populate_right_side(page._members[0]) This is a kind of weird phrasing. Why don't you use hasattr instead? > @@ -513,10 +499,13 @@ class CustomPartitioningSpoke(NormalSpoke): > # Devices just created by autopartitioning will be listed as unused > # since they are not yet a part of any known Root. > for device in self._unusedDevices(): > + if device.exists: > + continue > + > if device.format.type == "swap": > swaps.append(device) > > - if hasattr(device.format, "mountpoint"): > + if getattr(device.format, "mountpoint", None): > mounts[device.format.mountpoint] = device Likewise. - Chris _______________________________________________ Anaconda-devel-list mailing list Anaconda-devel-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/anaconda-devel-list
