[#MDL-20218] Should be looking at $PAGE->course when establishing settings for user context

Issue Details (XML | Word | Printable)

Key:	MDL-20218
Type:	Sub-task
Status:	Resolved
Resolution:	Fixed
Priority:	Minor
Assignee:	Sam Hemelryk
Reporter:	Sam Hemelryk
Votes:	0
Watchers:	0

Operations

Add/Edit UI Mockup to this issue

If you were logged in you would be able to see more operations.

Moodle

MDL-14632

Should be looking at $PAGE->course when establishing settings for user context

Created: 04/Sep/09 11:56 AM Updated: 08/Sep/09 10:47 AM

Return to search

Component/s:

Lib

Affects Version/s:

2.0

Fix Version/s:

2.0

Participants:	Sam Hemelryk
Security Level:	None
Difficulty:	Easy
Resolved date:	08/Sep/09
Affected Branches:	MOODLE_20_STABLE
Fixed Branches:	MOODLE_20_STABLE

Description

« Hide

Currently load_for_user is looking for the course id as a parameter when loading setting for the user context.
Really it should first be looking to see if $PAGE->course has been set first, and then failing over to the HTTP params.

All

Comments

Change History

Version Control

Sort Order:

[ Permalink | « Hide ]

Sam Hemelryk added a comment - 08/Sep/09 10:47 AM

Done and done, now looks for PAGE->course and if that is not set (possible if AJAX request) still looks for optional_param

All content on this web site is made available under the GNU General Public License, unless otherwise stated.