
from django.utils.translation import ugettext_lazy as _

import horizon

from openstack_dashboard.api import base
from openstack_dashboard.dashboards.admin import dashboard


class Ironic(horizon.Panel):
    name = _("Ironic Bare Metal Provisioning")
    slug = 'ironic'
    permissions = ('openstack.roles.admin',)

    def allowed(self, context):
        request = context['request']
        if not base.is_service_enabled(request, 'baremetal'):
            return False
        else:
            return super(Ironic, self).allowed(context)

    def nav(self, context):
        request = context['request']
        if not base.is_service_enabled(request, 'baremetal'):
            return False
        else:
            return True


dashboard.Admin.register(Ironic)


from django.utils.translation import ugettext_lazy as _

import horizon

from openstack_dashboard.dashboards.admin import dashboard


class Ironic(horizon.Panel):
    name = _("Ironic Bare Metal Provisioning")
    slug = 'ironic'
    permissions = ('openstack.roles.admin', 'openstack.services.baremetal',)


dashboard.Admin.register(Ironic)

Use permissions attribute to detect ironic service
Use permissions attribute to detect ironic service

Horizon implements a logic to enable/disable panel by permissions
defined in each panel class. This change replaces the current redundant
logic by that built-in feature to simplify how we define requirements
of the Ironic panels.

Change-Id: I4a9dabfea79c23155fb8986fe8386202d7474485

  
  from django.utils.translation import ugettext_lazy as _
  
  import horizon
  
- from openstack_dashboard.api import base
  from openstack_dashboard.dashboards.admin import dashboard
  
  
  class Ironic(horizon.Panel):
      name = _("Ironic Bare Metal Provisioning")
      slug = 'ironic'
+     permissions = ('openstack.roles.admin', 'openstack.services.baremetal',)
-     permissions = ('openstack.roles.admin',)
- 
-     def allowed(self, context):
-         request = context['request']
-         if not base.is_service_enabled(request, 'baremetal'):
-             return False
-         else:
-             return super(Ironic, self).allowed(context)
- 
-     def nav(self, context):
-         request = context['request']
-         if not base.is_service_enabled(request, 'baremetal'):
-             return False
-         else:
-             return True
  
  
  dashboard.Admin.register(Ironic)
  
Use permissions attribute to detect ironic service
## Code Before:

from django.utils.translation import ugettext_lazy as _

import horizon

from openstack_dashboard.api import base
from openstack_dashboard.dashboards.admin import dashboard


class Ironic(horizon.Panel):
    name = _("Ironic Bare Metal Provisioning")
    slug = 'ironic'
    permissions = ('openstack.roles.admin',)

    def allowed(self, context):
        request = context['request']
        if not base.is_service_enabled(request, 'baremetal'):
            return False
        else:
            return super(Ironic, self).allowed(context)

    def nav(self, context):
        request = context['request']
        if not base.is_service_enabled(request, 'baremetal'):
            return False
        else:
            return True


dashboard.Admin.register(Ironic)

## Instruction:
Use permissions attribute to detect ironic service
## Code After:

from django.utils.translation import ugettext_lazy as _

import horizon

from openstack_dashboard.dashboards.admin import dashboard


class Ironic(horizon.Panel):
    name = _("Ironic Bare Metal Provisioning")
    slug = 'ironic'
    permissions = ('openstack.roles.admin', 'openstack.services.baremetal',)


dashboard.Admin.register(Ironic)
