[MDL-65307] Clicking on "Get Predictions" in the navigation bar leads to error - Moodle Tracker

XML

Word

Printable

Details

Type: Bug
Status: Closed
Priority: Minor
Resolution: Fixed
Affects Version/s: 3.7
Fix Version/s: 3.7
Component/s: Analytics, Navigation
Labels:
- ci
- triaged

Affected Branches:
MOODLE_37_STABLE
Fixed Branches:
MOODLE_37_STABLE
Pull from Repository:
git://github.com/dmonllao/moodle.git
Pull Master Branch:
~~MDL-65307~~_master
Pull Master Diff URL:
https://github.com/dmonllao/moodle/compare/MDL-65307_master~...MDL-65307_master
Testing Instructions:
Hide

Log in as admin

Go to site admin > analytics > analytics settings

Disable 'onlycli'

Go to site admin > analytics > analytics models

Click on "Evaluate" for the model "Students at risk of dropping out"

The breadcrumb SHOULD contain a "Evaluate" item and it SHOULD be just text, NOT a link
Show
Log in as admin Go to site admin > analytics > analytics settings Disable 'onlycli' Go to site admin > analytics > analytics models Click on "Evaluate" for the model "Students at risk of dropping out" The breadcrumb SHOULD contain a "Evaluate" item and it SHOULD be just text, NOT a link

Description

While playing with ~~MDL-64783~~, it was detected that the "Get Predictions" link in the navigation bad (breadcrumb) does lead to missing sesskey problem.

Not sure if it's the unique case where an action is also clickable in the navigation bar, maybe it shouldn't, specially if you are already there, in the very same page (via POST) from the models page.

Reproduced in 3.7, it may affect other versions and other elements.

Attachments

- Sort By Name
- Sort By Date
- Ascending
- Descending
- Thumbnails
- List
- Download All

get_predictions_sesskey.png
76 kB
09/Apr/19 6:40 AM
MDL-65307.jpg
20 kB
16/Apr/19 9:58 AM

Issue Links

Discovered while testing

MDL-64783 Create a new model for upcoming activities due

Closed

Activity

People

Assignee:: David Monllaó

Reporter:: Eloy Lafuente (stronk7)

Integrator:: Eloy Lafuente (stronk7)

Tester:: Anna Carissa Sadia

Participants:: Anna Carissa Sadia, Bas Brands, CiBoT, David Monllaó, Eloy Lafuente (stronk7)

Component watchers:: David Woloszyn, Huong Nguyen, Jake Dallimore, Meirza, Michael Hawkins, Raquel Ortega, Safat Shahin, Stevani Andolo, David Woloszyn, Huong Nguyen, Jake Dallimore, Meirza, Michael Hawkins, Raquel Ortega, Safat Shahin, Stevani Andolo

Votes:: 0 Vote for this issue

Watchers:: 3 Start watching this issue

Dates

Created:: 09/Apr/19 6:38 AM

Updated:: 18/Apr/19 2:33 AM

Resolved:: 18/Apr/19 2:33 AM

Fix Release Date:: 20/May/19

Time Tracking

Estimated:

Not Specified

Remaining:

Logged:

1h 7m