Change in ovirt-engine[master]: webadmin: fix provider isReadOnlyAware flag

derez at redhat.com derez at redhat.com
Thu Apr 21 06:38:40 UTC 2016


Daniel Erez has submitted this change and it was merged.

Change subject: webadmin: fix provider isReadOnlyAware flag
......................................................................


webadmin: fix provider isReadOnlyAware flag

ProviderType -> isReadOnlyAware method returns a
wrong flag, which caused an exception when editing
a non 'ExternalNetworkProvider', and the 'Read Only'
checkbox was displays for all providers type.
Fixed by returning the correct flag.

Change-Id: Ie2c036548cb477abba617c01268eacec6af99ef3
Signed-off-by: Daniel Erez <derez at redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/ProviderType.java
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Alona Kaplan: Looks good to me, approved
  Daniel Erez: Verified
  Jenkins CI: Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/56356
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie2c036548cb477abba617c01268eacec6af99ef3
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list