Merge pull request #135 from ucfcdl/hotfix_oauth2_xml

bagofarms · bagofarms · commit 75903f9fe0f0 · 2016-04-22T09:08:44.000-04:00
Fixed bug with oauth2 process.  Changed link to udoit icon in xml.
diff --git a/public/index.php b/public/index.php
@@ -145,8 +145,6 @@
 		//If there is no user in the DB, send them to the Oauth2 process
 		$redirect = true;
 	}
-	//Otherwise, redirect to the Oauth2 process
-	$redirect = true;
 }
 
 // if the redirect key was invalid or we didn't have a redirect key, start the oauth2 process
diff --git a/public/udoit.xml.php b/public/udoit.xml.php
@@ -37,7 +37,7 @@
 	http://www.imsglobal.org/xsd/imslticp_v1p0 http://www.imsglobal.org/xsd/lti/ltiv1p0/imslticp_v1p0.xsd">
 	<blti:title>UDOIT</blti:title>
 	<blti:description>This tool allows you scan your courses and check for common accessibility issues.</blti:description>
-	<blti:icon><?= $servername ?>/assets/img/udoit_logo.png</blti:icon>
+	<blti:icon><?= $launch ?>assets/img/udoit_icon.png</blti:icon>
 	<blti:launch_url><?= $launch ?></blti:launch_url>
 	<blti:extensions platform="canvas.instructure.com">
 		<lticm:property name="tool_id">udoit</lticm:property>

Original file line number	Diff line number	Diff line change
`@@ -145,8 +145,6 @@`
`145`	`145`	`//If there is no user in the DB, send them to the Oauth2 process`
`146`	`146`	`$redirect = true;`
`147`	`147`	`}`
`148`		`- //Otherwise, redirect to the Oauth2 process`
`149`		`- $redirect = true;`
`150`	`148`	`}`
`151`	`149`
`152`	`150`	`// if the redirect key was invalid or we didn't have a redirect key, start the oauth2 process`