merge fix and additional comments for PR 773, generates and absolute …

…redirect path for usage inside subfolder installations #773

application/controller/LoginController.php

@@ -50,7 +50,7 @@ public function login()
  // check login status: if true, then redirect user to user/index, if false, then to login form again
  if ($login_successful) {
  if (Request::post('redirect')) {
- Redirect::to(ltrim(urldecode(Request::post('redirect')), '/'));
+ Redirect::toPreviousViewedPageAfterLogin(ltrim(urldecode(Request::post('redirect')), '/'));
  } else {
  Redirect::to('user/index');
  }

application/core/Redirect.php

@@ -7,6 +7,21 @@
  */
 class Redirect
 {
+ /**
+ * To the last visited page before user logged in (useful when people are on a certain page inside your application
+ * and then want to log in (to edit or comment something for example) and don't to be redirected to the main page).
+ *
+ * This is just a bulletproof version of Redirect::to(), redirecting to an ABSOLUTE URL path like
+ * "http:https://www.mydomain.com/user/profile", useful as people had problems with the RELATIVE URL path generated
+ * by Redirect::to() when using HUGE inside sub-folders.
+ *
+ * @param $path string
+ */
+ public static function toPreviousViewedPageAfterLogin($path)
+ {
+ header('location: http:https://' . $_SERVER['HTTP_HOST'] . '/' . $path);
+ }
+
  /**
  * To the homepage
  */
@@ -16,9 +31,15 @@ public static function home()
  }
 
  /**
- * To the defined page
+ * To the defined page, uses a relative path (like "user/profile")
+ *
+ * Redirects to a RELATIVE path, like "user/profile" (which works very fine unless you are using HUGE inside tricky
+ * sub-folder structures)
+ *
+ * @see https://github.com/panique/huge/issues/770
+ * @see https://github.com/panique/huge/issues/754
  *
- * @param $path
+ * @param $path string
  */
  public static function to($path)
  {