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

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

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 3.3
    • Fix Version/s: 3.3
    • Component/s: Glossary, Tags
    • Labels:
    • Testing Instructions:
      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
    • Affected Branches:
      MOODLE_33_STABLE
    • Fixed Branches:
      MOODLE_33_STABLE
    • Pull Master Branch:
      wip-MDL-58784-master

      Description

      As discovered in MDLQA-10972 and commented by Marina Glancy 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

              • Assignee:
                marina Marina Glancy
                Reporter:
                tsala Helen Foster
                Peer reviewer:
                Andrew Nicols
                Integrator:
                Dan Poltawski
                Tester:
                Ankit Agarwal
                Participants:
                Component watchers:
                Adrian Greeve, Mihail Geshoski, Peter Dias, Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona), Víctor Déniz Falcón
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  15/May/17