yahoo-eng-team team mailing list archive

[Liyingjun <liyingjun1988@xxxxxxxxx>]

Public bug reported:

we can move set_id_as_name_if_empty to the api loop for rule_list,firewall_list and policy_list to reduce a extra loop . 
https://github.com/openstack/horizon/blob/master/openstack_dashboard/dashboards/project/firewalls/tabs.py#L61
https://github.com/openstack/horizon/blob/master/openstack_dashboard/dashboards/project/firewalls/tabs.py#L47
https://github.com/openstack/horizon/blob/master/openstack_dashboard/dashboards/project/firewalls/tabs.py#L82

** Affects: horizon
     Importance: Undecided
     Assignee: Liyingjun (liyingjun)
         Status: New

** Changed in: horizon
     Assignee: (unassigned) => Liyingjun (liyingjun)

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1370986

Title:
  move set_id_as_name_if_empty to the api side for rule_list, fwaas_list
  and policy_list

Status in OpenStack Dashboard (Horizon):
  New

Bug description:
  we can move set_id_as_name_if_empty to the api loop for rule_list,firewall_list and policy_list to reduce a extra loop . 
  https://github.com/openstack/horizon/blob/master/openstack_dashboard/dashboards/project/firewalls/tabs.py#L61
  https://github.com/openstack/horizon/blob/master/openstack_dashboard/dashboards/project/firewalls/tabs.py#L47
  https://github.com/openstack/horizon/blob/master/openstack_dashboard/dashboards/project/firewalls/tabs.py#L82

To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1370986/+subscriptions