SlatePermutate Changeset - 58eeb176287a · protofusion repositories

Changeset - 58eeb176287a

Parent rev.

Child rev.

[Not reviewed]

default

0 2 0

Nathan Brink (binki) - 15 years ago 2010-10-09 20:56:31
ohnobinki@ohnopublishing.net

Prevent multiple calls to session_start() and use a named session to avoid colliding with other apps' sessions.

2 files changed with 29 insertions and 11 deletions:

inc/class.page.php

process.php

0 comments (0 inline, 0 general)

inc/class.page.php

➞

Show inline comments

@@ @@ -60,13 +60,13 @@ class page @@
    $ga_www = 'http://www.';
    if ($_SERVER['SERVER_PORT'] != 80)
      $ga_www = 'https://ssl.';
    $this->trackingcode = '<script type="text/javascript" src="' . $ga_www . 'google-analytics.com/ga.js" />' . "\n"
      . $this->trackingcode;
-    session_start();
+   page::session_start();
     if($immediate
        && $ntitle != "NOHEAD")
       $this->head();
     /* everything that needs sessions started to work: */
@@ @@ -262,12 +262,32 @@ class page @@
     exit();
+  }
   /**
    * \brief
    *   Start the PHP session by calling session_start().
+   *
    * Used to make sure that different areas of our code don't call
    * session_start() multiple times and to make it easier to ensure
    * that session_start() is called at least before it's needed.
    */
   public static function session_start()
+  {
     static $session_started = FALSE;
     if (!$session_started)
+      {
 	session_name('slate_permutate');
 	session_start();
 	$session_started = TRUE;
+      }
+  }
   /**
    * \brief
    *   Get the current school profile handle.
    */
   public function get_school()
+  {
     return $this->school;
+  }

process.php

➞

Show inline comments

 <?php
 session_start();
 require_once('inc/schedule_store.inc');
 require_once('inc/class.page.php');
 include_once 'class.schedule.php';
 include_once 'class.class.php';
 include_once 'class.section.php';
 function sortInputs($post){
 //	return array_filter($post['postData']); // Remove any null or unset items. Disabled as it kills day stuff, @FIXME and add day unset setting here (==0).
 	return $post['postData'];
+}
 // Converts a 5-element array into a nice string.
 // Supports multiple modes, prettiness, and searching for different indicators
 function arrayToDays($array, $mode = 'num', $pretty = false, $key = 1) {
 	$outString = '';
 	switch($mode){
 		case 'short':
@@ @@ -54,12 +46,17 @@ function arrayToDays($array, $mode = 'nu @@
+}
 function prettyTime($time){
 	return substr($time,0,strlen($time)-2) . ":" . substr($time,strlen($time)-2, strlen($time));
+}
 /*
  * The below code relies on sessions being started already.
  */
 page::session_start();
 $DEBUG = FALSE;
 if (isset($_GET['debug']))
   $DEBUG = $_GET['debug'];
 $schedule_store = schedule_store_init();
@@ @@ -81,18 +78,19 @@ if(!$DEBUG) @@
 	    /* user owns this schedule ID */
 	    schedule_store_delete($schedule_store, (int)$_GET['del']);
 	    unset($_SESSION['saved'][(int)$_GET['del']]);
+	  }
 	header('Location: input.php');
 	exit;
+      }
     else
+      {
 		$allClasses = new Schedule($_POST['postData']['name']);
-		foreach(sortInputs($_POST) as $class)
+		foreach($_POST['postData'] as $class)
+		{
 		  /*
 		   * Only add classes if the user added at least one
 		   * section to the class. We know that $class['name']
 		   * is not a section, so count() needs to be > 1 and
 		   * we need to skip over 'name' in our loop.
@@ @@ -124,13 +122,13 @@ if(!$DEBUG) @@
 		$allClasses->writeoutTables();
+      }
+  }
 else
+  {
 	echo '<pre>DEBUG OUTPUT: <br /><br />';
-	foreach(sortInputs($_POST) as $class) {
+	foreach($_POST['postData'] as $class) {
 		echo 'Class: ' . $class['name'] . '<br />';
 		foreach($class as $section)
 			if(is_array($section))
+			{
 				echo '---- Section that starts at ' . prettyTime($section['start']) . ' and ends at ' . prettyTime($section['end']) . '. This class meets on ';
 				echo arrayToDays($section['days'],'long',true) . '.<br />';

0 comments (0 inline, 0 general)