Merge lp:~whosdaz/ubuntu-qa-website/fix1194770 into lp:ubuntu-qa-website

[Matthew Allen]

I've set up redirects so if the path is '/user' it redirects to https://login.ubuntu.com/ (is that what SSO was referring to?).

Just be aware the implementation isn't perfect but it was the best I could do with redirects on the server side.

Furthermore, be aware that from what I can determine the link itself cannot be and changed as it is a limitation of drupal (as the User module is mandatory)

[Nicholas Skaggs (nskaggs)]

Nice; provided this works out a-ok with SSO, this is a great workaround.

[Nicholas Skaggs (nskaggs)]

Can you redirect user/*?

For instance, http://localhost/?q=user/login still shows me the page, so does http://localhost/?q=user/register, http://localhost/?q=user/password, etc.

review: Needs Fixing

[lp:~whosdaz/ubuntu-qa-website/fix1194770]

416. By Matthew Allen on 2016-01-12

Added redirects for missing urls

[Matthew Allen (whosdaz)]

Committing a fix as we speak, if there are further issues - message me here or by email. Won't be on IRC until tomorrow.

[Nicholas Skaggs (nskaggs)]

Looks good!

review: Approve

=== modified file 'modules/qawebsite/qawebsite.module'
--- modules/qawebsite/qawebsite.module	2016-01-05 17:23:40 +0000
+++ modules/qawebsite/qawebsite.module	2016-01-12 16:08:48 +0000
@@ -16,4 +16,29 @@
 along with this program; if not, write to the Free Software
 Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
 */
+
+
+//Work around to fix lp:1194770
+
+$path = drupal_get_path_alias($_GET['q']);
+echo $path;
+
+
+if($path == 'user' || $path == 'user/login' || $path == 'user/register' || $path == 'user/password'){
+	$redirect = array(
+		'#tag' => 'meta', // The #tag is the html tag - <link />
+		'#attributes' => array( // Set up an array of attributes inside the tag
+			'http-equiv' => 'refresh', 
+			'content' => '0; url=https://login.ubuntu.com/',
+			),
+		);
+
+	// Please note: this fix is purely a work around as a meta refresh is not reccomended by the W3C, so redirects should 
+	// implemented on the server side however this work around provides the functionality neccesary.
+	
+	drupal_add_html_head($redirect, 'redirect');
+}
+
+//
+
 ?>