Moodle

Adding comment to database entry leads to wrong entry

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.9.3
  • Fix Version/s: 1.9.5, 2.0
  • Labels:
    None

Description

In the Modules & Plugins database, search for a plugin, for example 'lightbox' and view the returned entry.

Click 'Add comment'.

You're now adding a comment to the '123flashchat module', where you'll notice others have had the same problem as there are a number of comments relating to different modules...

Issue Links

Activity

Hide
Helen Foster added a comment -

Paul, thanks for your report.

It seems that having &filter=1 in the URL (from doing the search previously) results in the problem of the comment being added to the wrong entry i.e.

http://moodle.org/mod/data/view.php?d=13&rid=1021 - comment added to correct entry
http://moodle.org/mod/data/view.php?d=13&rid=1021&filter=1 - comment added to wrong entry

Eloy, is it possible for this issue to be fixed in 1.9.5, as it seems to be causing an increasing amount of confusion in the modules and plugins database on moodle.org.

Show
Helen Foster added a comment - Paul, thanks for your report. It seems that having &filter=1 in the URL (from doing the search previously) results in the problem of the comment being added to the wrong entry i.e. http://moodle.org/mod/data/view.php?d=13&rid=1021 - comment added to correct entry http://moodle.org/mod/data/view.php?d=13&rid=1021&filter=1 - comment added to wrong entry Eloy, is it possible for this issue to be fixed in 1.9.5, as it seems to be causing an increasing amount of confusion in the modules and plugins database on moodle.org.
Hide
Eloy Lafuente (stronk7) added a comment -

Attaching patch that changes current behaviour of the "Add Comment" link from the "floating" page identification to use the "record->id" more deterministic one. Seems to be working ok here and I cannot imagine any drawback).

Ciao

Show
Eloy Lafuente (stronk7) added a comment - Attaching patch that changes current behaviour of the "Add Comment" link from the "floating" page identification to use the "record->id" more deterministic one. Seems to be working ok here and I cannot imagine any drawback). Ciao
Hide
Eloy Lafuente (stronk7) added a comment -

Assigning to Dongsheng and addressing for 1.9.5. Please test & apply.

Also, Dong, I'd recommend to, once applied, say to Martin about this (and MDL-18136) in order to get moodle.org database module updated asap.

Ciao

Show
Eloy Lafuente (stronk7) added a comment - Assigning to Dongsheng and addressing for 1.9.5. Please test & apply. Also, Dong, I'd recommend to, once applied, say to Martin about this (and MDL-18136) in order to get moodle.org database module updated asap. Ciao
Hide
Dongsheng Cai added a comment -

Worked fine for me, committed ,thanks, Eloy

Show
Dongsheng Cai added a comment - Worked fine for me, committed ,thanks, Eloy
Hide
Dongsheng Cai added a comment -

Fixed, please review.

Show
Dongsheng Cai added a comment - Fixed, please review.
Hide
Petr Škoda (skodak) added a comment -

reviewed code and tested, works fine, thanks!

Show
Petr Škoda (skodak) added a comment - reviewed code and tested, works fine, thanks!
Hide
Eloy Lafuente (stronk7) added a comment -

Just noticed this was reverted by mistake in:

http://cvs.moodle.org/moodle/mod/data/lib.php?r1=1.137.2.50&r2=1.137.2.51

So I've reapplied it. Ciao

Show
Eloy Lafuente (stronk7) added a comment - Just noticed this was reverted by mistake in: http://cvs.moodle.org/moodle/mod/data/lib.php?r1=1.137.2.50&r2=1.137.2.51 So I've reapplied it. Ciao

Dates

  • Created:
    Updated:
    Resolved: