Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-59898

Delete symbol missing in "Permissions in Course" - after ajax adding

    XMLWordPrintable

Details

    • MOODLE_32_STABLE, MOODLE_33_STABLE
    • MOODLE_32_STABLE, MOODLE_33_STABLE
    • wip-MDL-59898-master-test
    • Hide
      1. Go to a course and then go to [Course administration  ► Users  ► Permissions].
      2. Click the '+' symbol in the roles with permission column.
      3. Make sure that an 'X' symbol is next to the role that you selected (see screen shots).
      4. Click the delete icon and ensure that the role has now been deleted form that column.
      5. Do the same check for the prohibited column.
      Show
      Go to a course and then go to [Course administration  ► Users  ► Permissions] . Click the '+' symbol in the roles with permission column. Make sure that an 'X' symbol is next to the role that you selected (see screen shots). Click the delete icon and ensure that the role has now been deleted form that column. Do the same check for the prohibited column.

    Description

      As Admin when you go to "Home -> Courses -> MyCourse -> Users -> Permissions"
      and (wait for popupmenu) add an allowance for a role, no delete icon for this new role is shown next to it.
      For it to appear a refresh of the page is necessary.

      The issue seems to be a missing icon variable for the mustache template permissionmanager_role.
      At least the issue was resolved when i added following lines to lib/amd/scr/permissionmanager.js (js dev mode)

       

      //begins with line 106
      		 
      case 'allow':
      		 
      templatedata.spanclass = 'allowed';
      		 
      templatedata.linkclass = 'preventlink';
      		 
      templatedata.action = 'prevent';
      		 
      templatedata.icon = 't/delete'; // <-- Line added
      		 
      break;
      		 
      case 'prohibit':
      		 
      templatedata.spanclass = 'forbidden';
      		 
      templatedata.linkclass = 'unprohibitlink';
      		 
      templatedata.action = 'unprohibit';
      		 
      templatedata.icon = 't/delete'; // <-- Line added
      		 
      break;

      git: https://github.com/TobiGa/moodle/commit/81fd3d522b47d36cf93f00297aac219a56663cef

       

       

      Attachments

        Activity

          People

            toga Tobias Garske
            toga Tobias Garske
            Adrian Greeve Adrian Greeve
            David Monllaó David Monllaó
            Mark Nelson Mark Nelson
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:
              11/Sep/17