Alexander Wels has uploaded a new change for review.

Change subject: userportal,webadmin: Generify permission in models
......................................................................

userportal,webadmin: Generify permission in models

- After the GIN patch a lot of the permission list models where
  not properly generified, this patch fixes that.

Change-Id: I443d45d7bb1ffb69d239777d6c61f5a2fe0d2a2c
Signed-off-by: Alexander Wels <aw...@redhat.com>
---
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterHostListModel.java
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterListModel.java
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterVmListModel.java
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterClusterListModel.java
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterListModel.java
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/disks/DiskListModel.java
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostListModel.java
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostVmListModel.java
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/networks/NetworkListModel.java
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/pools/PoolListModel.java
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/pools/PoolVmListModel.java
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/profiles/CpuProfileListModel.java
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/profiles/DiskProfileListModel.java
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/profiles/ProfileListModel.java
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/profiles/VnicProfileListModel.java
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageListModel.java
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/templates/TemplateListModel.java
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/templates/TemplateVmListModel.java
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmListModel.java
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/volumes/VolumeListModel.java
20 files changed, 44 insertions(+), 39 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/14/35514/1

diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterHostListModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterHostListModel.java
index 9782ff2..34322f6 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterHostListModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterHostListModel.java
@@ -40,7 +40,7 @@
             final HostVmListModel hostVmListModel, final HostEventListModel 
hostEventListModel,
             final HostInterfaceListModel hostInterfaceListModel,
             final HostHardwareGeneralModel hostHardwareGeneralModel, final 
HostHooksListModel hostHooksListModel,
-            final PermissionListModel permissionListModel) {
+            final PermissionListModel<HostListModel> permissionListModel) {
         super(hostGeneralModel, hostGlusterSwiftListModel, 
hostBricksListModel, hostVmListModel, hostEventListModel,
                 hostInterfaceListModel, hostHardwareGeneralModel, 
hostHooksListModel, permissionListModel);
         setUpdateMomPolicyCommand(new UICommand("updateMomPolicyCommand", 
this)); //$NON-NLS-1$
diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterListModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterListModel.java
index c419809..4b7b2e5 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterListModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterListModel.java
@@ -188,7 +188,7 @@
             final ClusterAffinityGroupListModel clusterAffinityGroupListModel,
             final CpuProfileListModel cpuProfileListModel, final 
ClusterGeneralModel clusterGeneralModel,
             final ClusterNetworkListModel clusterNetworkListModel, final 
ClusterHostListModel clusterHostListModel,
-            final PermissionListModel permissionListModel) {
+            final PermissionListModel<ClusterListModel> permissionListModel) {
         this.clusterVmListModel = clusterVmListModel;
         this.clusterServiceModel = clusterServiceModel;
         this.clusterGlusterHookListModel = clusterGlusterHookListModel;
@@ -247,7 +247,7 @@
 
     private void setDetailList(final ClusterGeneralModel clusterGeneralModel,
             final ClusterNetworkListModel clusterNetworkListModel, final 
ClusterHostListModel clusterHostListModel,
-            final PermissionListModel permissionListModel) {
+            final PermissionListModel<ClusterListModel> permissionListModel) {
         List<EntityModel> list = new ArrayList<EntityModel>();
         list.add(clusterGeneralModel);
         list.add(clusterNetworkListModel);
diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterVmListModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterVmListModel.java
index 5610023..348811a 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterVmListModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterVmListModel.java
@@ -27,7 +27,7 @@
             VmSnapshotListModel vmSnapshotListModel,
             VmEventListModel vmEventListModel,
             VmAppListModel vmAppListModel,
-            PermissionListModel permissionListModel,
+            PermissionListModel<VmListModel> permissionListModel,
             VmAffinityGroupListModel vmAffinityGroupListModel,
             VmSessionsModel vmSessionsModel) {
         super(vmGeneralModel,
diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterClusterListModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterClusterListModel.java
index bf95b5d..eeb1b62 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterClusterListModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterClusterListModel.java
@@ -26,7 +26,7 @@
             ClusterGeneralModel clusterGeneralModel,
             ClusterNetworkListModel clusterNetworkListModel,
             ClusterHostListModel clusterHostListModel,
-            PermissionListModel permissionListModel) {
+            PermissionListModel<ClusterListModel> permissionListModel) {
         super(clusterVmListModel,
                 clusterServiceModel,
                 clusterGlusterHookListModel,
diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterListModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterListModel.java
index 2fa5701..4d07ac5 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterListModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterListModel.java
@@ -195,7 +195,7 @@
             final DataCenterNetworkQoSListModel dataCenterNetworkQoSListModel,
             final DataCenterStorageQosListModel dataCenterStorageQosListModel,
             final DataCenterCpuQosListModel dataCenterCpuQosListModel,
-            final PermissionListModel permissionListModel,
+            final PermissionListModel<DataCenterListModel> permissionListModel,
             final DataCenterEventListModel dataCenterEventListModel) {
         this.commonModelProvider = commonModelProvider;
         iscsiBondListModel = dataCenterIscsiBondListModel;
@@ -262,7 +262,7 @@
             final DataCenterNetworkQoSListModel dataCenterNetworkQoSListModel,
             final DataCenterStorageQosListModel dataCenterStorageQosListModel,
             final DataCenterCpuQosListModel dataCenterCpuQosListModel,
-            final PermissionListModel permissionListModel,
+            final PermissionListModel<DataCenterListModel> permissionListModel,
             final DataCenterEventListModel dataCenterEventListModel) {
         List<EntityModel> list = new ArrayList<EntityModel>();
         list.add(dataCenterStorageListModel);
diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/disks/DiskListModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/disks/DiskListModel.java
index 7bcf4a4..dcb289f 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/disks/DiskListModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/disks/DiskListModel.java
@@ -45,7 +45,6 @@
 
 import com.google.inject.Inject;
 
-@SuppressWarnings("unused")
 public class DiskListModel extends ListWithDetailsModel implements 
ISupportSystemTreeContext
 {
     private UICommand privateNewCommand;
@@ -175,7 +174,7 @@
     @Inject
     public DiskListModel(final DiskVmListModel diskVmListModel, final 
DiskTemplateListModel diskTemplateListModel,
             final DiskStorageListModel diskStorageListModel, final 
DiskGeneralModel diskGeneralModel,
-            final PermissionListModel permissionListModel) {
+            final PermissionListModel<DiskListModel> permissionListModel) {
         this.diskVmListModel = diskVmListModel;
         this.diskTemplateListModel = diskTemplateListModel;
         this.diskStorageListModel = diskStorageListModel;
@@ -206,7 +205,7 @@
     }
 
     private void setDetailList(final DiskGeneralModel diskGeneralModel,
-            final PermissionListModel permissionListModel) {
+            final PermissionListModel<DiskListModel> permissionListModel) {
         diskVmListModel.setIsAvailable(false);
         diskTemplateListModel.setIsAvailable(false);
         diskStorageListModel.setIsAvailable(false);
diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostListModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostListModel.java
index 7cfaba4..17e59b9 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostListModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostListModel.java
@@ -381,7 +381,7 @@
             final HostVmListModel hostVmListModel, final HostEventListModel 
hostEventListModel,
             final HostInterfaceListModel hostInterfaceListModel,
             final HostHardwareGeneralModel hostHardwareGeneralModel, final 
HostHooksListModel hostHooksListModel,
-            final PermissionListModel permissionListModel) {
+            final PermissionListModel<HostListModel> permissionListModel) {
         this.generalModel = hostGeneralModel;
         this.glusterSwiftModel = hostGlusterSwiftListModel;
         this.hostBricksListModel = hostBricksListModel;
@@ -425,7 +425,7 @@
 
     private void setDetailList(final HostInterfaceListModel 
hostInterfaceListModel,
             final HostHardwareGeneralModel hostHardwareGeneralModel, final 
HostHooksListModel hostHooksListModel,
-            final PermissionListModel permissionListModel) {
+            final PermissionListModel<HostListModel> permissionListModel) {
         generalModel.getRequestEditEvent().addListener(this);
         generalModel.getRequestGOToEventsTabEvent().addListener(this);
 
diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostVmListModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostVmListModel.java
index 00036fb..4293dea 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostVmListModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostVmListModel.java
@@ -29,7 +29,7 @@
             VmSnapshotListModel vmSnapshotListModel,
             VmEventListModel vmEventListModel,
             VmAppListModel vmAppListModel,
-            PermissionListModel permissionListModel,
+            PermissionListModel<VmListModel> permissionListModel,
             VmAffinityGroupListModel vmAffinityGroupListModel,
             VmSessionsModel vmSessionsModel) {
         super(vmGeneralModel, vmInterfaceListModel, vmDiskListModel, 
vmSnapshotListModel, vmEventListModel,
diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/networks/NetworkListModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/networks/NetworkListModel.java
index da5bf65..0941da3 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/networks/NetworkListModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/networks/NetworkListModel.java
@@ -59,7 +59,8 @@
             final NetworkGeneralModel networkGeneralModel, final 
NetworkProfileListModel networkProfileListModel,
             final NetworkClusterListModel networkClusterListModel,
             final NetworkHostListModel networkHostListModel, final 
NetworkVmListModel networkVmListModel,
-            final NetworkTemplateListModel networkTemplateListModel, final 
PermissionListModel permissionListModel) {
+            final NetworkTemplateListModel networkTemplateListModel,
+            final PermissionListModel<NetworkListModel> permissionListModel) {
         this.networkExternalSubnetListModel = networkExternalSubnetListModel;
         this.importNetworkModelProvider = importNetworkModelProvider;
         setDetailList(networkGeneralModel, networkProfileListModel, 
networkClusterListModel, networkHostListModel,
@@ -88,7 +89,8 @@
             final NetworkProfileListModel networkProfileListModel,
             final NetworkClusterListModel networkClusterListModel,
             final NetworkHostListModel networkHostListModel, final 
NetworkVmListModel networkVmListModel,
-            final NetworkTemplateListModel networkTemplateListModel, final 
PermissionListModel permissionListModel) {
+            final NetworkTemplateListModel networkTemplateListModel,
+            final PermissionListModel<NetworkListModel> permissionListModel) {
         List<EntityModel> list = new ArrayList<EntityModel>();
 
         list.add(networkGeneralModel);
diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/pools/PoolListModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/pools/PoolListModel.java
index 846aa0e..a4bc6e8 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/pools/PoolListModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/pools/PoolListModel.java
@@ -133,7 +133,7 @@
 
     @Inject
     public PoolListModel(final PoolGeneralModel poolGeneralModel, final 
PoolVmListModel poolVmListModel,
-            final PermissionListModel permissionListModel) {
+            final PermissionListModel<PoolListModel> permissionListModel) {
         setDetailList(poolGeneralModel, poolVmListModel, permissionListModel);
         setTitle(ConstantsManager.getInstance().getConstants().poolsTitle());
 
@@ -153,7 +153,7 @@
     }
 
     private void setDetailList(final PoolGeneralModel poolGeneralModel, final 
PoolVmListModel poolVmListModel,
-            final PermissionListModel permissionListModel) {
+            final PermissionListModel<PoolListModel> permissionListModel) {
         List<EntityModel> list = new ArrayList<EntityModel>();
         list.add(poolGeneralModel);
         list.add(poolVmListModel);
diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/pools/PoolVmListModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/pools/PoolVmListModel.java
index 6209660..56a0952 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/pools/PoolVmListModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/pools/PoolVmListModel.java
@@ -52,7 +52,7 @@
     public PoolVmListModel(final VmGeneralModel vmGeneralModel, final 
VmInterfaceListModel vmInterfaceListModel,
             final VmDiskListModel vmDiskListModel, final VmSnapshotListModel 
vmSnapshotListModel,
             final VmEventListModel vmEventListModel, final VmAppListModel 
vmAppListModel,
-            final PermissionListModel permissionListModel, final 
VmAffinityGroupListModel vmAffinityGroupListModel,
+            final PermissionListModel<VmListModel> permissionListModel, final 
VmAffinityGroupListModel vmAffinityGroupListModel,
             final VmSessionsModel vmSessionsModel) {
         super(vmGeneralModel, vmInterfaceListModel, vmDiskListModel, 
vmSnapshotListModel, vmEventListModel,
                 vmAppListModel, permissionListModel, vmAffinityGroupListModel, 
vmSessionsModel);
diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/profiles/CpuProfileListModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/profiles/CpuProfileListModel.java
index b9fc5a1..deb936c9 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/profiles/CpuProfileListModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/profiles/CpuProfileListModel.java
@@ -7,6 +7,7 @@
 import org.ovirt.engine.core.common.queries.VdcQueryType;
 import org.ovirt.engine.core.compat.Guid;
 import org.ovirt.engine.ui.uicommonweb.help.HelpTag;
+import org.ovirt.engine.ui.uicommonweb.models.clusters.ClusterListModel;
 import org.ovirt.engine.ui.uicommonweb.models.configure.PermissionListModel;
 import org.ovirt.engine.ui.uicompat.ConstantsManager;
 
@@ -15,7 +16,7 @@
 public class CpuProfileListModel extends ProfileListModel<CpuProfile, CpuQos, 
VDSGroup> {
 
     @Inject
-    public CpuProfileListModel(final PermissionListModel permissionListModel) {
+    public CpuProfileListModel(final PermissionListModel<ClusterListModel> 
permissionListModel) {
         super(permissionListModel);
         
setTitle(ConstantsManager.getInstance().getConstants().cpuProfileTitle());
         setHelpTag(HelpTag.cpu_profiles);
diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/profiles/DiskProfileListModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/profiles/DiskProfileListModel.java
index e4a1e8e..5bc1883 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/profiles/DiskProfileListModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/profiles/DiskProfileListModel.java
@@ -8,6 +8,7 @@
 import org.ovirt.engine.core.compat.Guid;
 import org.ovirt.engine.ui.uicommonweb.help.HelpTag;
 import org.ovirt.engine.ui.uicommonweb.models.configure.PermissionListModel;
+import org.ovirt.engine.ui.uicommonweb.models.storage.StorageListModel;
 import org.ovirt.engine.ui.uicompat.ConstantsManager;
 
 import com.google.inject.Inject;
@@ -15,7 +16,7 @@
 public class DiskProfileListModel extends ProfileListModel<DiskProfile, 
StorageQos, StorageDomain> {
 
     @Inject
-    public DiskProfileListModel(final PermissionListModel permissionListModel) 
{
+    public DiskProfileListModel(final PermissionListModel<StorageListModel> 
permissionListModel) {
         super(permissionListModel);
         
setTitle(ConstantsManager.getInstance().getConstants().diskProfilesTitle());
         setHelpTag(HelpTag.disk_profiles);
diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/profiles/ProfileListModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/profiles/ProfileListModel.java
index 64ca1d9..ab18f65 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/profiles/ProfileListModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/profiles/ProfileListModel.java
@@ -22,6 +22,7 @@
 import org.ovirt.engine.ui.uicommonweb.help.HelpTag;
 import org.ovirt.engine.ui.uicommonweb.models.EntityModel;
 import org.ovirt.engine.ui.uicommonweb.models.ListWithDetailsModel;
+import org.ovirt.engine.ui.uicommonweb.models.SearchableListModel;
 import org.ovirt.engine.ui.uicommonweb.models.configure.PermissionListModel;
 import org.ovirt.engine.ui.uicompat.ConstantsManager;
 import org.ovirt.engine.ui.uicompat.PropertyChangedEventArgs;
@@ -33,10 +34,10 @@
     private UICommand editCommand;
     private UICommand removeCommand;
     private Map<Guid, Q> qosMap;
-    final PermissionListModel permissionListModel;
+    final PermissionListModel<? extends SearchableListModel> 
permissionListModel;
 
     @Inject
-    public ProfileListModel(final PermissionListModel permissionListModel) {
+    public ProfileListModel(final PermissionListModel<? extends 
SearchableListModel> permissionListModel) {
         this.permissionListModel = permissionListModel;
         setDetailList();
         
setTitle(ConstantsManager.getInstance().getConstants().diskProfilesTitle());
@@ -246,7 +247,7 @@
         return qosMap.get(qosId);
     }
 
-    public PermissionListModel getPermissionListModel() {
+    public PermissionListModel<? extends SearchableListModel> 
getPermissionListModel() {
         return permissionListModel;
     }
 }
diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/profiles/VnicProfileListModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/profiles/VnicProfileListModel.java
index 87d9e7d..f46cc89 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/profiles/VnicProfileListModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/profiles/VnicProfileListModel.java
@@ -47,7 +47,7 @@
     public VnicProfileListModel(final Provider<CommonModel> 
commonModelProvider,
             final VnicProfileVmListModel vNicProfileVmListModel,
             final VnicProfileTemplateListModel vNicProfileTemplateListModel,
-            final PermissionListModel permissionListModel) {
+            final PermissionListModel<VnicProfileListModel> 
permissionListModel) {
         this.commonModelProvider = commonModelProvider;
         setDetailList(vNicProfileVmListModel, vNicProfileTemplateListModel, 
permissionListModel);
         
setTitle(ConstantsManager.getInstance().getConstants().vnicProfilesTitle());
@@ -73,7 +73,7 @@
 
     private void setDetailList(final VnicProfileVmListModel 
vNicProfileVmListModel,
             final VnicProfileTemplateListModel vNicProfileTemplateListModel,
-            final PermissionListModel permissionListModel) {
+            final PermissionListModel<VnicProfileListModel> 
permissionListModel) {
         List<EntityModel> list = new ArrayList<EntityModel>();
 
         list.add(vNicProfileVmListModel);
diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageListModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageListModel.java
index fcb785a..d789973 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageListModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageListModel.java
@@ -70,9 +70,7 @@
 
 import com.google.inject.Inject;
 
-@SuppressWarnings("unused")
-public class StorageListModel extends ListWithDetailsAndReportsModel 
implements ITaskTarget, ISupportSystemTreeContext
-{
+public class StorageListModel extends ListWithDetailsAndReportsModel 
implements ITaskTarget, ISupportSystemTreeContext {
 
     private UICommand privateNewDomainCommand;
 
@@ -174,7 +172,8 @@
             final StorageVmListModel storageVmListModel, final 
StorageTemplateListModel storageTemplateListModel,
             final StorageIsoListModel storageIsoListModel, final 
StorageDiskListModel storageDiskListModel,
             final StorageSnapshotListModel storageSnapshotListModel, final 
DiskProfileListModel diskProfileListModel,
-            final StorageEventListModel storageEventListModel, final 
PermissionListModel permissionListModel) {
+            final StorageEventListModel storageEventListModel,
+            final PermissionListModel<StorageListModel> permissionListModel) {
         generalModel = storageGeneralModel;
         dcListModel = storageDataCenterListModel;
         vmBackupModel = storageVmBackupModel;
@@ -210,7 +209,7 @@
     }
 
     private void setDetailList(final StorageEventListModel 
storageEventListModel,
-            final PermissionListModel permissionListModel) {
+            final PermissionListModel<StorageListModel> permissionListModel) {
         generalModel.setIsAvailable(false);
         dcListModel.setIsAvailable(false);
         this.vmBackupModel.setIsAvailable(false);
diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/templates/TemplateListModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/templates/TemplateListModel.java
index 06790b4..8ff87aa 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/templates/TemplateListModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/templates/TemplateListModel.java
@@ -117,7 +117,7 @@
             final TemplateVmListModel templateVmListModel, final 
TemplateInterfaceListModel templateInterfaceListModel,
             final TemplateStorageListModel templateStorageListModel,
             final TemplateDiskListModel templateDiskListModel, final 
TemplateEventListModel templateEventListModel,
-            final PermissionListModel permissionListModel) {
+            final PermissionListModel<TemplateListModel> permissionListModel) {
         this(templateGeneralModel, templateVmListModel, 
templateInterfaceListModel, templateStorageListModel,
                 templateDiskListModel, templateEventListModel, 
permissionListModel, 3);
     }
@@ -126,7 +126,7 @@
             final TemplateVmListModel templateVmListModel, final 
TemplateInterfaceListModel templateInterfaceListModel,
             final TemplateStorageListModel templateStorageListModel,
             final TemplateDiskListModel templateDiskListModel, final 
TemplateEventListModel templateEventListModel,
-            final PermissionListModel permissionListModel, int customPosition) 
{
+            final PermissionListModel<TemplateListModel> permissionListModel, 
int customPosition) {
         List<EntityModel> list = new ArrayList<EntityModel>();
         setDetailList(list, templateGeneralModel, templateVmListModel, 
templateInterfaceListModel,
                 templateStorageListModel);
@@ -382,7 +382,7 @@
 
     protected void addCustomModelsDetailModelList(final List<EntityModel> 
list, int customPosition,
             final TemplateDiskListModel templateDiskListModel, final 
TemplateEventListModel templateEventListModel,
-            final PermissionListModel permissionListModel) {
+            final PermissionListModel<TemplateListModel> permissionListModel) {
         templateDiskListModel.setSystemTreeContext(this);
         list.add(customPosition, templateDiskListModel);
         list.add(templateEventListModel);
diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/templates/TemplateVmListModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/templates/TemplateVmListModel.java
index a05ca1f..588204c 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/templates/TemplateVmListModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/templates/TemplateVmListModel.java
@@ -27,8 +27,8 @@
     public TemplateVmListModel(final VmGeneralModel vmGeneralModel, final 
VmInterfaceListModel vmInterfaceListModel,
             final VmDiskListModel vmDiskListModel, final VmSnapshotListModel 
vmSnapshotListModel,
             final VmEventListModel vmEventListModel, final VmAppListModel 
vmAppListModel,
-            final PermissionListModel permissionListModel, final 
VmAffinityGroupListModel vmAffinityGroupListModel,
-            final VmSessionsModel vmSessionsModel) {
+            final PermissionListModel<VmListModel> permissionListModel,
+            final VmAffinityGroupListModel vmAffinityGroupListModel, final 
VmSessionsModel vmSessionsModel) {
         super(vmGeneralModel, vmInterfaceListModel, vmDiskListModel, 
vmSnapshotListModel, vmEventListModel,
                 vmAppListModel, permissionListModel, vmAffinityGroupListModel, 
vmSessionsModel);
         
setTitle(ConstantsManager.getInstance().getConstants().virtualMachinesTitle());
diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmListModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmListModel.java
index 49697e8..5f92fac 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmListModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmListModel.java
@@ -458,7 +458,7 @@
     public VmListModel(final VmGeneralModel vmGeneralModel, final 
VmInterfaceListModel vmInterfaceListModel,
             final VmDiskListModel vmDiskListModel, final VmSnapshotListModel 
vmSnapshotListModel,
             final VmEventListModel vmEventListModel, final VmAppListModel 
vmAppListModel,
-            final PermissionListModel permissionListModel, final 
VmAffinityGroupListModel vmAffinityGroupListModel,
+            final PermissionListModel<VmListModel> permissionListModel, final 
VmAffinityGroupListModel vmAffinityGroupListModel,
             final VmSessionsModel vmSessionsModel) {
         setDetailList(vmGeneralModel, vmInterfaceListModel, vmDiskListModel, 
vmSnapshotListModel, vmEventListModel,
                 vmAppListModel, permissionListModel, vmAffinityGroupListModel, 
vmSessionsModel);
@@ -520,7 +520,7 @@
     private void setDetailList(final VmGeneralModel vmGeneralModel, final 
VmInterfaceListModel vmInterfaceListModel,
             final VmDiskListModel vmDiskListModel, final VmSnapshotListModel 
vmSnapshotListModel,
             final VmEventListModel vmEventListModel, final VmAppListModel 
vmAppListModel,
-            final PermissionListModel permissionListModel, final 
VmAffinityGroupListModel vmAffinityGroupListModel,
+            final PermissionListModel<VmListModel> permissionListModel, final 
VmAffinityGroupListModel vmAffinityGroupListModel,
             final VmSessionsModel vmSessionsModel) {
         List<EntityModel> list = new ArrayList<EntityModel>();
         list.add(vmGeneralModel);
diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/volumes/VolumeListModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/volumes/VolumeListModel.java
index a5f36d7..6861d88 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/volumes/VolumeListModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/volumes/VolumeListModel.java
@@ -193,7 +193,8 @@
 
     @Inject
     public VolumeListModel(final VolumeBrickListModel volumeBrickListModel, 
final VolumeGeneralModel volumeGeneralModel,
-            final VolumeParameterListModel volumeParameterListModel, final 
PermissionListModel permissionListModel,
+            final VolumeParameterListModel volumeParameterListModel,
+            final PermissionListModel<VolumeListModel> permissionListModel,
             final VolumeEventListModel volumeEventListModel, final 
VolumeGeoRepListModel geoRepListModel) {
         this.brickListModel = volumeBrickListModel;
         this.geoRepListModel = geoRepListModel;
@@ -231,7 +232,8 @@
     }
 
     private void setDetailList(final VolumeGeneralModel volumeGeneralModel,
-            final VolumeParameterListModel volumeParameterListModel, final 
PermissionListModel permissionListModel,
+            final VolumeParameterListModel volumeParameterListModel,
+            final PermissionListModel<VolumeListModel> permissionListModel,
             final VolumeEventListModel volumeEventListModel) {
 
         List<EntityModel> list = new ArrayList<EntityModel>();


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

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

Reply via email to