Change in ovirt-engine[master]: core: unify GetImagesList using ImageFileType.All

amureini at redhat.com amureini at redhat.com
Thu Apr 18 16:32:11 UTC 2013


Allon Mureinik has submitted this change and it was merged.

Change subject: core: unify GetImagesList using ImageFileType.All
......................................................................


core: unify GetImagesList using ImageFileType.All

It is possible to unify the two separate requests for Floppy and ISO
images using the enum ImageFileType.All. In order to do so the stored
procedures and the DAO methods have been updated to not filter by type
when ImageFileType.All is used.

Change-Id: I4e8fb85c3f0d58778b03fd0aa101d95bbeebf3f6
Signed-off-by: Federico Simoncelli <fsimonce at redhat.com>
---
M backend/manager/dbscripts/repo_files_meta_data_sp.sql
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/IsoDomainListSyncronizer.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/RepoFileMetaDataDAODbFacadeImpl.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageIsoListModel.java
4 files changed, 75 insertions(+), 84 deletions(-)

Approvals:
  Allon Mureinik: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I4e8fb85c3f0d58778b03fd0aa101d95bbeebf3f6
Gerrit-PatchSet: 7
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Ravi Nori <rnori at redhat.com>



More information about the Engine-commits mailing list