Details

    • Rank:
      17121

      Description

      Noticed this by chance. In load_current_settings() in /admin/roles/lib.php it appears that we're not closing the record set

        Activity

        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
        Andrew Davis added a comment -

        rebased

        Show
        Andrew Davis added a comment - rebased
        Hide
        Eloy Lafuente (stronk7) added a comment -

        Integrated, thanks!

        Show
        Eloy Lafuente (stronk7) added a comment - Integrated, thanks!
        Hide
        Helen Foster added a comment -

        Tested as described.

        Show
        Helen Foster added a comment - Tested as described.
        Hide
        Eloy Lafuente (stronk7) added a comment -

        Closing this, it's already part of upstream, thanks!

        Show
        Eloy Lafuente (stronk7) added a comment - Closing this, it's already part of upstream, thanks!

          People

          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: