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

add an "alphabetical order" option to the random glossary entry block

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.3
    • Fix Version/s: 2.5
    • Component/s: Blocks, Glossary
    • Labels:

      Description

      See this discussion: http://moodle.org/mod/forum/discuss.php?d=213034
      I am attaching a very simple patch to add an "alphabetical order" display option to the random glossary entry block.
      Joseph

        Gliffy Diagrams

          Activity

          Hide
          salvetore Michael de Raadt added a comment -

          Thanks for suggesting that and providing a patch.

          Show
          salvetore Michael de Raadt added a comment - Thanks for suggesting that and providing a patch.
          Hide
          rezeau Joseph Rézeau added a comment -

          starting review

          Show
          rezeau Joseph Rézeau added a comment - starting review
          Hide
          rezeau Joseph Rézeau added a comment -

          Hi Jason,

          On my local install of moodle I checked from your Repository: git://github.com/phalacee/moodle.git the wip-MDL-35842-master branch to my moodle 2.3.
          However, this throws the error: Table "course_format_options" does not exist
          The problem is that we are currently testing the patch for moodle 2.3 but your wip-MDL-35842-master branch seems to have been made for the Moodle master (aka 2.4) branch.
          Can you confirm this?
          Should I rather check your patch to my local 2.4 moodle install rather than 2.3?
          Joseph

          Show
          rezeau Joseph Rézeau added a comment - Hi Jason, On my local install of moodle I checked from your Repository: git://github.com/phalacee/moodle.git the wip- MDL-35842 -master branch to my moodle 2.3. However, this throws the error: Table "course_format_options" does not exist The problem is that we are currently testing the patch for moodle 2.3 but your wip- MDL-35842 -master branch seems to have been made for the Moodle master (aka 2.4) branch. Can you confirm this? Should I rather check your patch to my local 2.4 moodle install rather than 2.3? Joseph
          Hide
          phalacee Jason Fowler added a comment -

          Hi Joseph,

          My patch is only for 2.4, and this is an improvement, which means it should only be submitted for 2.4

          I have no idea what the course_format_options has to do with this patch in particular, but there has been a lot of work on course formats for 2.4 - maybe cherry-picking the patch back on to a 2.3 branch will avoid those issues.

          Hope this helps.

          Show
          phalacee Jason Fowler added a comment - Hi Joseph, My patch is only for 2.4, and this is an improvement, which means it should only be submitted for 2.4 I have no idea what the course_format_options has to do with this patch in particular, but there has been a lot of work on course formats for 2.4 - maybe cherry-picking the patch back on to a 2.3 branch will avoid those issues. Hope this helps.
          Hide
          rezeau Joseph Rézeau added a comment -

          Tested on a moodle 2.4 installation. Works as expected.

          Show
          rezeau Joseph Rézeau added a comment - Tested on a moodle 2.4 installation. Works as expected.
          Hide
          rezeau Joseph Rézeau added a comment -

          works as expected (moodle 2.4)

          Show
          rezeau Joseph Rézeau added a comment - works as expected (moodle 2.4)
          Hide
          phalacee Jason Fowler added a comment -

          Thanks Joseph, pushing for integration now.

          Show
          phalacee Jason Fowler added a comment - Thanks Joseph, pushing for integration now.
          Hide
          samhemelryk Sam Hemelryk added a comment -

          Thanks Jason, this has been integrated now.

          Show
          samhemelryk Sam Hemelryk added a comment - Thanks Jason, this has been integrated now.
          Hide
          abgreeve Adrian Greeve added a comment -

          Tested on the master integration branch.
          Works as described.
          Test passed.

          Show
          abgreeve Adrian Greeve added a comment - Tested on the master integration branch. Works as described. Test passed.
          Hide
          stronk7 Eloy Lafuente (stronk7) added a comment -

          And your fantastic code has met core, hope they become good friends for a long period.

          Closing, thanks!

          Show
          stronk7 Eloy Lafuente (stronk7) added a comment - And your fantastic code has met core, hope they become good friends for a long period. Closing, thanks!
          Hide
          marycooch Mary Cooch added a comment -

          Removing the qa_test_required label as there is a test for this here MDLQA-5256 ready for the next cycle.

          Show
          marycooch Mary Cooch added a comment - Removing the qa_test_required label as there is a test for this here MDLQA-5256 ready for the next cycle.
          Hide
          marycooch Mary Cooch added a comment - - edited

          Removing docs_required as this is now documented here http://docs.moodle.org/25/en/Random_glossary_entry_block

          Show
          marycooch Mary Cooch added a comment - - edited Removing docs_required as this is now documented here http://docs.moodle.org/25/en/Random_glossary_entry_block

            People

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

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                14/May/13