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

Security overview report backported to 1.9 uses 2.0 API

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.9.3
    • Fix Version/s: 1.9.4
    • Component/s: Administration
    • Labels:
      None
    • Environment:
      Spotted at PostgreSQL 8.3
    • Difficulty:
      Easy
    • Affected Branches:
      MOODLE_19_STABLE
    • Fixed Branches:
      MOODLE_19_STABLE

      Description

      admin/report/security has been recently backported from HEAD to 19_STABLE. However, some code remained using new DB API returned values. Also, some links do not work in 1.9.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              mudrd8mz David Mudrák added a comment -

              Fixes:

              • admin/roles/manage.php instead of new admin/roles/define.php
              • results returned by old get_recordset_sql() must be iterated using rs_fetch_next_record() instead of new foreach()
              Show
              mudrd8mz David Mudrák added a comment - Fixes: admin/roles/manage.php instead of new admin/roles/define.php results returned by old get_recordset_sql() must be iterated using rs_fetch_next_record() instead of new foreach()
              Hide
              mudrd8mz David Mudrák added a comment -

              Fixed in CVS, please review and close

              Show
              mudrd8mz David Mudrák added a comment - Fixed in CVS, please review and close
              Hide
              skodak Petr Skoda added a comment -

              ooops, thanks going to review later today

              Show
              skodak Petr Skoda added a comment - ooops, thanks going to review later today
              Hide
              skodak Petr Skoda added a comment -

              thanks!

              Show
              skodak Petr Skoda added a comment - thanks!

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    28/Jan/09