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
    • Rank:
      38334

      Description

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

        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: