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

Administration block displays link to Legacy course files when it should not

    XMLWordPrintable

Details

    • Bug
    • Resolution: Fixed
    • Minor
    • 3.2.1
    • 3.2
    • Files API
    • MOODLE_32_STABLE
    • MOODLE_32_STABLE
    • MDL-57256-master-legacylink
    • Hide
      1. Set up a course with Legacy files enabled
      2. TEST: Make sure that as an admin/teacher with the capability to manage legacy files, you can see the link.
      3. Switch role to one that does not have the capability to manage legacy files (student, guest)
      4. TEST: Make sure the link is not displayed if the user does not have the capability.
      Show
      Set up a course with Legacy files enabled TEST: Make sure that as an admin/teacher with the capability to manage legacy files, you can see the link. Switch role to one that does not have the capability to manage legacy files (student, guest) TEST: Make sure the link is not displayed if the user does not have the capability.

    Description

      As discovered by tsala in MDLSITE-4849 while testing moodle.org upgrade, there is a link "Legacy course files" displayed in the Administration block for students and guests accounts. Following the link as a student may lead to an error page "Sorry, but you do not currently have permissions to do that (Manage files)"

      Attachments

        Issue Links

          is a regression caused by

          New Feature - A new Moodle feature which has yet to be developed. MDL-55415 New Web Services for retrieving navigation and administration options available

          • Minor - Minor loss of function where workaround is possible
          • Closed
          will help resolve

          Sub-task - Issues are sometimes broken into multiple sub-tasks. MDLSITE-4849 Admin block includes 'Legacy course files' link

          • Low - Cosmetic problems, nice-to-have features
          • Closed

          Activity

            People

              mudrd8mz David Mudrák (@mudrd8mz)
              mudrd8mz David Mudrák (@mudrd8mz)
              Juan Leyva Juan Leyva
              Dan Poltawski Dan Poltawski
              John Okely John Okely
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                9/Jan/17