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

Inconsistency in who can view unapproved glossary entries on tag index page

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 3.3
    • 3.3
    • Glossary, Tags
    • MOODLE_33_STABLE
    • MOODLE_33_STABLE
    • wip-MDL-58784-master
    • Hide
      1. Create a glossary in a course, set automatic approval to No
      2. As a student add an entry with a tag
      3. Go to http://yourmoodlesite/tag , make sure you can see your entry when you click on a tag name
      4. As another student in the same course go to tags page, make sure you can NOT see this entry when you click on a tag name
      5. As a teacher go to tags page, make sure you can see your entry when you click on a tag name
      6. As a teacher approve entry
      7. As another student make sure you now can see the tagged entry on tags page
      Show
      Create a glossary in a course, set automatic approval to No As a student add an entry with a tag Go to http://yourmoodlesite/tag , make sure you can see your entry when you click on a tag name As another student in the same course go to tags page, make sure you can NOT see this entry when you click on a tag name As a teacher go to tags page, make sure you can see your entry when you click on a tag name As a teacher approve entry As another student make sure you now can see the tagged entry on tags page

    Description

      As discovered in MDLQA-10972 and commented by marina in MDL-57456:

      yes there is some inconsistency now.
      We either should not show any unapproved glossary entries on the tag search page or show them to everybody who can see the entries (author and user with capability to approve).
      At the moment author can not see tagged entries if they are not approved but teacher can

      Attachments

        Issue Links

          Activity

            People

              marina Marina Glancy
              tsala Helen Foster
              Andrew Lyons Andrew Lyons
              Dan Poltawski Dan Poltawski
              Ankit Agarwal Ankit Agarwal
              Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Laurent David, Raquel Ortega, Sara Arjona (@sarjona)
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                15/May/17