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

Clarify search_references[_count]() methods usage and add tests for them

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: 2.3, 2.4
    • Fix Version/s: 2.3.1
    • Component/s: Files API
    • Labels:
      None
    • Testing Instructions:
      Hide

      Testing difficulty: trivial

      Make sure that the filemanager correctly reports the number and the list of aliases that refer to a given file.

      Show
      Testing difficulty: trivial Make sure that the filemanager correctly reports the number and the list of aliases that refer to a given file.
    • Affected Branches:
      MOODLE_23_STABLE, MOODLE_24_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE
    • Pull from Repository:

      Description

      Methods like file_storage::search_references() or search_references_count() need to accept $repositoryid too. The reference field is supposed to be unique per repository instance, not world-wide.

        Attachments

          Issue Links

          Discovered while testing

          Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-34194 Some annoyances while creating aliases from filesystem repository

          • Minor - Minor loss of function where workaround is possible
          • Closed
          has a non-specific relationship to

          Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-33568 count_records_sql does not return an integer

          • Minor - Minor loss of function where workaround is possible
          • Closed
          will help resolve

          Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-33368 Improve filelib references unit test coverage

          • Minor - Minor loss of function where workaround is possible
          • Closed

            Activity

              People

              Assignee:
              mudrd8mz David Mudrák (@mudrd8mz)
              Reporter:
              mudrd8mz David Mudrák (@mudrd8mz)
              Integrator:
              Aparup Banerjee
              Tester:
              Eloy Lafuente (stronk7)
              Participants:
              Component watchers:
              Matteo Scaramuccia, Andrew Nicols, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                9/Jul/12