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(a)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(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>