Itamar Heim has posted comments on this change.

Change subject: core: Eliminated use of StringHelper.EqOp
......................................................................


Patch Set 2: (1 inline comment)

....................................................
File 
backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/AdUserConditionFieldAutoCompleter.java
Line 15:         // building the ColumnName Dict
Line 16:         columnNameDict.put("NAME", "$GIVENNAME");
Line 17:         columnNameDict.put("USERNAME", "$SAMACCOUNTNAME");
Line 18: 
Line 19:         if 
("LDAP".equalsIgnoreCase(SyntaxCheckerFactory.getConfigAuthenticationMethod())) 
{
yes, leave as is. not used right now, but we are working on making this 
relevant again right now, so a good reference, and not harmful.
Line 20:             mVerbs.put("LASTNAME", "LASTNAME");
Line 21:             mVerbs.put("DEPARTMENT", "DEPARTMENT");
Line 22:             mVerbs.put("TITLE", "TITLE");
Line 23: 


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Iaf3d65db824f31df05dc7b0eda5f9c24274cf53c
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tal Nisan <tni...@redhat.com>
Gerrit-Reviewer: Alissa Bonas <abo...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Ayal Baron <aba...@redhat.com>
Gerrit-Reviewer: Itamar Heim <ih...@redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipc...@redhat.com>
Gerrit-Reviewer: Tal Nisan <tni...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to