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

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

    Details

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

      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);

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            salvetore Michael de Raadt added a comment -

            Thanks for reporting that and for providing a solution.

            Show
            salvetore Michael de Raadt added a comment - Thanks for reporting that and for providing a solution.
            Hide
            stronk7 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
            stronk7 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
            samhemelryk Sam Hemelryk added a comment -

            Thanks Ankit, this has been integrated now.

            Cheers
            Sam

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

            Tested during integration

            Show
            samhemelryk Sam Hemelryk added a comment - Tested during integration
            Hide
            stronk7 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
            stronk7 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:
                  Fix Release Date:
                  9/Jan/12