Change in ovirt-engine[master]: core: simplify changeCaseDisplay method in Search

yzaslavs at redhat.com yzaslavs at redhat.com
Fri May 23 01:00:15 UTC 2014


Yair Zaslavsky has submitted this change and it was merged.

Change subject: core: simplify changeCaseDisplay method in Search
......................................................................


core: simplify changeCaseDisplay method in Search

Simplify changeCaseDisplay method in search autocompleter.
Part of the expression was processing an empty string which has no
purpose.
Added one more test case to the existing unit test.

Change-Id: Ib2bc968c5fc724c64fc45804dd847c29994b7851
Signed-off-by: Alissa Bonas <abonas at redhat.com>
---
M backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/SearchObjectsBaseAutoCompleter.java
M backend/manager/modules/searchbackend/src/test/java/org/ovirt/engine/core/searchbackend/SearchObjectsBaseAutoCompleterTest.java
2 files changed, 2 insertions(+), 1 deletion(-)

Approvals:
  Alissa Bonas: Verified
  Yair Zaslavsky: Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/27723
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib2bc968c5fc724c64fc45804dd847c29994b7851
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alissa Bonas <abonas at redhat.com>
Gerrit-Reviewer: Alissa Bonas <abonas at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list