From fca1dabf2611d461bbeed68777adb24c2fa3b59e Mon Sep 17 00:00:00 2001 From: liaozd Date: Fri, 25 Nov 2016 21:40:46 +0800 Subject: [PATCH] implements: make twice list iterations into one Closes-Bug: 1644817 Change-Id: Ie97659230ad59f56a0285e7f693683bde32dca77 --- openstack_dashboard/dashboards/project/images/utils.py | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/openstack_dashboard/dashboards/project/images/utils.py b/openstack_dashboard/dashboards/project/images/utils.py index 63760f0668..9bfee8cfa4 100644 --- a/openstack_dashboard/dashboards/project/images/utils.py +++ b/openstack_dashboard/dashboards/project/images/utils.py @@ -67,15 +67,14 @@ def get_available_images(request, project_id=None, images_cache=None): images = owned_images + public_images - # Remove duplicate images image_ids = [] final_images = [] for image in images: - if image.id not in image_ids: + if image.id not in image_ids and \ + image.container_format not in ('aki', 'ari'): image_ids.append(image.id) final_images.append(image) - return [image for image in final_images - if image.container_format not in ('aki', 'ari')] + return final_images def image_field_data(request, include_empty_option=False):