Skip to content

Commit

Permalink
Bug Fixes: Update to fix LTI provider as session not existing under s…
Browse files Browse the repository at this point in the history
…ome circumstances.

git-svn-id: https://xerteonlinetoolkits.googlecode.com/svn/trunk@449 912cdd6b-5c7d-d5a7-a2ba-d0f0cdb91641
  • Loading branch information
Simon Atack committed Oct 9, 2012
1 parent adf42fd commit 3a43818
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 1 deletion.
6 changes: 5 additions & 1 deletion LTI/index.php
Expand Up @@ -16,6 +16,10 @@
require_once '../' . $xerte_toolkits_site->php_library_path . "url_library.php";


//error_reporting(E_ALL);
//ini_set(display_errors,"ON");


$mysql_id = database_connect("LTI database connect success", "LTI database connect fail");


Expand Down Expand Up @@ -99,7 +103,7 @@ function auth($update = false) {


} else {
if(isset($_SESSION['toolkits_logon_username'])) {
if(!isset($_SESSION['toolkits_logon_username'])) {
$_SESSION['toolkits_logon_username']=$returned[0];
}
$time1 = strtotime($returned[1]);
Expand Down
4 changes: 4 additions & 0 deletions play.php
Expand Up @@ -241,7 +241,11 @@ function check_security_type($security_setting)
} else {
$lti = new UoN_LTI($mysqli);
}
if(session_id()=='') {
session_start();
}
$lti->init_lti();

}


Expand Down

0 comments on commit 3a43818

Please sign in to comment.