[MDL-33603] Database entries are always linked to wrong entry - Moodle Tracker

XML

Word

Printable

Testing Instructions:
Hide

Add a few entries to a database module

Make sure to include ##more## in your template so that you can see the more icon.

Click on "view as list"

Change the "Sort by" column and make sure that the first entry is different to the default. (clicking "Descending" should work)

Click on any of the "more" icons

[Test]
It takes you to the correct page with the entry details.
Show
Add a few entries to a database module Make sure to include ##more## in your template so that you can see the more icon. Click on "view as list" Change the "Sort by" column and make sure that the first entry is different to the default. (clicking "Descending" should work) Click on any of the "more" icons [Test] It takes you to the correct page with the entry details.
Affected Branches:
MOODLE_22_STABLE, MOODLE_23_STABLE
Fixed Branches:
MOODLE_21_STABLE, MOODLE_22_STABLE
Pull from Repository:
git://github.com/abgreeve/moodle.git
Pull Master Branch:
wip-~~MDL-33603~~-master-v2
Pull Master Diff URL:
https://github.com/abgreeve/moodle/compare/wip-MDL-33603-master-v2

Replication steps:-

Add a few entries to a database module
Make sure to include ##more## in your template so that you can see the more icon.
Click on "view as list"
Change the "Sort by" column and make sure that the first entry is different to the default. (clicking "Descending" should work)
Click on any of the "more" icons

Expected result:-
It takes you to the correct page with the entry details

Actual result:-
It takes to a page with details of some random entry.

Because of this issue all the databases on moodle.org are completely useless, such as Moodle Jobs, Moodle Buzz etc

For example http://moodle.org/mod/data/view.php?d=50 , try accessing any package from that url.

has been marked as being related by

MDL-38225 regression: database entries linked incorrectly (backport MDL-33603)

is a QA test written for

MDL-33797 Improve the database module paging system.

is duplicated by

MDL-33709 Database error in Moodle jobs on Moodle.org

MDL-33764 Database Search results display the wrong entry

Participants:: Adrian Greeve, Ankit Agarwal, Chris Follin, Dan Marsden, Dan Poltawski, Eloy Lafuente (stronk7), Fernando Oliveira, Helen Foster, Luis Gómez Caballero, Martin Dougiamas, Michael de Raadt, Mike Churchward, Patrick Pollet, Paul Holden, Rajesh Taneja, Séverin Terrier

Component watchers:: Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias, Sujith Haridasan