Alon Bar-Lev has posted comments on this change.

Change subject: aaa: disabling "go" search button when there are no namespaces
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.ovirt.org/#/c/32213/1/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetAvailableNamespacesQuery.java
File 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetAvailableNamespacesQuery.java:

Line 27:     @Override
Line 28:     protected void executeQueryCommand() {
Line 29:         HashMap<String, List<String>> namespacesMap = new HashMap<>();
Line 30:         for (ExtensionProxy authz: 
EngineExtensionsManager.getInstance().getExtensionsByService(Authz.class.getName()))
 {
Line 31:             for (String namespace : 
authz.getContext().get(Authz.ContextKeys.AVAILABLE_NAMESPACES, 
Arrays.asList(""))) {
why not empty list?
Line 32:                 MultiValueMapUtils.addToMap(AuthzUtils.getName(authz), 
namespace, namespacesMap);
Line 33: 
Line 34:             }
Line 35:         }


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I4eb1ed2648f3dccc39af9adf409d8c0ec73b2a6f
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yair Zaslavsky <yzasl...@redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to