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

Glossary filter JS popup uselessly links to itself for recursive glossary entry

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.3, 2.7.3, 2.8.1, 2.8.6, 2.9
    • Fix Version/s: 2.8.7, 2.9.1
    • Component/s: Glossary
    • Labels:
    • Testing Instructions:
      Hide
      Test 1
      1. Enable the glossary filter via Site administration > Plugins > Filters > Manage filters.
      2. Create two glossary entries that contain the definition and link to each other (ensure both have 'This entry should be automatically linked' checked). Example -

        Concept: Moodle
        Definition: Moodle is better than Blackboard
         
        Concept: Blackboard
        Definition: Blackboard is not as good as Moodle
        

      3. Edit a course section and add the concepts to the description.
      4. Click on one link and ensure it shows one JS popup with the definition of the concept.
      5. Make sure that concept is not linked to itself in popup, ie is not clickable.
      6. Click on the link of the other concept in the pop-up and ensure another pop-up is created with a slight offset.
      Test 2
      1. Repeat the above on a few mobile devices and ensure all is good.
      Show
      Test 1 Enable the glossary filter via Site administration > Plugins > Filters > Manage filters. Create two glossary entries that contain the definition and link to each other (ensure both have 'This entry should be automatically linked' checked). Example - Concept: Moodle Definition: Moodle is better than Blackboard   Concept: Blackboard Definition: Blackboard is not as good as Moodle Edit a course section and add the concepts to the description. Click on one link and ensure it shows one JS popup with the definition of the concept. Make sure that concept is not linked to itself in popup, ie is not clickable. Click on the link of the other concept in the pop-up and ensure another pop-up is created with a slight offset. Test 2 Repeat the above on a few mobile devices and ensure all is good.
    • Affected Branches:
      MOODLE_23_STABLE, MOODLE_27_STABLE, MOODLE_28_STABLE, MOODLE_29_STABLE
    • Fixed Branches:
      MOODLE_28_STABLE, MOODLE_29_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-33765_master_offset

      Description

      As discovered in MDL-32279. The JS popup to a glossary entry is filtered, causing a link to itself.

      Replication steps:

      1. Enable glossary filter @ admin->plugins->filters->manage. (set to content and headings)
      2. Create one glossary entry with the definition containing the concept
      3. Create one forum post containing some of the concepts created in 3/. Save it.
      4. Clicking the link shows one JS popup with the definition of the concept

      Expected result:
      The concept is not linked to itself in the JS popup definition

      Actual result:
      The concept is linked to itself in the JS popup definition

        Attachments

          Issue Links

            Activity

              People

              • Votes:
                1 Vote for this issue
                Watchers:
                7 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  6/Jul/15