mahara-contributors team mailing list archive
-
mahara-contributors team
-
Mailing list archive
-
Message #03722
[Bug 628113] Re: Using javascript for Back link
I think for this, the javascript is the best solution.
We can have no button for browsers with javascript disabled.
For those with javascript, add in the button dynamically with JS
This approach may upset some of you, but there are several other places
where the non javascript get slightly less functionality.
Cheers,
Hugh
--
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
https://bugs.launchpad.net/bugs/628113
Title:
Using javascript for Back link
Status in Mahara ePortfolio:
Confirmed
Bug description:
Now we are using $_SERVER['HTTP_REFERER'] to get a previous view page.
But for example the following 2nd case, we can not go back to the Top Page.
1. Top Page > Links and Resources > Site view 001 > Back > Top Page
2. Top Page > Links and Resources > Site view 001 > Public view by user001 > Back > Site view 001 > Back > public view by user001 > Back > Site view 001
So how about changing view/view.php as below?
File: view/view.php
Line: 179
[ Before ]
if ($USER->is_logged_in() && !empty($_SERVER['HTTP_REFERER'])) {
$page = get_config('wwwroot') . 'view/view.php?id=' . $viewid . ($new ? '&new=1' : '');
if ($_SERVER['HTTP_REFERER'] != $page) {
$smarty->assign('backurl', $_SERVER['HTTP_REFERER']);
}
}
[ After ]
if ($USER->is_logged_in() && !empty($_SERVER['HTTP_REFERER'])) {
$page = get_config('wwwroot') . 'view/view.php?id=' . $viewid . ($new ? '&new=1' : '');
if ($_SERVER['HTTP_REFERER'] != $page) {
$smarty->assign('backurl', 'javascript:history.back()');
}
}
References