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

not able to edit records using the phpMyAdmin plugin for moodle

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 1.8
    • Fix Version/s: None
    • Component/s: Administration
    • Labels:
      None
    • Database:
      MySQL
    • Affected Branches:
      MOODLE_18_STABLE

      Description

      Bug only appeared when phpMyAdmin version was updated to 2.9.x

      Edit a record, make a change to a field and then save. Changes not saved and 2 new (blank) records are added to the table.
      Deletion of records also throws an error.

      PHP Notice messages thrown at time of saving changes:
      [Thu Jan 11 09:10:34 2007] [error] PHP Notice: Undefined variable: session_max_rows in /home/moodtest/public_html/admin/mysql/tbl_replace.php on line 80
      [Thu Jan 11 09:10:34 2007] [error] PHP Notice: Undefined variable: disp_direction in /home/moodtest/public_html/admin/mysql/tbl_replace.php on line 81
      [Thu Jan 11 09:10:34 2007] [error] PHP Notice: Undefined variable: repeat_cells in /home/moodtest/public_html/admin/mysql/tbl_replace.php on line 82

      Tested on user_info_field and tags table.

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            skodak Petr Skoda added a comment -

            Confirming. I guess the problem is not in our integration, it is IMHO caused by the recent security improvements in phpmyadmin in handling of global variables. I have found some more similar problems. I will have to make a default install first to file proper bug reports upstream

            Show
            skodak Petr Skoda added a comment - Confirming. I guess the problem is not in our integration, it is IMHO caused by the recent security improvements in phpmyadmin in handling of global variables. I have found some more similar problems. I will have to make a default install first to file proper bug reports upstream
            Hide
            dougiamas Martin Dougiamas added a comment -

            Yes I'm sticking to the old version ...

            Can we roll-back our CVS copy to the last known working version?

            Show
            dougiamas Martin Dougiamas added a comment - Yes I'm sticking to the old version ... Can we roll-back our CVS copy to the last known working version?
            Hide
            skodak Petr Skoda added a comment -

            Old versions have several published security problems, I am working on solution now...

            Show
            skodak Petr Skoda added a comment - Old versions have several published security problems, I am working on solution now...
            Hide
            skodak Petr Skoda added a comment -

            fixed in cvs, apparently the problem was caused by deregisterglobals hack for 1.5, it works now but 1.5.x is not supported anymore because phpmyadmin is not compatible(==sacure) with register globals emulated in 1.5.x

            imported new 2.9.2 with security fixes from upstream

            closing now, please reopen if needed.

            Show
            skodak Petr Skoda added a comment - fixed in cvs, apparently the problem was caused by deregisterglobals hack for 1.5, it works now but 1.5.x is not supported anymore because phpmyadmin is not compatible(==sacure) with register globals emulated in 1.5.x imported new 2.9.2 with security fixes from upstream closing now, please reopen if needed.

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: