Details

      Description

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

        Gliffy Diagrams

          Attachments

            Activity

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

            rebased

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

            Integrated, thanks!

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

            Tested as described.

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

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

            Show
            stronk7 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:
                  Fix Release Date:
                  1/Aug/11