[#MDL-17548] MNET jump links in forum emails have incorrect path when wwwroot contains a path

Issue Details (XML | Word | Printable)

Key:	MDL-17548
Type:	Bug
Status:	Closed
Resolution:	Fixed
Priority:	Major
Assignee:	Martin Dougiamas
Reporter:	Jonathan Harker
Votes:	1
Watchers:	3

Operations

Add/Edit UI Mockup to this issue

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

Moodle

MNET jump links in forum emails have incorrect path when wwwroot contains a path

Created: 08/Dec/08 08:15 AM Updated: 25/Mar/09 12:53 AM

Return to search

Component/s:

Forum, Messages, Networking

Affects Version/s:

1.9.3, 2.0

Fix Version/s:

1.9.4

File Attachments:

MDL-17548.HEAD.patch (1.0 kB)
2.

MDL-17548.MOODLE_19_STABLE.patch (1.0 kB)

Participants:	Dan Poltawski, Eloy Lafuente (stronk7), Jonathan Harker, Martin Dougiamas and Peter Bulmer
Security Level:	None
QA Assignee:	Petr Skoda
Resolved date:	23/Mar/09
Affected Branches:	MOODLE_19_STABLE, MOODLE_20_STABLE
Fixed Branches:	MOODLE_19_STABLE

Description

« Hide

If $CFG->wwwroot contains a path component (e.g. /moodle in http://contentprovider.com/moodle or similar), then the MNET jump URLs to the forum threads on the CP that get written into forum emails contain a double-up of the path - on the end of hostwwwroot and the beginning of wantsurl. For instance:

http://idprovider.com/mnet/jump.php?hostwwwroot=http://contentprovider.com/moodle&wantsurl=/moodle/mod/forum/discuss.php...

Attached fix.

All

Comments

Change History

Version Control

Sort Order:

jonathanharker committed 1 file to 'Moodle CVS' - 15/Dec/08 07:50 AM

~~MDL-17548~~ MNET: Fix email links for sites with path component in wwwroot

Where Moodle sites had a path in their wwwroot, the MNET function that
forced remote users to go via their identity provider (to make sure they
were logged in) previously directed the user back to a URL like
contentprovider.com/moodle/moodle/mod/forum/view.php?f=7 where there
should only be one /moodle in the middle of the URL.

MODIFY

mnet/lib.php

Rev. 1.31

(+10 -2 lines)

jonathanharker committed 1 file to 'Moodle CVS' on branch 'MOODLE_19_STABLE' - 15/Dec/08 08:40 AM

~~MDL-17548~~ MNET: Fix email links for sites with path component in wwwroot

Where Moodle sites have a path in their wwwroot, the MNET function that
forced remote users to go via their identity provider (to make sure they
were logged in) previously directed the user back to a URL like
contentprovider.com/moodle/moodle/mod/forum/view.php?f=7 where there
should only be one /moodle in the middle of the URL.

Slightly tweaked from HEAD to support PHP4

MODIFY

mnet/lib.php

Rev. 1.16.2.9

(+10 -2 lines)

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