Maor Lipchuk has uploaded a new change for review.

Change subject: core: Modify constructor to be compatible with caller method
......................................................................

core: Modify constructor to be compatible with caller method

ConnectDomainToStorageCommand is being called without context
although its constructor is only supported with context parameter,
so we will always get runtime exception when the command is being
called.

The proposed fix is to add a constructor which also supports to get only
paramteres

Change-Id: Ic120c4b33b40a122895c2088b8365f26c05eb517
Signed-off-by: Maor Lipchuk <mlipc...@redhat.com>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/ConnectDomainToStorageCommand.java
1 file changed, 4 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/54/29554/1

diff --git 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/ConnectDomainToStorageCommand.java
 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/ConnectDomainToStorageCommand.java
index 9789e7f..fd49871 100644
--- 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/ConnectDomainToStorageCommand.java
+++ 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/ConnectDomainToStorageCommand.java
@@ -12,6 +12,10 @@
 @NonTransactiveCommandAttribute
 public class ConnectDomainToStorageCommand<T extends 
StorageDomainPoolParametersBase> extends StorageDomainCommandBase<T> {
 
+    public ConnectDomainToStorageCommand(T parameters) {
+        super(parameters, null);
+    }
+
     public ConnectDomainToStorageCommand(T parameters, CommandContext 
cmdContext) {
         super(parameters, cmdContext);
     }


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic120c4b33b40a122895c2088b8365f26c05eb517
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Maor Lipchuk <mlipc...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to