Moodle
  1. Moodle
  2. MDL-31738

Missing function mod_glossary_get_file_info() : needed Recent Files

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1, 2.2
    • Fix Version/s: 2.1.5, 2.2.2
    • Component/s: Files API, Glossary
    • Labels:
    • Testing Instructions:
      Hide

      1. Create Glossary activity
      2. Add entry with embedded images
      3. Start editing description to any other activity and make sure that you can see in File Picker "Recent Files" the files you used in Glossary

      4. Create Database activity
      5. Create file and picture fields
      6. Create an entry with attached files
      7. Start editing description to any other activity and make sure that you can see in File Picker "Recent Files" the files you used in Database

      Show
      1. Create Glossary activity 2. Add entry with embedded images 3. Start editing description to any other activity and make sure that you can see in File Picker "Recent Files" the files you used in Glossary 4. Create Database activity 5. Create file and picture fields 6. Create an entry with attached files 7. Start editing description to any other activity and make sure that you can see in File Picker "Recent Files" the files you used in Database
    • Affected Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Pull Master Branch:
      wip-MDL-31738-master

      Description

      and now I discovered that mod_glossary_get_file_info() is also missing.
      I will definitely look carefully now at all modules

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            Marina Glancy added a comment -

            also mod_data !

            Show
            Marina Glancy added a comment - also mod_data !
            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
            Eloy Lafuente (stronk7) added a comment -

            Integrated thanks! (21, 22 & master)

            Side comment: Not important enough to reject but note that get_context_instance() is deprecated since 2.2.

            Show
            Eloy Lafuente (stronk7) added a comment - Integrated thanks! (21, 22 & master) Side comment: Not important enough to reject but note that get_context_instance() is deprecated since 2.2.
            Hide
            Adrian Greeve added a comment -

            Tested in master 2.2 and 2.1
            Worked well.
            Thanks Marina.

            Show
            Adrian Greeve added a comment - Tested in master 2.2 and 2.1 Worked well. Thanks Marina.
            Hide
            Eloy Lafuente (stronk7) added a comment -

            Well,

            I wish I said it every time
            you do the things you do.
            You always lend a helping hand,
            and I'm filled with gratitude.

            You are strong and generous
            for each and everyone one of us.
            I am eternally grateful,
            I cannot say thanks enough.

            Sorry for the (un)cool bit above, lol. Closing this as fixed. Ciao

            Show
            Eloy Lafuente (stronk7) added a comment - Well, I wish I said it every time you do the things you do. You always lend a helping hand, and I'm filled with gratitude. You are strong and generous for each and everyone one of us. I am eternally grateful, I cannot say thanks enough. Sorry for the (un)cool bit above, lol. Closing this as fixed. Ciao

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: