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

Reuse get_all_sections() function in get_array_of_activities()

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Won't Fix
    • Affects Version/s: 2.3
    • Fix Version/s: 2.3.2
    • Component/s: Course
    • Labels:

      Description

      This could potentially mean less DB queries (which is always a good thing), also promotes code reuse.

      The only problem I could see is due to get_all_sections() caching the sections where get_array_of_activities() may be expected to reload them. Testing has brought up no issues however.

        Attachments

          Activity

            People

            Assignee:
            sry_not4sale Aaron Barnes
            Reporter:
            sry_not4sale Aaron Barnes
            Peer reviewer:
            Sam Hemelryk
            Integrator:
            Dan Poltawski
            Tester:
            Dan Poltawski
            Participants:
            Component watchers:
            Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Sara Arjona (@sarjona)
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              10/Sep/12