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

CreateOvfStoresForStorageDomainCommand 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 remove the context from the
CreateOvfStoresForStorageDomainCommand constructor

Change-Id: Ib2cd44301ccf992541ecd0f9572fd7afde4a123d
Signed-off-by: Maor Lipchuk <[email protected]>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CreateOvfStoresForStorageDomainCommand.java
1 file changed, 2 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/15/29315/1

diff --git 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CreateOvfStoresForStorageDomainCommand.java
 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CreateOvfStoresForStorageDomainCommand.java
index 9af157e..72cb47f 100644
--- 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CreateOvfStoresForStorageDomainCommand.java
+++ 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CreateOvfStoresForStorageDomainCommand.java
@@ -1,7 +1,5 @@
 package org.ovirt.engine.core.bll;
 
-import org.ovirt.engine.core.bll.context.CommandContext;
-
 import java.util.List;
 
 import org.ovirt.engine.core.bll.utils.PermissionSubject;
@@ -22,8 +20,8 @@
 @NonTransactiveCommandAttribute
 public class CreateOvfStoresForStorageDomainCommand<T extends 
CreateOvfStoresForStorageDomainCommandParameters> extends CommandBase<T> {
 
-    public CreateOvfStoresForStorageDomainCommand(T parameters, CommandContext 
cmdContext) {
-        super(parameters, cmdContext);
+    public CreateOvfStoresForStorageDomainCommand(T parameters) {
+        super(parameters);
     }
 
     @Override


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib2cd44301ccf992541ecd0f9572fd7afde4a123d
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Maor Lipchuk <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to