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
Committers, 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()');
      }
  }

_______________________________________________
Mailing list: https://launchpad.net/~mahara-core
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~mahara-core
More help   : https://help.launchpad.net/ListHelp

Reply via email to