We launched new forums in March 2019—join us there. In a hurry for help with your website? Get Help Now!
    • 28042 ☆ A M B ☆
    • 24,524 Posts
    I’ve noticed a number of permission issues (at least in 3 and 4). l3mnu.php, for example, has this:

    <?php if($modx->hasPermission('new_user') || $modx->hasPermission('edit_user') || $modx->hasPermission('new_role') || $modx->hasPermission('edit_role') || $modx->hasPermission('access_permissions')) { ?>
    <td class="menuHeader"><?php echo $_lang["users"]; ?></td>
    <?php } ?>
    


    It doesn’t allow for a role that allows web user management, but not manager user management. The same in the actual links for the user management section;

    <?php if($modx->hasPermission('new_user') || $modx->hasPermission('edit_user') || $modx->hasPermission('new_role') || $modx->hasPermission('edit_role') || $modx->hasPermission('access_permissions')) { ?>
    <td valign="top">
    


    Is there some reason for this, or is it just an oversight and should get noted in the bug tracker?

    I’ve also reworked l4mnu.php so that it actually works, and works in IE as well.
      Studying MODX in the desert - http://sottwell.com
      Tips and Tricks from the MODX Forums and Slack Channels - http://modxcookbook.com
      Join the Slack Community - http://modx.org
      • 32963
      • 1,732 Posts
      Seems like a bug to me
        xWisdom
        www.xwisdomhtml.com
        The fear of the Lord is the beginning of wisdom:
        MODx Co-Founder - Create and do more with less.
        • 25663 MODX Staff
        • 12,272 Posts
        Fix committed, thatnks Susan!
          Ryan Thrash, MODX Co-Founder
          Follow me on Twitter at @rthrash or catch my occasional unofficial thoughts at thrash.me
          • 33337
          • 3,975 Posts
          hmm... thanks Ryan and Susan..

          actually, l3mnu.php is made in *my hands*, so most of time code *slips out*... laugh

          *aham*
            Zaigham R - MODX Professional | Skype | Email | Twitter

            Digging the interwebs for #MODX gems and bringing it to you. modx.link
            • 28042 ☆ A M B ☆
            • 24,524 Posts
            And the l4mnu was my baby...hmmm, maybe there is a connection between this and two of my three kids being ex-cons here?
              Studying MODX in the desert - http://sottwell.com
              Tips and Tricks from the MODX Forums and Slack Channels - http://modxcookbook.com
              Join the Slack Community - http://modx.org