Moodle
  1. Moodle
  2. MDL-30198

Title label "Current glossary" on "Import entries from XML file" page needs to be fixed.

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0, 2.1
    • Fix Version/s: 2.1.4, 2.2.1
    • Component/s: Glossary
    • Labels:
    • Rank:
      26122

      Description

      The title label "Current glossary" on "Import entries from XML file" page needs to be fixed as below.

      Program file: /mod/glossary/import_form.php
      Line: 20

      before
      $mform->addElement('select', 'dest', get_string('currentglossary', 'glossary'), $options);
      
      after
      $mform->addElement('select', 'dest', get_string('destination', 'glossary'), $options);
      

        Activity

        Hide
        Michael de Raadt added a comment -

        Thanks for reporting that and for providing a solution.

        Show
        Michael de Raadt added a comment - Thanks for reporting that and for providing a solution.
        Hide
        Eloy Lafuente (stronk7) added a comment -

        The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

        TIA and ciao

        Show
        Eloy Lafuente (stronk7) added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
        Hide
        Sam Hemelryk added a comment -

        Thanks Ankit, this has been integrated now.

        Cheers
        Sam

        Show
        Sam Hemelryk added a comment - Thanks Ankit, this has been integrated now. Cheers Sam
        Hide
        Sam Hemelryk added a comment -

        Tested during integration

        Show
        Sam Hemelryk added a comment - Tested during integration
        Hide
        Eloy Lafuente (stronk7) added a comment -

        Whoever decided one week was worth 14 days had really one bad idea. Anyway, the nightmare is over, so thanks for your, once again, amazing contributions. Many, many thanks!

        Now... disconnect, relax and enjoy the next days, yay!

        Closing...ciao

        Show
        Eloy Lafuente (stronk7) added a comment - Whoever decided one week was worth 14 days had really one bad idea. Anyway, the nightmare is over, so thanks for your, once again, amazing contributions. Many, many thanks! Now... disconnect, relax and enjoy the next days, yay! Closing...ciao

          People

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

            Dates

            • Created:
              Updated:
              Resolved: