Tal Nisan has submitted this change and it was merged. Change subject: webadmin: there are no such things as ghost in MacRangeWidget ...................................................................... webadmin: there are no such things as ghost in MacRangeWidget 'isGhost' mechanism is faulty. Correct behavior is, acording to colleges and code notes, that there's at most one ghost and it has to be the last record. Currently implementation allows existence of multiple ghosts, which needs not to be trailing records. For simplicity, in this dialog we assume that there cannot exist ghosts, until ghost problem is solved generally Change-Id: I8091cedfd3cf51dfcb93bb7c7de45234088bfdd8 Bug-Url: https://bugzilla.redhat.com/1217940 Signed-off-by: Martin Mucha <mmucha@redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/macpool/MacRangeWidget.java 1 file changed, 1 insertion(+), 3 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Martin Mucha: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/49966 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8091cedfd3cf51dfcb93bb7c7de45234088bfdd8 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Martin Mucha <mmucha@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Mucha <mmucha@redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan@redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>