xpressengine / xe-core

libs/phpmailer/smtp.php

Doc Comments +7 added lines

@@ -107,6 +107,7 @@ 
    * SMTP CODE SUCCESS: 220
    * SMTP CODE FAILURE: 421
    * @access public
+   * @param string $host
    * @return bool
    */
   public function Connect($host,$port=0,$tval=30) {
@@ -205,6 +206,8 @@ 
    * Performs SMTP authentication.  Must be run after running the
    * Hello() method.  Returns true if successfully authenticated.
    * @access public
+   * @param string $username
+   * @param string $password
    * @return bool
    */
   public function Authenticate($username, $password) {
@@ -321,6 +324,7 @@ 
    * SMTP CODE FAILURE: 451,554
    * SMTP CODE ERROR  : 500,501,503,421
    * @access public
+   * @param string $msg_data
    * @return bool
    */
   public function Data($msg_data) {
@@ -489,6 +493,8 @@ 
   /**
    * Sends a HELO/EHLO command.
    * @access private
+   * @param string $hello
+   * @param string $host
    * @return bool
    */
   private function SendHello($hello, $host) {
@@ -529,6 +535,7 @@ 
    * SMTP CODE SUCCESS: 552,451,452
    * SMTP CODE SUCCESS: 500,501,421
    * @access public
+   * @param string $from
    * @return bool
    */
   public function Mail($from) {

Indentation +470 added lines, -470 removed lines

@@ -85,11 +85,11 @@ 
    * @return void
    */
   public function __construct() {
-    $this->smtp_conn = 0;
-    $this->error = null;
-    $this->helo_rply = null;
+	$this->smtp_conn = 0;
+	$this->error = null;
+	$this->helo_rply = null;
 
-    $this->do_debug = 0;
+	$this->do_debug = 0;
   }
 
   /////////////////////////////////////////////////
@@ -110,50 +110,50 @@ 
    * @return bool
    */
   public function Connect($host,$port=0,$tval=30) {
-    // set the error val to null so there is no confusion
-    $this->error = null;
-
-    // make sure we are __not__ connected
-    if($this->connected()) {
-      // already connected, generate error
-      $this->error = array("error" => "Already connected to a server");
-      return false;
-    }
-
-    if(empty($port)) {
-      $port = $this->SMTP_PORT;
-    }
-
-    // connect to the smtp server
-    $this->smtp_conn = @fsockopen($host,    // the host of the server
-                                 $port,    // the port to use
-                                 $errno,   // error number if any
-                                 $errstr,  // error message if any
-                                 $tval);   // give up after ? secs
-    // verify we connected properly
-    if(empty($this->smtp_conn)) {
-      $this->error = array("error" => "Failed to connect to server",
-                           "errno" => $errno,
-                           "errstr" => $errstr);
-      if($this->do_debug >= 1) {
-        echo "SMTP -> ERROR: " . $this->error["error"] . ": $errstr ($errno)" . $this->CRLF . '<br />';
-      }
-      return false;
-    }
-
-    // SMTP server can take longer to respond, give longer timeout for first read
-    // Windows does not have support for this timeout function
-    if(substr(PHP_OS, 0, 3) != "WIN")
-     socket_set_timeout($this->smtp_conn, $tval, 0);
-
-    // get any announcement
-    $announce = $this->get_lines();
-
-    if($this->do_debug >= 2) {
-      echo "SMTP -> FROM SERVER:" . $announce . $this->CRLF . '<br />';
-    }
-
-    return true;
+	// set the error val to null so there is no confusion
+	$this->error = null;
+
+	// make sure we are __not__ connected
+	if($this->connected()) {
+	  // already connected, generate error
+	  $this->error = array("error" => "Already connected to a server");
+	  return false;
+	}
+
+	if(empty($port)) {
+	  $port = $this->SMTP_PORT;
+	}
+
+	// connect to the smtp server
+	$this->smtp_conn = @fsockopen($host,    // the host of the server
+								 $port,    // the port to use
+								 $errno,   // error number if any
+								 $errstr,  // error message if any
+								 $tval);   // give up after ? secs
+	// verify we connected properly
+	if(empty($this->smtp_conn)) {
+	  $this->error = array("error" => "Failed to connect to server",
+						   "errno" => $errno,
+						   "errstr" => $errstr);
+	  if($this->do_debug >= 1) {
+		echo "SMTP -> ERROR: " . $this->error["error"] . ": $errstr ($errno)" . $this->CRLF . '<br />';
+	  }
+	  return false;
+	}
+
+	// SMTP server can take longer to respond, give longer timeout for first read
+	// Windows does not have support for this timeout function
+	if(substr(PHP_OS, 0, 3) != "WIN")
+	 socket_set_timeout($this->smtp_conn, $tval, 0);
+
+	// get any announcement
+	$announce = $this->get_lines();
+
+	if($this->do_debug >= 2) {
+	  echo "SMTP -> FROM SERVER:" . $announce . $this->CRLF . '<br />';
+	}
+
+	return true;
   }
 
   /**
@@ -166,39 +166,39 @@ 
    * @return bool success
    */
   public function StartTLS() {
-    $this->error = null; # to avoid confusion
-
-    if(!$this->connected()) {
-      $this->error = array("error" => "Called StartTLS() without being connected");
-      return false;
-    }
-
-    fputs($this->smtp_conn,"STARTTLS" . $this->CRLF);
-
-    $rply = $this->get_lines();
-    $code = substr($rply,0,3);
-
-    if($this->do_debug >= 2) {
-      echo "SMTP -> FROM SERVER:" . $rply . $this->CRLF . '<br />';
-    }
-
-    if($code != 220) {
-      $this->error =
-         array("error"     => "STARTTLS not accepted from server",
-               "smtp_code" => $code,
-               "smtp_msg"  => substr($rply,4));
-      if($this->do_debug >= 1) {
-        echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';
-      }
-      return false;
-    }
-
-    //Begin encrypted connection
-    if(!stream_socket_enable_crypto($this->smtp_conn, true, STREAM_CRYPTO_METHOD_TLS_CLIENT)) {
-      return false;
-    }
-
-    return true;
+	$this->error = null; # to avoid confusion
+
+	if(!$this->connected()) {
+	  $this->error = array("error" => "Called StartTLS() without being connected");
+	  return false;
+	}
+
+	fputs($this->smtp_conn,"STARTTLS" . $this->CRLF);
+
+	$rply = $this->get_lines();
+	$code = substr($rply,0,3);
+
+	if($this->do_debug >= 2) {
+	  echo "SMTP -> FROM SERVER:" . $rply . $this->CRLF . '<br />';
+	}
+
+	if($code != 220) {
+	  $this->error =
+		 array("error"     => "STARTTLS not accepted from server",
+			   "smtp_code" => $code,
+			   "smtp_msg"  => substr($rply,4));
+	  if($this->do_debug >= 1) {
+		echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';
+	  }
+	  return false;
+	}
+
+	//Begin encrypted connection
+	if(!stream_socket_enable_crypto($this->smtp_conn, true, STREAM_CRYPTO_METHOD_TLS_CLIENT)) {
+	  return false;
+	}
+
+	return true;
   }
 
   /**
@@ -208,58 +208,58 @@ 
    * @return bool
    */
   public function Authenticate($username, $password) {
-    // Start authentication
-    fputs($this->smtp_conn,"AUTH LOGIN" . $this->CRLF);
-
-    $rply = $this->get_lines();
-    $code = substr($rply,0,3);
-
-    if($code != 334) {
-      $this->error =
-        array("error" => "AUTH not accepted from server",
-              "smtp_code" => $code,
-              "smtp_msg" => substr($rply,4));
-      if($this->do_debug >= 1) {
-        echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';
-      }
-      return false;
-    }
-
-    // Send encoded username
-    fputs($this->smtp_conn, base64_encode($username) . $this->CRLF);
-
-    $rply = $this->get_lines();
-    $code = substr($rply,0,3);
-
-    if($code != 334) {
-      $this->error =
-        array("error" => "Username not accepted from server",
-              "smtp_code" => $code,
-              "smtp_msg" => substr($rply,4));
-      if($this->do_debug >= 1) {
-        echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';
-      }
-      return false;
-    }
-
-    // Send encoded password
-    fputs($this->smtp_conn, base64_encode($password) . $this->CRLF);
-
-    $rply = $this->get_lines();
-    $code = substr($rply,0,3);
-
-    if($code != 235) {
-      $this->error =
-        array("error" => "Password not accepted from server",
-              "smtp_code" => $code,
-              "smtp_msg" => substr($rply,4));
-      if($this->do_debug >= 1) {
-        echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';
-      }
-      return false;
-    }
-
-    return true;
+	// Start authentication
+	fputs($this->smtp_conn,"AUTH LOGIN" . $this->CRLF);
+
+	$rply = $this->get_lines();
+	$code = substr($rply,0,3);
+
+	if($code != 334) {
+	  $this->error =
+		array("error" => "AUTH not accepted from server",
+			  "smtp_code" => $code,
+			  "smtp_msg" => substr($rply,4));
+	  if($this->do_debug >= 1) {
+		echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';
+	  }
+	  return false;
+	}
+
+	// Send encoded username
+	fputs($this->smtp_conn, base64_encode($username) . $this->CRLF);
+
+	$rply = $this->get_lines();
+	$code = substr($rply,0,3);
+
+	if($code != 334) {
+	  $this->error =
+		array("error" => "Username not accepted from server",
+			  "smtp_code" => $code,
+			  "smtp_msg" => substr($rply,4));
+	  if($this->do_debug >= 1) {
+		echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';
+	  }
+	  return false;
+	}
+
+	// Send encoded password
+	fputs($this->smtp_conn, base64_encode($password) . $this->CRLF);
+
+	$rply = $this->get_lines();
+	$code = substr($rply,0,3);
+
+	if($code != 235) {
+	  $this->error =
+		array("error" => "Password not accepted from server",
+			  "smtp_code" => $code,
+			  "smtp_msg" => substr($rply,4));
+	  if($this->do_debug >= 1) {
+		echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';
+	  }
+	  return false;
+	}
+
+	return true;
   }
 
   /**
@@ -268,19 +268,19 @@ 
    * @return bool
    */
   public function Connected() {
-    if(!empty($this->smtp_conn)) {
-      $sock_status = socket_get_status($this->smtp_conn);
-      if($sock_status["eof"]) {
-        // the socket is valid but we are not connected
-        if($this->do_debug >= 1) {
-            echo "SMTP -> NOTICE:" . $this->CRLF . "EOF caught while checking if connected";
-        }
-        $this->Close();
-        return false;
-      }
-      return true; // everything looks good
-    }
-    return false;
+	if(!empty($this->smtp_conn)) {
+	  $sock_status = socket_get_status($this->smtp_conn);
+	  if($sock_status["eof"]) {
+		// the socket is valid but we are not connected
+		if($this->do_debug >= 1) {
+			echo "SMTP -> NOTICE:" . $this->CRLF . "EOF caught while checking if connected";
+		}
+		$this->Close();
+		return false;
+	  }
+	  return true; // everything looks good
+	}
+	return false;
   }
 
   /**
@@ -291,13 +291,13 @@ 
    * @return void
    */
   public function Close() {
-    $this->error = null; // so there is no confusion
-    $this->helo_rply = null;
-    if(!empty($this->smtp_conn)) {
-      // close the connection and cleanup
-      fclose($this->smtp_conn);
-      $this->smtp_conn = 0;
-    }
+	$this->error = null; // so there is no confusion
+	$this->helo_rply = null;
+	if(!empty($this->smtp_conn)) {
+	  // close the connection and cleanup
+	  fclose($this->smtp_conn);
+	  $this->smtp_conn = 0;
+	}
   }
 
   /////////////////////////////////////////////////
@@ -324,35 +324,35 @@ 
    * @return bool
    */
   public function Data($msg_data) {
-    $this->error = null; // so no confusion is caused
-
-    if(!$this->connected()) {
-      $this->error = array(
-              "error" => "Called Data() without being connected");
-      return false;
-    }
-
-    fputs($this->smtp_conn,"DATA" . $this->CRLF);
-
-    $rply = $this->get_lines();
-    $code = substr($rply,0,3);
-
-    if($this->do_debug >= 2) {
-      echo "SMTP -> FROM SERVER:" . $rply . $this->CRLF . '<br />';
-    }
-
-    if($code != 354) {
-      $this->error =
-        array("error" => "DATA command not accepted from server",
-              "smtp_code" => $code,
-              "smtp_msg" => substr($rply,4));
-      if($this->do_debug >= 1) {
-        echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';
-      }
-      return false;
-    }
-
-    /* the server is ready to accept data!
+	$this->error = null; // so no confusion is caused
+
+	if(!$this->connected()) {
+	  $this->error = array(
+			  "error" => "Called Data() without being connected");
+	  return false;
+	}
+
+	fputs($this->smtp_conn,"DATA" . $this->CRLF);
+
+	$rply = $this->get_lines();
+	$code = substr($rply,0,3);
+
+	if($this->do_debug >= 2) {
+	  echo "SMTP -> FROM SERVER:" . $rply . $this->CRLF . '<br />';
+	}
+
+	if($code != 354) {
+	  $this->error =
+		array("error" => "DATA command not accepted from server",
+			  "smtp_code" => $code,
+			  "smtp_msg" => substr($rply,4));
+	  if($this->do_debug >= 1) {
+		echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';
+	  }
+	  return false;
+	}
+
+	/* the server is ready to accept data!
      * according to rfc 821 we should not send more than 1000
      * including the CRLF
      * characters on a single line so we will break the data up
@@ -363,12 +363,12 @@ 
      * line. NOTE: this does not count towards limit.
      */
 
-    // normalize the line breaks so we know the explode works
-    $msg_data = str_replace("\r\n","\n",$msg_data);
-    $msg_data = str_replace("\r","\n",$msg_data);
-    $lines = explode("\n",$msg_data);
+	// normalize the line breaks so we know the explode works
+	$msg_data = str_replace("\r\n","\n",$msg_data);
+	$msg_data = str_replace("\r","\n",$msg_data);
+	$lines = explode("\n",$msg_data);
 
-    /* we need to find a good way to determine is headers are
+	/* we need to find a good way to determine is headers are
      * in the msg_data or if it is a straight msg body
      * currently I am assuming rfc 822 definitions of msg headers
      * and if the first field of the first line (':' sperated)
@@ -377,76 +377,76 @@ 
      * headers.
      */
 
-    $field = substr($lines[0],0,strpos($lines[0],":"));
-    $in_headers = false;
-    if(!empty($field) && !strstr($field," ")) {
-      $in_headers = true;
-    }
-
-    $max_line_length = 998; // used below; set here for ease in change
-
-    while(list(,$line) = @each($lines)) {
-      $lines_out = null;
-      if($line == "" && $in_headers) {
-        $in_headers = false;
-      }
-      // ok we need to break this line up into several smaller lines
-      while(strlen($line) > $max_line_length) {
-        $pos = strrpos(substr($line,0,$max_line_length)," ");
-
-        // Patch to fix DOS attack
-        if(!$pos) {
-          $pos = $max_line_length - 1;
-          $lines_out[] = substr($line,0,$pos);
-          $line = substr($line,$pos);
-        } else {
-          $lines_out[] = substr($line,0,$pos);
-          $line = substr($line,$pos + 1);
-        }
-
-        /* if we are processing headers we need to
+	$field = substr($lines[0],0,strpos($lines[0],":"));
+	$in_headers = false;
+	if(!empty($field) && !strstr($field," ")) {
+	  $in_headers = true;
+	}
+
+	$max_line_length = 998; // used below; set here for ease in change
+
+	while(list(,$line) = @each($lines)) {
+	  $lines_out = null;
+	  if($line == "" && $in_headers) {
+		$in_headers = false;
+	  }
+	  // ok we need to break this line up into several smaller lines
+	  while(strlen($line) > $max_line_length) {
+		$pos = strrpos(substr($line,0,$max_line_length)," ");
+
+		// Patch to fix DOS attack
+		if(!$pos) {
+		  $pos = $max_line_length - 1;
+		  $lines_out[] = substr($line,0,$pos);
+		  $line = substr($line,$pos);
+		} else {
+		  $lines_out[] = substr($line,0,$pos);
+		  $line = substr($line,$pos + 1);
+		}
+
+		/* if we are processing headers we need to
          * add a LWSP-char to the front of the new line
          * rfc 822 on long msg headers
          */
-        if($in_headers) {
-          $line = "\t" . $line;
-        }
-      }
-      $lines_out[] = $line;
-
-      // send the lines to the server
-      while(list(,$line_out) = @each($lines_out)) {
-        if(strlen($line_out) > 0)
-        {
-          if(substr($line_out, 0, 1) == ".") {
-            $line_out = "." . $line_out;
-          }
-        }
-        fputs($this->smtp_conn,$line_out . $this->CRLF);
-      }
-    }
-
-    // message data has been sent
-    fputs($this->smtp_conn, $this->CRLF . "." . $this->CRLF);
-
-    $rply = $this->get_lines();
-    $code = substr($rply,0,3);
-
-    if($this->do_debug >= 2) {
-      echo "SMTP -> FROM SERVER:" . $rply . $this->CRLF . '<br />';
-    }
-
-    if($code != 250) {
-      $this->error =
-        array("error" => "DATA not accepted from server",
-              "smtp_code" => $code,
-              "smtp_msg" => substr($rply,4));
-      if($this->do_debug >= 1) {
-        echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';
-      }
-      return false;
-    }
-    return true;
+		if($in_headers) {
+		  $line = "\t" . $line;
+		}
+	  }
+	  $lines_out[] = $line;
+
+	  // send the lines to the server
+	  while(list(,$line_out) = @each($lines_out)) {
+		if(strlen($line_out) > 0)
+		{
+		  if(substr($line_out, 0, 1) == ".") {
+			$line_out = "." . $line_out;
+		  }
+		}
+		fputs($this->smtp_conn,$line_out . $this->CRLF);
+	  }
+	}
+
+	// message data has been sent
+	fputs($this->smtp_conn, $this->CRLF . "." . $this->CRLF);
+
+	$rply = $this->get_lines();
+	$code = substr($rply,0,3);
+
+	if($this->do_debug >= 2) {
+	  echo "SMTP -> FROM SERVER:" . $rply . $this->CRLF . '<br />';
+	}
+
+	if($code != 250) {
+	  $this->error =
+		array("error" => "DATA not accepted from server",
+			  "smtp_code" => $code,
+			  "smtp_msg" => substr($rply,4));
+	  if($this->do_debug >= 1) {
+		echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';
+	  }
+	  return false;
+	}
+	return true;
   }
 
   /**
@@ -462,28 +462,28 @@ 
    * @return bool
    */
   public function Hello($host = '') {
-    $this->error = null; // so no confusion is caused
-
-    if(!$this->connected()) {
-      $this->error = array(
-            "error" => "Called Hello() without being connected");
-      return false;
-    }
-
-    // if hostname for HELO was not specified send default
-    if(empty($host)) {
-      // determine appropriate default to send to server
-      $host = "localhost";
-    }
-
-    // Send extended hello first (RFC 2821)
-    if(!$this->SendHello("EHLO", $host)) {
-      if(!$this->SendHello("HELO", $host)) {
-          return false;
-    }
-    }
-
-    return true;
+	$this->error = null; // so no confusion is caused
+
+	if(!$this->connected()) {
+	  $this->error = array(
+			"error" => "Called Hello() without being connected");
+	  return false;
+	}
+
+	// if hostname for HELO was not specified send default
+	if(empty($host)) {
+	  // determine appropriate default to send to server
+	  $host = "localhost";
+	}
+
+	// Send extended hello first (RFC 2821)
+	if(!$this->SendHello("EHLO", $host)) {
+	  if(!$this->SendHello("HELO", $host)) {
+		  return false;
+	}
+	}
+
+	return true;
   }
 
   /**
@@ -492,29 +492,29 @@ 
    * @return bool
    */
   private function SendHello($hello, $host) {
-    fputs($this->smtp_conn, $hello . " " . $host . $this->CRLF);
+	fputs($this->smtp_conn, $hello . " " . $host . $this->CRLF);
 
-    $rply = $this->get_lines();
-    $code = substr($rply,0,3);
+	$rply = $this->get_lines();
+	$code = substr($rply,0,3);
 
-    if($this->do_debug >= 2) {
-      echo "SMTP -> FROM SERVER: " . $rply . $this->CRLF . '<br />';
-    }
+	if($this->do_debug >= 2) {
+	  echo "SMTP -> FROM SERVER: " . $rply . $this->CRLF . '<br />';
+	}
 
-    if($code != 250) {
-      $this->error =
-        array("error" => $hello . " not accepted from server",
-              "smtp_code" => $code,
-              "smtp_msg" => substr($rply,4));
-      if($this->do_debug >= 1) {
-        echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';
-      }
-      return false;
-    }
+	if($code != 250) {
+	  $this->error =
+		array("error" => $hello . " not accepted from server",
+			  "smtp_code" => $code,
+			  "smtp_msg" => substr($rply,4));
+	  if($this->do_debug >= 1) {
+		echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';
+	  }
+	  return false;
+	}
 
-    $this->helo_rply = $rply;
+	$this->helo_rply = $rply;
 
-    return true;
+	return true;
   }
 
   /**
@@ -532,35 +532,35 @@ 
    * @return bool
    */
   public function Mail($from) {
-    $this->error = null; // so no confusion is caused
-
-    if(!$this->connected()) {
-      $this->error = array(
-              "error" => "Called Mail() without being connected");
-      return false;
-    }
-
-    $useVerp = ($this->do_verp ? "XVERP" : "");
-    fputs($this->smtp_conn,"MAIL FROM:<" . $from . ">" . $useVerp . $this->CRLF);
-
-    $rply = $this->get_lines();
-    $code = substr($rply,0,3);
-
-    if($this->do_debug >= 2) {
-      echo "SMTP -> FROM SERVER:" . $rply . $this->CRLF . '<br />';
-    }
-
-    if($code != 250) {
-      $this->error =
-        array("error" => "MAIL not accepted from server",
-              "smtp_code" => $code,
-              "smtp_msg" => substr($rply,4));
-      if($this->do_debug >= 1) {
-        echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';
-      }
-      return false;
-    }
-    return true;
+	$this->error = null; // so no confusion is caused
+
+	if(!$this->connected()) {
+	  $this->error = array(
+			  "error" => "Called Mail() without being connected");
+	  return false;
+	}
+
+	$useVerp = ($this->do_verp ? "XVERP" : "");
+	fputs($this->smtp_conn,"MAIL FROM:<" . $from . ">" . $useVerp . $this->CRLF);
+
+	$rply = $this->get_lines();
+	$code = substr($rply,0,3);
+
+	if($this->do_debug >= 2) {
+	  echo "SMTP -> FROM SERVER:" . $rply . $this->CRLF . '<br />';
+	}
+
+	if($code != 250) {
+	  $this->error =
+		array("error" => "MAIL not accepted from server",
+			  "smtp_code" => $code,
+			  "smtp_msg" => substr($rply,4));
+	  if($this->do_debug >= 1) {
+		echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';
+	  }
+	  return false;
+	}
+	return true;
   }
 
   /**
@@ -575,44 +575,44 @@ 
    * @return bool
    */
   public function Quit($close_on_error=true) {
-    $this->error = null; // so there is no confusion
-
-    if(!$this->connected()) {
-      $this->error = array(
-              "error" => "Called Quit() without being connected");
-      return false;
-    }
-
-    // send the quit command to the server
-    fputs($this->smtp_conn,"quit" . $this->CRLF);
-
-    // get any good-bye messages
-    $byemsg = $this->get_lines();
-
-    if($this->do_debug >= 2) {
-      echo "SMTP -> FROM SERVER:" . $byemsg . $this->CRLF . '<br />';
-    }
-
-    $rval = true;
-    $e = null;
-
-    $code = substr($byemsg,0,3);
-    if($code != 221) {
-      // use e as a tmp var cause Close will overwrite $this->error
-      $e = array("error" => "SMTP server rejected quit command",
-                 "smtp_code" => $code,
-                 "smtp_rply" => substr($byemsg,4));
-      $rval = false;
-      if($this->do_debug >= 1) {
-        echo "SMTP -> ERROR: " . $e["error"] . ": " . $byemsg . $this->CRLF . '<br />';
-      }
-    }
-
-    if(empty($e) || $close_on_error) {
-      $this->Close();
-    }
-
-    return $rval;
+	$this->error = null; // so there is no confusion
+
+	if(!$this->connected()) {
+	  $this->error = array(
+			  "error" => "Called Quit() without being connected");
+	  return false;
+	}
+
+	// send the quit command to the server
+	fputs($this->smtp_conn,"quit" . $this->CRLF);
+
+	// get any good-bye messages
+	$byemsg = $this->get_lines();
+
+	if($this->do_debug >= 2) {
+	  echo "SMTP -> FROM SERVER:" . $byemsg . $this->CRLF . '<br />';
+	}
+
+	$rval = true;
+	$e = null;
+
+	$code = substr($byemsg,0,3);
+	if($code != 221) {
+	  // use e as a tmp var cause Close will overwrite $this->error
+	  $e = array("error" => "SMTP server rejected quit command",
+				 "smtp_code" => $code,
+				 "smtp_rply" => substr($byemsg,4));
+	  $rval = false;
+	  if($this->do_debug >= 1) {
+		echo "SMTP -> ERROR: " . $e["error"] . ": " . $byemsg . $this->CRLF . '<br />';
+	  }
+	}
+
+	if(empty($e) || $close_on_error) {
+	  $this->Close();
+	}
+
+	return $rval;
   }
 
   /**
@@ -628,34 +628,34 @@ 
    * @return bool
    */
   public function Recipient($to) {
-    $this->error = null; // so no confusion is caused
-
-    if(!$this->connected()) {
-      $this->error = array(
-              "error" => "Called Recipient() without being connected");
-      return false;
-    }
-
-    fputs($this->smtp_conn,"RCPT TO:<" . $to . ">" . $this->CRLF);
-
-    $rply = $this->get_lines();
-    $code = substr($rply,0,3);
-
-    if($this->do_debug >= 2) {
-      echo "SMTP -> FROM SERVER:" . $rply . $this->CRLF . '<br />';
-    }
-
-    if($code != 250 && $code != 251) {
-      $this->error =
-        array("error" => "RCPT not accepted from server",
-              "smtp_code" => $code,
-              "smtp_msg" => substr($rply,4));
-      if($this->do_debug >= 1) {
-        echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';
-      }
-      return false;
-    }
-    return true;
+	$this->error = null; // so no confusion is caused
+
+	if(!$this->connected()) {
+	  $this->error = array(
+			  "error" => "Called Recipient() without being connected");
+	  return false;
+	}
+
+	fputs($this->smtp_conn,"RCPT TO:<" . $to . ">" . $this->CRLF);
+
+	$rply = $this->get_lines();
+	$code = substr($rply,0,3);
+
+	if($this->do_debug >= 2) {
+	  echo "SMTP -> FROM SERVER:" . $rply . $this->CRLF . '<br />';
+	}
+
+	if($code != 250 && $code != 251) {
+	  $this->error =
+		array("error" => "RCPT not accepted from server",
+			  "smtp_code" => $code,
+			  "smtp_msg" => substr($rply,4));
+	  if($this->do_debug >= 1) {
+		echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';
+	  }
+	  return false;
+	}
+	return true;
   }
 
   /**
@@ -671,35 +671,35 @@ 
    * @return bool
    */
   public function Reset() {
-    $this->error = null; // so no confusion is caused
-
-    if(!$this->connected()) {
-      $this->error = array(
-              "error" => "Called Reset() without being connected");
-      return false;
-    }
-
-    fputs($this->smtp_conn,"RSET" . $this->CRLF);
-
-    $rply = $this->get_lines();
-    $code = substr($rply,0,3);
-
-    if($this->do_debug >= 2) {
-      echo "SMTP -> FROM SERVER:" . $rply . $this->CRLF . '<br />';
-    }
-
-    if($code != 250) {
-      $this->error =
-        array("error" => "RSET failed",
-              "smtp_code" => $code,
-              "smtp_msg" => substr($rply,4));
-      if($this->do_debug >= 1) {
-        echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';
-      }
-      return false;
-    }
-
-    return true;
+	$this->error = null; // so no confusion is caused
+
+	if(!$this->connected()) {
+	  $this->error = array(
+			  "error" => "Called Reset() without being connected");
+	  return false;
+	}
+
+	fputs($this->smtp_conn,"RSET" . $this->CRLF);
+
+	$rply = $this->get_lines();
+	$code = substr($rply,0,3);
+
+	if($this->do_debug >= 2) {
+	  echo "SMTP -> FROM SERVER:" . $rply . $this->CRLF . '<br />';
+	}
+
+	if($code != 250) {
+	  $this->error =
+		array("error" => "RSET failed",
+			  "smtp_code" => $code,
+			  "smtp_msg" => substr($rply,4));
+	  if($this->do_debug >= 1) {
+		echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';
+	  }
+	  return false;
+	}
+
+	return true;
   }
 
   /**
@@ -719,34 +719,34 @@ 
    * @return bool
    */
   public function SendAndMail($from) {
-    $this->error = null; // so no confusion is caused
-
-    if(!$this->connected()) {
-      $this->error = array(
-          "error" => "Called SendAndMail() without being connected");
-      return false;
-    }
-
-    fputs($this->smtp_conn,"SAML FROM:" . $from . $this->CRLF);
-
-    $rply = $this->get_lines();
-    $code = substr($rply,0,3);
-
-    if($this->do_debug >= 2) {
-      echo "SMTP -> FROM SERVER:" . $rply . $this->CRLF . '<br />';
-    }
-
-    if($code != 250) {
-      $this->error =
-        array("error" => "SAML not accepted from server",
-              "smtp_code" => $code,
-              "smtp_msg" => substr($rply,4));
-      if($this->do_debug >= 1) {
-        echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';
-      }
-      return false;
-    }
-    return true;
+	$this->error = null; // so no confusion is caused
+
+	if(!$this->connected()) {
+	  $this->error = array(
+		  "error" => "Called SendAndMail() without being connected");
+	  return false;
+	}
+
+	fputs($this->smtp_conn,"SAML FROM:" . $from . $this->CRLF);
+
+	$rply = $this->get_lines();
+	$code = substr($rply,0,3);
+
+	if($this->do_debug >= 2) {
+	  echo "SMTP -> FROM SERVER:" . $rply . $this->CRLF . '<br />';
+	}
+
+	if($code != 250) {
+	  $this->error =
+		array("error" => "SAML not accepted from server",
+			  "smtp_code" => $code,
+			  "smtp_msg" => substr($rply,4));
+	  if($this->do_debug >= 1) {
+		echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';
+	  }
+	  return false;
+	}
+	return true;
   }
 
   /**
@@ -763,22 +763,22 @@ 
    * @return bool
    */
   public function Turn() {
-    $this->error = array("error" => "This method, TURN, of the SMTP ".
-                                    "is not implemented");
-    if($this->do_debug >= 1) {
-      echo "SMTP -> NOTICE: " . $this->error["error"] . $this->CRLF . '<br />';
-    }
-    return false;
+	$this->error = array("error" => "This method, TURN, of the SMTP ".
+									"is not implemented");
+	if($this->do_debug >= 1) {
+	  echo "SMTP -> NOTICE: " . $this->error["error"] . $this->CRLF . '<br />';
+	}
+	return false;
   }
 
   /**
-  * Get the current error
+   * Get the current error
    * @access public
-  * @return array
+   * @return array
    */
   public function getError() {
-    return $this->error;
-    }
+	return $this->error;
+	}
 
   /////////////////////////////////////////////////
   // INTERNAL FUNCTIONS
@@ -794,20 +794,20 @@ 
    * @return string
    */
   private function get_lines() {
-    $data = "";
-    while($str = @fgets($this->smtp_conn,515)) {
-      if($this->do_debug >= 4) {
-        echo "SMTP -> get_lines(): \$data was \"$data\"" . $this->CRLF . '<br />';
-        echo "SMTP -> get_lines(): \$str is \"$str\"" . $this->CRLF . '<br />';
-      }
-      $data .= $str;
-      if($this->do_debug >= 4) {
-        echo "SMTP -> get_lines(): \$data is \"$data\"" . $this->CRLF . '<br />';
-      }
-      // if 4th character is a space, we are done reading, break the loop
-      if(substr($str,3,1) == " ") { break; }
-    }
-    return $data;
+	$data = "";
+	while($str = @fgets($this->smtp_conn,515)) {
+	  if($this->do_debug >= 4) {
+		echo "SMTP -> get_lines(): \$data was \"$data\"" . $this->CRLF . '<br />';
+		echo "SMTP -> get_lines(): \$str is \"$str\"" . $this->CRLF . '<br />';
+	  }
+	  $data .= $str;
+	  if($this->do_debug >= 4) {
+		echo "SMTP -> get_lines(): \$data is \"$data\"" . $this->CRLF . '<br />';
+	  }
+	  // if 4th character is a space, we are done reading, break the loop
+	  if(substr($str,3,1) == " ") { break; }
+	}
+	return $data;
   }
 
 }

Braces +3 added lines, -2 removed lines

@@ -143,8 +143,9 @@
 
     // SMTP server can take longer to respond, give longer timeout for first read
     // Windows does not have support for this timeout function
-    if(substr(PHP_OS, 0, 3) != "WIN")
-     socket_set_timeout($this->smtp_conn, $tval, 0);
+    if(substr(PHP_OS, 0, 3) != "WIN") {
+         socket_set_timeout($this->smtp_conn, $tval, 0);
+    }
 
     // get any announcement
     $announce = $this->get_lines();

Spacing +158 added lines, -158 removed lines

@@ -63,7 +63,7 @@ 
    *  Sets whether debugging is turned on
    *  @var bool
    */
-  public $do_debug;       // the level of debug to perform
+  public $do_debug; // the level of debug to perform
 
   /**
    *  Sets VERP use on/off (default is off)
@@ -75,9 +75,9 @@ 
   // PROPERTIES, PRIVATE AND PROTECTED
   /////////////////////////////////////////////////
 
-  private $smtp_conn;      // the socket to the server
-  private $error;          // error if any on the last call
-  private $helo_rply;      // the reply the server sent to us for HELO
+  private $smtp_conn; // the socket to the server
+  private $error; // error if any on the last call
+  private $helo_rply; // the reply the server sent to us for HELO
 
   /**
    * Initialize the class so that the data is in a known state.
@@ -109,48 +109,48 @@ 
    * @access public
    * @return bool
    */
-  public function Connect($host,$port=0,$tval=30) {
+  public function Connect($host, $port = 0, $tval = 30) {
     // set the error val to null so there is no confusion
     $this->error = null;
 
     // make sure we are __not__ connected
-    if($this->connected()) {
+    if ($this->connected()) {
       // already connected, generate error
       $this->error = array("error" => "Already connected to a server");
       return false;
     }
 
-    if(empty($port)) {
+    if (empty($port)) {
       $port = $this->SMTP_PORT;
     }
 
     // connect to the smtp server
-    $this->smtp_conn = @fsockopen($host,    // the host of the server
-                                 $port,    // the port to use
-                                 $errno,   // error number if any
-                                 $errstr,  // error message if any
-                                 $tval);   // give up after ? secs
+    $this->smtp_conn = @fsockopen($host, // the host of the server
+                                 $port, // the port to use
+                                 $errno, // error number if any
+                                 $errstr, // error message if any
+                                 $tval); // give up after ? secs
     // verify we connected properly
-    if(empty($this->smtp_conn)) {
+    if (empty($this->smtp_conn)) {
       $this->error = array("error" => "Failed to connect to server",
                            "errno" => $errno,
                            "errstr" => $errstr);
-      if($this->do_debug >= 1) {
-        echo "SMTP -> ERROR: " . $this->error["error"] . ": $errstr ($errno)" . $this->CRLF . '<br />';
+      if ($this->do_debug >= 1) {
+        echo "SMTP -> ERROR: ".$this->error["error"].": $errstr ($errno)".$this->CRLF.'<br />';
       }
       return false;
     }
 
     // SMTP server can take longer to respond, give longer timeout for first read
     // Windows does not have support for this timeout function
-    if(substr(PHP_OS, 0, 3) != "WIN")
+    if (substr(PHP_OS, 0, 3) != "WIN")
      socket_set_timeout($this->smtp_conn, $tval, 0);
 
     // get any announcement
     $announce = $this->get_lines();
 
-    if($this->do_debug >= 2) {
-      echo "SMTP -> FROM SERVER:" . $announce . $this->CRLF . '<br />';
+    if ($this->do_debug >= 2) {
+      echo "SMTP -> FROM SERVER:".$announce.$this->CRLF.'<br />';
     }
 
     return true;
@@ -168,33 +168,33 @@ 
   public function StartTLS() {
     $this->error = null; # to avoid confusion
 
-    if(!$this->connected()) {
+    if (!$this->connected()) {
       $this->error = array("error" => "Called StartTLS() without being connected");
       return false;
     }
 
-    fputs($this->smtp_conn,"STARTTLS" . $this->CRLF);
+    fputs($this->smtp_conn, "STARTTLS".$this->CRLF);
 
     $rply = $this->get_lines();
-    $code = substr($rply,0,3);
+    $code = substr($rply, 0, 3);
 
-    if($this->do_debug >= 2) {
-      echo "SMTP -> FROM SERVER:" . $rply . $this->CRLF . '<br />';
+    if ($this->do_debug >= 2) {
+      echo "SMTP -> FROM SERVER:".$rply.$this->CRLF.'<br />';
     }
 
-    if($code != 220) {
+    if ($code != 220) {
       $this->error =
          array("error"     => "STARTTLS not accepted from server",
                "smtp_code" => $code,
-               "smtp_msg"  => substr($rply,4));
-      if($this->do_debug >= 1) {
-        echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';
+               "smtp_msg"  => substr($rply, 4));
+      if ($this->do_debug >= 1) {
+        echo "SMTP -> ERROR: ".$this->error["error"].": ".$rply.$this->CRLF.'<br />';
       }
       return false;
     }
 
     //Begin encrypted connection
-    if(!stream_socket_enable_crypto($this->smtp_conn, true, STREAM_CRYPTO_METHOD_TLS_CLIENT)) {
+    if (!stream_socket_enable_crypto($this->smtp_conn, true, STREAM_CRYPTO_METHOD_TLS_CLIENT)) {
       return false;
     }
 
@@ -209,52 +209,52 @@ 
    */
   public function Authenticate($username, $password) {
     // Start authentication
-    fputs($this->smtp_conn,"AUTH LOGIN" . $this->CRLF);
+    fputs($this->smtp_conn, "AUTH LOGIN".$this->CRLF);
 
     $rply = $this->get_lines();
-    $code = substr($rply,0,3);
+    $code = substr($rply, 0, 3);
 
-    if($code != 334) {
+    if ($code != 334) {
       $this->error =
         array("error" => "AUTH not accepted from server",
               "smtp_code" => $code,
-              "smtp_msg" => substr($rply,4));
-      if($this->do_debug >= 1) {
-        echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';
+              "smtp_msg" => substr($rply, 4));
+      if ($this->do_debug >= 1) {
+        echo "SMTP -> ERROR: ".$this->error["error"].": ".$rply.$this->CRLF.'<br />';
       }
       return false;
     }
 
     // Send encoded username
-    fputs($this->smtp_conn, base64_encode($username) . $this->CRLF);
+    fputs($this->smtp_conn, base64_encode($username).$this->CRLF);
 
     $rply = $this->get_lines();
-    $code = substr($rply,0,3);
+    $code = substr($rply, 0, 3);
 
-    if($code != 334) {
+    if ($code != 334) {
       $this->error =
         array("error" => "Username not accepted from server",
               "smtp_code" => $code,
-              "smtp_msg" => substr($rply,4));
-      if($this->do_debug >= 1) {
-        echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';
+              "smtp_msg" => substr($rply, 4));
+      if ($this->do_debug >= 1) {
+        echo "SMTP -> ERROR: ".$this->error["error"].": ".$rply.$this->CRLF.'<br />';
       }
       return false;
     }
 
     // Send encoded password
-    fputs($this->smtp_conn, base64_encode($password) . $this->CRLF);
+    fputs($this->smtp_conn, base64_encode($password).$this->CRLF);
 
     $rply = $this->get_lines();
-    $code = substr($rply,0,3);
+    $code = substr($rply, 0, 3);
 
-    if($code != 235) {
+    if ($code != 235) {
       $this->error =
         array("error" => "Password not accepted from server",
               "smtp_code" => $code,
-              "smtp_msg" => substr($rply,4));
-      if($this->do_debug >= 1) {
-        echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';
+              "smtp_msg" => substr($rply, 4));
+      if ($this->do_debug >= 1) {
+        echo "SMTP -> ERROR: ".$this->error["error"].": ".$rply.$this->CRLF.'<br />';
       }
       return false;
     }
@@ -268,12 +268,12 @@ 
    * @return bool
    */
   public function Connected() {
-    if(!empty($this->smtp_conn)) {
+    if (!empty($this->smtp_conn)) {
       $sock_status = socket_get_status($this->smtp_conn);
-      if($sock_status["eof"]) {
+      if ($sock_status["eof"]) {
         // the socket is valid but we are not connected
-        if($this->do_debug >= 1) {
-            echo "SMTP -> NOTICE:" . $this->CRLF . "EOF caught while checking if connected";
+        if ($this->do_debug >= 1) {
+            echo "SMTP -> NOTICE:".$this->CRLF."EOF caught while checking if connected";
         }
         $this->Close();
         return false;
@@ -293,7 +293,7 @@ 
   public function Close() {
     $this->error = null; // so there is no confusion
     $this->helo_rply = null;
-    if(!empty($this->smtp_conn)) {
+    if (!empty($this->smtp_conn)) {
       // close the connection and cleanup
       fclose($this->smtp_conn);
       $this->smtp_conn = 0;
@@ -326,28 +326,28 @@ 
   public function Data($msg_data) {
     $this->error = null; // so no confusion is caused
 
-    if(!$this->connected()) {
+    if (!$this->connected()) {
       $this->error = array(
               "error" => "Called Data() without being connected");
       return false;
     }
 
-    fputs($this->smtp_conn,"DATA" . $this->CRLF);
+    fputs($this->smtp_conn, "DATA".$this->CRLF);
 
     $rply = $this->get_lines();
-    $code = substr($rply,0,3);
+    $code = substr($rply, 0, 3);
 
-    if($this->do_debug >= 2) {
-      echo "SMTP -> FROM SERVER:" . $rply . $this->CRLF . '<br />';
+    if ($this->do_debug >= 2) {
+      echo "SMTP -> FROM SERVER:".$rply.$this->CRLF.'<br />';
     }
 
-    if($code != 354) {
+    if ($code != 354) {
       $this->error =
         array("error" => "DATA command not accepted from server",
               "smtp_code" => $code,
-              "smtp_msg" => substr($rply,4));
-      if($this->do_debug >= 1) {
-        echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';
+              "smtp_msg" => substr($rply, 4));
+      if ($this->do_debug >= 1) {
+        echo "SMTP -> ERROR: ".$this->error["error"].": ".$rply.$this->CRLF.'<br />';
       }
       return false;
     }
@@ -364,9 +364,9 @@ 
      */
 
     // normalize the line breaks so we know the explode works
-    $msg_data = str_replace("\r\n","\n",$msg_data);
-    $msg_data = str_replace("\r","\n",$msg_data);
-    $lines = explode("\n",$msg_data);
+    $msg_data = str_replace("\r\n", "\n", $msg_data);
+    $msg_data = str_replace("\r", "\n", $msg_data);
+    $lines = explode("\n", $msg_data);
 
     /* we need to find a good way to determine is headers are
      * in the msg_data or if it is a straight msg body
@@ -377,72 +377,72 @@ 
      * headers.
      */
 
-    $field = substr($lines[0],0,strpos($lines[0],":"));
+    $field = substr($lines[0], 0, strpos($lines[0], ":"));
     $in_headers = false;
-    if(!empty($field) && !strstr($field," ")) {
+    if (!empty($field) && !strstr($field, " ")) {
       $in_headers = true;
     }
 
     $max_line_length = 998; // used below; set here for ease in change
 
-    while(list(,$line) = @each($lines)) {
+    while (list(,$line) = @each($lines)) {
       $lines_out = null;
-      if($line == "" && $in_headers) {
+      if ($line == "" && $in_headers) {
         $in_headers = false;
       }
       // ok we need to break this line up into several smaller lines
-      while(strlen($line) > $max_line_length) {
-        $pos = strrpos(substr($line,0,$max_line_length)," ");
+      while (strlen($line) > $max_line_length) {
+        $pos = strrpos(substr($line, 0, $max_line_length), " ");
 
         // Patch to fix DOS attack
-        if(!$pos) {
+        if (!$pos) {
           $pos = $max_line_length - 1;
-          $lines_out[] = substr($line,0,$pos);
-          $line = substr($line,$pos);
+          $lines_out[] = substr($line, 0, $pos);
+          $line = substr($line, $pos);
         } else {
-          $lines_out[] = substr($line,0,$pos);
-          $line = substr($line,$pos + 1);
+          $lines_out[] = substr($line, 0, $pos);
+          $line = substr($line, $pos + 1);
         }
 
         /* if we are processing headers we need to
          * add a LWSP-char to the front of the new line
          * rfc 822 on long msg headers
          */
-        if($in_headers) {
-          $line = "\t" . $line;
+        if ($in_headers) {
+          $line = "\t".$line;
         }
       }
       $lines_out[] = $line;
 
       // send the lines to the server
-      while(list(,$line_out) = @each($lines_out)) {
-        if(strlen($line_out) > 0)
+      while (list(,$line_out) = @each($lines_out)) {
+        if (strlen($line_out) > 0)
         {
-          if(substr($line_out, 0, 1) == ".") {
-            $line_out = "." . $line_out;
+          if (substr($line_out, 0, 1) == ".") {
+            $line_out = ".".$line_out;
           }
         }
-        fputs($this->smtp_conn,$line_out . $this->CRLF);
+        fputs($this->smtp_conn, $line_out.$this->CRLF);
       }
     }
 
     // message data has been sent
-    fputs($this->smtp_conn, $this->CRLF . "." . $this->CRLF);
+    fputs($this->smtp_conn, $this->CRLF.".".$this->CRLF);
 
     $rply = $this->get_lines();
-    $code = substr($rply,0,3);
+    $code = substr($rply, 0, 3);
 
-    if($this->do_debug >= 2) {
-      echo "SMTP -> FROM SERVER:" . $rply . $this->CRLF . '<br />';
+    if ($this->do_debug >= 2) {
+      echo "SMTP -> FROM SERVER:".$rply.$this->CRLF.'<br />';
     }
 
-    if($code != 250) {
+    if ($code != 250) {
       $this->error =
         array("error" => "DATA not accepted from server",
               "smtp_code" => $code,
-              "smtp_msg" => substr($rply,4));
-      if($this->do_debug >= 1) {
-        echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';
+              "smtp_msg" => substr($rply, 4));
+      if ($this->do_debug >= 1) {
+        echo "SMTP -> ERROR: ".$this->error["error"].": ".$rply.$this->CRLF.'<br />';
       }
       return false;
     }
@@ -464,21 +464,21 @@ 
   public function Hello($host = '') {
     $this->error = null; // so no confusion is caused
 
-    if(!$this->connected()) {
+    if (!$this->connected()) {
       $this->error = array(
             "error" => "Called Hello() without being connected");
       return false;
     }
 
     // if hostname for HELO was not specified send default
-    if(empty($host)) {
+    if (empty($host)) {
       // determine appropriate default to send to server
       $host = "localhost";
     }
 
     // Send extended hello first (RFC 2821)
-    if(!$this->SendHello("EHLO", $host)) {
-      if(!$this->SendHello("HELO", $host)) {
+    if (!$this->SendHello("EHLO", $host)) {
+      if (!$this->SendHello("HELO", $host)) {
           return false;
     }
     }
@@ -492,22 +492,22 @@ 
    * @return bool
    */
   private function SendHello($hello, $host) {
-    fputs($this->smtp_conn, $hello . " " . $host . $this->CRLF);
+    fputs($this->smtp_conn, $hello." ".$host.$this->CRLF);
 
     $rply = $this->get_lines();
-    $code = substr($rply,0,3);
+    $code = substr($rply, 0, 3);
 
-    if($this->do_debug >= 2) {
-      echo "SMTP -> FROM SERVER: " . $rply . $this->CRLF . '<br />';
+    if ($this->do_debug >= 2) {
+      echo "SMTP -> FROM SERVER: ".$rply.$this->CRLF.'<br />';
     }
 
-    if($code != 250) {
+    if ($code != 250) {
       $this->error =
-        array("error" => $hello . " not accepted from server",
+        array("error" => $hello." not accepted from server",
               "smtp_code" => $code,
-              "smtp_msg" => substr($rply,4));
-      if($this->do_debug >= 1) {
-        echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';
+              "smtp_msg" => substr($rply, 4));
+      if ($this->do_debug >= 1) {
+        echo "SMTP -> ERROR: ".$this->error["error"].": ".$rply.$this->CRLF.'<br />';
       }
       return false;
     }
@@ -534,29 +534,29 @@ 
   public function Mail($from) {
     $this->error = null; // so no confusion is caused
 
-    if(!$this->connected()) {
+    if (!$this->connected()) {
       $this->error = array(
               "error" => "Called Mail() without being connected");
       return false;
     }
 
     $useVerp = ($this->do_verp ? "XVERP" : "");
-    fputs($this->smtp_conn,"MAIL FROM:<" . $from . ">" . $useVerp . $this->CRLF);
+    fputs($this->smtp_conn, "MAIL FROM:<".$from.">".$useVerp.$this->CRLF);
 
     $rply = $this->get_lines();
-    $code = substr($rply,0,3);
+    $code = substr($rply, 0, 3);
 
-    if($this->do_debug >= 2) {
-      echo "SMTP -> FROM SERVER:" . $rply . $this->CRLF . '<br />';
+    if ($this->do_debug >= 2) {
+      echo "SMTP -> FROM SERVER:".$rply.$this->CRLF.'<br />';
     }
 
-    if($code != 250) {
+    if ($code != 250) {
       $this->error =
         array("error" => "MAIL not accepted from server",
               "smtp_code" => $code,
-              "smtp_msg" => substr($rply,4));
-      if($this->do_debug >= 1) {
-        echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';
+              "smtp_msg" => substr($rply, 4));
+      if ($this->do_debug >= 1) {
+        echo "SMTP -> ERROR: ".$this->error["error"].": ".$rply.$this->CRLF.'<br />';
       }
       return false;
     }
@@ -574,41 +574,41 @@ 
    * @access public
    * @return bool
    */
-  public function Quit($close_on_error=true) {
+  public function Quit($close_on_error = true) {
     $this->error = null; // so there is no confusion
 
-    if(!$this->connected()) {
+    if (!$this->connected()) {
       $this->error = array(
               "error" => "Called Quit() without being connected");
       return false;
     }
 
     // send the quit command to the server
-    fputs($this->smtp_conn,"quit" . $this->CRLF);
+    fputs($this->smtp_conn, "quit".$this->CRLF);
 
     // get any good-bye messages
     $byemsg = $this->get_lines();
 
-    if($this->do_debug >= 2) {
-      echo "SMTP -> FROM SERVER:" . $byemsg . $this->CRLF . '<br />';
+    if ($this->do_debug >= 2) {
+      echo "SMTP -> FROM SERVER:".$byemsg.$this->CRLF.'<br />';
     }
 
     $rval = true;
     $e = null;
 
-    $code = substr($byemsg,0,3);
-    if($code != 221) {
+    $code = substr($byemsg, 0, 3);
+    if ($code != 221) {
       // use e as a tmp var cause Close will overwrite $this->error
       $e = array("error" => "SMTP server rejected quit command",
                  "smtp_code" => $code,
-                 "smtp_rply" => substr($byemsg,4));
+                 "smtp_rply" => substr($byemsg, 4));
       $rval = false;
-      if($this->do_debug >= 1) {
-        echo "SMTP -> ERROR: " . $e["error"] . ": " . $byemsg . $this->CRLF . '<br />';
+      if ($this->do_debug >= 1) {
+        echo "SMTP -> ERROR: ".$e["error"].": ".$byemsg.$this->CRLF.'<br />';
       }
     }
 
-    if(empty($e) || $close_on_error) {
+    if (empty($e) || $close_on_error) {
       $this->Close();
     }
 
@@ -630,28 +630,28 @@ 
   public function Recipient($to) {
     $this->error = null; // so no confusion is caused
 
-    if(!$this->connected()) {
+    if (!$this->connected()) {
       $this->error = array(
               "error" => "Called Recipient() without being connected");
       return false;
     }
 
-    fputs($this->smtp_conn,"RCPT TO:<" . $to . ">" . $this->CRLF);
+    fputs($this->smtp_conn, "RCPT TO:<".$to.">".$this->CRLF);
 
     $rply = $this->get_lines();
-    $code = substr($rply,0,3);
+    $code = substr($rply, 0, 3);
 
-    if($this->do_debug >= 2) {
-      echo "SMTP -> FROM SERVER:" . $rply . $this->CRLF . '<br />';
+    if ($this->do_debug >= 2) {
+      echo "SMTP -> FROM SERVER:".$rply.$this->CRLF.'<br />';
     }
 
-    if($code != 250 && $code != 251) {
+    if ($code != 250 && $code != 251) {
       $this->error =
         array("error" => "RCPT not accepted from server",
               "smtp_code" => $code,
-              "smtp_msg" => substr($rply,4));
-      if($this->do_debug >= 1) {
-        echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';
+              "smtp_msg" => substr($rply, 4));
+      if ($this->do_debug >= 1) {
+        echo "SMTP -> ERROR: ".$this->error["error"].": ".$rply.$this->CRLF.'<br />';
       }
       return false;
     }
@@ -673,28 +673,28 @@ 
   public function Reset() {
     $this->error = null; // so no confusion is caused
 
-    if(!$this->connected()) {
+    if (!$this->connected()) {
       $this->error = array(
               "error" => "Called Reset() without being connected");
       return false;
     }
 
-    fputs($this->smtp_conn,"RSET" . $this->CRLF);
+    fputs($this->smtp_conn, "RSET".$this->CRLF);
 
     $rply = $this->get_lines();
-    $code = substr($rply,0,3);
+    $code = substr($rply, 0, 3);
 
-    if($this->do_debug >= 2) {
-      echo "SMTP -> FROM SERVER:" . $rply . $this->CRLF . '<br />';
+    if ($this->do_debug >= 2) {
+      echo "SMTP -> FROM SERVER:".$rply.$this->CRLF.'<br />';
     }
 
-    if($code != 250) {
+    if ($code != 250) {
       $this->error =
         array("error" => "RSET failed",
               "smtp_code" => $code,
-              "smtp_msg" => substr($rply,4));
-      if($this->do_debug >= 1) {
-        echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';
+              "smtp_msg" => substr($rply, 4));
+      if ($this->do_debug >= 1) {
+        echo "SMTP -> ERROR: ".$this->error["error"].": ".$rply.$this->CRLF.'<br />';
       }
       return false;
     }
@@ -721,28 +721,28 @@ 
   public function SendAndMail($from) {
     $this->error = null; // so no confusion is caused
 
-    if(!$this->connected()) {
+    if (!$this->connected()) {
       $this->error = array(
           "error" => "Called SendAndMail() without being connected");
       return false;
     }
 
-    fputs($this->smtp_conn,"SAML FROM:" . $from . $this->CRLF);
+    fputs($this->smtp_conn, "SAML FROM:".$from.$this->CRLF);
 
     $rply = $this->get_lines();
-    $code = substr($rply,0,3);
+    $code = substr($rply, 0, 3);
 
-    if($this->do_debug >= 2) {
-      echo "SMTP -> FROM SERVER:" . $rply . $this->CRLF . '<br />';
+    if ($this->do_debug >= 2) {
+      echo "SMTP -> FROM SERVER:".$rply.$this->CRLF.'<br />';
     }
 
-    if($code != 250) {
+    if ($code != 250) {
       $this->error =
         array("error" => "SAML not accepted from server",
               "smtp_code" => $code,
-              "smtp_msg" => substr($rply,4));
-      if($this->do_debug >= 1) {
-        echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';
+              "smtp_msg" => substr($rply, 4));
+      if ($this->do_debug >= 1) {
+        echo "SMTP -> ERROR: ".$this->error["error"].": ".$rply.$this->CRLF.'<br />';
       }
       return false;
     }
@@ -765,8 +765,8 @@ 
   public function Turn() {
     $this->error = array("error" => "This method, TURN, of the SMTP ".
                                     "is not implemented");
-    if($this->do_debug >= 1) {
-      echo "SMTP -> NOTICE: " . $this->error["error"] . $this->CRLF . '<br />';
+    if ($this->do_debug >= 1) {
+      echo "SMTP -> NOTICE: ".$this->error["error"].$this->CRLF.'<br />';
     }
     return false;
   }
@@ -795,17 +795,17 @@ 
    */
   private function get_lines() {
     $data = "";
-    while($str = @fgets($this->smtp_conn,515)) {
-      if($this->do_debug >= 4) {
-        echo "SMTP -> get_lines(): \$data was \"$data\"" . $this->CRLF . '<br />';
-        echo "SMTP -> get_lines(): \$str is \"$str\"" . $this->CRLF . '<br />';
+    while ($str = @fgets($this->smtp_conn, 515)) {
+      if ($this->do_debug >= 4) {
+        echo "SMTP -> get_lines(): \$data was \"$data\"".$this->CRLF.'<br />';
+        echo "SMTP -> get_lines(): \$str is \"$str\"".$this->CRLF.'<br />';
       }
       $data .= $str;
-      if($this->do_debug >= 4) {
-        echo "SMTP -> get_lines(): \$data is \"$data\"" . $this->CRLF . '<br />';
+      if ($this->do_debug >= 4) {
+        echo "SMTP -> get_lines(): \$data is \"$data\"".$this->CRLF.'<br />';
       }
       // if 4th character is a space, we are done reading, break the loop
-      if(substr($str,3,1) == " ") { break; }
+      if (substr($str, 3, 1) == " ") { break; }
     }
     return $data;
   }

libs/tar.class.php

Doc Comments +24 added lines

@@ -84,6 +84,10 @@ 
     // Computes the unsigned Checksum of a file's header
     // to try to ensure valid file
     // PRIVATE ACCESS FUNCTION
+
+    /**
+     * @param string $bytestring
+     */
     function __computeUnsignedChecksum($bytestring) {
         for($i=0; $i<512; $i++)
             $unsigned_chksum += ord($bytestring[$i]);
@@ -97,6 +101,10 @@ 
 
     // Converts a NULL padded string to a non-NULL padded string
     // PRIVATE ACCESS FUNCTION
+
+    /**
+     * @param string $string
+     */
     function __parseNullPaddedString($string) {
         $position = strpos($string,chr(0));
 		if(!$position)
@@ -357,6 +365,10 @@ 
 
 
     // Open a TAR file
+
+    /**
+     * @param string $filename
+     */
     function openTAR($filename) {
         // Clear any values from previous tar archives
         unset($this->filename);
@@ -393,6 +405,10 @@ 
 
 
     // Retrieves information about a file in the current tar archive
+
+    /**
+     * @param string $filename
+     */
     function getFile($filename) {
         if($this->numFiles > 0) {
             foreach($this->files as $key => $information) {
@@ -467,6 +483,10 @@ 
 
 
     // Add a file to the tar archive
+
+    /**
+     * @param string $filename
+     */
     function addFile($filename,$from=null,$to=null) {
         // Make sure the file we are adding exists!
         if(!file_exists($filename))
@@ -555,6 +575,10 @@ 
 
 
     // Saves tar archive to a different file than the current file
+
+    /**
+     * @param boolean $useGzip
+     */
     function toTar($filename,$useGzip) {
         if(!$filename)
             return false;

Indentation +431 added lines, -431 removed lines

@@ -63,99 +63,99 @@ 
 */
 
 class tar {
-    // Unprocessed Archive Information
-    var $filename;
-    var $isGzipped;
-    var $tar_file;
-
-    // Processed Archive Information
-    var $files;
-    var $directories;
-    var $numFiles;
-    var $numDirectories;
-
-
-    // Class Constructor -- Does nothing...
-    function tar() {
-        return true;
-    }
-
-
-    // Computes the unsigned Checksum of a file's header
-    // to try to ensure valid file
-    // PRIVATE ACCESS FUNCTION
-    function __computeUnsignedChecksum($bytestring) {
-        for($i=0; $i<512; $i++)
-            $unsigned_chksum += ord($bytestring[$i]);
-        for($i=0; $i<8; $i++)
-            $unsigned_chksum -= ord($bytestring[148 + $i]);
-        $unsigned_chksum += ord(" ") * 8;
-
-        return $unsigned_chksum;
-    }
-
-
-    // Converts a NULL padded string to a non-NULL padded string
-    // PRIVATE ACCESS FUNCTION
-    function __parseNullPaddedString($string) {
-        $position = strpos($string,chr(0));
+	// Unprocessed Archive Information
+	var $filename;
+	var $isGzipped;
+	var $tar_file;
+
+	// Processed Archive Information
+	var $files;
+	var $directories;
+	var $numFiles;
+	var $numDirectories;
+
+
+	// Class Constructor -- Does nothing...
+	function tar() {
+		return true;
+	}
+
+
+	// Computes the unsigned Checksum of a file's header
+	// to try to ensure valid file
+	// PRIVATE ACCESS FUNCTION
+	function __computeUnsignedChecksum($bytestring) {
+		for($i=0; $i<512; $i++)
+			$unsigned_chksum += ord($bytestring[$i]);
+		for($i=0; $i<8; $i++)
+			$unsigned_chksum -= ord($bytestring[148 + $i]);
+		$unsigned_chksum += ord(" ") * 8;
+
+		return $unsigned_chksum;
+	}
+
+
+	// Converts a NULL padded string to a non-NULL padded string
+	// PRIVATE ACCESS FUNCTION
+	function __parseNullPaddedString($string) {
+		$position = strpos($string,chr(0));
 		if(!$position)
 		{
 			$position = strlen($string);
 		}
-        return substr($string,0,$position);
-    }
+		return substr($string,0,$position);
+	}
 
 
-    // This function parses the current TAR file
-    // PRIVATE ACCESS FUNCTION
-    function __parseTar() {
-        // Read Files from archive
-        $tar_length = strlen($this->tar_file);
-        $main_offset = 0;
+	// This function parses the current TAR file
+	// PRIVATE ACCESS FUNCTION
+	function __parseTar() {
+		// Read Files from archive
+		$tar_length = strlen($this->tar_file);
+		$main_offset = 0;
 		$flag_longlink = false;
-        while($main_offset < $tar_length) {
-            // If we read a block of 512 nulls, we are at the end of the archive
-            if(substr($this->tar_file,$main_offset,512) == str_repeat(chr(0),512))
-                break;
+		while($main_offset < $tar_length) {
+			// If we read a block of 512 nulls, we are at the end of the archive
+			if(substr($this->tar_file,$main_offset,512) == str_repeat(chr(0),512))
+				break;
 
-            // Parse file name
-            $file_name      = $this->__parseNullPaddedString(substr($this->tar_file,$main_offset,100));
+			// Parse file name
+			$file_name      = $this->__parseNullPaddedString(substr($this->tar_file,$main_offset,100));
 
-            // Parse the file mode
-            $file_mode      = substr($this->tar_file,$main_offset + 100,8);
+			// Parse the file mode
+			$file_mode      = substr($this->tar_file,$main_offset + 100,8);
 
-            // Parse the file user ID
-            $file_uid       = octdec(substr($this->tar_file,$main_offset + 108,8));
+			// Parse the file user ID
+			$file_uid       = octdec(substr($this->tar_file,$main_offset + 108,8));
 
-            // Parse the file group ID
-            $file_gid       = octdec(substr($this->tar_file,$main_offset + 116,8));
+			// Parse the file group ID
+			$file_gid       = octdec(substr($this->tar_file,$main_offset + 116,8));
 
-            // Parse the file size
-            $file_size      = octdec(substr($this->tar_file,$main_offset + 124,12));
+			// Parse the file size
+			$file_size      = octdec(substr($this->tar_file,$main_offset + 124,12));
 
-            // Parse the file update time - unix timestamp format
-            $file_time      = octdec(substr($this->tar_file,$main_offset + 136,12));
+			// Parse the file update time - unix timestamp format
+			$file_time      = octdec(substr($this->tar_file,$main_offset + 136,12));
 
-            // Parse Checksum
-            $file_chksum        = octdec(substr($this->tar_file,$main_offset + 148,6));
+			// Parse Checksum
+			$file_chksum        = octdec(substr($this->tar_file,$main_offset + 148,6));
 
-            // Parse user name
-            $file_uname     = $this->__parseNullPaddedString(substr($this->tar_file,$main_offset + 265,32));
+			// Parse user name
+			$file_uname     = $this->__parseNullPaddedString(substr($this->tar_file,$main_offset + 265,32));
 
-            // Parse Group name
-            $file_gname     = $this->__parseNullPaddedString(substr($this->tar_file,$main_offset + 297,32));
+			// Parse Group name
+			$file_gname     = $this->__parseNullPaddedString(substr($this->tar_file,$main_offset + 297,32));
 
-            $file_type = substr($this->tar_file,$main_offset + 156,1);
+			$file_type = substr($this->tar_file,$main_offset + 156,1);
 
-            // Make sure our file is valid
-            if($this->__computeUnsignedChecksum(substr($this->tar_file,$main_offset,512)) != $file_chksum)
-                return false;
+			// Make sure our file is valid
+			if($this->__computeUnsignedChecksum(substr($this->tar_file,$main_offset,512)) != $file_chksum)
+				return false;
 
-            // Parse File Contents
-            $file_contents      = substr($this->tar_file,$main_offset + 512,$file_size);
+			// Parse File Contents
+			$file_contents      = substr($this->tar_file,$main_offset + 512,$file_size);
 
-            /*  ### Unused Header Information ###
+			/*  ### Unused Header Information ###
                 $activeFile["typeflag"]     = substr($this->tar_file,$main_offset + 156,1);
                 $activeFile["linkname"]     = substr($this->tar_file,$main_offset + 157,100);
                 $activeFile["magic"]        = substr($this->tar_file,$main_offset + 257,6);
@@ -228,363 +228,363 @@ 
 				$flag_longlink = false;
 			}
 
-            // Move our offset the number of blocks we have processed
-            $main_offset += 512 + (ceil($file_size / 512) * 512);
-        }
-
-        return true;
-    }
-
-
-    // Read a non gzipped tar file in for processing
-    // PRIVATE ACCESS FUNCTION
-    function __readTar($filename='') {
-        // Set the filename to load
-        if(!$filename)
-            $filename = $this->filename;
-
-        // Read in the TAR file
-        $fp = fopen($filename,"rb");
-        $this->tar_file = fread($fp,filesize($filename));
-        fclose($fp);
-
-        if($this->tar_file[0] == chr(31) && $this->tar_file[1] == chr(139) && $this->tar_file[2] == chr(8)) {
-            if(!function_exists("gzinflate"))
-                return false;
-
-            $this->isGzipped = TRUE;
-
-            $this->tar_file = gzinflate(substr($this->tar_file,10,-4));
-        }
-
-        // Parse the TAR file
-        $this->__parseTar();
-
-        return true;
-    }
-
-
-    // Generates a TAR file from the processed data
-    // PRIVATE ACCESS FUNCTION
-    function __generateTAR() {
-        // Clear any data currently in $this->tar_file
-        unset($this->tar_file);
-
-        // Generate Records for each directory, if we have directories
-        if($this->numDirectories > 0) {
-            foreach($this->directories as $key => $information) {
-                unset($header);
-
-                // Generate tar header for this directory
-                // Filename, Permissions, UID, GID, size, Time, checksum, typeflag, linkname, magic, version, user name, group name, devmajor, devminor, prefix, end
-                $header .= str_pad($information["name"],100,chr(0));
-                $header .= str_pad(decoct($information["mode"]),7,"0",STR_PAD_LEFT) . chr(0);
-                $header .= str_pad(decoct($information["user_id"]),7,"0",STR_PAD_LEFT) . chr(0);
-                $header .= str_pad(decoct($information["group_id"]),7,"0",STR_PAD_LEFT) . chr(0);
-                $header .= str_pad(decoct(0),11,"0",STR_PAD_LEFT) . chr(0);
-                $header .= str_pad(decoct($information["time"]),11,"0",STR_PAD_LEFT) . chr(0);
-                $header .= str_repeat(" ",8);
-                $header .= "5";
-                $header .= str_repeat(chr(0),100);
-                $header .= str_pad("ustar",6,chr(32));
-                $header .= chr(32) . chr(0);
-                $header .= str_pad("",32,chr(0));
-                $header .= str_pad("",32,chr(0));
-                $header .= str_repeat(chr(0),8);
-                $header .= str_repeat(chr(0),8);
-                $header .= str_repeat(chr(0),155);
-                $header .= str_repeat(chr(0),12);
-
-                // Compute header checksum
-                $checksum = str_pad(decoct($this->__computeUnsignedChecksum($header)),6,"0",STR_PAD_LEFT);
-                for($i=0; $i<6; $i++) {
-                    $header[(148 + $i)] = substr($checksum,$i,1);
-                }
-                $header[154] = chr(0);
-                $header[155] = chr(32);
-
-                // Add new tar formatted data to tar file contents
-                $this->tar_file .= $header;
-            }
-        }
-
-        // Generate Records for each file, if we have files (We should...)
-        if($this->numFiles > 0) {
-            foreach($this->files as $key => $information) {
-                unset($header);
-
-                // Generate the TAR header for this file
-                // Filename, Permissions, UID, GID, size, Time, checksum, typeflag, linkname, magic, version, user name, group name, devmajor, devminor, prefix, end
-                $header .= str_pad($information["name"],100,chr(0));
-                $header .= str_pad(decoct($information["mode"]),7,"0",STR_PAD_LEFT) . chr(0);
-                $header .= str_pad(decoct($information["user_id"]),7,"0",STR_PAD_LEFT) . chr(0);
-                $header .= str_pad(decoct($information["group_id"]),7,"0",STR_PAD_LEFT) . chr(0);
-                $header .= str_pad(decoct($information["size"]),11,"0",STR_PAD_LEFT) . chr(0);
-                $header .= str_pad(decoct($information["time"]),11,"0",STR_PAD_LEFT) . chr(0);
-                $header .= str_repeat(" ",8);
-                $header .= "0";
-                $header .= str_repeat(chr(0),100);
-                $header .= str_pad("ustar",6,chr(32));
-                $header .= chr(32) . chr(0);
-                $header .= str_pad($information["user_name"],32,chr(0));    // How do I get a file's user name from PHP?
-                $header .= str_pad($information["group_name"],32,chr(0));   // How do I get a file's group name from PHP?
-                $header .= str_repeat(chr(0),8);
-                $header .= str_repeat(chr(0),8);
-                $header .= str_repeat(chr(0),155);
-                $header .= str_repeat(chr(0),12);
-
-                // Compute header checksum
-                $checksum = str_pad(decoct($this->__computeUnsignedChecksum($header)),6,"0",STR_PAD_LEFT);
-                for($i=0; $i<6; $i++) {
-                    $header[(148 + $i)] = substr($checksum,$i,1);
-                }
-                $header[154] = chr(0);
-                $header[155] = chr(32);
-
-                // Pad file contents to byte count divisible by 512
-                $file_contents = str_pad($information["file"],(ceil($information["size"] / 512) * 512),chr(0));
-
-                // Add new tar formatted data to tar file contents
-                $this->tar_file .= $header . $file_contents;
-            }
-        }
-
-        // Add 512 bytes of NULLs to designate EOF
-        $this->tar_file .= str_repeat(chr(0),512);
-
-        return true;
-    }
-
-
-    // Open a TAR file
-    function openTAR($filename) {
-        // Clear any values from previous tar archives
-        unset($this->filename);
-        unset($this->isGzipped);
-        unset($this->tar_file);
-        unset($this->files);
-        unset($this->directories);
-        unset($this->numFiles);
-        unset($this->numDirectories);
-
-        // If the tar file doesn't exist...
-        if(!file_exists($filename))
-            return false;
-
-        $this->filename = $filename;
-
-        // Parse this file
-        $this->__readTar();
-
-        return true;
-    }
-
-
-    // Appends a tar file to the end of the currently opened tar file
-    function appendTar($filename) {
-        // If the tar file doesn't exist...
-        if(!file_exists($filename))
-            return false;
-
-        $this->__readTar($filename);
-
-        return true;
-    }
-
-
-    // Retrieves information about a file in the current tar archive
-    function getFile($filename) {
-        if($this->numFiles > 0) {
-            foreach($this->files as $key => $information) {
-                if($information["name"] == $filename)
-                    return $information;
-            }
-        }
-
-        return false;
-    }
-
-
-    // Retrieves information about a directory in the current tar archive
-    function getDirectory($dirname) {
-        if($this->numDirectories > 0) {
-            foreach($this->directories as $key => $information) {
-                if($information["name"] == $dirname)
-                    return $information;
-            }
-        }
-
-        return false;
-    }
-
-
-    // Check if this tar archive contains a specific file
-    function containsFile($filename) {
-        if($this->numFiles > 0) {
-            foreach($this->files as $key => $information) {
-                if($information["name"] == $filename)
-                    return true;
-            }
-        }
-
-        return false;
-    }
-
-
-    // Check if this tar archive contains a specific directory
-    function containsDirectory($dirname) {
-        if($this->numDirectories > 0) {
-            foreach($this->directories as $key => $information) {
-                if($information["name"] == $dirname)
-                    return true;
-            }
-        }
-
-        return false;
-    }
-
-
-    // Add a directory to this tar archive
-    function addDirectory($dirname) {
-        if(!file_exists($dirname))
-            return false;
-
-        // Get directory information
-        $file_information = stat($dirname);
-
-        // Add directory to processed data
-        $this->numDirectories++;
-        $activeDir      = &$this->directories[];
-        $activeDir["name"]  = $dirname;
-        $activeDir["mode"]  = $file_information["mode"];
-        $activeDir["time"]  = $file_information["time"];
-        $activeDir["user_id"]   = $file_information["uid"];
-        $activeDir["group_id"]  = $file_information["gid"];
-        $activeDir["checksum"]  = $checksum;
-
-        return true;
-    }
-
-
-    // Add a file to the tar archive
-    function addFile($filename,$from=null,$to=null) {
-        // Make sure the file we are adding exists!
-        if(!file_exists($filename))
-            return false;
-
-        if(filesize($filename)==0)
-            return false;
-
-        // Make sure there are no other files in the archive that have this same filename
-        if($this->containsFile($filename))
-            return false;
-
-        // Get file information
-        $file_information = stat($filename);
-
-        // Read in the file's contents
-        $fp = fopen($filename,"rb");
-        $file_contents = fread($fp,filesize($filename));
-        fclose($fp);
-
-        if($from && $to){
-            $file_contents = str_replace($from,$to,$file_contents);
-            $file_information["size"] = strlen($file_contents);
-        }
-
-        // Add file to processed data
-        $this->numFiles++;
-        $activeFile         = &$this->files[];
-        $activeFile["name"]     = $filename;
-        $activeFile["mode"]     = $file_information["mode"];
-        $activeFile["user_id"]      = $file_information["uid"];
-        $activeFile["group_id"]     = $file_information["gid"];
-        $activeFile["size"]     = $file_information["size"];
-        $activeFile["time"]     = $file_information["mtime"];
-        $activeFile["checksum"]     = $checksum;
-        $activeFile["user_name"]    = "";
-        $activeFile["group_name"]   = "";
-        $activeFile["file"]     = $file_contents;
-
-        return true;
-    }
-
-
-    // Remove a file from the tar archive
-    function removeFile($filename) {
-        if($this->numFiles > 0) {
-            foreach($this->files as $key => $information) {
-                if($information["name"] == $filename) {
-                    $this->numFiles--;
-                    unset($this->files[$key]);
-                    return true;
-                }
-            }
-        }
-
-        return false;
-    }
-
-
-    // Remove a directory from the tar archive
-    function removeDirectory($dirname) {
-        if($this->numDirectories > 0) {
-            foreach($this->directories as $key => $information) {
-                if($information["name"] == $dirname) {
-                    $this->numDirectories--;
-                    unset($this->directories[$key]);
-                    return true;
-                }
-            }
-        }
-
-        return false;
-    }
-
-
-    // Write the currently loaded tar archive to disk
-    function saveTar() {
-        if(!$this->filename)
-            return false;
-
-        // Write tar to current file using specified gzip compression
-        $this->toTar($this->filename,$this->isGzipped);
-
-        return true;
-    }
-
-
-    // Saves tar archive to a different file than the current file
-    function toTar($filename,$useGzip) {
-        if(!$filename)
-            return false;
-
-        // Encode processed files into TAR file format
-        $this->__generateTar();
-
-        // GZ Compress the data if we need to
-        if($useGzip) {
-            // Make sure we have gzip support
-            if(!function_exists("gzencode"))
-                return false;
-
-            $file = gzencode($this->tar_file);
-        } else {
-            $file = $this->tar_file;
-        }
-
-        // Write the TAR file
-        $fp = fopen($filename,"wb");
-        fwrite($fp,$file);
-        fclose($fp);
-
-        return true;
-    }
-
-
-    function toTarStream() {
-        $this->__generateTar();
-        return $this->tar_file;
-    }
+			// Move our offset the number of blocks we have processed
+			$main_offset += 512 + (ceil($file_size / 512) * 512);
+		}
+
+		return true;
+	}
+
+
+	// Read a non gzipped tar file in for processing
+	// PRIVATE ACCESS FUNCTION
+	function __readTar($filename='') {
+		// Set the filename to load
+		if(!$filename)
+			$filename = $this->filename;
+
+		// Read in the TAR file
+		$fp = fopen($filename,"rb");
+		$this->tar_file = fread($fp,filesize($filename));
+		fclose($fp);
+
+		if($this->tar_file[0] == chr(31) && $this->tar_file[1] == chr(139) && $this->tar_file[2] == chr(8)) {
+			if(!function_exists("gzinflate"))
+				return false;
+
+			$this->isGzipped = TRUE;
+
+			$this->tar_file = gzinflate(substr($this->tar_file,10,-4));
+		}
+
+		// Parse the TAR file
+		$this->__parseTar();
+
+		return true;
+	}
+
+
+	// Generates a TAR file from the processed data
+	// PRIVATE ACCESS FUNCTION
+	function __generateTAR() {
+		// Clear any data currently in $this->tar_file
+		unset($this->tar_file);
+
+		// Generate Records for each directory, if we have directories
+		if($this->numDirectories > 0) {
+			foreach($this->directories as $key => $information) {
+				unset($header);
+
+				// Generate tar header for this directory
+				// Filename, Permissions, UID, GID, size, Time, checksum, typeflag, linkname, magic, version, user name, group name, devmajor, devminor, prefix, end
+				$header .= str_pad($information["name"],100,chr(0));
+				$header .= str_pad(decoct($information["mode"]),7,"0",STR_PAD_LEFT) . chr(0);
+				$header .= str_pad(decoct($information["user_id"]),7,"0",STR_PAD_LEFT) . chr(0);
+				$header .= str_pad(decoct($information["group_id"]),7,"0",STR_PAD_LEFT) . chr(0);
+				$header .= str_pad(decoct(0),11,"0",STR_PAD_LEFT) . chr(0);
+				$header .= str_pad(decoct($information["time"]),11,"0",STR_PAD_LEFT) . chr(0);
+				$header .= str_repeat(" ",8);
+				$header .= "5";
+				$header .= str_repeat(chr(0),100);
+				$header .= str_pad("ustar",6,chr(32));
+				$header .= chr(32) . chr(0);
+				$header .= str_pad("",32,chr(0));
+				$header .= str_pad("",32,chr(0));
+				$header .= str_repeat(chr(0),8);
+				$header .= str_repeat(chr(0),8);
+				$header .= str_repeat(chr(0),155);
+				$header .= str_repeat(chr(0),12);
+
+				// Compute header checksum
+				$checksum = str_pad(decoct($this->__computeUnsignedChecksum($header)),6,"0",STR_PAD_LEFT);
+				for($i=0; $i<6; $i++) {
+					$header[(148 + $i)] = substr($checksum,$i,1);
+				}
+				$header[154] = chr(0);
+				$header[155] = chr(32);
+
+				// Add new tar formatted data to tar file contents
+				$this->tar_file .= $header;
+			}
+		}
+
+		// Generate Records for each file, if we have files (We should...)
+		if($this->numFiles > 0) {
+			foreach($this->files as $key => $information) {
+				unset($header);
+
+				// Generate the TAR header for this file
+				// Filename, Permissions, UID, GID, size, Time, checksum, typeflag, linkname, magic, version, user name, group name, devmajor, devminor, prefix, end
+				$header .= str_pad($information["name"],100,chr(0));
+				$header .= str_pad(decoct($information["mode"]),7,"0",STR_PAD_LEFT) . chr(0);
+				$header .= str_pad(decoct($information["user_id"]),7,"0",STR_PAD_LEFT) . chr(0);
+				$header .= str_pad(decoct($information["group_id"]),7,"0",STR_PAD_LEFT) . chr(0);
+				$header .= str_pad(decoct($information["size"]),11,"0",STR_PAD_LEFT) . chr(0);
+				$header .= str_pad(decoct($information["time"]),11,"0",STR_PAD_LEFT) . chr(0);
+				$header .= str_repeat(" ",8);
+				$header .= "0";
+				$header .= str_repeat(chr(0),100);
+				$header .= str_pad("ustar",6,chr(32));
+				$header .= chr(32) . chr(0);
+				$header .= str_pad($information["user_name"],32,chr(0));    // How do I get a file's user name from PHP?
+				$header .= str_pad($information["group_name"],32,chr(0));   // How do I get a file's group name from PHP?
+				$header .= str_repeat(chr(0),8);
+				$header .= str_repeat(chr(0),8);
+				$header .= str_repeat(chr(0),155);
+				$header .= str_repeat(chr(0),12);
+
+				// Compute header checksum
+				$checksum = str_pad(decoct($this->__computeUnsignedChecksum($header)),6,"0",STR_PAD_LEFT);
+				for($i=0; $i<6; $i++) {
+					$header[(148 + $i)] = substr($checksum,$i,1);
+				}
+				$header[154] = chr(0);
+				$header[155] = chr(32);
+
+				// Pad file contents to byte count divisible by 512
+				$file_contents = str_pad($information["file"],(ceil($information["size"] / 512) * 512),chr(0));
+
+				// Add new tar formatted data to tar file contents
+				$this->tar_file .= $header . $file_contents;
+			}
+		}
+
+		// Add 512 bytes of NULLs to designate EOF
+		$this->tar_file .= str_repeat(chr(0),512);
+
+		return true;
+	}
+
+
+	// Open a TAR file
+	function openTAR($filename) {
+		// Clear any values from previous tar archives
+		unset($this->filename);
+		unset($this->isGzipped);
+		unset($this->tar_file);
+		unset($this->files);
+		unset($this->directories);
+		unset($this->numFiles);
+		unset($this->numDirectories);
+
+		// If the tar file doesn't exist...
+		if(!file_exists($filename))
+			return false;
+
+		$this->filename = $filename;
+
+		// Parse this file
+		$this->__readTar();
+
+		return true;
+	}
+
+
+	// Appends a tar file to the end of the currently opened tar file
+	function appendTar($filename) {
+		// If the tar file doesn't exist...
+		if(!file_exists($filename))
+			return false;
+
+		$this->__readTar($filename);
+
+		return true;
+	}
+
+
+	// Retrieves information about a file in the current tar archive
+	function getFile($filename) {
+		if($this->numFiles > 0) {
+			foreach($this->files as $key => $information) {
+				if($information["name"] == $filename)
+					return $information;
+			}
+		}
+
+		return false;
+	}
+
+
+	// Retrieves information about a directory in the current tar archive
+	function getDirectory($dirname) {
+		if($this->numDirectories > 0) {
+			foreach($this->directories as $key => $information) {
+				if($information["name"] == $dirname)
+					return $information;
+			}
+		}
+
+		return false;
+	}
+
+
+	// Check if this tar archive contains a specific file
+	function containsFile($filename) {
+		if($this->numFiles > 0) {
+			foreach($this->files as $key => $information) {
+				if($information["name"] == $filename)
+					return true;
+			}
+		}
+
+		return false;
+	}
+
+
+	// Check if this tar archive contains a specific directory
+	function containsDirectory($dirname) {
+		if($this->numDirectories > 0) {
+			foreach($this->directories as $key => $information) {
+				if($information["name"] == $dirname)
+					return true;
+			}
+		}
+
+		return false;
+	}
+
+
+	// Add a directory to this tar archive
+	function addDirectory($dirname) {
+		if(!file_exists($dirname))
+			return false;
+
+		// Get directory information
+		$file_information = stat($dirname);
+
+		// Add directory to processed data
+		$this->numDirectories++;
+		$activeDir      = &$this->directories[];
+		$activeDir["name"]  = $dirname;
+		$activeDir["mode"]  = $file_information["mode"];
+		$activeDir["time"]  = $file_information["time"];
+		$activeDir["user_id"]   = $file_information["uid"];
+		$activeDir["group_id"]  = $file_information["gid"];
+		$activeDir["checksum"]  = $checksum;
+
+		return true;
+	}
+
+
+	// Add a file to the tar archive
+	function addFile($filename,$from=null,$to=null) {
+		// Make sure the file we are adding exists!
+		if(!file_exists($filename))
+			return false;
+
+		if(filesize($filename)==0)
+			return false;
+
+		// Make sure there are no other files in the archive that have this same filename
+		if($this->containsFile($filename))
+			return false;
+
+		// Get file information
+		$file_information = stat($filename);
+
+		// Read in the file's contents
+		$fp = fopen($filename,"rb");
+		$file_contents = fread($fp,filesize($filename));
+		fclose($fp);
+
+		if($from && $to){
+			$file_contents = str_replace($from,$to,$file_contents);
+			$file_information["size"] = strlen($file_contents);
+		}
+
+		// Add file to processed data
+		$this->numFiles++;
+		$activeFile         = &$this->files[];
+		$activeFile["name"]     = $filename;
+		$activeFile["mode"]     = $file_information["mode"];
+		$activeFile["user_id"]      = $file_information["uid"];
+		$activeFile["group_id"]     = $file_information["gid"];
+		$activeFile["size"]     = $file_information["size"];
+		$activeFile["time"]     = $file_information["mtime"];
+		$activeFile["checksum"]     = $checksum;
+		$activeFile["user_name"]    = "";
+		$activeFile["group_name"]   = "";
+		$activeFile["file"]     = $file_contents;
+
+		return true;
+	}
+
+
+	// Remove a file from the tar archive
+	function removeFile($filename) {
+		if($this->numFiles > 0) {
+			foreach($this->files as $key => $information) {
+				if($information["name"] == $filename) {
+					$this->numFiles--;
+					unset($this->files[$key]);
+					return true;
+				}
+			}
+		}
+
+		return false;
+	}
+
+
+	// Remove a directory from the tar archive
+	function removeDirectory($dirname) {
+		if($this->numDirectories > 0) {
+			foreach($this->directories as $key => $information) {
+				if($information["name"] == $dirname) {
+					$this->numDirectories--;
+					unset($this->directories[$key]);
+					return true;
+				}
+			}
+		}
+
+		return false;
+	}
+
+
+	// Write the currently loaded tar archive to disk
+	function saveTar() {
+		if(!$this->filename)
+			return false;
+
+		// Write tar to current file using specified gzip compression
+		$this->toTar($this->filename,$this->isGzipped);
+
+		return true;
+	}
+
+
+	// Saves tar archive to a different file than the current file
+	function toTar($filename,$useGzip) {
+		if(!$filename)
+			return false;
+
+		// Encode processed files into TAR file format
+		$this->__generateTar();
+
+		// GZ Compress the data if we need to
+		if($useGzip) {
+			// Make sure we have gzip support
+			if(!function_exists("gzencode"))
+				return false;
+
+			$file = gzencode($this->tar_file);
+		} else {
+			$file = $this->tar_file;
+		}
+
+		// Write the TAR file
+		$fp = fopen($filename,"wb");
+		fwrite($fp,$file);
+		fclose($fp);
+
+		return true;
+	}
+
+
+	function toTarStream() {
+		$this->__generateTar();
+		return $this->tar_file;
+	}
 }
 ?>

Braces +60 added lines, -44 removed lines

@@ -85,10 +85,12 @@ 
     // to try to ensure valid file
     // PRIVATE ACCESS FUNCTION
     function __computeUnsignedChecksum($bytestring) {
-        for($i=0; $i<512; $i++)
-            $unsigned_chksum += ord($bytestring[$i]);
-        for($i=0; $i<8; $i++)
-            $unsigned_chksum -= ord($bytestring[148 + $i]);
+        for($i=0; $i<512; $i++) {
+                    $unsigned_chksum += ord($bytestring[$i]);
+        }
+        for($i=0; $i<8; $i++) {
+                    $unsigned_chksum -= ord($bytestring[148 + $i]);
+        }
         $unsigned_chksum += ord(" ") * 8;
 
         return $unsigned_chksum;
@@ -116,8 +118,9 @@ 
 		$flag_longlink = false;
         while($main_offset < $tar_length) {
             // If we read a block of 512 nulls, we are at the end of the archive
-            if(substr($this->tar_file,$main_offset,512) == str_repeat(chr(0),512))
-                break;
+            if(substr($this->tar_file,$main_offset,512) == str_repeat(chr(0),512)) {
+                            break;
+            }
 
             // Parse file name
             $file_name      = $this->__parseNullPaddedString(substr($this->tar_file,$main_offset,100));
@@ -149,8 +152,9 @@ 
             $file_type = substr($this->tar_file,$main_offset + 156,1);
 
             // Make sure our file is valid
-            if($this->__computeUnsignedChecksum(substr($this->tar_file,$main_offset,512)) != $file_chksum)
-                return false;
+            if($this->__computeUnsignedChecksum(substr($this->tar_file,$main_offset,512)) != $file_chksum) {
+                            return false;
+            }
 
             // Parse File Contents
             $file_contents      = substr($this->tar_file,$main_offset + 512,$file_size);
@@ -170,8 +174,7 @@ 
 			{
 				$flag_longlink = true;
 				$longlink_name = $this->__parseNullPaddedString($file_contents);
-			}
-			elseif($file_type == '0') {
+			} elseif($file_type == '0') {
 				// Increment number of files
 				$this->numFiles++;
 
@@ -182,8 +185,7 @@ 
 				if($flag_longlink)
 				{
 					$activeFile["name"]     = $longlink_name;
-				}
-				else
+				} else
 				{
 					$activeFile["name"]     = $file_name;
 				}
@@ -211,8 +213,7 @@ 
 				if($flag_longlink)
 				{
 					$activeDir["name"]     = $longlink_name;
-				}
-				else
+				} else
 				{
 					$activeDir["name"]     = $file_name;
 				}
@@ -240,8 +241,9 @@ 
     // PRIVATE ACCESS FUNCTION
     function __readTar($filename='') {
         // Set the filename to load
-        if(!$filename)
-            $filename = $this->filename;
+        if(!$filename) {
+                    $filename = $this->filename;
+        }
 
         // Read in the TAR file
         $fp = fopen($filename,"rb");
@@ -249,8 +251,9 @@ 
         fclose($fp);
 
         if($this->tar_file[0] == chr(31) && $this->tar_file[1] == chr(139) && $this->tar_file[2] == chr(8)) {
-            if(!function_exists("gzinflate"))
-                return false;
+            if(!function_exists("gzinflate")) {
+                            return false;
+            }
 
             $this->isGzipped = TRUE;
 
@@ -368,8 +371,9 @@ 
         unset($this->numDirectories);
 
         // If the tar file doesn't exist...
-        if(!file_exists($filename))
-            return false;
+        if(!file_exists($filename)) {
+                    return false;
+        }
 
         $this->filename = $filename;
 
@@ -383,8 +387,9 @@ 
     // Appends a tar file to the end of the currently opened tar file
     function appendTar($filename) {
         // If the tar file doesn't exist...
-        if(!file_exists($filename))
-            return false;
+        if(!file_exists($filename)) {
+                    return false;
+        }
 
         $this->__readTar($filename);
 
@@ -396,8 +401,9 @@ 
     function getFile($filename) {
         if($this->numFiles > 0) {
             foreach($this->files as $key => $information) {
-                if($information["name"] == $filename)
-                    return $information;
+                if($information["name"] == $filename) {
+                                    return $information;
+                }
             }
         }
 
@@ -409,8 +415,9 @@ 
     function getDirectory($dirname) {
         if($this->numDirectories > 0) {
             foreach($this->directories as $key => $information) {
-                if($information["name"] == $dirname)
-                    return $information;
+                if($information["name"] == $dirname) {
+                                    return $information;
+                }
             }
         }
 
@@ -422,8 +429,9 @@ 
     function containsFile($filename) {
         if($this->numFiles > 0) {
             foreach($this->files as $key => $information) {
-                if($information["name"] == $filename)
-                    return true;
+                if($information["name"] == $filename) {
+                                    return true;
+                }
             }
         }
 
@@ -435,8 +443,9 @@ 
     function containsDirectory($dirname) {
         if($this->numDirectories > 0) {
             foreach($this->directories as $key => $information) {
-                if($information["name"] == $dirname)
-                    return true;
+                if($information["name"] == $dirname) {
+                                    return true;
+                }
             }
         }
 
@@ -446,8 +455,9 @@ 
 
     // Add a directory to this tar archive
     function addDirectory($dirname) {
-        if(!file_exists($dirname))
-            return false;
+        if(!file_exists($dirname)) {
+                    return false;
+        }
 
         // Get directory information
         $file_information = stat($dirname);
@@ -469,15 +479,18 @@ 
     // Add a file to the tar archive
     function addFile($filename,$from=null,$to=null) {
         // Make sure the file we are adding exists!
-        if(!file_exists($filename))
-            return false;
+        if(!file_exists($filename)) {
+                    return false;
+        }
 
-        if(filesize($filename)==0)
-            return false;
+        if(filesize($filename)==0) {
+                    return false;
+        }
 
         // Make sure there are no other files in the archive that have this same filename
-        if($this->containsFile($filename))
-            return false;
+        if($this->containsFile($filename)) {
+                    return false;
+        }
 
         // Get file information
         $file_information = stat($filename);
@@ -544,8 +557,9 @@ 
 
     // Write the currently loaded tar archive to disk
     function saveTar() {
-        if(!$this->filename)
-            return false;
+        if(!$this->filename) {
+                    return false;
+        }
 
         // Write tar to current file using specified gzip compression
         $this->toTar($this->filename,$this->isGzipped);
@@ -556,8 +570,9 @@ 
 
     // Saves tar archive to a different file than the current file
     function toTar($filename,$useGzip) {
-        if(!$filename)
-            return false;
+        if(!$filename) {
+                    return false;
+        }
 
         // Encode processed files into TAR file format
         $this->__generateTar();
@@ -565,8 +580,9 @@ 
         // GZ Compress the data if we need to
         if($useGzip) {
             // Make sure we have gzip support
-            if(!function_exists("gzencode"))
-                return false;
+            if(!function_exists("gzencode")) {
+                            return false;
+            }
 
             $file = gzencode($this->tar_file);
         } else {

Spacing +120 added lines, -120 removed lines

@@ -85,9 +85,9 @@ 
     // to try to ensure valid file
     // PRIVATE ACCESS FUNCTION
     function __computeUnsignedChecksum($bytestring) {
-        for($i=0; $i<512; $i++)
+        for ($i = 0; $i < 512; $i++)
             $unsigned_chksum += ord($bytestring[$i]);
-        for($i=0; $i<8; $i++)
+        for ($i = 0; $i < 8; $i++)
             $unsigned_chksum -= ord($bytestring[148 + $i]);
         $unsigned_chksum += ord(" ") * 8;
 
@@ -98,12 +98,12 @@ 
     // Converts a NULL padded string to a non-NULL padded string
     // PRIVATE ACCESS FUNCTION
     function __parseNullPaddedString($string) {
-        $position = strpos($string,chr(0));
-		if(!$position)
+        $position = strpos($string, chr(0));
+		if (!$position)
 		{
 			$position = strlen($string);
 		}
-        return substr($string,0,$position);
+        return substr($string, 0, $position);
     }
 
 
@@ -114,46 +114,46 @@ 
         $tar_length = strlen($this->tar_file);
         $main_offset = 0;
 		$flag_longlink = false;
-        while($main_offset < $tar_length) {
+        while ($main_offset < $tar_length) {
             // If we read a block of 512 nulls, we are at the end of the archive
-            if(substr($this->tar_file,$main_offset,512) == str_repeat(chr(0),512))
+            if (substr($this->tar_file, $main_offset, 512) == str_repeat(chr(0), 512))
                 break;
 
             // Parse file name
-            $file_name      = $this->__parseNullPaddedString(substr($this->tar_file,$main_offset,100));
+            $file_name      = $this->__parseNullPaddedString(substr($this->tar_file, $main_offset, 100));
 
             // Parse the file mode
-            $file_mode      = substr($this->tar_file,$main_offset + 100,8);
+            $file_mode      = substr($this->tar_file, $main_offset + 100, 8);
 
             // Parse the file user ID
-            $file_uid       = octdec(substr($this->tar_file,$main_offset + 108,8));
+            $file_uid       = octdec(substr($this->tar_file, $main_offset + 108, 8));
 
             // Parse the file group ID
-            $file_gid       = octdec(substr($this->tar_file,$main_offset + 116,8));
+            $file_gid       = octdec(substr($this->tar_file, $main_offset + 116, 8));
 
             // Parse the file size
-            $file_size      = octdec(substr($this->tar_file,$main_offset + 124,12));
+            $file_size      = octdec(substr($this->tar_file, $main_offset + 124, 12));
 
             // Parse the file update time - unix timestamp format
-            $file_time      = octdec(substr($this->tar_file,$main_offset + 136,12));
+            $file_time      = octdec(substr($this->tar_file, $main_offset + 136, 12));
 
             // Parse Checksum
-            $file_chksum        = octdec(substr($this->tar_file,$main_offset + 148,6));
+            $file_chksum = octdec(substr($this->tar_file, $main_offset + 148, 6));
 
             // Parse user name
-            $file_uname     = $this->__parseNullPaddedString(substr($this->tar_file,$main_offset + 265,32));
+            $file_uname     = $this->__parseNullPaddedString(substr($this->tar_file, $main_offset + 265, 32));
 
             // Parse Group name
-            $file_gname     = $this->__parseNullPaddedString(substr($this->tar_file,$main_offset + 297,32));
+            $file_gname     = $this->__parseNullPaddedString(substr($this->tar_file, $main_offset + 297, 32));
 
-            $file_type = substr($this->tar_file,$main_offset + 156,1);
+            $file_type = substr($this->tar_file, $main_offset + 156, 1);
 
             // Make sure our file is valid
-            if($this->__computeUnsignedChecksum(substr($this->tar_file,$main_offset,512)) != $file_chksum)
+            if ($this->__computeUnsignedChecksum(substr($this->tar_file, $main_offset, 512)) != $file_chksum)
                 return false;
 
             // Parse File Contents
-            $file_contents      = substr($this->tar_file,$main_offset + 512,$file_size);
+            $file_contents = substr($this->tar_file, $main_offset + 512, $file_size);
 
             /*  ### Unused Header Information ###
                 $activeFile["typeflag"]     = substr($this->tar_file,$main_offset + 156,1);
@@ -166,12 +166,12 @@ 
                 $activeFile["endheader"]    = substr($this->tar_file,$main_offset + 500,12);
             */
 
-			if(strtolower($file_type) == 'l' || $file_name == '././@LongLink')
+			if (strtolower($file_type) == 'l' || $file_name == '././@LongLink')
 			{
 				$flag_longlink = true;
 				$longlink_name = $this->__parseNullPaddedString($file_contents);
 			}
-			elseif($file_type == '0') {
+			elseif ($file_type == '0') {
 				// Increment number of files
 				$this->numFiles++;
 
@@ -179,13 +179,13 @@ 
 				$activeFile = &$this->files[];
 
 				// Asign Values
-				if($flag_longlink)
+				if ($flag_longlink)
 				{
-					$activeFile["name"]     = $longlink_name;
+					$activeFile["name"] = $longlink_name;
 				}
 				else
 				{
-					$activeFile["name"]     = $file_name;
+					$activeFile["name"] = $file_name;
 				}
 				$activeFile["type"]     = $file_type;
 				$activeFile["mode"]     = $file_mode;
@@ -196,11 +196,11 @@ 
 				$activeFile["user_name"]    = $file_uname;
 				$activeFile["group_name"]   = $file_gname;
 				$activeFile["checksum"]     = $file_chksum;
-				$activeFile["file"]     = $file_contents;
+				$activeFile["file"] = $file_contents;
 
 				$flag_longlink = false;
 
-			} elseif($file_type == '5') {
+			} elseif ($file_type == '5') {
 				// Increment number of directories
 				$this->numDirectories++;
 
@@ -208,9 +208,9 @@ 
 				$activeDir = &$this->directories[];
 
 				// Assign values
-				if($flag_longlink)
+				if ($flag_longlink)
 				{
-					$activeDir["name"]     = $longlink_name;
+					$activeDir["name"] = $longlink_name;
 				}
 				else
 				{
@@ -238,23 +238,23 @@ 
 
     // Read a non gzipped tar file in for processing
     // PRIVATE ACCESS FUNCTION
-    function __readTar($filename='') {
+    function __readTar($filename = '') {
         // Set the filename to load
-        if(!$filename)
+        if (!$filename)
             $filename = $this->filename;
 
         // Read in the TAR file
-        $fp = fopen($filename,"rb");
-        $this->tar_file = fread($fp,filesize($filename));
+        $fp = fopen($filename, "rb");
+        $this->tar_file = fread($fp, filesize($filename));
         fclose($fp);
 
-        if($this->tar_file[0] == chr(31) && $this->tar_file[1] == chr(139) && $this->tar_file[2] == chr(8)) {
-            if(!function_exists("gzinflate"))
+        if ($this->tar_file[0] == chr(31) && $this->tar_file[1] == chr(139) && $this->tar_file[2] == chr(8)) {
+            if (!function_exists("gzinflate"))
                 return false;
 
             $this->isGzipped = TRUE;
 
-            $this->tar_file = gzinflate(substr($this->tar_file,10,-4));
+            $this->tar_file = gzinflate(substr($this->tar_file, 10, -4));
         }
 
         // Parse the TAR file
@@ -271,34 +271,34 @@ 
         unset($this->tar_file);
 
         // Generate Records for each directory, if we have directories
-        if($this->numDirectories > 0) {
-            foreach($this->directories as $key => $information) {
+        if ($this->numDirectories > 0) {
+            foreach ($this->directories as $key => $information) {
                 unset($header);
 
                 // Generate tar header for this directory
                 // Filename, Permissions, UID, GID, size, Time, checksum, typeflag, linkname, magic, version, user name, group name, devmajor, devminor, prefix, end
-                $header .= str_pad($information["name"],100,chr(0));
-                $header .= str_pad(decoct($information["mode"]),7,"0",STR_PAD_LEFT) . chr(0);
-                $header .= str_pad(decoct($information["user_id"]),7,"0",STR_PAD_LEFT) . chr(0);
-                $header .= str_pad(decoct($information["group_id"]),7,"0",STR_PAD_LEFT) . chr(0);
-                $header .= str_pad(decoct(0),11,"0",STR_PAD_LEFT) . chr(0);
-                $header .= str_pad(decoct($information["time"]),11,"0",STR_PAD_LEFT) . chr(0);
-                $header .= str_repeat(" ",8);
+                $header .= str_pad($information["name"], 100, chr(0));
+                $header .= str_pad(decoct($information["mode"]), 7, "0", STR_PAD_LEFT).chr(0);
+                $header .= str_pad(decoct($information["user_id"]), 7, "0", STR_PAD_LEFT).chr(0);
+                $header .= str_pad(decoct($information["group_id"]), 7, "0", STR_PAD_LEFT).chr(0);
+                $header .= str_pad(decoct(0), 11, "0", STR_PAD_LEFT).chr(0);
+                $header .= str_pad(decoct($information["time"]), 11, "0", STR_PAD_LEFT).chr(0);
+                $header .= str_repeat(" ", 8);
                 $header .= "5";
-                $header .= str_repeat(chr(0),100);
-                $header .= str_pad("ustar",6,chr(32));
-                $header .= chr(32) . chr(0);
-                $header .= str_pad("",32,chr(0));
-                $header .= str_pad("",32,chr(0));
-                $header .= str_repeat(chr(0),8);
-                $header .= str_repeat(chr(0),8);
-                $header .= str_repeat(chr(0),155);
-                $header .= str_repeat(chr(0),12);
+                $header .= str_repeat(chr(0), 100);
+                $header .= str_pad("ustar", 6, chr(32));
+                $header .= chr(32).chr(0);
+                $header .= str_pad("", 32, chr(0));
+                $header .= str_pad("", 32, chr(0));
+                $header .= str_repeat(chr(0), 8);
+                $header .= str_repeat(chr(0), 8);
+                $header .= str_repeat(chr(0), 155);
+                $header .= str_repeat(chr(0), 12);
 
                 // Compute header checksum
-                $checksum = str_pad(decoct($this->__computeUnsignedChecksum($header)),6,"0",STR_PAD_LEFT);
-                for($i=0; $i<6; $i++) {
-                    $header[(148 + $i)] = substr($checksum,$i,1);
+                $checksum = str_pad(decoct($this->__computeUnsignedChecksum($header)), 6, "0", STR_PAD_LEFT);
+                for ($i = 0; $i < 6; $i++) {
+                    $header[(148 + $i)] = substr($checksum, $i, 1);
                 }
                 $header[154] = chr(0);
                 $header[155] = chr(32);
@@ -309,48 +309,48 @@ 
         }
 
         // Generate Records for each file, if we have files (We should...)
-        if($this->numFiles > 0) {
-            foreach($this->files as $key => $information) {
+        if ($this->numFiles > 0) {
+            foreach ($this->files as $key => $information) {
                 unset($header);
 
                 // Generate the TAR header for this file
                 // Filename, Permissions, UID, GID, size, Time, checksum, typeflag, linkname, magic, version, user name, group name, devmajor, devminor, prefix, end
-                $header .= str_pad($information["name"],100,chr(0));
-                $header .= str_pad(decoct($information["mode"]),7,"0",STR_PAD_LEFT) . chr(0);
-                $header .= str_pad(decoct($information["user_id"]),7,"0",STR_PAD_LEFT) . chr(0);
-                $header .= str_pad(decoct($information["group_id"]),7,"0",STR_PAD_LEFT) . chr(0);
-                $header .= str_pad(decoct($information["size"]),11,"0",STR_PAD_LEFT) . chr(0);
-                $header .= str_pad(decoct($information["time"]),11,"0",STR_PAD_LEFT) . chr(0);
-                $header .= str_repeat(" ",8);
+                $header .= str_pad($information["name"], 100, chr(0));
+                $header .= str_pad(decoct($information["mode"]), 7, "0", STR_PAD_LEFT).chr(0);
+                $header .= str_pad(decoct($information["user_id"]), 7, "0", STR_PAD_LEFT).chr(0);
+                $header .= str_pad(decoct($information["group_id"]), 7, "0", STR_PAD_LEFT).chr(0);
+                $header .= str_pad(decoct($information["size"]), 11, "0", STR_PAD_LEFT).chr(0);
+                $header .= str_pad(decoct($information["time"]), 11, "0", STR_PAD_LEFT).chr(0);
+                $header .= str_repeat(" ", 8);
                 $header .= "0";
-                $header .= str_repeat(chr(0),100);
-                $header .= str_pad("ustar",6,chr(32));
-                $header .= chr(32) . chr(0);
-                $header .= str_pad($information["user_name"],32,chr(0));    // How do I get a file's user name from PHP?
-                $header .= str_pad($information["group_name"],32,chr(0));   // How do I get a file's group name from PHP?
-                $header .= str_repeat(chr(0),8);
-                $header .= str_repeat(chr(0),8);
-                $header .= str_repeat(chr(0),155);
-                $header .= str_repeat(chr(0),12);
+                $header .= str_repeat(chr(0), 100);
+                $header .= str_pad("ustar", 6, chr(32));
+                $header .= chr(32).chr(0);
+                $header .= str_pad($information["user_name"], 32, chr(0)); // How do I get a file's user name from PHP?
+                $header .= str_pad($information["group_name"], 32, chr(0)); // How do I get a file's group name from PHP?
+                $header .= str_repeat(chr(0), 8);
+                $header .= str_repeat(chr(0), 8);
+                $header .= str_repeat(chr(0), 155);
+                $header .= str_repeat(chr(0), 12);
 
                 // Compute header checksum
-                $checksum = str_pad(decoct($this->__computeUnsignedChecksum($header)),6,"0",STR_PAD_LEFT);
-                for($i=0; $i<6; $i++) {
-                    $header[(148 + $i)] = substr($checksum,$i,1);
+                $checksum = str_pad(decoct($this->__computeUnsignedChecksum($header)), 6, "0", STR_PAD_LEFT);
+                for ($i = 0; $i < 6; $i++) {
+                    $header[(148 + $i)] = substr($checksum, $i, 1);
                 }
                 $header[154] = chr(0);
                 $header[155] = chr(32);
 
                 // Pad file contents to byte count divisible by 512
-                $file_contents = str_pad($information["file"],(ceil($information["size"] / 512) * 512),chr(0));
+                $file_contents = str_pad($information["file"], (ceil($information["size"] / 512) * 512), chr(0));
 
                 // Add new tar formatted data to tar file contents
-                $this->tar_file .= $header . $file_contents;
+                $this->tar_file .= $header.$file_contents;
             }
         }
 
         // Add 512 bytes of NULLs to designate EOF
-        $this->tar_file .= str_repeat(chr(0),512);
+        $this->tar_file .= str_repeat(chr(0), 512);
 
         return true;
     }
@@ -368,7 +368,7 @@ 
         unset($this->numDirectories);
 
         // If the tar file doesn't exist...
-        if(!file_exists($filename))
+        if (!file_exists($filename))
             return false;
 
         $this->filename = $filename;
@@ -383,7 +383,7 @@ 
     // Appends a tar file to the end of the currently opened tar file
     function appendTar($filename) {
         // If the tar file doesn't exist...
-        if(!file_exists($filename))
+        if (!file_exists($filename))
             return false;
 
         $this->__readTar($filename);
@@ -394,9 +394,9 @@ 
 
     // Retrieves information about a file in the current tar archive
     function getFile($filename) {
-        if($this->numFiles > 0) {
-            foreach($this->files as $key => $information) {
-                if($information["name"] == $filename)
+        if ($this->numFiles > 0) {
+            foreach ($this->files as $key => $information) {
+                if ($information["name"] == $filename)
                     return $information;
             }
         }
@@ -407,9 +407,9 @@ 
 
     // Retrieves information about a directory in the current tar archive
     function getDirectory($dirname) {
-        if($this->numDirectories > 0) {
-            foreach($this->directories as $key => $information) {
-                if($information["name"] == $dirname)
+        if ($this->numDirectories > 0) {
+            foreach ($this->directories as $key => $information) {
+                if ($information["name"] == $dirname)
                     return $information;
             }
         }
@@ -420,9 +420,9 @@ 
 
     // Check if this tar archive contains a specific file
     function containsFile($filename) {
-        if($this->numFiles > 0) {
-            foreach($this->files as $key => $information) {
-                if($information["name"] == $filename)
+        if ($this->numFiles > 0) {
+            foreach ($this->files as $key => $information) {
+                if ($information["name"] == $filename)
                     return true;
             }
         }
@@ -433,9 +433,9 @@ 
 
     // Check if this tar archive contains a specific directory
     function containsDirectory($dirname) {
-        if($this->numDirectories > 0) {
-            foreach($this->directories as $key => $information) {
-                if($information["name"] == $dirname)
+        if ($this->numDirectories > 0) {
+            foreach ($this->directories as $key => $information) {
+                if ($information["name"] == $dirname)
                     return true;
             }
         }
@@ -446,7 +446,7 @@ 
 
     // Add a directory to this tar archive
     function addDirectory($dirname) {
-        if(!file_exists($dirname))
+        if (!file_exists($dirname))
             return false;
 
         // Get directory information
@@ -454,7 +454,7 @@ 
 
         // Add directory to processed data
         $this->numDirectories++;
-        $activeDir      = &$this->directories[];
+        $activeDir = &$this->directories[];
         $activeDir["name"]  = $dirname;
         $activeDir["mode"]  = $file_information["mode"];
         $activeDir["time"]  = $file_information["time"];
@@ -467,34 +467,34 @@ 
 
 
     // Add a file to the tar archive
-    function addFile($filename,$from=null,$to=null) {
+    function addFile($filename, $from = null, $to = null) {
         // Make sure the file we are adding exists!
-        if(!file_exists($filename))
+        if (!file_exists($filename))
             return false;
 
-        if(filesize($filename)==0)
+        if (filesize($filename) == 0)
             return false;
 
         // Make sure there are no other files in the archive that have this same filename
-        if($this->containsFile($filename))
+        if ($this->containsFile($filename))
             return false;
 
         // Get file information
         $file_information = stat($filename);
 
         // Read in the file's contents
-        $fp = fopen($filename,"rb");
-        $file_contents = fread($fp,filesize($filename));
+        $fp = fopen($filename, "rb");
+        $file_contents = fread($fp, filesize($filename));
         fclose($fp);
 
-        if($from && $to){
-            $file_contents = str_replace($from,$to,$file_contents);
+        if ($from && $to) {
+            $file_contents = str_replace($from, $to, $file_contents);
             $file_information["size"] = strlen($file_contents);
         }
 
         // Add file to processed data
         $this->numFiles++;
-        $activeFile         = &$this->files[];
+        $activeFile = &$this->files[];
         $activeFile["name"]     = $filename;
         $activeFile["mode"]     = $file_information["mode"];
         $activeFile["user_id"]      = $file_information["uid"];
@@ -504,7 +504,7 @@ 
         $activeFile["checksum"]     = $checksum;
         $activeFile["user_name"]    = "";
         $activeFile["group_name"]   = "";
-        $activeFile["file"]     = $file_contents;
+        $activeFile["file"] = $file_contents;
 
         return true;
     }
@@ -512,9 +512,9 @@ 
 
     // Remove a file from the tar archive
     function removeFile($filename) {
-        if($this->numFiles > 0) {
-            foreach($this->files as $key => $information) {
-                if($information["name"] == $filename) {
+        if ($this->numFiles > 0) {
+            foreach ($this->files as $key => $information) {
+                if ($information["name"] == $filename) {
                     $this->numFiles--;
                     unset($this->files[$key]);
                     return true;
@@ -528,9 +528,9 @@ 
 
     // Remove a directory from the tar archive
     function removeDirectory($dirname) {
-        if($this->numDirectories > 0) {
-            foreach($this->directories as $key => $information) {
-                if($information["name"] == $dirname) {
+        if ($this->numDirectories > 0) {
+            foreach ($this->directories as $key => $information) {
+                if ($information["name"] == $dirname) {
                     $this->numDirectories--;
                     unset($this->directories[$key]);
                     return true;
@@ -544,28 +544,28 @@ 
 
     // Write the currently loaded tar archive to disk
     function saveTar() {
-        if(!$this->filename)
+        if (!$this->filename)
             return false;
 
         // Write tar to current file using specified gzip compression
-        $this->toTar($this->filename,$this->isGzipped);
+        $this->toTar($this->filename, $this->isGzipped);
 
         return true;
     }
 
 
     // Saves tar archive to a different file than the current file
-    function toTar($filename,$useGzip) {
-        if(!$filename)
+    function toTar($filename, $useGzip) {
+        if (!$filename)
             return false;
 
         // Encode processed files into TAR file format
         $this->__generateTar();
 
         // GZ Compress the data if we need to
-        if($useGzip) {
+        if ($useGzip) {
             // Make sure we have gzip support
-            if(!function_exists("gzencode"))
+            if (!function_exists("gzencode"))
                 return false;
 
             $file = gzencode($this->tar_file);
@@ -574,8 +574,8 @@ 
         }
 
         // Write the TAR file
-        $fp = fopen($filename,"wb");
-        fwrite($fp,$file);
+        $fp = fopen($filename, "wb");
+        fwrite($fp, $file);
         fclose($fp);
 
         return true;

modules/addon/addon.admin.model.php

Doc Comments +1 added lines, -1 removed lines

@@ -141,7 +141,7 @@
 	 * @param string $addon Name to get information
 	 * @param int $site_srl Site srl
 	 * @param string $gtype site or global
-	 * @return object Returns a information
+	 * @return null|stdClass Returns a information
 	 */
 	function getAddonInfoXml($addon, $site_srl = 0, $gtype = 'site')
 	{

Spacing +53 added lines, -53 removed lines

@@ -27,7 +27,7 @@ 
 	function getAddonPath($addon_name)
 	{
 		$class_path = sprintf('./addons/%s/', $addon_name);
-		if(is_dir($class_path))
+		if (is_dir($class_path))
 		{
 			return $class_path;
 		}
@@ -44,7 +44,7 @@ 
 		$addonList = $this->getAddonList(0, 'site');
 
 		$oAutoinstallModel = getModel('autoinstall');
-		foreach($addonList as $key => $addon)
+		foreach ($addonList as $key => $addon)
 		{
 			// get easyinstall remove url
 			$packageSrl = $oAutoinstallModel->getPackageSrlByPath($addon->path);
@@ -55,7 +55,7 @@ 
 			$addonList[$key]->need_update = $package[$packageSrl]->need_update;
 
 			// get easyinstall update url
-			if($addonList[$key]->need_update == 'Y')
+			if ($addonList[$key]->need_update == 'Y')
 			{
 				$addonList[$key]->update_url = $oAutoinstallModel->getUpdateUrlByPackageSrl($packageSrl);
 			}
@@ -78,7 +78,7 @@ 
 		// Downloaded and installed add-on to the list of Wanted
 		$searched_list = FileHandler::readDir('./addons', '/^([a-zA-Z0-9-_]+)$/');
 		$searched_count = count($searched_list);
-		if(!$searched_count)
+		if (!$searched_count)
 		{
 			return;
 		}
@@ -87,11 +87,11 @@ 
 
 		$oAddonAdminController = getAdminController('addon');
 
-		for($i = 0; $i < $searched_count; $i++)
+		for ($i = 0; $i < $searched_count; $i++)
 		{
 			// Add the name of
 			$addon_name = $searched_list[$i];
-			if($addon_name == "smartphone")
+			if ($addon_name == "smartphone")
 			{
 				continue;
 			}
@@ -100,7 +100,7 @@ 
 			// Wanted information on the add-on
 			$info = $this->getAddonInfoXml($addon_name, $site_srl, $gtype);
 			
-			if(!$info) $info = new stdClass();
+			if (!$info) $info = new stdClass();
 
 			$info->addon = $addon_name;
 			$info->path = $path;
@@ -108,7 +108,7 @@ 
 			$info->mactivated = FALSE;
 			$info->fixed = FALSE;
 			// Check if a permossion is granted entered in DB
-			if(!in_array($addon_name, array_keys($inserted_addons)))
+			if (!in_array($addon_name, array_keys($inserted_addons)))
 			{
 				// If not, type in the DB type (model, perhaps because of the hate doing this haneungeo .. ㅡ. ㅜ)
 				$oAddonAdminController->doInsert($addon_name, $site_srl, $type);
@@ -116,15 +116,15 @@ 
 			}
 			else
 			{
-				if($inserted_addons[$addon_name]->is_used == 'Y')
+				if ($inserted_addons[$addon_name]->is_used == 'Y')
 				{
 					$info->activated = TRUE;
 				}
-				if($inserted_addons[$addon_name]->is_used_m == 'Y')
+				if ($inserted_addons[$addon_name]->is_used_m == 'Y')
 				{
 					$info->mactivated = TRUE;
 				}
-				if($gtype == 'global' && $inserted_addons[$addon_name]->is_fixed == 'Y')
+				if ($gtype == 'global' && $inserted_addons[$addon_name]->is_fixed == 'Y')
 				{
 					$info->fixed = TRUE;
 				}
@@ -147,14 +147,14 @@ 
 	{
 		// Get a path of the requested module. Return if not exists.
 		$addon_path = $this->getAddonPath($addon);
-		if(!$addon_path)
+		if (!$addon_path)
 		{
 			return;
 		}
 
 		// Read the xml file for module skin information
 		$xml_file = sprintf("%sconf/info.xml", FileHandler::getRealpath($addon_path));
-		if(!file_exists($xml_file))
+		if (!file_exists($xml_file))
 		{
 			return;
 		}
@@ -163,7 +163,7 @@ 
 		$tmp_xml_obj = $oXmlParser->loadXmlFile($xml_file);
 		$xml_obj = $tmp_xml_obj->addon;
 
-		if(!$xml_obj)
+		if (!$xml_obj)
 		{
 			return;
 		}
@@ -171,7 +171,7 @@ 
 		// DB is set to bring history
 		$db_args = new stdClass();
 		$db_args->addon = $addon;
-		if($gtype == 'global')
+		if ($gtype == 'global')
 		{
 			$output = executeQuery('addon.getAddonInfo', $db_args);
 		}
@@ -183,7 +183,7 @@ 
 		$extra_vals = unserialize($output->data->extra_vars);
 
 		$addon_info = new stdClass();
-		if($extra_vals->mid_list)
+		if ($extra_vals->mid_list)
 		{
 			$addon_info->mid_list = $extra_vals->mid_list;
 		}
@@ -192,13 +192,13 @@ 
 			$addon_info->mid_list = array();
 		}
 
-		if($extra_vals->xe_run_method)
+		if ($extra_vals->xe_run_method)
 		{
 			$addon_info->xe_run_method = $extra_vals->xe_run_method;
 		}
 
 		// Add information
-		if($xml_obj->version && $xml_obj->attrs->version == '0.2')
+		if ($xml_obj->version && $xml_obj->attrs->version == '0.2')
 		{
 			// addon format v0.2
 			$date_obj = new stdClass();
@@ -213,7 +213,7 @@ 
 			$addon_info->license = $xml_obj->license->body;
 			$addon_info->license_link = $xml_obj->license->attrs->link;
 
-			if(!is_array($xml_obj->author))
+			if (!is_array($xml_obj->author))
 			{
 				$author_list = array();
 				$author_list[] = $xml_obj->author;
@@ -224,7 +224,7 @@ 
 			}
 
 			$addon_info->author = array();
-			foreach($author_list as $author)
+			foreach ($author_list as $author)
 			{
 				$author_obj = new stdClass();
 				$author_obj->name = $author->name->body;
@@ -234,39 +234,39 @@ 
 			}
 
 			// Expand the variable order
-			if($xml_obj->extra_vars)
+			if ($xml_obj->extra_vars)
 			{
 				$extra_var_groups = $xml_obj->extra_vars->group;
-				if(!$extra_var_groups)
+				if (!$extra_var_groups)
 				{
 					$extra_var_groups = $xml_obj->extra_vars;
 				}
-				if(!is_array($extra_var_groups))
+				if (!is_array($extra_var_groups))
 				{
 					$extra_var_groups = array($extra_var_groups);
 				}
 
-				foreach($extra_var_groups as $group)
+				foreach ($extra_var_groups as $group)
 				{
 					$extra_vars = $group->var;
-					if(!is_array($group->var))
+					if (!is_array($group->var))
 					{
 						$extra_vars = array($group->var);
 					}
 
-					foreach($extra_vars as $key => $val)
+					foreach ($extra_vars as $key => $val)
 					{
-						if(!$val)
+						if (!$val)
 						{
 							continue;
 						}
 
 						$obj = new stdClass();
-						if(!$val->attrs)
+						if (!$val->attrs)
 						{
 							$val->attrs = new stdClass();
 						}
-						if(!$val->attrs->type)
+						if (!$val->attrs->type)
 						{
 							$val->attrs->type = 'text';
 						}
@@ -276,26 +276,26 @@ 
 						$obj->title = $val->title->body;
 						$obj->type = $val->attrs->type;
 						$obj->description = $val->description->body;
-						if($obj->name)
+						if ($obj->name)
 						{
 							$obj->value = $extra_vals->{$obj->name};
 						}
-						if(strpos($obj->value, '|@|') != FALSE)
+						if (strpos($obj->value, '|@|') != FALSE)
 						{
 							$obj->value = explode('|@|', $obj->value);
 						}
-						if($obj->type == 'mid_list' && !is_array($obj->value))
+						if ($obj->type == 'mid_list' && !is_array($obj->value))
 						{
 							$obj->value = array($obj->value);
 						}
 
 						// 'Select'type obtained from the option list.
-						if($val->options && !is_array($val->options))
+						if ($val->options && !is_array($val->options))
 						{
 							$val->options = array($val->options);
 						}
 
-						for($i = 0, $c = count($val->options); $i < $c; $i++)
+						for ($i = 0, $c = count($val->options); $i < $c; $i++)
 						{
 							$obj->options[$i] = new stdClass();
 							$obj->options[$i]->title = $val->options[$i]->title->body;
@@ -328,30 +328,30 @@ 
 			$addon_info->author = array();
 			$addon_info->author[] = $author_obj;
 
-			if($xml_obj->extra_vars)
+			if ($xml_obj->extra_vars)
 			{
 				// Expand the variable order
 				$extra_var_groups = $xml_obj->extra_vars->group;
-				if(!$extra_var_groups)
+				if (!$extra_var_groups)
 				{
 					$extra_var_groups = $xml_obj->extra_vars;
 				}
-				if(!is_array($extra_var_groups))
+				if (!is_array($extra_var_groups))
 				{
 					$extra_var_groups = array($extra_var_groups);
 				}
-				foreach($extra_var_groups as $group)
+				foreach ($extra_var_groups as $group)
 				{
 					$extra_vars = $group->var;
-					if(!is_array($group->var))
+					if (!is_array($group->var))
 					{
 						$extra_vars = array($group->var);
 					}
 
 					$addon_info->extra_vars = array();
-					foreach($extra_vars as $key => $val)
+					foreach ($extra_vars as $key => $val)
 					{
-						if(!$val)
+						if (!$val)
 						{
 							continue;
 						}
@@ -363,26 +363,26 @@ 
 						$obj->title = $val->title->body;
 						$obj->type = $val->type->body ? $val->type->body : 'text';
 						$obj->description = $val->description->body;
-						if($obj->name)
+						if ($obj->name)
 						{
 							$obj->value = $extra_vals->{$obj->name};
 						}
-						if(strpos($obj->value, '|@|') != false)
+						if (strpos($obj->value, '|@|') != false)
 						{
 							$obj->value = explode('|@|', $obj->value);
 						}
-						if($obj->type == 'mid_list' && !is_array($obj->value))
+						if ($obj->type == 'mid_list' && !is_array($obj->value))
 						{
 							$obj->value = array($obj->value);
 						}
 						// 'Select'type obtained from the option list.
-						if($val->options && !is_array($val->options))
+						if ($val->options && !is_array($val->options))
 						{
 							$val->options = array($val->options);
 						}
 
 						$obj->options = array();
-						for($i = 0, $c = count($val->options); $i < $c; $i++)
+						for ($i = 0, $c = count($val->options); $i < $c; $i++)
 						{
 							$obj->options[$i]->title = $val->options[$i]->title->body;
 							$obj->options[$i]->value = $val->options[$i]->value->body;
@@ -406,7 +406,7 @@ 
 	{
 		$args = new stdClass();
 		$args->list_order = 'addon';
-		if($gtype == 'global')
+		if ($gtype == 'global')
 		{
 			$output = executeQueryArray('addon.getAddons', $args);
 		}
@@ -415,14 +415,14 @@ 
 			$args->site_srl = $site_srl;
 			$output = executeQueryArray('addon.getSiteAddons', $args);
 		}
-		if(!$output->data)
+		if (!$output->data)
 		{
 			return array();
 		}
 
 		$activated_count = count($output->data);
 		$addon_list = array();
-		for($i = 0; $i < $activated_count; $i++)
+		for ($i = 0; $i < $activated_count; $i++)
 		{
 			$addon = $output->data[$i];
 			$addon_list[$addon->addon] = $addon;
@@ -443,9 +443,9 @@ 
 	{
 		$args = new stdClass();
 		$args->addon = $addon;
-		if($gtype == 'global')
+		if ($gtype == 'global')
 		{
-			if($type == "pc")
+			if ($type == "pc")
 			{
 				$output = executeQuery('addon.getAddonIsActivated', $args);
 			}
@@ -457,7 +457,7 @@ 
 		else
 		{
 			$args->site_srl = $site_srl;
-			if($type == "pc")
+			if ($type == "pc")
 			{
 				$output = executeQuery('addon.getSiteAddonIsActivated', $args);
 			}
@@ -466,7 +466,7 @@ 
 				$output = executeQuery('addon.getSiteMAddonIsActivated', $args);
 			}
 		}
-		if($output->data->count > 0)
+		if ($output->data->count > 0)
 		{
 			return TRUE;
 		}

Braces +12 added lines, -19 removed lines

@@ -100,7 +100,9 @@ 
 			// Wanted information on the add-on
 			$info = $this->getAddonInfoXml($addon_name, $site_srl, $gtype);
 			
-			if(!$info) $info = new stdClass();
+			if(!$info) {
+				$info = new stdClass();
+			}
 
 			$info->addon = $addon_name;
 			$info->path = $path;
@@ -113,8 +115,7 @@ 
 				// If not, type in the DB type (model, perhaps because of the hate doing this haneungeo .. ㅡ. ㅜ)
 				$oAddonAdminController->doInsert($addon_name, $site_srl, $type);
 				// Is activated
-			}
-			else
+			} else
 			{
 				if($inserted_addons[$addon_name]->is_used == 'Y')
 				{
@@ -174,8 +175,7 @@ 
 		if($gtype == 'global')
 		{
 			$output = executeQuery('addon.getAddonInfo', $db_args);
-		}
-		else
+		} else
 		{
 			$db_args->site_srl = $site_srl;
 			$output = executeQuery('addon.getSiteAddonInfo', $db_args);
@@ -186,8 +186,7 @@ 
 		if($extra_vals->mid_list)
 		{
 			$addon_info->mid_list = $extra_vals->mid_list;
-		}
-		else
+		} else
 		{
 			$addon_info->mid_list = array();
 		}
@@ -217,8 +216,7 @@ 
 			{
 				$author_list = array();
 				$author_list[] = $xml_obj->author;
-			}
-			else
+			} else
 			{
 				$author_list = $xml_obj->author;
 			}
@@ -306,8 +304,7 @@ 
 					}
 				}
 			}
-		}
-		else
+		} else
 		{
 			// addon format 0.1
 			$addon_info = new stdClass();
@@ -409,8 +406,7 @@ 
 		if($gtype == 'global')
 		{
 			$output = executeQueryArray('addon.getAddons', $args);
-		}
-		else
+		} else
 		{
 			$args->site_srl = $site_srl;
 			$output = executeQueryArray('addon.getSiteAddons', $args);
@@ -448,20 +444,17 @@ 
 			if($type == "pc")
 			{
 				$output = executeQuery('addon.getAddonIsActivated', $args);
-			}
-			else
+			} else
 			{
 				$output = executeQuery('addon.getMAddonIsActivated', $args);
 			}
-		}
-		else
+		} else
 		{
 			$args->site_srl = $site_srl;
 			if($type == "pc")
 			{
 				$output = executeQuery('addon.getSiteAddonIsActivated', $args);
-			}
-			else
+			} else
 			{
 				$output = executeQuery('addon.getSiteMAddonIsActivated', $args);
 			}

modules/admin/admin.admin.view.php

Doc Comments +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
 
 	/**
 	 * Initilization
-	 * @return void
+	 * @return ModuleObject|null
 	 */
 	function init()
 	{

Indentation +1 added lines, -1 removed lines

@@ -566,7 +566,7 @@
 	/**
 	 * Retrun server environment to XML string
 	 * @return object
-	*/
+	 */
 	function dispAdminViewServerEnv()
 	{
 		$info = array();

Spacing +77 added lines, -77 removed lines

@@ -28,7 +28,7 @@ 
 	{
 		$db_info = Context::getDBInfo();
 
-		if(strpos($db_info->default_url, 'xn--') !== FALSE)
+		if (strpos($db_info->default_url, 'xn--') !== FALSE)
 		{
 			$xe_default_url = Context::decodeIdna($db_info->default_url);
 		}
@@ -48,13 +48,13 @@ 
 		// forbit access if the user is not an administrator
 		$oMemberModel = getModel('member');
 		$logged_info = $oMemberModel->getLoggedInfo();
-		if($logged_info->is_admin != 'Y')
+		if ($logged_info->is_admin != 'Y')
 		{
 			return $this->stop("msg_is_not_administrator");
 		}
 
 		// change into administration layout
-		$this->setTemplatePath($this->module_path . 'tpl');
+		$this->setTemplatePath($this->module_path.'tpl');
 		$this->setLayoutPath($this->getTemplatePath());
 		$this->setLayoutFile('layout.html');
 
@@ -74,11 +74,11 @@ 
 		Context::set('use_db_session', $db_info->use_db_session == 'N' ? 'N' : 'Y');
 		Context::set('use_mobile_view', $db_info->use_mobile_view == 'Y' ? 'Y' : 'N');
 		Context::set('use_ssl', $db_info->use_ssl ? $db_info->use_ssl : "none");
-		if($db_info->http_port)
+		if ($db_info->http_port)
 		{
 			Context::set('http_port', $db_info->http_port);
 		}
-		if($db_info->https_port)
+		if ($db_info->https_port)
 		{
 			Context::set('https_port', $db_info->https_port);
 		}
@@ -94,7 +94,7 @@ 
 	function checkEasyinstall()
 	{
 		$lastTime = (int) FileHandler::readFile($this->easyinstallCheckFile);
-		if($lastTime > $_SERVER['REQUEST_TIME'] - 60 * 60 * 24 * 30)
+		if ($lastTime > $_SERVER['REQUEST_TIME'] - 60 * 60 * 24 * 30)
 		{
 			return;
 		}
@@ -108,14 +108,14 @@ 
 		$lUpdateDoc = $xml_lUpdate->parse($buff);
 		$updateDate = $lUpdateDoc->response->updatedate->body;
 
-		if(!$updateDate)
+		if (!$updateDate)
 		{
 			$this->_markingCheckEasyinstall();
 			return;
 		}
 
 		$item = $oAutoinstallModel->getLatestPackage();
-		if(!$item || $item->updatedate < $updateDate)
+		if (!$item || $item->updatedate < $updateDate)
 		{
 			$oController = getAdminController('autoinstall');
 			$oController->_updateinfo();
@@ -144,7 +144,7 @@ 
 
 		// Check is_shortcut column
 		$oDB = DB::getInstance();
-		if(!$oDB->isColumnExists('menu_item', 'is_shortcut'))
+		if (!$oDB->isColumnExists('menu_item', 'is_shortcut'))
 		{
 			return;
 		}
@@ -162,9 +162,9 @@ 
 		$currentAct = Context::get('act');
 		$subMenuTitle = '';
 
-		foreach((array) $moduleActionInfo->menu as $key => $value)
+		foreach ((array) $moduleActionInfo->menu as $key => $value)
 		{
-			if(isset($value->acts) && is_array($value->acts) && in_array($currentAct, $value->acts))
+			if (isset($value->acts) && is_array($value->acts) && in_array($currentAct, $value->acts))
 			{
 				$subMenuTitle = $value->title;
 				break;
@@ -173,21 +173,21 @@ 
 		// get current menu's srl(=parentSrl)
 		$parentSrl = 0;
 		$oMenuAdminConroller = getAdminController('menu');
-		foreach((array) $menu->list as $parentKey => $parentMenu)
+		foreach ((array) $menu->list as $parentKey => $parentMenu)
 		{
-			if(!is_array($parentMenu['list']) || !count($parentMenu['list']))
+			if (!is_array($parentMenu['list']) || !count($parentMenu['list']))
 			{
 				continue;
 			}
-			if($parentMenu['href'] == '#' && count($parentMenu['list']))
+			if ($parentMenu['href'] == '#' && count($parentMenu['list']))
 			{
 				$firstChild = current($parentMenu['list']);
 				$menu->list[$parentKey]['href'] = $firstChild['href'];
 			}
 
-			foreach($parentMenu['list'] as $childKey => $childMenu)
+			foreach ($parentMenu['list'] as $childKey => $childMenu)
 			{
-				if($subMenuTitle == $childMenu['text'] && $parentSrl == 0)
+				if ($subMenuTitle == $childMenu['text'] && $parentSrl == 0)
 				{
 					$parentSrl = $childMenu['parent_srl'];
 				}
@@ -200,7 +200,7 @@ 
 		$gnbTitleInfo->adminTitle = $objConfig->adminTitle ? $objConfig->adminTitle : 'XE Admin';
 		$gnbTitleInfo->adminLogo = $objConfig->adminLogo ? $objConfig->adminLogo : 'modules/admin/tpl/img/xe.h1.png';
 
-		$browserTitle = ($subMenuTitle ? $subMenuTitle : 'Dashboard') . ' - ' . $gnbTitleInfo->adminTitle;
+		$browserTitle = ($subMenuTitle ? $subMenuTitle : 'Dashboard').' - '.$gnbTitleInfo->adminTitle;
 
 		// Get list of favorite
 		$oAdminAdminModel = getAdminModel('admin');
@@ -211,7 +211,7 @@ 
 		// move from index method, because use in admin footer
 		$newest_news_url = sprintf("http://news.xpressengine.com/%s/news.php?version=%s&package=%s", _XE_LOCATION_, __XE_VERSION__, _XE_PACKAGE_);
 		$cache_file = sprintf("%sfiles/cache/newest_news.%s.cache.php", _XE_PATH_, _XE_LOCATION_);
-		if(!file_exists($cache_file) || filemtime($cache_file) + 60 * 60 < $_SERVER['REQUEST_TIME'])
+		if (!file_exists($cache_file) || filemtime($cache_file) + 60 * 60 < $_SERVER['REQUEST_TIME'])
 		{
 			// Considering if data cannot be retrieved due to network problem, modify filemtime to prevent trying to reload again when refreshing administration page
 			// Ensure to access the administration page even though news cannot be displayed
@@ -219,20 +219,20 @@ 
 			FileHandler::getRemoteFile($newest_news_url, $cache_file, null, 1, 'GET', 'text/html', array('REQUESTURL' => getFullUrl('')));
 		}
 
-		if(file_exists($cache_file))
+		if (file_exists($cache_file))
 		{
 			$oXml = new XmlParser();
 			$buff = $oXml->parse(FileHandler::readFile($cache_file));
 
 			$item = $buff->zbxe_news->item;
-			if($item)
+			if ($item)
 			{
-				if(!is_array($item))
+				if (!is_array($item))
 				{
 					$item = array($item);
 				}
 
-				foreach($item as $key => $val)
+				foreach ($item as $key => $val)
 				{
 					$obj = new stdClass();
 					$obj->title = $val->body;
@@ -241,7 +241,7 @@ 
 					$news[] = $obj;
 				}
 				Context::set('news', $news);
-				if(isset($news) && is_array($news))
+				if (isset($news) && is_array($news))
 				{
 					Context::set('latestVersion', array_shift($news));
 				}
@@ -264,7 +264,7 @@ 
 	function dispAdminIndex()
 	{
 		$db_info = Context::getDBInfo();
-		Context::set('db_info',$db_info);
+		Context::set('db_info', $db_info);
 
 		// Get statistics
 		$args = new stdClass();
@@ -301,9 +301,9 @@ 
 		$args = new stdClass();
 		$args->list_count = 5;
 		$output = $oCommentModel->getNewestCommentList($args, $columnList);
-		if(is_array($output))
+		if (is_array($output))
 		{
-			foreach($output AS $key => $value)
+			foreach ($output AS $key => $value)
 			{
 				$value->content = strip_tags($value->content);
 			}
@@ -314,17 +314,17 @@ 
 		// Get list of modules
 		$oModuleModel = getModel('module');
 		$module_list = $oModuleModel->getModuleList();
-		if(is_array($module_list))
+		if (is_array($module_list))
 		{
 			$needUpdate = FALSE;
 			$addTables = FALSE;
-			foreach($module_list AS $key => $value)
+			foreach ($module_list AS $key => $value)
 			{
-				if($value->need_install)
+				if ($value->need_install)
 				{
 					$addTables = TRUE;
 				}
-				if($value->need_update)
+				if ($value->need_update)
 				{
 					$needUpdate = TRUE;
 				}
@@ -335,12 +335,12 @@ 
 		$oAutoinstallAdminModel = getAdminModel('autoinstall');
 		$needUpdateList = $oAutoinstallAdminModel->getNeedUpdateList();
 
-		if(is_array($needUpdateList))
+		if (is_array($needUpdateList))
 		{
-			foreach($needUpdateList AS $key => $value)
+			foreach ($needUpdateList AS $key => $value)
 			{
 				$helpUrl = './admin/help/index.html#';
-				switch($value->type)
+				switch ($value->type)
 				{
 					case 'addon':
 						$helpUrl .= 'UMAN_terminology_addon';
@@ -367,8 +367,8 @@ 
 
 		$site_module_info = Context::get('site_module_info');
 		$oAddonAdminModel = getAdminModel('addon');
-		$counterAddonActivated = $oAddonAdminModel->isActivatedAddon('counter', $site_module_info->site_srl );
-		if(!$counterAddonActivated)
+		$counterAddonActivated = $oAddonAdminModel->isActivatedAddon('counter', $site_module_info->site_srl);
+		if (!$counterAddonActivated)
 		{
 			$columnList = array('member_srl', 'nick_name', 'user_name', 'user_id', 'email_address');
 			$args = new stdClass;
@@ -391,9 +391,9 @@ 
 
 		// gathering enviroment check
 		$mainVersion = join('.', array_slice(explode('.', __XE_VERSION__), 0, 2));
-		$path = FileHandler::getRealPath('./files/env/' . $mainVersion);
+		$path = FileHandler::getRealPath('./files/env/'.$mainVersion);
 		$isEnviromentGatheringAgreement = FALSE;
-		if(file_exists($path))
+		if (file_exists($path))
 		{
 			$isEnviromentGatheringAgreement = TRUE;
 		}
@@ -403,7 +403,7 @@ 
 		$isLicenseAgreement = FALSE;
 		$path = FileHandler::getRealPath('./files/env/license_agreement');
 		$isLicenseAgreement = FALSE;
-		if(file_exists($path))
+		if (file_exists($path))
 		{
 			$isLicenseAgreement = TRUE;
 		}
@@ -425,7 +425,7 @@ 
 
 		Context::set('selected_lang', $db_info->lang_type);
 
-		if(strpos($db_info->default_url, 'xn--') !== FALSE)
+		if (strpos($db_info->default_url, 'xn--') !== FALSE)
 		{
 			$db_info->default_url = Context::decodeIdna($db_info->default_url);
 		}
@@ -434,9 +434,9 @@ 
 
 		// site lock
 		Context::set('IP', $_SERVER['REMOTE_ADDR']);
-		if(!$db_info->sitelock_title) $db_info->sitelock_title = 'Maintenance in progress...';
-		if(!in_array('127.0.0.1', $db_info->sitelock_whitelist)) $db_info->sitelock_whitelist[] = '127.0.0.1';
-		if(!in_array($_SERVER['REMOTE_ADDR'], $db_info->sitelock_whitelist)) $db_info->sitelock_whitelist[] = $_SERVER['REMOTE_ADDR'];
+		if (!$db_info->sitelock_title) $db_info->sitelock_title = 'Maintenance in progress...';
+		if (!in_array('127.0.0.1', $db_info->sitelock_whitelist)) $db_info->sitelock_whitelist[] = '127.0.0.1';
+		if (!in_array($_SERVER['REMOTE_ADDR'], $db_info->sitelock_whitelist)) $db_info->sitelock_whitelist[] = $_SERVER['REMOTE_ADDR'];
 		$db_info->sitelock_whitelist = array_unique($db_info->sitelock_whitelist);
 		Context::set('remote_addr', $_SERVER['REMOTE_ADDR']);
 		Context::set('use_sitelock', $db_info->use_sitelock);
@@ -447,7 +447,7 @@ 
 		Context::set('sitelock_whitelist', $whitelist);
 
 
-		if($db_info->admin_ip_list) $admin_ip_list = implode("\r\n", $db_info->admin_ip_list);
+		if ($db_info->admin_ip_list) $admin_ip_list = implode("\r\n", $db_info->admin_ip_list);
 		else $admin_ip_list = '';
 		Context::set('admin_ip_list', $admin_ip_list);
 
@@ -470,7 +470,7 @@ 
 		Context::set('htmlFooter', htmlspecialchars($config->htmlFooter));
 
 		// embed filter
-		require_once(_XE_PATH_ . 'classes/security/EmbedFilter.class.php');
+		require_once(_XE_PATH_.'classes/security/EmbedFilter.class.php');
 		$oEmbedFilter = EmbedFilter::getInstance();
 		context::set('embed_white_object', implode(PHP_EOL, $oEmbedFilter->whiteUrlList));
 		context::set('embed_white_iframe', implode(PHP_EOL, $oEmbedFilter->whiteIframeUrlList));
@@ -529,36 +529,36 @@ 
 	 */
 	function showSendEnv()
 	{
-		if(Context::getResponseMethod() != 'HTML')
+		if (Context::getResponseMethod() != 'HTML')
 		{
 			return;
 		}
 
 		$server = 'http://collect.xpressengine.com/env/img.php?';
 		$path = './files/env/';
-		$install_env = $path . 'install';
+		$install_env = $path.'install';
 		$mainVersion = join('.', array_slice(explode('.', __XE_VERSION__), 0, 2));
 
-		if(file_exists(FileHandler::getRealPath($install_env)))
+		if (file_exists(FileHandler::getRealPath($install_env)))
 		{
 			$oAdminAdminModel = getAdminModel('admin');
 			$params = $oAdminAdminModel->getEnv('INSTALL');
-			$img = sprintf('<img src="%s" alt="" style="height:0px;width:0px" />', $server . $params);
+			$img = sprintf('<img src="%s" alt="" style="height:0px;width:0px" />', $server.$params);
 			Context::addHtmlFooter($img);
 
-			FileHandler::writeFile($path . $mainVersion, '1');
+			FileHandler::writeFile($path.$mainVersion, '1');
 		}
-		else if(isset($_SESSION['enviroment_gather']) && !file_exists(FileHandler::getRealPath($path . $mainVersion)))
+		else if (isset($_SESSION['enviroment_gather']) && !file_exists(FileHandler::getRealPath($path.$mainVersion)))
 		{
-			if($_SESSION['enviroment_gather'] == 'Y')
+			if ($_SESSION['enviroment_gather'] == 'Y')
 			{
 				$oAdminAdminModel = getAdminModel('admin');
 				$params = $oAdminAdminModel->getEnv();
-				$img = sprintf('<img src="%s" alt="" style="height:0px;width:0px" />', $server . $params);
+				$img = sprintf('<img src="%s" alt="" style="height:0px;width:0px" />', $server.$params);
 				Context::addHtmlFooter($img);
 			}
 
-			FileHandler::writeFile($path . $mainVersion, '1');
+			FileHandler::writeFile($path.$mainVersion, '1');
 			unset($_SESSION['enviroment_gather']);
 		}
 	}
@@ -576,64 +576,64 @@ 
 		$tmp = explode("&", $envInfo);
 		$arrInfo = array();
 		$xe_check_env = array();
-		foreach($tmp as $value) {
+		foreach ($tmp as $value) {
 			$arr = explode("=", $value);
-			if($arr[0]=="type") {
+			if ($arr[0] == "type") {
 				continue;
-			}elseif($arr[0]=="phpext" ) {
+			}elseif ($arr[0] == "phpext") {
 				$str = urldecode($arr[1]);
-				$xe_check_env[$arr[0]]= str_replace("|", ", ", $str);
-			} elseif($arr[0]=="module" ) {
+				$xe_check_env[$arr[0]] = str_replace("|", ", ", $str);
+			} elseif ($arr[0] == "module") {
 				$str = urldecode($arr[1]);
 				$arrModuleName = explode("|", $str);
 				$oModuleModel = getModel("module");
 				$mInfo = array();
-				foreach($arrModuleName as $moduleName) {
+				foreach ($arrModuleName as $moduleName) {
 					$moduleInfo = $oModuleModel->getModuleInfoXml($moduleName);
 					$mInfo[] = "{$moduleName}({$moduleInfo->version})";
 				}
-				$xe_check_env[$arr[0]]= join(", ", $mInfo);
-			} elseif($arr[0]=="addon") {
+				$xe_check_env[$arr[0]] = join(", ", $mInfo);
+			} elseif ($arr[0] == "addon") {
 				$str = urldecode($arr[1]);
 				$arrAddonName = explode("|", $str);
 				$oAddonModel = getAdminModel("addon");
 				$mInfo = array();
-				foreach($arrAddonName as $addonName) {
+				foreach ($arrAddonName as $addonName) {
 					$addonInfo = $oAddonModel->getAddonInfoXml($addonName);
 					$mInfo[] = "{$addonName}({$addonInfo->version})";
 				}
-				$xe_check_env[$arr[0]]= join(", ", $mInfo);
-			} elseif($arr[0]=="widget") {
+				$xe_check_env[$arr[0]] = join(", ", $mInfo);
+			} elseif ($arr[0] == "widget") {
 				$str = urldecode($arr[1]);
 				$arrWidgetName = explode("|", $str);
 				$oWidgetModel = getModel("widget");
 				$mInfo = array();
-				foreach($arrWidgetName as $widgetName) {
+				foreach ($arrWidgetName as $widgetName) {
 					$widgetInfo = $oWidgetModel->getWidgetInfo($widgetName);
 					$mInfo[] = "{$widgetName}({$widgetInfo->version})";
 				}
-				$xe_check_env[$arr[0]]= join(", ", $mInfo);
-			} elseif($arr[0]=="widgetstyle") {
+				$xe_check_env[$arr[0]] = join(", ", $mInfo);
+			} elseif ($arr[0] == "widgetstyle") {
 				$str = urldecode($arr[1]);
 				$arrWidgetstyleName = explode("|", $str);
 				$oWidgetModel = getModel("widget");
 				$mInfo = array();
-				foreach($arrWidgetstyleName as $widgetstyleName) {
+				foreach ($arrWidgetstyleName as $widgetstyleName) {
 					$widgetstyleInfo = $oWidgetModel->getWidgetStyleInfo($widgetstyleName);
 					$mInfo[] = "{$widgetstyleName}({$widgetstyleInfo->version})";
 				}
-				$xe_check_env[$arr[0]]= join(", ", $mInfo);
+				$xe_check_env[$arr[0]] = join(", ", $mInfo);
 
-			} elseif($arr[0]=="layout") {
+			} elseif ($arr[0] == "layout") {
 				$str = urldecode($arr[1]);
 				$arrLayoutName = explode("|", $str);
 				$oLayoutModel = getModel("layout");
 				$mInfo = array();
-				foreach($arrLayoutName as $layoutName) {
+				foreach ($arrLayoutName as $layoutName) {
 					$layoutInfo = $oLayoutModel->getLayoutInfo($layoutName);
 					$mInfo[] = "{$layoutName}({$layoutInfo->version})";
 				}
-				$xe_check_env[$arr[0]]= join(", ", $mInfo);
+				$xe_check_env[$arr[0]] = join(", ", $mInfo);
 			} else {
 				$xe_check_env[$arr[0]] = urldecode($arr[1]);
 			}
@@ -647,15 +647,15 @@ 
 		$php_core['memory_limit'] = "{$ini_info['memory_limit']['local_value']}";
 		$info['PHP_Core'] = $php_core;
 
-		$str_info = "[XE Server Environment " . date("Y-m-d") . "]\n\n";
+		$str_info = "[XE Server Environment ".date("Y-m-d")."]\n\n";
 		$str_info .= "realpath : ".realpath('./')."\n";
-		foreach( $info as $key=>$value )
+		foreach ($info as $key=>$value)
 		{
-			if( is_array( $value ) == false ) {
+			if (is_array($value) == false) {
 				$str_info .= "{$key} : {$value}\n";
 			} else {
 				//$str_info .= "\n{$key} \n";
-				foreach( $value as $key2=>$value2 )
+				foreach ($value as $key2=>$value2)
 					$str_info .= "{$key2} : {$value2}\n";
 			}
 		}
@@ -672,7 +672,7 @@ 
 		Context::set('use_rewrite', $useRewrite); 
 
 		// nginx 체크, rewrite 사용법 안내
-		if($useRewrite == 'N' && stripos($_SERVER['SERVER_SOFTWARE'], 'nginx') !== false) Context::set('use_nginx', 'Y');
+		if ($useRewrite == 'N' && stripos($_SERVER['SERVER_SOFTWARE'], 'nginx') !== false) Context::set('use_nginx', 'Y');
 		
 		Context::set('useable', $oInstallController->checkInstallEnv());
 		$this->setTemplateFile('check_env.html');

Braces +23 added lines, -13 removed lines

@@ -31,8 +31,7 @@ 
 		if(strpos($db_info->default_url, 'xn--') !== FALSE)
 		{
 			$xe_default_url = Context::decodeIdna($db_info->default_url);
-		}
-		else
+		} else
 		{
 			$xe_default_url = $db_info->default_url;
 		}
@@ -434,9 +433,15 @@ 
 
 		// site lock
 		Context::set('IP', $_SERVER['REMOTE_ADDR']);
-		if(!$db_info->sitelock_title) $db_info->sitelock_title = 'Maintenance in progress...';
-		if(!in_array('127.0.0.1', $db_info->sitelock_whitelist)) $db_info->sitelock_whitelist[] = '127.0.0.1';
-		if(!in_array($_SERVER['REMOTE_ADDR'], $db_info->sitelock_whitelist)) $db_info->sitelock_whitelist[] = $_SERVER['REMOTE_ADDR'];
+		if(!$db_info->sitelock_title) {
+			$db_info->sitelock_title = 'Maintenance in progress...';
+		}
+		if(!in_array('127.0.0.1', $db_info->sitelock_whitelist)) {
+			$db_info->sitelock_whitelist[] = '127.0.0.1';
+		}
+		if(!in_array($_SERVER['REMOTE_ADDR'], $db_info->sitelock_whitelist)) {
+			$db_info->sitelock_whitelist[] = $_SERVER['REMOTE_ADDR'];
+		}
 		$db_info->sitelock_whitelist = array_unique($db_info->sitelock_whitelist);
 		Context::set('remote_addr', $_SERVER['REMOTE_ADDR']);
 		Context::set('use_sitelock', $db_info->use_sitelock);
@@ -447,8 +452,11 @@ 
 		Context::set('sitelock_whitelist', $whitelist);
 
 
-		if($db_info->admin_ip_list) $admin_ip_list = implode("\r\n", $db_info->admin_ip_list);
-		else $admin_ip_list = '';
+		if($db_info->admin_ip_list) {
+			$admin_ip_list = implode("\r\n", $db_info->admin_ip_list);
+		} else {
+			$admin_ip_list = '';
+		}
 		Context::set('admin_ip_list', $admin_ip_list);
 
 		Context::set('lang_selected', Context::loadLangSelected());
@@ -547,8 +555,7 @@ 
 			Context::addHtmlFooter($img);
 
 			FileHandler::writeFile($path . $mainVersion, '1');
-		}
-		else if(isset($_SESSION['enviroment_gather']) && !file_exists(FileHandler::getRealPath($path . $mainVersion)))
+		} else if(isset($_SESSION['enviroment_gather']) && !file_exists(FileHandler::getRealPath($path . $mainVersion)))
 		{
 			if($_SESSION['enviroment_gather'] == 'Y')
 			{
@@ -580,7 +587,7 @@ 
 			$arr = explode("=", $value);
 			if($arr[0]=="type") {
 				continue;
-			}elseif($arr[0]=="phpext" ) {
+			} elseif($arr[0]=="phpext" ) {
 				$str = urldecode($arr[1]);
 				$xe_check_env[$arr[0]]= str_replace("|", ", ", $str);
 			} elseif($arr[0]=="module" ) {
@@ -655,8 +662,9 @@ 
 				$str_info .= "{$key} : {$value}\n";
 			} else {
 				//$str_info .= "\n{$key} \n";
-				foreach( $value as $key2=>$value2 )
-					$str_info .= "{$key2} : {$value2}\n";
+				foreach( $value as $key2=>$value2 ) {
+									$str_info .= "{$key2} : {$value2}\n";
+				}
 			}
 		}
 
@@ -672,7 +680,9 @@ 
 		Context::set('use_rewrite', $useRewrite); 
 
 		// nginx 체크, rewrite 사용법 안내
-		if($useRewrite == 'N' && stripos($_SERVER['SERVER_SOFTWARE'], 'nginx') !== false) Context::set('use_nginx', 'Y');
+		if($useRewrite == 'N' && stripos($_SERVER['SERVER_SOFTWARE'], 'nginx') !== false) {
+			Context::set('use_nginx', 'Y');
+		}
 		
 		Context::set('useable', $oInstallController->checkInstallEnv());
 		$this->setTemplateFile('check_env.html');

modules/adminlogging/adminlogging.controller.php

Doc Comments +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
 
 	/**
 	 * Initialization
-	 * @return void
+	 * @return ModuleObject|null
 	 */
 	function init()
 	{

Spacing +2 added lines, -2 removed lines

@@ -21,7 +21,7 @@ 
 		// forbit access if the user is not an administrator
 		$oMemberModel = getModel('member');
 		$logged_info = $oMemberModel->getLoggedInfo();
-		if($logged_info->is_admin != 'Y')
+		if ($logged_info->is_admin != 'Y')
 		{
 			return $this->stop("msg_is_not_administrator");
 		}
@@ -33,7 +33,7 @@ 
 	 */
 	function insertLog($module, $act)
 	{
-		if(!$module || !$act)
+		if (!$module || !$act)
 		{
 			return;
 		}

modules/autoinstall/autoinstall.admin.model.php

Doc Comments +7 added lines

@@ -15,6 +15,9 @@ 
 
 	/**
 	 * Pre process parameters
+	 * @param string $order_target
+	 * @param string $order_type
+	 * @param string $page
 	 */
 	function preProcParam(&$order_target, &$order_type, &$page)
 	{
@@ -115,6 +118,10 @@ 
 
 	/**
 	 * Get Package List
+	 * @param string $type
+	 * @param string $search_keyword
+	 * @param integer $category_srl
+	 * @param string $parent_program
 	 */
 	function getPackageList($type, $order_target = 'newest', $order_type = 'desc', $page = '1', $search_keyword = NULL, $category_srl = NULL, $parent_program = NULL)
 	{

Braces +10 added lines, -20 removed lines

@@ -71,8 +71,7 @@ 
 		if($type == 'P')
 		{
 			$category_srl = $this->layout_category_srl;
-		}
-		else
+		} else
 		{
 			$category_srl = $this->mobile_layout_category_srl;
 		}
@@ -103,8 +102,7 @@ 
 		if($type == 'P')
 		{
 			$category_srl = $this->module_skin_category_srl;
-		}
-		else
+		} else
 		{
 			$category_srl = $this->module_mobile_skin_category_srl;
 		}
@@ -121,13 +119,11 @@ 
 		if($type == 'menu')
 		{
 			$params["act"] = "getResourceapiMenuPackageList";
-		}
-		elseif($type == 'skin')
+		} elseif($type == 'skin')
 		{
 			$params["act"] = "getResourceapiSkinPackageList";
 			$params['parent_program'] = $parent_program;
-		}
-		else
+		} else
 		{
 			$params["act"] = "getResourceapiPackagelist";
 		}
@@ -172,15 +168,13 @@ 
 		if($output->toBool()==TRUE)
 		{
 			$is_authed = 1;
-		}
-		else
+		} else
 		{
 			$ftp_info = Context::getFTPInfo();
 			if(!$ftp_info->ftp_root_path)
 			{
 				$is_authed = -1;
-			}
-			else
+			} else
 			{
 				$is_authed = (int) isset($_SESSION['ftp_password']);
 			}
@@ -216,8 +210,7 @@ 
 			if($packageInfo->type == 'core')
 			{
 				$title = 'XpressEngine';
-			}
-			else
+			} else
 			{
 				$configFile = $oModel->getConfigFilePath($packageInfo->type);
 				$xmlDoc = $xml->loadXmlFile(FileHandler::getRealPath($package->path) . $configFile);
@@ -238,8 +231,7 @@ 
 						$type = "layout";
 					}
 					$title = $xmlDoc->{$type}->title->body;
-				}
-				else
+				} else
 				{
 					$pathInfo = explode('/', $package->path);
 					$title = $pathInfo[count($pathInfo) - 1];
@@ -303,8 +295,7 @@ 
 					{
 						$package->depends[$key]->installed = FALSE;
 						$package->package_srl .= "," . $dep->package_srl;
-					}
-					else
+					} else
 					{
 						$package->depends[$key]->installed = TRUE;
 						$package->depends[$key]->cur_version = $packages[$dep->package_srl]->current_version;
@@ -318,8 +309,7 @@ 
 								$package->contain_core = TRUE;
 								$package->contain_core_version = $dep->version;
 							}
-						}
-						else
+						} else
 						{
 							$package->need_update = FALSE;
 						}

Spacing +43 added lines, -43 removed lines

@@ -19,19 +19,19 @@ 
 	function preProcParam(&$order_target, &$order_type, &$page)
 	{
 		$order_target_array = array('newest' => 1, 'download' => 1, 'popular' => 1);
-		if(!isset($order_target_array[$order_target]))
+		if (!isset($order_target_array[$order_target]))
 		{
 			$order_target = 'newest';
 		}
 
 		$order_type_array = array('asc' => 1, 'desc' => 1);
-		if(!isset($order_type_array[$order_type]))
+		if (!isset($order_type_array[$order_type]))
 		{
 			$order_type = 'desc';
 		}
 
 		$page = (int) $page;
-		if($page < 1)
+		if ($page < 1)
 		{
 			$page = 1;
 		}
@@ -63,12 +63,12 @@ 
 
 		$type_array = array('M' => 1, 'P' => 1);
 		$type = Context::get('type');
-		if(!isset($type_array[$type]))
+		if (!isset($type_array[$type]))
 		{
 			$type = 'P';
 		}
 
-		if($type == 'P')
+		if ($type == 'P')
 		{
 			$category_srl = $this->layout_category_srl;
 		}
@@ -95,12 +95,12 @@ 
 
 		$type_array = array('M' => 1, 'P' => 1);
 		$type = Context::get('type');
-		if(!isset($type_array[$type]))
+		if (!isset($type_array[$type]))
 		{
 			$type = 'P';
 		}
 
-		if($type == 'P')
+		if ($type == 'P')
 		{
 			$category_srl = $this->module_skin_category_srl;
 		}
@@ -118,11 +118,11 @@ 
 	 */
 	function getPackageList($type, $order_target = 'newest', $order_type = 'desc', $page = '1', $search_keyword = NULL, $category_srl = NULL, $parent_program = NULL)
 	{
-		if($type == 'menu')
+		if ($type == 'menu')
 		{
 			$params["act"] = "getResourceapiMenuPackageList";
 		}
-		elseif($type == 'skin')
+		elseif ($type == 'skin')
 		{
 			$params["act"] = "getResourceapiSkinPackageList";
 			$params['parent_program'] = $parent_program;
@@ -137,18 +137,18 @@ 
 		$params["order_type"] = $order_type;
 		$params["page"] = $page;
 
-		if($category_srl)
+		if ($category_srl)
 		{
 			$params["category_srl"] = $category_srl;
 		}
 
-		if($search_keyword)
+		if ($search_keyword)
 		{
 			$params["search_keyword"] = $search_keyword;
 		}
 
 		$xmlDoc = XmlGenerater::getXmlDoc($params);
-		if($xmlDoc && $xmlDoc->response->packagelist->item)
+		if ($xmlDoc && $xmlDoc->response->packagelist->item)
 		{
 			$item_list = $oAdminView->rearranges($xmlDoc->response->packagelist->item);
 			$this->add('item_list', $item_list);
@@ -169,14 +169,14 @@ 
 		
 		$is_authed = 0;
 		$output = $oAdminModel->checkUseDirectModuleInstall($package);
-		if($output->toBool()==TRUE)
+		if ($output->toBool() == TRUE)
 		{
 			$is_authed = 1;
 		}
 		else
 		{
 			$ftp_info = Context::getFTPInfo();
-			if(!$ftp_info->ftp_root_path)
+			if (!$ftp_info->ftp_root_path)
 			{
 				$is_authed = -1;
 			}
@@ -196,14 +196,14 @@ 
 	{
 		$oModel = getModel('autoinstall');
 		$output = executeQueryArray('autoinstall.getNeedUpdate');
-		if(!is_array($output->data))
+		if (!is_array($output->data))
 		{
 			return NULL;
 		}
 
 		$result = array();
 		$xml = new XmlParser();
-		foreach($output->data as $package)
+		foreach ($output->data as $package)
 		{
 			$packageSrl = $package->package_srl;
 
@@ -213,27 +213,27 @@ 
 			$packageInfo->type = $oModel->getTypeFromPath($package->path);
 			$packageInfo->url = $oModel->getUpdateUrlByPackageSrl($package->package_srl);
 
-			if($packageInfo->type == 'core')
+			if ($packageInfo->type == 'core')
 			{
 				$title = 'XpressEngine';
 			}
 			else
 			{
 				$configFile = $oModel->getConfigFilePath($packageInfo->type);
-				$xmlDoc = $xml->loadXmlFile(FileHandler::getRealPath($package->path) . $configFile);
+				$xmlDoc = $xml->loadXmlFile(FileHandler::getRealPath($package->path).$configFile);
 
-				if($xmlDoc)
+				if ($xmlDoc)
 				{
 					$type = $packageInfo->type;
-					if($type == "drcomponent")
+					if ($type == "drcomponent")
 					{
 						$type = "component";
 					}
-					if($type == "style" || $type == "m.skin")
+					if ($type == "style" || $type == "m.skin")
 					{
 						$type = "skin";
 					}
-					if($type == "m.layout")
+					if ($type == "m.layout")
 					{
 						$type = "layout";
 					}
@@ -267,7 +267,7 @@ 
 		$oModel = getModel('autoinstall');
 
 		$targetpackages = array();
-		if($xmlDoc)
+		if ($xmlDoc)
 		{
 			$xmlPackage = $xmlDoc->response->package;
 			$package = new stdClass();
@@ -276,15 +276,15 @@ 
 			$package->package_description = $xmlPackage->package_description->body;
 			$package->version = $xmlPackage->version->body;
 			$package->path = $xmlPackage->path->body;
-			if($xmlPackage->depends)
+			if ($xmlPackage->depends)
 			{
-				if(!is_array($xmlPackage->depends->item))
+				if (!is_array($xmlPackage->depends->item))
 				{
 					$xmlPackage->depends->item = array($xmlPackage->depends->item);
 				}
 
 				$package->depends = array();
-				foreach($xmlPackage->depends->item as $item)
+				foreach ($xmlPackage->depends->item as $item)
 				{
 					$dep_item = new stdClass();
 					$dep_item->package_srl = $item->package_srl->body;
@@ -297,23 +297,23 @@ 
 
 				$packages = $oModel->getInstalledPackages(array_keys($targetpackages));
 				$package->deplist = "";
-				foreach($package->depends as $key => $dep)
+				foreach ($package->depends as $key => $dep)
 				{
-					if(!$packages[$dep->package_srl])
+					if (!$packages[$dep->package_srl])
 					{
 						$package->depends[$key]->installed = FALSE;
-						$package->package_srl .= "," . $dep->package_srl;
+						$package->package_srl .= ",".$dep->package_srl;
 					}
 					else
 					{
 						$package->depends[$key]->installed = TRUE;
 						$package->depends[$key]->cur_version = $packages[$dep->package_srl]->current_version;
-						if(version_compare($dep->version, $packages[$dep->package_srl]->current_version, ">"))
+						if (version_compare($dep->version, $packages[$dep->package_srl]->current_version, ">"))
 						{
 							$package->depends[$key]->need_update = TRUE;
-							$package->package_srl .= "," . $dep->package_srl;
+							$package->package_srl .= ",".$dep->package_srl;
 
-							if($dep->path === '.')
+							if ($dep->path === '.')
 							{
 								$package->contain_core = TRUE;
 								$package->contain_core_version = $dep->version;
@@ -328,14 +328,14 @@ 
 			}
 
 			$installedPackage = $oModel->getInstalledPackage($packageSrl);
-			if($installedPackage)
+			if ($installedPackage)
 			{
 				$package->installed = TRUE;
 				$package->cur_version = $installedPackage->current_version;
 				$package->need_update = version_compare($package->version, $installedPackage->current_version, ">");
 			}
 
-			if($package->path === '.')
+			if ($package->path === '.')
 			{
 				$package->contain_core = TRUE;
 				$package->contain_core_version = $package->version;
@@ -351,7 +351,7 @@ 
 	public function getAutoInstallAdminInstallInfo()
 	{
 		$packageSrl = Context::get('package_srl');
-		if(!$packageSrl)
+		if (!$packageSrl)
 		{
 			return new BaseObject(-1, 'msg_invalid_request');
 		}
@@ -365,23 +365,23 @@ 
 		$directModuleInstall = TRUE;
 		$arrUnwritableDir = array();
 		$output = $this->isWritableDir($package->path);
-		if($output->toBool()==FALSE)
+		if ($output->toBool() == FALSE)
 		{
 			$directModuleInstall = FALSE;
 			$arrUnwritableDir[] = $output->get('path');
 		}
 
-		foreach($package->depends as $dep)
+		foreach ($package->depends as $dep)
 		{
 			$output = $this->isWritableDir($dep->path);
-			if($output->toBool()==FALSE)
+			if ($output->toBool() == FALSE)
 			{
 				$directModuleInstall = FALSE;
 				$arrUnwritableDir[] = $output->get('path');
 			}
 		}
 
-		if($directModuleInstall==FALSE)
+		if ($directModuleInstall == FALSE)
 		{
 			$output = new BaseObject(-1, 'msg_direct_inall_invalid');
 			$output->add('path', $arrUnwritableDir);
@@ -396,17 +396,17 @@ 
 		$path_list = explode('/', dirname($path));
 		$real_path = './';
 
-		while($path_list)
+		while ($path_list)
 		{
-			$check_path = realpath($real_path . implode('/', $path_list));
-			if(FileHandler::isDir($check_path))
+			$check_path = realpath($real_path.implode('/', $path_list));
+			if (FileHandler::isDir($check_path))
 			{
 				break;
 			}
 			array_pop($path_list);
 		}
 
-		if(FileHandler::isWritableDir($check_path)==FALSE)
+		if (FileHandler::isWritableDir($check_path) == FALSE)
 		{
 			$output = new BaseObject(-1, 'msg_unwritable_directory');
 			$output->add('path', FileHandler::getRealPath($check_path));

modules/board/board.admin.model.php

Doc Comments +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
 
 	/**
 	 * Get the board module admin simple setting page
-	 * @return void
+	 * @return null|string
 	 */
 	public function getBoardAdminSimpleSetup($moduleSrl, $setupUrl)
 	{

Spacing +3 added lines, -3 removed lines

@@ -25,7 +25,7 @@ 
 	 */
 	public function getBoardAdminSimpleSetup($moduleSrl, $setupUrl)
 	{
-		if(!$moduleSrl)
+		if (!$moduleSrl)
 		{
 			return;
 		}
@@ -35,7 +35,7 @@ 
 		$oModuleModel = getModel('module');
 		$moduleInfo = $oModuleModel->getModuleInfoByModuleSrl($moduleSrl);
 		$moduleInfo->use_status = explode('|@|', $moduleInfo->use_status);
-		if($moduleInfo)
+		if ($moduleInfo)
 		{
 			Context::set('module_info', $moduleInfo);
 		}
@@ -46,7 +46,7 @@ 
 		Context::set('document_status_list', $documentStatusList);
 
 		// set order target list
-		foreach($this->order_target AS $key)
+		foreach ($this->order_target AS $key)
 		{
 			$order_target[$key] = Context::getLang($key);
 		}

modules/board/board.api.php

Doc Comments +9 added lines

@@ -87,6 +87,9 @@ 
 		$oModule->add('comment_list',$this->arrangeComment(Context::get('comment_list')));

 	}

 

+	/**
+	 * @param string $content_list
+	 */
 	function arrangeContentList($content_list) {

 		$output = array();

 		if(count($content_list)) {

@@ -120,6 +123,9 @@ 
 		return $output;

 	}

 

+	/**
+	 * @param string $comment_list
+	 */
 	function arrangeComment($comment_list) {

 		$output = array();

 		if(count($comment_list) > 0 ) {

@@ -133,6 +139,9 @@ 
 	}

 

 

+	/**
+	 * @param string $file_list
+	 */
 	function arrangeFile($file_list) {

 		$output = array();

 		if(count($file_list) > 0) {

Spacing +27 added lines, -27 removed lines

@@ -18,7 +18,7 @@ 
 	 * @brief notice list

 	 **/

 	function dispBoardNoticeList(&$oModule) {

-		 $oModule->add('notice_list',$this->arrangeContentList(Context::get('notice_list')));

+		 $oModule->add('notice_list', $this->arrangeContentList(Context::get('notice_list')));

 	}

 

 

@@ -29,11 +29,11 @@ 
 		$api_type = Context::get('api_type');

 		$document_list = $this->arrangeContentList(Context::get('document_list'));

 

-		if($api_type =='summary')

+		if ($api_type == 'summary')

 		{

 			$content_cut_size = Context::get('content_cut_size');

-			$content_cut_size = $content_cut_size?$content_cut_size:50;

-			foreach($document_list as $k=>$v)

+			$content_cut_size = $content_cut_size ? $content_cut_size : 50;

+			foreach ($document_list as $k=>$v)

 			{

 				$oDocument = new documentItem();

 				$oDocument->setAttribute($v, false);

@@ -42,8 +42,8 @@ 
 			}

 		}

 

-		$oModule->add('document_list',$document_list);

-		$oModule->add('page_navigation',Context::get('page_navigation'));

+		$oModule->add('document_list', $document_list);

+		$oModule->add('page_navigation', Context::get('page_navigation'));

 	}

 

 

@@ -51,7 +51,7 @@ 
 	 * @brief category list

 	 **/

 	function dispBoardCategoryList(&$oModule) {

-		$oModule->add('category_list',Context::get('category_list'));

+		$oModule->add('category_list', Context::get('category_list'));

 	}

 

 	/**

@@ -60,11 +60,11 @@ 
 	function dispBoardContentView(&$oModule) {

 		$oDocument = Context::get('oDocument');

 		$extra_vars = $oDocument->getExtraVars();
-		if($oDocument->isGranted())
+		if ($oDocument->isGranted())
 		{

-			$oDocument->add('extra_vars',$this->arrangeExtraVars($extra_vars));
+			$oDocument->add('extra_vars', $this->arrangeExtraVars($extra_vars));
 		}

-		$oModule->add('oDocument',$this->arrangeContent($oDocument));

+		$oModule->add('oDocument', $this->arrangeContent($oDocument));

 	}

 

 

@@ -73,7 +73,7 @@ 
 	 **/

 	function dispBoardContentFileList(&$oModule) {

 		$oDocument = Context::get('oDocument');
-		if($oDocument->isAccessible())
+		if ($oDocument->isAccessible())
 		{
 			$oModule->add('file_list', $this->arrangeFile(Context::get('file_list')));
 		}
@@ -88,20 +88,20 @@ 
 	 * @brief tag list

 	 **/

 	function dispBoardTagList(&$oModule) {

-		$oModule->add('tag_list',Context::get('tag_list'));

+		$oModule->add('tag_list', Context::get('tag_list'));

 	}

 

 	/**

 	 * @brief comments list

 	 **/

 	function dispBoardContentCommentList(&$oModule) {

-		$oModule->add('comment_list',$this->arrangeComment(Context::get('comment_list')));

+		$oModule->add('comment_list', $this->arrangeComment(Context::get('comment_list')));

 	}

 

 	function arrangeContentList($content_list) {

 		$output = array();

-		if(count($content_list)) {

-			foreach($content_list as $key => $val) $output[] = $this->arrangeContent($val);

+		if (count($content_list)) {

+			foreach ($content_list as $key => $val) $output[] = $this->arrangeContent($val);

 		}

 		return $output;

 	}

@@ -110,16 +110,16 @@ 
 	function arrangeContent($content) {

 		$oBoardView = getView('board');

 		$output = new stdClass;

-		if($content){

-			$output = $content->gets('document_srl','category_srl','member_srl','nick_name','title','content','tags','readed_count','voted_count','blamed_count','comment_count','regdate','last_update','extra_vars','status');

+		if ($content) {

+			$output = $content->gets('document_srl', 'category_srl', 'member_srl', 'nick_name', 'title', 'content', 'tags', 'readed_count', 'voted_count', 'blamed_count', 'comment_count', 'regdate', 'last_update', 'extra_vars', 'status');

 

-			if(!$oBoardView->grant->view)

+			if (!$oBoardView->grant->view)

 			{

 				unset($output->content);

 				unset($output->tags);

 				unset($output->extra_vars);

 			}
-			if(!$content->isAccessible())
+			if (!$content->isAccessible())
 			{
 				$output->content = Context::getLang('msg_is_secret');
 			}

@@ -137,11 +137,11 @@ 
 

 	function arrangeComment($comment_list) {

 		$output = array();

-		if(count($comment_list) > 0 ) {

-			foreach($comment_list as $key => $val){

+		if (count($comment_list) > 0) {

+			foreach ($comment_list as $key => $val) {

 				$item = null;

-				$item = $val->gets('comment_srl','parent_srl','depth','nick_name','content','is_secret','voted_count','blamed_count','regdate','last_update');

-				if(!$val->isAccessible())
+				$item = $val->gets('comment_srl', 'parent_srl', 'depth', 'nick_name', 'content', 'is_secret', 'voted_count', 'blamed_count', 'regdate', 'last_update');

+				if (!$val->isAccessible())
 				{
 					$item->content = Context::getLang('msg_is_secret');
 				}
@@ -154,8 +154,8 @@ 
 

 	function arrangeFile($file_list) {

 		$output = array();

-		if(count($file_list) > 0) {

-			foreach($file_list as $key => $val){

+		if (count($file_list) > 0) {

+			foreach ($file_list as $key => $val) {

 				$item = new stdClass;

 				$item->download_count = $val->download_count;

 				$item->source_filename = $val->source_filename;

@@ -169,8 +169,8 @@ 
 

 	function arrangeExtraVars($list) {

 		$output = array();

-		if(count($list)) {

-			foreach($list as $key => $val){

+		if (count($list)) {

+			foreach ($list as $key => $val) {

 				$item = new stdClass;

 				$item->name = $val->name;

 				$item->type = $val->type;

Braces +4 added lines, -3 removed lines

@@ -76,8 +76,7 @@ 
 		if($oDocument->isAccessible())
 		{
 			$oModule->add('file_list', $this->arrangeFile(Context::get('file_list')));
-		}
-		else
+		} else
 		{
 			$oModule->add('file_list', array());
 		}

@@ -101,7 +100,9 @@ 
 	function arrangeContentList($content_list) {

 		$output = array();

 		if(count($content_list)) {

-			foreach($content_list as $key => $val) $output[] = $this->arrangeContent($val);

+			foreach($content_list as $key => $val) {
+				$output[] = $this->arrangeContent($val);
+			}
 		}

 		return $output;

 	}

modules/board/board.view.php

Doc Comments +3 added lines

@@ -747,6 +747,9 @@
 		$this->setTemplateFile('write_form');
 	}
 
+	/**
+	 * @param ModuleObject $oDocumentModel
+	 */
 	function _getStatusNameList(&$oDocumentModel)
 	{
 		$resultList = array();

Braces +62 added lines, -38 removed lines

@@ -53,18 +53,15 @@ 
 			if($this->module_info->hide_category)
 			{
 				$this->module_info->use_category = ($this->module_info->hide_category == 'Y') ? 'N' : 'Y';
-			}
-			else if($this->module_info->use_category)
+			} else if($this->module_info->use_category)
 			{
 				$this->module_info->hide_category = ($this->module_info->use_category == 'Y') ? 'N' : 'Y';
-			}
-			else
+			} else
 			{
 				$this->module_info->hide_category = 'N';
 				$this->module_info->use_category = 'Y';
 			}
-		}
-		else
+		} else
 		{
 			$this->module_info->hide_category = 'Y';
 			$this->module_info->use_category = 'N';
@@ -84,8 +81,7 @@ 
 				$this->grant->write_comment = FALSE;
 				$this->grant->view = FALSE;
 			}
-		}
-		else
+		} else
 		{
 			$this->consultation = FALSE;
 		}
@@ -160,13 +156,17 @@ 
 		 * add extra vaiables to the search options
 		 **/
 		// use search options on the template (the search options key has been declared, based on the language selected)
-		foreach($this->search_option as $opt) $search_option[$opt] = Context::getLang($opt);
+		foreach($this->search_option as $opt) {
+			$search_option[$opt] = Context::getLang($opt);
+		}
 		$extra_keys = Context::get('extra_keys');
 		if($extra_keys)
 		{
 			foreach($extra_keys as $key => $val)
 			{
-				if($val->search == 'Y') $search_option['extra_vars'.$val->idx] = $val->name;
+				if($val->search == 'Y') {
+					$search_option['extra_vars'.$val->idx] = $val->name;
+				}
 			}
 		}
 		// remove a search option that is not public in member config
@@ -175,8 +175,9 @@ 
 		{
 			if(in_array($signupFormElement->title, $search_option))
 			{
-				if($signupFormElement->isPublic == 'N')
-					unset($search_option[$signupFormElement->name]);
+				if($signupFormElement->isPublic == 'N') {
+									unset($search_option[$signupFormElement->name]);
+				}
 			}
 		}
 		Context::set('search_option', $search_option);
@@ -194,7 +195,9 @@ 
 		// list config, columnList setting
 		$oBoardModel = getModel('board');
 		$this->listConfig = $oBoardModel->getListConfig($this->module_info->module_srl);
-		if(!$this->listConfig) $this->listConfig = array();
+		if(!$this->listConfig) {
+			$this->listConfig = array();
+		}
 		$this->_makeListColumnList();
 
 		// display the notice list
@@ -265,7 +268,9 @@ 
 				}
 
 				// check the manage grant
-				if($this->grant->manager) $oDocument->setGrant();
+				if($this->grant->manager) {
+					$oDocument->setGrant();
+				}
 
 				// if the consultation function is enabled, and the document is not a notice
 				if($this->consultation && !$oDocument->isNotice())
@@ -286,8 +291,7 @@ 
 					}
 				}
 
-			}
-			else
+			} else
 			{
 				// if the document is not existed, then alert a warning message
 				Context::set('document_srl','',true);
@@ -297,8 +301,7 @@ 
 		/**
 		 * if the document is not existed, get an empty document
 		 **/
-		}
-		else
+		} else
 		{
 			$oDocument = $oDocumentModel->getDocument(0);
 		}
@@ -313,8 +316,7 @@ 
 				$oDocument = $oDocumentModel->getDocument(0);
 				Context::set('document_srl','',true);
 				$this->alertMessage('msg_not_permitted');
-			}
-			else
+			} else
 			{
 				// add the document title to the browser
 				Context::addBrowserTitle($oDocument->getTitleText());
@@ -368,13 +370,16 @@ 
 		$downloadGrantCount = 0;
 		if(is_array($file_module_config->download_grant))
 		{
-			foreach($file_module_config->download_grant AS $value)
-				if($value) $downloadGrantCount++;
+			foreach($file_module_config->download_grant AS $value) {
+							if($value) $downloadGrantCount++;
+			}
 		}
 
 		if(is_array($file_module_config->download_grant) && $downloadGrantCount>0)
 		{
-			if(!Context::get('is_logged')) return $this->stop('msg_not_permitted_download');
+			if(!Context::get('is_logged')) {
+				return $this->stop('msg_not_permitted_download');
+			}
 			$logged_info = Context::get('logged_info');
 			if($logged_info->is_admin != 'Y')
 			{
@@ -397,7 +402,9 @@ 
 							break;
 						}
 					}
-					if(!$is_permitted) return $this->stop('msg_not_permitted_download');
+					if(!$is_permitted) {
+						return $this->stop('msg_not_permitted_download');
+					}
 				}
 			}
 		}
@@ -560,7 +567,9 @@ 
 				'allow_trackback', 'notify_message', 'status', 'comment_status');
 		$this->columnList = array_intersect($configColumList, $tableColumnList);
 
-		if(in_array('summary', $configColumList)) array_push($this->columnList, 'content');
+		if(in_array('summary', $configColumList)) {
+			array_push($this->columnList, 'content');
+		}
 
 		// default column list add
 		$defaultColumn = array('document_srl', 'module_srl', 'category_srl', 'lang_code', 'member_srl', 'last_update', 'comment_count', 'trackback_count', 'uploaded_count', 'status', 'regdate', 'title_bold', 'title_color');
@@ -654,8 +663,7 @@ 
 			{
 				$logged_info = Context::get('logged_info');
 				$group_srls = array_keys($logged_info->group_list);
-			}
-			else
+			} else
 			{
 				$group_srls = array();
 			}
@@ -672,10 +680,14 @@ 
 					{
 						$category_group_srls = explode(',',$category->group_srls);
 						$is_granted = FALSE;
-						if(count(array_intersect($group_srls, $category_group_srls))) $is_granted = TRUE;
+						if(count(array_intersect($group_srls, $category_group_srls))) {
+							$is_granted = TRUE;
+						}
 
 					}
-					if($is_granted) $category_list[$category_srl] = $category;
+					if($is_granted) {
+						$category_list[$category_srl] = $category;
+					}
 				}
 			}
 			Context::set('category_list', $category_list);
@@ -686,7 +698,9 @@ 
 		$oDocument = $oDocumentModel->getDocument(0, $this->grant->manager);
 		$oDocument->setDocument($document_srl);
 
-		if($oDocument->get('module_srl') == $oDocument->get('member_srl')) $savedDoc = TRUE;
+		if($oDocument->get('module_srl') == $oDocument->get('member_srl')) {
+			$savedDoc = TRUE;
+		}
 		$oDocument->add('module_srl', $this->module_srl);
 
 		if($oDocument->isExists() && $this->module_info->protect_content=="Y" && $oDocument->get('comment_count')>0 && $this->grant->manager==false)
@@ -712,17 +726,20 @@ 
 				if( !$logged_info )
 				{
 					return $this->dispBoardMessage('msg_not_permitted');
-				}
-				else if (($oPointModel->getPoint($logged_info->member_srl) + $pointForInsert )< 0 )
+				} else if (($oPointModel->getPoint($logged_info->member_srl) + $pointForInsert )< 0 )
 				{
 					return $this->dispBoardMessage('msg_not_enough_point');
 				}
 			}
 		}
-		if(!$oDocument->get('status')) $oDocument->add('status', $oDocumentModel->getDefaultStatus());
+		if(!$oDocument->get('status')) {
+			$oDocument->add('status', $oDocumentModel->getDefaultStatus());
+		}
 
 		$statusList = $this->_getStatusNameList($oDocumentModel);
-		if(count($statusList) > 0) Context::set('status_list', $statusList);
+		if(count($statusList) > 0) {
+			Context::set('status_list', $statusList);
+		}
 
 		// get Document status config value
 		Context::set('document_srl',$document_srl);
@@ -733,13 +750,18 @@ 
 		$oDocumentController->addXmlJsFilter($this->module_info->module_srl);
 
 		// if the document exists, then setup extra variabels on context
-		if($oDocument->isExists() && !$savedDoc) Context::set('extra_keys', $oDocument->getExtraVars());
+		if($oDocument->isExists() && !$savedDoc) {
+			Context::set('extra_keys', $oDocument->getExtraVars());
+		}
 
 		/**
 		 * add JS filters
 		 **/
-		if(Context::get('logged_info')->is_admin=='Y') Context::addJsFilter($this->module_path.'tpl/filter', 'insert_admin.xml');
-		else Context::addJsFilter($this->module_path.'tpl/filter', 'insert.xml');
+		if(Context::get('logged_info')->is_admin=='Y') {
+			Context::addJsFilter($this->module_path.'tpl/filter', 'insert_admin.xml');
+		} else {
+			Context::addJsFilter($this->module_path.'tpl/filter', 'insert.xml');
+		}
 
 		$oSecurity = new Security();
 		$oSecurity->encodeHTML('category_list.text', 'category_list.title');
@@ -1054,7 +1076,9 @@ 
 	function dispBoardMessage($msg_code)
 	{
 		$msg = Context::getLang($msg_code);
-		if(!$msg) $msg = $msg_code;
+		if(!$msg) {
+			$msg = $msg_code;
+		}
 		Context::set('message', $msg);
 		$this->setTemplateFile('message');
 	}

Spacing +152 added lines, -152 removed lines

@@ -23,15 +23,15 @@ 
 		/**
 		 * setup the module general information
 		 **/
-		if($this->module_info->list_count)
+		if ($this->module_info->list_count)
 		{
 			$this->list_count = $this->module_info->list_count;
 		}
-		if($this->module_info->search_list_count)
+		if ($this->module_info->search_list_count)
 		{
 			$this->search_list_count = $this->module_info->search_list_count;
 		}
-		if($this->module_info->page_count)
+		if ($this->module_info->page_count)
 		{
 			$this->page_count = $this->module_info->page_count;
 		}
@@ -41,20 +41,20 @@ 
 		$oDocumentModel = getModel('document');
 
 		$statusList = $this->_getStatusNameList($oDocumentModel);
-		if(isset($statusList['SECRET']))
+		if (isset($statusList['SECRET']))
 		{
 			$this->module_info->secret = 'Y';
 		}
 
 		// use_category <=1.5.x, hide_category >=1.7.x
 		$count_category = count($oDocumentModel->getCategoryList($this->module_info->module_srl));
-		if($count_category)
+		if ($count_category)
 		{
-			if($this->module_info->hide_category)
+			if ($this->module_info->hide_category)
 			{
 				$this->module_info->use_category = ($this->module_info->hide_category == 'Y') ? 'N' : 'Y';
 			}
-			else if($this->module_info->use_category)
+			else if ($this->module_info->use_category)
 			{
 				$this->module_info->hide_category = ($this->module_info->use_category == 'Y') ? 'N' : 'Y';
 			}
@@ -74,10 +74,10 @@ 
 		 * check the consultation function, if the user is admin then swich off consultation function
 		 * if the user is not logged, then disppear write document/write comment./ view document
 		 **/
-		if($this->module_info->consultation == 'Y' && !$this->grant->manager && !$this->grant->consultation_read)
+		if ($this->module_info->consultation == 'Y' && !$this->grant->manager && !$this->grant->consultation_read)
 		{
 			$this->consultation = TRUE;
-			if(!Context::get('is_logged'))
+			if (!Context::get('is_logged'))
 			{
 				$this->grant->list = FALSE;
 				$this->grant->write_document = FALSE;
@@ -94,11 +94,11 @@ 
 		 * setup the template path based on the skin
 		 * the default skin is default
 		 **/
-		$template_path = sprintf("%sskins/%s/",$this->module_path, $this->module_info->skin);
-		if(!is_dir($template_path)||!$this->module_info->skin)
+		$template_path = sprintf("%sskins/%s/", $this->module_path, $this->module_info->skin);
+		if (!is_dir($template_path) || !$this->module_info->skin)
 		{
 			$this->module_info->skin = 'default';
-			$template_path = sprintf("%sskins/%s/",$this->module_path, $this->module_info->skin);
+			$template_path = sprintf("%sskins/%s/", $this->module_path, $this->module_info->skin);
 		}
 		$this->setTemplatePath($template_path);
 
@@ -114,7 +114,7 @@ 
 		 **/
 		if (is_array($extra_keys))
 		{
-			foreach($extra_keys as $val)
+			foreach ($extra_keys as $val)
 			{
 				$this->order_target[] = $val->eid;
 			}
@@ -127,9 +127,9 @@ 
 
 		// remove [document_srl]_cpage from get_vars
 		$args = Context::getRequestVars();
-		foreach($args as $name => $value)
+		foreach ($args as $name => $value)
 		{
-			if(preg_match('/[0-9]+_cpage/', $name))
+			if (preg_match('/[0-9]+_cpage/', $name))
 			{
 				Context::set($name, '', TRUE);
 				Context::set($name, $value);
@@ -145,7 +145,7 @@ 
 		/**
 		 * check the access grant (all the grant has been set by the module object)
 		 **/
-		if(!$this->grant->access || !$this->grant->list)
+		if (!$this->grant->access || !$this->grant->list)
 		{
 			return $this->dispBoardMessage('msg_not_permitted');
 		}
@@ -160,22 +160,22 @@ 
 		 * add extra vaiables to the search options
 		 **/
 		// use search options on the template (the search options key has been declared, based on the language selected)
-		foreach($this->search_option as $opt) $search_option[$opt] = Context::getLang($opt);
+		foreach ($this->search_option as $opt) $search_option[$opt] = Context::getLang($opt);
 		$extra_keys = Context::get('extra_keys');
-		if($extra_keys)
+		if ($extra_keys)
 		{
-			foreach($extra_keys as $key => $val)
+			foreach ($extra_keys as $key => $val)
 			{
-				if($val->search == 'Y') $search_option['extra_vars'.$val->idx] = $val->name;
+				if ($val->search == 'Y') $search_option['extra_vars'.$val->idx] = $val->name;
 			}
 		}
 		// remove a search option that is not public in member config
 		$memberConfig = getModel('module')->getModuleConfig('member');
-		foreach($memberConfig->signupForm as $signupFormElement)
+		foreach ($memberConfig->signupForm as $signupFormElement)
 		{
-			if(in_array($signupFormElement->title, $search_option))
+			if (in_array($signupFormElement->title, $search_option))
 			{
-				if($signupFormElement->isPublic == 'N')
+				if ($signupFormElement->isPublic == 'N')
 					unset($search_option[$signupFormElement->name]);
 			}
 		}
@@ -183,7 +183,7 @@ 
 
 		$oDocumentModel = getModel('document');
 		$statusNameList = $this->_getStatusNameList($oDocumentModel);
-		if(count($statusNameList) > 0)
+		if (count($statusNameList) > 0)
 		{
 			Context::set('status_list', $statusNameList);
 		}
@@ -194,7 +194,7 @@ 
 		// list config, columnList setting
 		$oBoardModel = getModel('board');
 		$this->listConfig = $oBoardModel->getListConfig($this->module_info->module_srl);
-		if(!$this->listConfig) $this->listConfig = array();
+		if (!$this->listConfig) $this->listConfig = array();
 		$this->_makeListColumnList();
 
 		// display the notice list
@@ -218,12 +218,12 @@ 
 	/**
 	 * @brief display the category list
 	 **/
-	function dispBoardCategoryList(){
+	function dispBoardCategoryList() {
 		// check if the use_category option is enabled
-		if($this->module_info->use_category=='Y')
+		if ($this->module_info->use_category == 'Y')
 		{
 			// check the grant
-			if(!$this->grant->list)
+			if (!$this->grant->list)
 			{
 				Context::set('category_list', array());
 				return;
@@ -240,7 +240,7 @@ 
 	/**
 	 * @brief display the board conent view
 	 **/
-	function dispBoardContentView(){
+	function dispBoardContentView() {
 		// get the variable value
 		$document_srl = Context::get('document_srl');
 		$page = Context::get('page');
@@ -251,36 +251,36 @@ 
 		/**
 		 * if the document exists, then get the document information
 		 **/
-		if($document_srl)
+		if ($document_srl)
 		{
 			$oDocument = $oDocumentModel->getDocument($document_srl, false, true);
 
 			// if the document is existed
-			if($oDocument->isExists())
+			if ($oDocument->isExists())
 			{
 				// if the module srl is not consistent
-				if($oDocument->get('module_srl')!=$this->module_info->module_srl )
+				if ($oDocument->get('module_srl') != $this->module_info->module_srl)
 				{
 					return $this->stop('msg_invalid_request');
 				}
 
 				// check the manage grant
-				if($this->grant->manager) $oDocument->setGrant();
+				if ($this->grant->manager) $oDocument->setGrant();
 
 				// if the consultation function is enabled, and the document is not a notice
-				if($this->consultation && !$oDocument->isNotice())
+				if ($this->consultation && !$oDocument->isNotice())
 				{
 					$logged_info = Context::get('logged_info');
-					if(abs($oDocument->get('member_srl')) != $logged_info->member_srl)
+					if (abs($oDocument->get('member_srl')) != $logged_info->member_srl)
 					{
 						$oDocument = $oDocumentModel->getDocument(0);
 					}
 				}
 
 				// if the document is TEMP saved, check Grant
-				if($oDocument->getStatus() == 'TEMP')
+				if ($oDocument->getStatus() == 'TEMP')
 				{
-					if(!$oDocument->isGranted())
+					if (!$oDocument->isGranted())
 					{
 						$oDocument = $oDocumentModel->getDocument(0);
 					}
@@ -290,7 +290,7 @@ 
 			else
 			{
 				// if the document is not existed, then alert a warning message
-				Context::set('document_srl','',true);
+				Context::set('document_srl', '', true);
 				$this->alertMessage('msg_not_founded');
 			}
 
@@ -306,12 +306,12 @@ 
 		/**
 		 *check the document view grant
 		 **/
-		if($oDocument->isExists())
+		if ($oDocument->isExists())
 		{
-			if(!$this->grant->view && !$oDocument->isGranted())
+			if (!$this->grant->view && !$oDocument->isGranted())
 			{
 				$oDocument = $oDocumentModel->getDocument(0);
-				Context::set('document_srl','',true);
+				Context::set('document_srl', '', true);
 				$this->alertMessage('msg_not_permitted');
 			}
 			else
@@ -320,15 +320,15 @@ 
 				Context::addBrowserTitle($oDocument->getTitleText());
 
 				// update the document view count (if the document is not secret)
-				if(!$oDocument->isSecret() || $oDocument->isGranted())
+				if (!$oDocument->isSecret() || $oDocument->isGranted())
 				{
 					$oDocument->updateReadedCount();
 				}
 
 				// disappear the document if it is secret
-				if($oDocument->isSecret() && !$oDocument->isGranted())
+				if ($oDocument->isSecret() && !$oDocument->isGranted())
 				{
-					$oDocument->add('content',Context::getLang('thisissecret'));
+					$oDocument->add('content', Context::getLang('thisissecret'));
 				}
 			}
 		}
@@ -348,11 +348,11 @@ 
 	/**
 	 * @brief  display the document file list (can be used by API)
 	 **/
-	function dispBoardContentFileList(){
+	function dispBoardContentFileList() {
 		/**
 		 * check the access grant (all the grant has been set by the module object)
 		 **/
-		if(!$this->grant->access)
+		if (!$this->grant->access)
 		{
 			return $this->dispBoardMessage('msg_not_permitted');
 		}
@@ -363,41 +363,41 @@ 
 		// Check if a permission for file download is granted
 		// Get configurations (using module model object)
 		$oModuleModel = getModel('module');
-		$file_module_config = $oModuleModel->getModulePartConfig('file',$this->module_srl);
+		$file_module_config = $oModuleModel->getModulePartConfig('file', $this->module_srl);
 		
 		$downloadGrantCount = 0;
-		if(is_array($file_module_config->download_grant))
+		if (is_array($file_module_config->download_grant))
 		{
-			foreach($file_module_config->download_grant AS $value)
-				if($value) $downloadGrantCount++;
+			foreach ($file_module_config->download_grant AS $value)
+				if ($value) $downloadGrantCount++;
 		}
 
-		if(is_array($file_module_config->download_grant) && $downloadGrantCount>0)
+		if (is_array($file_module_config->download_grant) && $downloadGrantCount > 0)
 		{
-			if(!Context::get('is_logged')) return $this->stop('msg_not_permitted_download');
+			if (!Context::get('is_logged')) return $this->stop('msg_not_permitted_download');
 			$logged_info = Context::get('logged_info');
-			if($logged_info->is_admin != 'Y')
+			if ($logged_info->is_admin != 'Y')
 			{
-				$oModuleModel =& getModel('module');
+				$oModuleModel = & getModel('module');
 				$columnList = array('module_srl', 'site_srl');
 				$module_info = $oModuleModel->getModuleInfoByModuleSrl($this->module_srl, $columnList);
 
-				if(!$oModuleModel->isSiteAdmin($logged_info, $module_info->site_srl))
+				if (!$oModuleModel->isSiteAdmin($logged_info, $module_info->site_srl))
 				{
-					$oMemberModel =& getModel('member');
+					$oMemberModel = & getModel('member');
 					$member_groups = $oMemberModel->getMemberGroups($logged_info->member_srl, $module_info->site_srl);
 
 					$is_permitted = false;
-					for($i=0;$i<count($file_module_config->download_grant);$i++)
+					for ($i = 0; $i < count($file_module_config->download_grant); $i++)
 					{
 						$group_srl = $file_module_config->download_grant[$i];
-						if($member_groups[$group_srl])
+						if ($member_groups[$group_srl])
 						{
 							$is_permitted = true;
 							break;
 						}
 					}
-					if(!$is_permitted) return $this->stop('msg_not_permitted_download');
+					if (!$is_permitted) return $this->stop('msg_not_permitted_download');
 				}
 			}
 		}
@@ -406,7 +406,7 @@ 
 		$document_srl = Context::get('document_srl');
 		$oDocument = $oDocumentModel->getDocument($document_srl);
 		Context::set('oDocument', $oDocument);
-		Context::set('file_list',$oDocument->getUploadedFiles());
+		Context::set('file_list', $oDocument->getUploadedFiles());
 
 		$oSecurity = new Security();
 		$oSecurity->encodeHTML('file_list..source_filename');
@@ -415,7 +415,7 @@ 
 	/**
 	 * @brief display the document comment list (can be used by API)
 	 **/
-	function dispBoardContentCommentList(){
+	function dispBoardContentCommentList() {
 		// check document view grant
 		$this->dispBoardContentView();
 
@@ -425,26 +425,26 @@ 
 		$comment_list = $oDocument->getComments();
 
 		// setup the comment list
-		if(is_array($comment_list))
+		if (is_array($comment_list))
 		{
-			foreach($comment_list as $key => $val)
+			foreach ($comment_list as $key => $val)
 			{
-				if(!$val->isAccessible())
+				if (!$val->isAccessible())
 				{
-					$val->add('content',Context::getLang('thisissecret'));
+					$val->add('content', Context::getLang('thisissecret'));
 				}
 			}
 		}
-		Context::set('comment_list',$comment_list);
+		Context::set('comment_list', $comment_list);
 
 	}
 
 	/**
 	 * @brief display notice list (can be used by API)
 	 **/
-	function dispBoardNoticeList(){
+	function dispBoardNoticeList() {
 		// check the grant
-		if(!$this->grant->list)
+		if (!$this->grant->list)
 		{
 			Context::set('notice_list', array());
 			return;
@@ -460,15 +460,15 @@ 
 	/**
 	 * @brief display board content list
 	 **/
-	function dispBoardContentList(){
+	function dispBoardContentList() {
 		// check the grant
-		if(!$this->grant->list)
+		if (!$this->grant->list)
 		{
 			Context::set('document_list', array());
 			Context::set('total_count', 0);
 			Context::set('total_page', 1);
 			Context::set('page', 1);
-			Context::set('page_navigation', new PageHandler(0,0,1,10));
+			Context::set('page_navigation', new PageHandler(0, 0, 1, 10));
 			return;
 		}
 
@@ -486,17 +486,17 @@ 
 		$args->search_keyword = Context::get('search_keyword');
 
 		$search_option = Context::get('search_option');
-		if($search_option==FALSE)
+		if ($search_option == FALSE)
 		{
 			$search_option = $this->search_option;
 		}
-		if(isset($search_option[$args->search_target])==FALSE)
+		if (isset($search_option[$args->search_target]) == FALSE)
 		{
 			$args->search_target = '';
 		}
 
 		// if the category is enabled, then get the category
-		if($this->module_info->use_category=='Y')
+		if ($this->module_info->use_category == 'Y')
 		{
 			$args->category_srl = Context::get('category');
 		}
@@ -504,21 +504,21 @@ 
 		// setup the sort index and order index
 		$args->sort_index = Context::get('sort_index');
 		$args->order_type = Context::get('order_type');
-		if(!in_array($args->sort_index, $this->order_target))
+		if (!in_array($args->sort_index, $this->order_target))
 		{
-			$args->sort_index = $this->module_info->order_target?$this->module_info->order_target:'list_order';
+			$args->sort_index = $this->module_info->order_target ? $this->module_info->order_target : 'list_order';
 		}
-		if(!in_array($args->order_type, array('asc','desc')))
+		if (!in_array($args->order_type, array('asc', 'desc')))
 		{
-			$args->order_type = $this->module_info->order_type?$this->module_info->order_type:'asc';
+			$args->order_type = $this->module_info->order_type ? $this->module_info->order_type : 'asc';
 		}
 
 		// set the current page of documents
 		$document_srl = Context::get('document_srl');
-		if(!$args->page && $document_srl)
+		if (!$args->page && $document_srl)
 		{
 			$oDocument = $oDocumentModel->getDocument($document_srl);
-			if($oDocument->isExists() && !$oDocument->isNotice())
+			if ($oDocument->isExists() && !$oDocument->isNotice())
 			{
 				$page = $oDocumentModel->getDocumentPage($oDocument, $args);
 				Context::set('page', $page);
@@ -527,21 +527,21 @@ 
 		}
 
 		// setup the list count to be serach list count, if the category or search keyword has been set
-		if($args->category_srl || $args->search_keyword)
+		if ($args->category_srl || $args->search_keyword)
 		{
 			$args->list_count = $this->search_list_count;
 		}
 
 		// if the consultation function is enabled,  the get the logged user information
-		if($this->consultation)
+		if ($this->consultation)
 		{
 			$logged_info = Context::get('logged_info');
 			$args->member_srl = $logged_info->member_srl;
 
-			if($this->module_info->use_anonymous === 'Y')
+			if ($this->module_info->use_anonymous === 'Y')
 			{
 				unset($args->member_srl);
-				$args->member_srls = $logged_info->member_srl . ',' . $logged_info->member_srl * -1;
+				$args->member_srls = $logged_info->member_srl.','.$logged_info->member_srl * -1;
 			}
 		}
 
@@ -567,18 +567,18 @@ 
 				'allow_trackback', 'notify_message', 'status', 'comment_status');
 		$this->columnList = array_intersect($configColumList, $tableColumnList);
 
-		if(in_array('summary', $configColumList)) array_push($this->columnList, 'content');
+		if (in_array('summary', $configColumList)) array_push($this->columnList, 'content');
 
 		// default column list add
 		$defaultColumn = array('document_srl', 'module_srl', 'category_srl', 'lang_code', 'member_srl', 'last_update', 'comment_count', 'trackback_count', 'uploaded_count', 'status', 'regdate', 'title_bold', 'title_color');
 
 		//TODO guestbook, blog style supports legacy codes.
-		if($this->module_info->skin == 'xe_guestbook' || $this->module_info->default_style == 'blog')
+		if ($this->module_info->skin == 'xe_guestbook' || $this->module_info->default_style == 'blog')
 		{
 			$defaultColumn = $tableColumnList;
 		}
 
-		if (in_array('last_post', $configColumList)){
+		if (in_array('last_post', $configColumList)) {
 			array_push($this->columnList, 'last_updater');
 		}
 
@@ -590,9 +590,9 @@ 
 		$this->columnList = array_unique(array_merge($this->columnList, $defaultColumn));
 
 		// add table name
-		foreach($this->columnList as $no => $value)
+		foreach ($this->columnList as $no => $value)
 		{
-			$this->columnList[$no] = 'documents.' . $value;
+			$this->columnList[$no] = 'documents.'.$value;
 		}
 	}
 
@@ -602,7 +602,7 @@ 
 	function dispBoardTagList()
 	{
 		// check if there is not grant fot view list, then alert an warning message
-		if(!$this->grant->list)
+		if (!$this->grant->list)
 		{
 			return $this->dispBoardMessage('msg_not_permitted');
 		}
@@ -616,14 +616,14 @@ 
 		$output = $oTagModel->getTagList($obj);
 
 		// automatically order
-		if(count($output->data))
+		if (count($output->data))
 		{
 			$numbers = array_keys($output->data);
 			shuffle($numbers);
 
-			if(count($output->data))
+			if (count($output->data))
 			{
-				foreach($numbers as $k => $v)
+				foreach ($numbers as $k => $v)
 				{
 					$tag_list[] = $output->data[$v];
 				}
@@ -644,7 +644,7 @@ 
 	function dispBoardWrite()
 	{
 		// check grant
-		if(!$this->grant->write_document)
+		if (!$this->grant->write_document)
 		{
 			return $this->dispBoardMessage('msg_not_permitted');
 		}
@@ -654,10 +654,10 @@ 
 		/**
 		 * check if the category option is enabled not not
 		 **/
-		if($this->module_info->use_category=='Y')
+		if ($this->module_info->use_category == 'Y')
 		{
 			// get the user group information
-			if(Context::get('is_logged'))
+			if (Context::get('is_logged'))
 			{
 				$logged_info = Context::get('logged_info');
 				$group_srls = array_keys($logged_info->group_list);
@@ -670,19 +670,19 @@ 
 
 			// check the grant after obtained the category list
 			$normal_category_list = $oDocumentModel->getCategoryList($this->module_srl);
-			if(count($normal_category_list))
+			if (count($normal_category_list))
 			{
-				foreach($normal_category_list as $category_srl => $category)
+				foreach ($normal_category_list as $category_srl => $category)
 				{
 					$is_granted = TRUE;
-					if($category->group_srls)
+					if ($category->group_srls)
 					{
-						$category_group_srls = explode(',',$category->group_srls);
+						$category_group_srls = explode(',', $category->group_srls);
 						$is_granted = FALSE;
-						if(count(array_intersect($group_srls, $category_group_srls))) $is_granted = TRUE;
+						if (count(array_intersect($group_srls, $category_group_srls))) $is_granted = TRUE;
 
 					}
-					if($is_granted) $category_list[$category_srl] = $category;
+					if ($is_granted) $category_list[$category_srl] = $category;
 				}
 			}
 			Context::set('category_list', $category_list);
@@ -693,46 +693,46 @@ 
 		$oDocument = $oDocumentModel->getDocument(0, $this->grant->manager);
 		$oDocument->setDocument($document_srl);
 
-		if($oDocument->get('module_srl') == $oDocument->get('member_srl')) $savedDoc = TRUE;
+		if ($oDocument->get('module_srl') == $oDocument->get('member_srl')) $savedDoc = TRUE;
 		$oDocument->add('module_srl', $this->module_srl);
 
-		if($oDocument->isExists() && $this->module_info->protect_content=="Y" && $oDocument->get('comment_count')>0 && $this->grant->manager==false)
+		if ($oDocument->isExists() && $this->module_info->protect_content == "Y" && $oDocument->get('comment_count') > 0 && $this->grant->manager == false)
 		{
 			return new BaseObject(-1, 'msg_protect_content');
 		}
 
 		// if the document is not granted, then back to the password input form
 		$oModuleModel = getModel('module');
-		if($oDocument->isExists()&&!$oDocument->isGranted())
+		if ($oDocument->isExists() && !$oDocument->isGranted())
 		{
 			return $this->setTemplateFile('input_password_form');
 		}
 
-		if(!$oDocument->isExists())
+		if (!$oDocument->isExists())
 		{
-			$point_config = $oModuleModel->getModulePartConfig('point',$this->module_srl);
+			$point_config = $oModuleModel->getModulePartConfig('point', $this->module_srl);
 			$logged_info = Context::get('logged_info');
 			$oPointModel = getModel('point');
 			$pointForInsert = $point_config["insert_document"];
-			if($pointForInsert < 0)
+			if ($pointForInsert < 0)
 			{
-				if( !$logged_info )
+				if (!$logged_info)
 				{
 					return $this->dispBoardMessage('msg_not_permitted');
 				}
-				else if (($oPointModel->getPoint($logged_info->member_srl) + $pointForInsert )< 0 )
+				else if (($oPointModel->getPoint($logged_info->member_srl) + $pointForInsert) < 0)
 				{
 					return $this->dispBoardMessage('msg_not_enough_point');
 				}
 			}
 		}
-		if(!$oDocument->get('status')) $oDocument->add('status', $oDocumentModel->getDefaultStatus());
+		if (!$oDocument->get('status')) $oDocument->add('status', $oDocumentModel->getDefaultStatus());
 
 		$statusList = $this->_getStatusNameList($oDocumentModel);
-		if(count($statusList) > 0) Context::set('status_list', $statusList);
+		if (count($statusList) > 0) Context::set('status_list', $statusList);
 
 		// get Document status config value
-		Context::set('document_srl',$document_srl);
+		Context::set('document_srl', $document_srl);
 		Context::set('oDocument', $oDocument);
 
 		// apply xml_js_filter on header
@@ -740,12 +740,12 @@ 
 		$oDocumentController->addXmlJsFilter($this->module_info->module_srl);
 
 		// if the document exists, then setup extra variabels on context
-		if($oDocument->isExists() && !$savedDoc) Context::set('extra_keys', $oDocument->getExtraVars());
+		if ($oDocument->isExists() && !$savedDoc) Context::set('extra_keys', $oDocument->getExtraVars());
 
 		/**
 		 * add JS filters
 		 **/
-		if(Context::get('logged_info')->is_admin=='Y') Context::addJsFilter($this->module_path.'tpl/filter', 'insert_admin.xml');
+		if (Context::get('logged_info')->is_admin == 'Y') Context::addJsFilter($this->module_path.'tpl/filter', 'insert_admin.xml');
 		else Context::addJsFilter($this->module_path.'tpl/filter', 'insert.xml');
 
 		$oSecurity = new Security();
@@ -757,14 +757,14 @@ 
 	function _getStatusNameList(&$oDocumentModel)
 	{
 		$resultList = array();
-		if(!empty($this->module_info->use_status))
+		if (!empty($this->module_info->use_status))
 		{
 			$statusNameList = $oDocumentModel->getStatusNameList();
 			$statusList = explode('|@|', $this->module_info->use_status);
 
-			if(is_array($statusList))
+			if (is_array($statusList))
 			{
-				foreach($statusList as $key => $value)
+				foreach ($statusList as $key => $value)
 				{
 					$resultList[$value] = $statusNameList[$value];
 				}
@@ -779,7 +779,7 @@ 
 	function dispBoardDelete()
 	{
 		// check grant
-		if(!$this->grant->write_document)
+		if (!$this->grant->write_document)
 		{
 			return $this->dispBoardMessage('msg_not_permitted');
 		}
@@ -788,30 +788,30 @@ 
 		$document_srl = Context::get('document_srl');
 
 		// if document exists, get the document information
-		if($document_srl)
+		if ($document_srl)
 		{
 			$oDocumentModel = getModel('document');
 			$oDocument = $oDocumentModel->getDocument($document_srl);
 		}
 
 		// if the document is not existed, then back to the board content page
-		if(!$oDocument || !$oDocument->isExists())
+		if (!$oDocument || !$oDocument->isExists())
 		{
 			return $this->dispBoardContent();
 		}
 
 		// if the document is not granted, then back to the password input form
-		if(!$oDocument->isGranted())
+		if (!$oDocument->isGranted())
 		{
 			return $this->setTemplateFile('input_password_form');
 		}
 
-		if($this->module_info->protect_content=="Y" && $oDocument->get('comment_count')>0 && $this->grant->manager==false)
+		if ($this->module_info->protect_content == "Y" && $oDocument->get('comment_count') > 0 && $this->grant->manager == false)
 		{
 			return $this->dispBoardMessage('msg_protect_content');
 		}
 
-		Context::set('oDocument',$oDocument);
+		Context::set('oDocument', $oDocument);
 
 		/**
 		 * add JS filters
@@ -829,7 +829,7 @@ 
 		$document_srl = Context::get('document_srl');
 
 		// check grant
-		if(!$this->grant->write_comment)
+		if (!$this->grant->write_comment)
 		{
 			return $this->dispBoardMessage('msg_not_permitted');
 		}
@@ -837,13 +837,13 @@ 
 		// get the document information
 		$oDocumentModel = getModel('document');
 		$oDocument = $oDocumentModel->getDocument($document_srl);
-		if(!$oDocument->isExists())
+		if (!$oDocument->isExists())
 		{
 			return $this->dispBoardMessage('msg_invalid_request');
 		}
 
 		// Check allow comment
-		if(!$oDocument->allowComment())
+		if (!$oDocument->allowComment())
 		{
 			return $this->dispBoardMessage('msg_not_allow_comment');
 		}
@@ -855,9 +855,9 @@ 
 		$oComment->add('module_srl', $this->module_srl);
 
 		// setup document variables on context
-		Context::set('oDocument',$oDocument);
-		Context::set('oSourceComment',$oSourceComment);
-		Context::set('oComment',$oComment);
+		Context::set('oDocument', $oDocument);
+		Context::set('oSourceComment', $oSourceComment);
+		Context::set('oComment', $oComment);
 
 		/**
 		 * add JS filter
@@ -873,7 +873,7 @@ 
 	function dispBoardReplyComment()
 	{
 		// check grant
-		if(!$this->grant->write_comment)
+		if (!$this->grant->write_comment)
 		{
 			return $this->dispBoardMessage('msg_not_permitted');
 		}
@@ -882,7 +882,7 @@ 
 		$parent_srl = Context::get('comment_srl');
 
 		// if the parent comment is not existed
-		if(!$parent_srl)
+		if (!$parent_srl)
 		{
 			return new BaseObject(-1, 'msg_invalid_request');
 		}
@@ -892,11 +892,11 @@ 
 		$oSourceComment = $oCommentModel->getComment($parent_srl, $this->grant->manager);
 
 		// if the comment is not existed, opoup an error message
-		if(!$oSourceComment->isExists())
+		if (!$oSourceComment->isExists())
 		{
 			return $this->dispBoardMessage('msg_invalid_request');
 		}
-		if(Context::get('document_srl') && $oSourceComment->get('document_srl') != Context::get('document_srl'))
+		if (Context::get('document_srl') && $oSourceComment->get('document_srl') != Context::get('document_srl'))
 		{
 			return $this->dispBoardMessage('msg_invalid_request');
 		}
@@ -904,7 +904,7 @@ 
 		// Check allow comment
 		$oDocumentModel = getModel('document');
 		$oDocument = $oDocumentModel->getDocument($oSourceComment->get('document_srl'));
-		if(!$oDocument->allowComment())
+		if (!$oDocument->allowComment())
 		{
 			return $this->dispBoardMessage('msg_not_allow_comment');
 		}
@@ -915,9 +915,9 @@ 
 		$oComment->add('document_srl', $oSourceComment->get('document_srl'));
 
 		// setup comment variables
-		Context::set('oSourceComment',$oSourceComment);
-		Context::set('oComment',$oComment);
-		Context::set('module_srl',$this->module_info->module_srl);
+		Context::set('oSourceComment', $oSourceComment);
+		Context::set('oComment', $oComment);
+		Context::set('module_srl', $this->module_info->module_srl);
 
 		/**
 		 * add JS filters
@@ -933,7 +933,7 @@ 
 	function dispBoardModifyComment()
 	{
 		// check grant
-		if(!$this->grant->write_comment)
+		if (!$this->grant->write_comment)
 		{
 			return $this->dispBoardMessage('msg_not_permitted');
 		}
@@ -943,7 +943,7 @@ 
 		$comment_srl = Context::get('comment_srl');
 
 		// if the comment is not existed
-		if(!$comment_srl)
+		if (!$comment_srl)
 		{
 			return new BaseObject(-1, 'msg_invalid_request');
 		}
@@ -953,13 +953,13 @@ 
 		$oComment = $oCommentModel->getComment($comment_srl, $this->grant->manager);
 
 		// if the comment is not exited, alert an error message
-		if(!$oComment->isExists())
+		if (!$oComment->isExists())
 		{
 			return $this->dispBoardMessage('msg_invalid_request');
 		}
 
 		// if the comment is not granted, then back to the password input form
-		if(!$oComment->isGranted())
+		if (!$oComment->isGranted())
 		{
 			return $this->setTemplateFile('input_password_form');
 		}
@@ -982,7 +982,7 @@ 
 	function dispBoardDeleteComment()
 	{
 		// check grant
-		if(!$this->grant->write_comment)
+		if (!$this->grant->write_comment)
 		{
 			return $this->dispBoardMessage('msg_not_permitted');
 		}
@@ -991,25 +991,25 @@ 
 		$comment_srl = Context::get('comment_srl');
 
 		// if the comment exists, then get the comment information
-		if($comment_srl)
+		if ($comment_srl)
 		{
 			$oCommentModel = getModel('comment');
 			$oComment = $oCommentModel->getComment($comment_srl, $this->grant->manager);
 		}
 
 		// if the comment is not existed, then back to the board content page
-		if(!$oComment->isExists() )
+		if (!$oComment->isExists())
 		{
 			return $this->dispBoardContent();
 		}
 
 		// if the comment is not granted, then back to the password input form
-		if(!$oComment->isGranted())
+		if (!$oComment->isGranted())
 		{
 			return $this->setTemplateFile('input_password_form');
 		}
 
-		Context::set('oComment',$oComment);
+		Context::set('oComment', $oComment);
 
 		/**
 		 * add JS filters
@@ -1026,7 +1026,7 @@ 
 	{
 		$oTrackbackModel = getModel('trackback');
 
-		if(!$oTrackbackModel)
+		if (!$oTrackbackModel)
 		{
 			return;
 		}
@@ -1040,7 +1040,7 @@ 
 		$trackback = $output->data;
 
 		// if no trackback, then display the board content
-		if(!$trackback)
+		if (!$trackback)
 		{
 			return $this->dispBoardContent();
 		}
@@ -1061,7 +1061,7 @@ 
 	function dispBoardMessage($msg_code)
 	{
 		$msg = Context::getLang($msg_code);
-		if(!$msg) $msg = $msg_code;
+		if (!$msg) $msg = $msg_code;
 		Context::set('message', $msg);
 		$this->setTemplateFile('message');
 	}
@@ -1072,8 +1072,8 @@ 
 	 **/
 	function alertMessage($message)
 	{
-		$script =  sprintf('<script> jQuery(function(){ alert("%s"); } );</script>', Context::getLang($message));
-		Context::addHtmlFooter( $script );
+		$script = sprintf('<script> jQuery(function(){ alert("%s"); } );</script>', Context::getLang($message));
+		Context::addHtmlFooter($script);
 	}
 
 }