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 <abo...@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 <abo...@redhat.com>
Gerrit-Reviewer: Alissa Bonas <abo...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourf...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to