yes


On 24 April 2014 11:47, Alain Flaus <notifications@github.com> wrote:

> In Resources/views/Block/Dashboard/block_media.html.twig:
>
> > @@ -5,30 +5,32 @@
> >  {% block block_title %}{{ 'block.dashboard.media.title' | trans }}{% endblock %}
> >
> >  {% block block_content %}
> > -    <div class="span6">
> > -        <div class="block-content-block block-content-block-media">
> > -            <h4>{{ 'block.dashboard.media.medias.title' | trans }}</h4>
> > -            <a href="{{ mediaAdmin.generateUrl('create')}}" class="btn btn-small">
> > -                <i class="icon-plus"></i>&nbsp;
> > -                {%- trans from 'SonataAdminBundle' %}link_add{% endtrans -%}
> > -            </a>
> > -            <a href="{{ mediaAdmin.generateUrl('list')}}" class="btn btn-small">
> > -                <i class="icon-list"></i>&nbsp;
> > -                {%- trans from 'SonataAdminBundle' %}link_list{% endtrans -%}
> > -            </a>
> > +    {% if (is_granted('ROLE_SUPER_ADMIN') or is_granted('ROLE_ADMIN_MEDIA')) %}
>
> Instead of testing the two differents roles, I think it would be better if
> the ROLE_SUPER_ADMIN should have the ROLE_ADMIN_MEDIA in is hierarchy. No ?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/prestaconcept/PrestaSonataAdminExtendedBundle/pull/8/files#r11941734>
> .
>