Moodle

not able to edit records using the phpMyAdmin plugin for moodle

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Minor 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.

Activity

Hide
Petr Škoda (skodak) 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
Petr Škoda (skodak) 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
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
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
Petr Škoda (skodak) added a comment -

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

Show
Petr Škoda (skodak) added a comment - Old versions have several published security problems, I am working on solution now...
Hide
Petr Škoda (skodak) 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
Petr Škoda (skodak) 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

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: