AyeCode / invoicing

includes/class-wpinv-session.php

Doc Comments +2 added lines, -4 removed lines

@@ -164,8 +164,7 @@ 
 	 * @since 1.0.0
 	 *
 	 * @param string|array $key Session key.
-	 * @param integer $value Session variable.
-	 * @return string Session variable.
+	 * @return boolean Session variable.
 	 */
 	public function un_set( $key ) {
 		if ( empty( $key ) ) {
@@ -201,8 +200,7 @@ 
 	 * @since 1.0.0
 	 *
 	 * @param string $key Session key.
-	 * @param integer $value Session variable.
-	 * @return string Session variable.
+	 * @return boolean Session variable.
 	 */
 	public function is_set( $key ) {
 		$key = sanitize_key( $key );

Indentation +274 added lines, -274 removed lines

@@ -8,291 +8,291 @@
  * @since 1.0.0
  */
 class WPInv_Session {
-	/**
-	 * Holds our session data.
-	 *
-	 * @var array
-	 * @access private
-	 * @since 1.0.0
-	 */
-	private $session;
-
-	/**
-	 * Whether to use PHP $_SESSION or WP_Session.
-	 *
-	 * @var bool
-	 * @access private
-	 * @since 1.0.0
-	 */
-	private $use_php_sessions = false;
-
-	/**
-	 * Session index prefix.
-	 *
-	 * @var string
-	 * @access private
-	 * @since 1.0.0
-	 */
-	private $prefix = '';
-
-	/**
-	 * Get things started.
-	 *
-	 * Defines our WP_Session constants, includes the necessary libraries and
-	 * retrieves the WP Session instance.
-	 *
-	 * @since 1.0.0
-	 */
-	public function __construct() {
-		$this->use_php_sessions = $this->use_php_sessions();
-
-		if ( $this->use_php_sessions ) {
-			if ( is_multisite() ) {
-				$this->prefix = '_' . get_current_blog_id();
-			}
-
-			// Use PHP SESSION (must be enabled via the WPINV_USE_PHP_SESSIONS constant)
-			add_action( 'init', array( $this, 'maybe_start_session' ), -2 );
-		} else {
-			// Use WP_Session (default)
-			if ( !defined( 'WP_SESSION_COOKIE' ) ) {
-				define( 'WP_SESSION_COOKIE', 'wpinv_wp_session' );
-			}
-
-			if ( !class_exists( 'Recursive_ArrayAccess' ) ) {
-				require_once WPINV_PLUGIN_DIR . 'includes/libraries/wp-session/class-recursive-arrayaccess.php';
-			}
+    /**
+     * Holds our session data.
+     *
+     * @var array
+     * @access private
+     * @since 1.0.0
+     */
+    private $session;
+
+    /**
+     * Whether to use PHP $_SESSION or WP_Session.
+     *
+     * @var bool
+     * @access private
+     * @since 1.0.0
+     */
+    private $use_php_sessions = false;
+
+    /**
+     * Session index prefix.
+     *
+     * @var string
+     * @access private
+     * @since 1.0.0
+     */
+    private $prefix = '';
+
+    /**
+     * Get things started.
+     *
+     * Defines our WP_Session constants, includes the necessary libraries and
+     * retrieves the WP Session instance.
+     *
+     * @since 1.0.0
+     */
+    public function __construct() {
+        $this->use_php_sessions = $this->use_php_sessions();
+
+        if ( $this->use_php_sessions ) {
+            if ( is_multisite() ) {
+                $this->prefix = '_' . get_current_blog_id();
+            }
+
+            // Use PHP SESSION (must be enabled via the WPINV_USE_PHP_SESSIONS constant)
+            add_action( 'init', array( $this, 'maybe_start_session' ), -2 );
+        } else {
+            // Use WP_Session (default)
+            if ( !defined( 'WP_SESSION_COOKIE' ) ) {
+                define( 'WP_SESSION_COOKIE', 'wpinv_wp_session' );
+            }
+
+            if ( !class_exists( 'Recursive_ArrayAccess' ) ) {
+                require_once WPINV_PLUGIN_DIR . 'includes/libraries/wp-session/class-recursive-arrayaccess.php';
+            }
             
-			if ( !class_exists( 'WP_Session_Utils' ) ) {
-				require_once WPINV_PLUGIN_DIR . 'includes/libraries/wp-session/class-wp-session-utils.php';
-			}
+            if ( !class_exists( 'WP_Session_Utils' ) ) {
+                require_once WPINV_PLUGIN_DIR . 'includes/libraries/wp-session/class-wp-session-utils.php';
+            }
             
-			if ( defined( 'WP_CLI' ) && WP_CLI && !class_exists( 'WP_Session_Command' ) ) {
-				require_once WPINV_PLUGIN_DIR . 'includes/libraries/wp-session/wp-cli.php';
-			}
-
-			if ( !class_exists( 'WP_Session' ) ) {
-				require_once WPINV_PLUGIN_DIR . 'includes/libraries/wp-session/class-wp-session.php';
-				require_once WPINV_PLUGIN_DIR . 'includes/libraries/wp-session/wp-session.php';
-			}
-
-			add_filter( 'wp_session_expiration_variant', array( $this, 'set_expiration_variant_time' ), 99999 );
-			add_filter( 'wp_session_expiration', array( $this, 'set_expiration_time' ), 99999 );
-		}
-
-		if ( empty( $this->session ) && ! $this->use_php_sessions ) {
-			add_action( 'plugins_loaded', array( $this, 'init' ), -1 );
-		} else {
-			add_action( 'init', array( $this, 'init' ), -1 );
-		}
-	}
-
-	/**
-	 * Setup the WP_Session instance.
-	 *
-	 * @access public
-	 * @since 1.0.0
-	 * @return void
-	 */
-	public function init() {
-		if ( $this->use_php_sessions ) {
-			$this->session = isset( $_SESSION['wpi' . $this->prefix ] ) && is_array( $_SESSION['wpi' . $this->prefix ] ) ? $_SESSION['wpi' . $this->prefix ] : array();
-		} else {
-			$this->session = WP_Session::get_instance();
-		}
-
-		return $this->session;
-	}
-
-	/**
-	 * Retrieve session ID.
-	 *
-	 * @access public
-	 * @since 1.0.0
-	 * @return string Session ID
-	 */
-	public function get_id() {
-		if ( $this->use_php_sessions ) {
-			$session_id = !empty( $_SESSION ) && function_exists( 'session_id' ) ? session_id() : NULL;
-		} else {
-			$session_id = !empty( $this->session ) && isset( $this->session->session_id ) ? $this->session->session_id : NULL;
-		}
-		return $session_id;
-	}
-
-	/**
-	 * Retrieve a session variable.
-	 *
-	 * @access public
-	 * @since 1.0.0
-	 * @param string $key Session key
-	 * @return string Session variable
-	 */
-	public function get( $key ) {
-		$key = sanitize_key( $key );
-		return isset( $this->session[ $key ] ) ? maybe_unserialize( $this->session[ $key ] ) : false;
-	}
-
-	/**
-	 * Set a session variable
-	 *
-	 * @since 1.0.0
-	 *
-	 * @param string $key Session key
-	 * @param integer $value Session variable
-	 * @return string Session variable
-	 */
-	public function set( $key, $value ) {
-		$key = sanitize_key( $key );
-
-		if ( is_array( $value ) ) {
-			$this->session[ $key ] = maybe_serialize( $value );
-		} else {
-			$this->session[ $key ] = $value;
-		}
-
-		if ( $this->use_php_sessions ) {
-			$_SESSION['wpi' . $this->prefix ] = $this->session;
-		}
-
-		return $this->session[ $key ];
-	}
+            if ( defined( 'WP_CLI' ) && WP_CLI && !class_exists( 'WP_Session_Command' ) ) {
+                require_once WPINV_PLUGIN_DIR . 'includes/libraries/wp-session/wp-cli.php';
+            }
+
+            if ( !class_exists( 'WP_Session' ) ) {
+                require_once WPINV_PLUGIN_DIR . 'includes/libraries/wp-session/class-wp-session.php';
+                require_once WPINV_PLUGIN_DIR . 'includes/libraries/wp-session/wp-session.php';
+            }
+
+            add_filter( 'wp_session_expiration_variant', array( $this, 'set_expiration_variant_time' ), 99999 );
+            add_filter( 'wp_session_expiration', array( $this, 'set_expiration_time' ), 99999 );
+        }
+
+        if ( empty( $this->session ) && ! $this->use_php_sessions ) {
+            add_action( 'plugins_loaded', array( $this, 'init' ), -1 );
+        } else {
+            add_action( 'init', array( $this, 'init' ), -1 );
+        }
+    }
+
+    /**
+     * Setup the WP_Session instance.
+     *
+     * @access public
+     * @since 1.0.0
+     * @return void
+     */
+    public function init() {
+        if ( $this->use_php_sessions ) {
+            $this->session = isset( $_SESSION['wpi' . $this->prefix ] ) && is_array( $_SESSION['wpi' . $this->prefix ] ) ? $_SESSION['wpi' . $this->prefix ] : array();
+        } else {
+            $this->session = WP_Session::get_instance();
+        }
+
+        return $this->session;
+    }
+
+    /**
+     * Retrieve session ID.
+     *
+     * @access public
+     * @since 1.0.0
+     * @return string Session ID
+     */
+    public function get_id() {
+        if ( $this->use_php_sessions ) {
+            $session_id = !empty( $_SESSION ) && function_exists( 'session_id' ) ? session_id() : NULL;
+        } else {
+            $session_id = !empty( $this->session ) && isset( $this->session->session_id ) ? $this->session->session_id : NULL;
+        }
+        return $session_id;
+    }
+
+    /**
+     * Retrieve a session variable.
+     *
+     * @access public
+     * @since 1.0.0
+     * @param string $key Session key
+     * @return string Session variable
+     */
+    public function get( $key ) {
+        $key = sanitize_key( $key );
+        return isset( $this->session[ $key ] ) ? maybe_unserialize( $this->session[ $key ] ) : false;
+    }
+
+    /**
+     * Set a session variable
+     *
+     * @since 1.0.0
+     *
+     * @param string $key Session key
+     * @param integer $value Session variable
+     * @return string Session variable
+     */
+    public function set( $key, $value ) {
+        $key = sanitize_key( $key );
+
+        if ( is_array( $value ) ) {
+            $this->session[ $key ] = maybe_serialize( $value );
+        } else {
+            $this->session[ $key ] = $value;
+        }
+
+        if ( $this->use_php_sessions ) {
+            $_SESSION['wpi' . $this->prefix ] = $this->session;
+        }
+
+        return $this->session[ $key ];
+    }
 	
-	/**
-	 * Unset a session variable.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @param string|array $key Session key.
-	 * @param integer $value Session variable.
-	 * @return string Session variable.
-	 */
-	public function un_set( $key ) {
-		if ( empty( $key ) ) {
-			return false;
-		}
+    /**
+     * Unset a session variable.
+     *
+     * @since 1.0.0
+     *
+     * @param string|array $key Session key.
+     * @param integer $value Session variable.
+     * @return string Session variable.
+     */
+    public function un_set( $key ) {
+        if ( empty( $key ) ) {
+            return false;
+        }
 		
-		if ( is_array( $key ) ) {
-			foreach ($key as $index) {
-				$index = sanitize_key( $index );
+        if ( is_array( $key ) ) {
+            foreach ($key as $index) {
+                $index = sanitize_key( $index );
 			
-				if ( $index && isset( $this->session[ $index ] ) ) {
-					unset( $this->session[ $index ] );
-				}
-			}
-		} else {
-			$key = sanitize_key( $key );
+                if ( $index && isset( $this->session[ $index ] ) ) {
+                    unset( $this->session[ $index ] );
+                }
+            }
+        } else {
+            $key = sanitize_key( $key );
 			
-			if ( isset( $this->session[ $key ] ) ) {
-				unset( $this->session[ $key ] );
-			}
-		}
+            if ( isset( $this->session[ $key ] ) ) {
+                unset( $this->session[ $key ] );
+            }
+        }
 
-		if ( $this->use_php_sessions ) {
-			$_SESSION['wpi' . $this->prefix ] = $this->session;
-		}
+        if ( $this->use_php_sessions ) {
+            $_SESSION['wpi' . $this->prefix ] = $this->session;
+        }
 
-		return true;
-	}
+        return true;
+    }
 	
-	/**
-	 * Check a session variable is set or not.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @param string $key Session key.
-	 * @param integer $value Session variable.
-	 * @return string Session variable.
-	 */
-	public function is_set( $key ) {
-		$key = sanitize_key( $key );
+    /**
+     * Check a session variable is set or not.
+     *
+     * @since 1.0.0
+     *
+     * @param string $key Session key.
+     * @param integer $value Session variable.
+     * @return string Session variable.
+     */
+    public function is_set( $key ) {
+        $key = sanitize_key( $key );
 		
-		if ( empty( $key ) ) {
-			return false;
-		}
-
-		if ( isset( $this->session[ $key ] ) ) {
-			return true;
-		}
-
-		return false;
-	}
-
-	/**
-	 * Force the cookie expiration variant time to 23 hours
-	 *
-	 * @access public
-	 * @since 1.0.0
-	 * @param int $exp Default expiration (1 hour)
-	 * @return int
-	 */
-	public function set_expiration_variant_time( $exp ) {
-		return ( 30 * 60 * 23 );
-	}
-
-	/**
-	 * Force the cookie expiration time to 24 hours
-	 *
-	 * @access public
-	 * @since 1.0.0
-	 * @param int $exp Default expiration (1 hour)
-	 * @return int
-	 */
-	public function set_expiration_time( $exp ) {
-		return ( 30 * 60 * 24 );
-	}
-
-	/**
-	 * Starts a new session if one hasn't started yet.
-	 *
-	 * @return boolean
-	 * Checks to see if the server supports PHP sessions
-	 * or if the WPINV_USE_PHP_SESSIONS constant is defined
-	 *
-	 * @access public
-	 * @since 1.0.0
-	 * @return boolean $ret True if we are using PHP sessions, false otherwise
-	 */
-	public function use_php_sessions() {
-		$ret = false;
-
-		// If the database variable is already set, no need to run autodetection
-		$wpinv_use_php_sessions = (bool)get_option( 'wpinv_use_php_sessions' );
-
-		if (!$wpinv_use_php_sessions ) {
-			// Attempt to detect if the server supports PHP sessions
-			if ( function_exists( 'session_start' ) && ! ini_get( 'safe_mode' ) ) {
-				$this->set( 'wpinv_use_php_sessions', 1 );
+        if ( empty( $key ) ) {
+            return false;
+        }
+
+        if ( isset( $this->session[ $key ] ) ) {
+            return true;
+        }
+
+        return false;
+    }
+
+    /**
+     * Force the cookie expiration variant time to 23 hours
+     *
+     * @access public
+     * @since 1.0.0
+     * @param int $exp Default expiration (1 hour)
+     * @return int
+     */
+    public function set_expiration_variant_time( $exp ) {
+        return ( 30 * 60 * 23 );
+    }
+
+    /**
+     * Force the cookie expiration time to 24 hours
+     *
+     * @access public
+     * @since 1.0.0
+     * @param int $exp Default expiration (1 hour)
+     * @return int
+     */
+    public function set_expiration_time( $exp ) {
+        return ( 30 * 60 * 24 );
+    }
+
+    /**
+     * Starts a new session if one hasn't started yet.
+     *
+     * @return boolean
+     * Checks to see if the server supports PHP sessions
+     * or if the WPINV_USE_PHP_SESSIONS constant is defined
+     *
+     * @access public
+     * @since 1.0.0
+     * @return boolean $ret True if we are using PHP sessions, false otherwise
+     */
+    public function use_php_sessions() {
+        $ret = false;
+
+        // If the database variable is already set, no need to run autodetection
+        $wpinv_use_php_sessions = (bool)get_option( 'wpinv_use_php_sessions' );
+
+        if (!$wpinv_use_php_sessions ) {
+            // Attempt to detect if the server supports PHP sessions
+            if ( function_exists( 'session_start' ) && ! ini_get( 'safe_mode' ) ) {
+                $this->set( 'wpinv_use_php_sessions', 1 );
 				
-				if ( $this->get( 'wpinv_use_php_sessions' ) ) {
-					$ret = true;
-
-					// Set the database option
-					update_option( 'wpinv_use_php_sessions', true );
-				}
-			}
-		} else {
-			$ret = $wpinv_use_php_sessions;
-		}
-
-		// Enable or disable PHP Sessions based on the WPINV_USE_PHP_SESSIONS constant
-		if ( defined( 'WPINV_USE_PHP_SESSIONS' ) && WPINV_USE_PHP_SESSIONS ) {
-			$ret = true;
-		} else if ( defined( 'WPINV_USE_PHP_SESSIONS' ) && ! WPINV_USE_PHP_SESSIONS ) {
-			$ret = false;
-		}
-
-		return (bool) apply_filters( 'wpinv_use_php_sessions', $ret );
-	}
-
-	/**
-	 * Starts a new session if one hasn't started yet.
-	 */
-	public function maybe_start_session() {
-		if ( !session_id() && !headers_sent() ) {
-			session_start();
-		}
-	}
+                if ( $this->get( 'wpinv_use_php_sessions' ) ) {
+                    $ret = true;
+
+                    // Set the database option
+                    update_option( 'wpinv_use_php_sessions', true );
+                }
+            }
+        } else {
+            $ret = $wpinv_use_php_sessions;
+        }
+
+        // Enable or disable PHP Sessions based on the WPINV_USE_PHP_SESSIONS constant
+        if ( defined( 'WPINV_USE_PHP_SESSIONS' ) && WPINV_USE_PHP_SESSIONS ) {
+            $ret = true;
+        } else if ( defined( 'WPINV_USE_PHP_SESSIONS' ) && ! WPINV_USE_PHP_SESSIONS ) {
+            $ret = false;
+        }
+
+        return (bool) apply_filters( 'wpinv_use_php_sessions', $ret );
+    }
+
+    /**
+     * Starts a new session if one hasn't started yet.
+     */
+    public function maybe_start_session() {
+        if ( !session_id() && !headers_sent() ) {
+            session_start();
+        }
+    }
 }
 
 global $wpi_session;

Spacing +60 added lines, -60 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 // Exit if accessed directly.
-if (!defined( 'ABSPATH' ) ) exit;
+if (!defined('ABSPATH')) exit;
 
 /**
  * WPInv_Session Class.
@@ -46,44 +46,44 @@ 
 	public function __construct() {
 		$this->use_php_sessions = $this->use_php_sessions();
 
-		if ( $this->use_php_sessions ) {
-			if ( is_multisite() ) {
+		if ($this->use_php_sessions) {
+			if (is_multisite()) {
 				$this->prefix = '_' . get_current_blog_id();
 			}
 
 			// Use PHP SESSION (must be enabled via the WPINV_USE_PHP_SESSIONS constant)
-			add_action( 'init', array( $this, 'maybe_start_session' ), -2 );
+			add_action('init', array($this, 'maybe_start_session'), -2);
 		} else {
 			// Use WP_Session (default)
-			if ( !defined( 'WP_SESSION_COOKIE' ) ) {
-				define( 'WP_SESSION_COOKIE', 'wpinv_wp_session' );
+			if (!defined('WP_SESSION_COOKIE')) {
+				define('WP_SESSION_COOKIE', 'wpinv_wp_session');
 			}
 
-			if ( !class_exists( 'Recursive_ArrayAccess' ) ) {
+			if (!class_exists('Recursive_ArrayAccess')) {
 				require_once WPINV_PLUGIN_DIR . 'includes/libraries/wp-session/class-recursive-arrayaccess.php';
 			}
             
-			if ( !class_exists( 'WP_Session_Utils' ) ) {
+			if (!class_exists('WP_Session_Utils')) {
 				require_once WPINV_PLUGIN_DIR . 'includes/libraries/wp-session/class-wp-session-utils.php';
 			}
             
-			if ( defined( 'WP_CLI' ) && WP_CLI && !class_exists( 'WP_Session_Command' ) ) {
+			if (defined('WP_CLI') && WP_CLI && !class_exists('WP_Session_Command')) {
 				require_once WPINV_PLUGIN_DIR . 'includes/libraries/wp-session/wp-cli.php';
 			}
 
-			if ( !class_exists( 'WP_Session' ) ) {
+			if (!class_exists('WP_Session')) {
 				require_once WPINV_PLUGIN_DIR . 'includes/libraries/wp-session/class-wp-session.php';
 				require_once WPINV_PLUGIN_DIR . 'includes/libraries/wp-session/wp-session.php';
 			}
 
-			add_filter( 'wp_session_expiration_variant', array( $this, 'set_expiration_variant_time' ), 99999 );
-			add_filter( 'wp_session_expiration', array( $this, 'set_expiration_time' ), 99999 );
+			add_filter('wp_session_expiration_variant', array($this, 'set_expiration_variant_time'), 99999);
+			add_filter('wp_session_expiration', array($this, 'set_expiration_time'), 99999);
 		}
 
-		if ( empty( $this->session ) && ! $this->use_php_sessions ) {
-			add_action( 'plugins_loaded', array( $this, 'init' ), -1 );
+		if (empty($this->session) && !$this->use_php_sessions) {
+			add_action('plugins_loaded', array($this, 'init'), -1);
 		} else {
-			add_action( 'init', array( $this, 'init' ), -1 );
+			add_action('init', array($this, 'init'), -1);
 		}
 	}
 
@@ -95,8 +95,8 @@ 
 	 * @return void
 	 */
 	public function init() {
-		if ( $this->use_php_sessions ) {
-			$this->session = isset( $_SESSION['wpi' . $this->prefix ] ) && is_array( $_SESSION['wpi' . $this->prefix ] ) ? $_SESSION['wpi' . $this->prefix ] : array();
+		if ($this->use_php_sessions) {
+			$this->session = isset($_SESSION['wpi' . $this->prefix]) && is_array($_SESSION['wpi' . $this->prefix]) ? $_SESSION['wpi' . $this->prefix] : array();
 		} else {
 			$this->session = WP_Session::get_instance();
 		}
@@ -112,10 +112,10 @@ 
 	 * @return string Session ID
 	 */
 	public function get_id() {
-		if ( $this->use_php_sessions ) {
-			$session_id = !empty( $_SESSION ) && function_exists( 'session_id' ) ? session_id() : NULL;
+		if ($this->use_php_sessions) {
+			$session_id = !empty($_SESSION) && function_exists('session_id') ? session_id() : NULL;
 		} else {
-			$session_id = !empty( $this->session ) && isset( $this->session->session_id ) ? $this->session->session_id : NULL;
+			$session_id = !empty($this->session) && isset($this->session->session_id) ? $this->session->session_id : NULL;
 		}
 		return $session_id;
 	}
@@ -128,9 +128,9 @@ 
 	 * @param string $key Session key
 	 * @return string Session variable
 	 */
-	public function get( $key ) {
-		$key = sanitize_key( $key );
-		return isset( $this->session[ $key ] ) ? maybe_unserialize( $this->session[ $key ] ) : false;
+	public function get($key) {
+		$key = sanitize_key($key);
+		return isset($this->session[$key]) ? maybe_unserialize($this->session[$key]) : false;
 	}
 
 	/**
@@ -142,20 +142,20 @@ 
 	 * @param integer $value Session variable
 	 * @return string Session variable
 	 */
-	public function set( $key, $value ) {
-		$key = sanitize_key( $key );
+	public function set($key, $value) {
+		$key = sanitize_key($key);
 
-		if ( is_array( $value ) ) {
-			$this->session[ $key ] = maybe_serialize( $value );
+		if (is_array($value)) {
+			$this->session[$key] = maybe_serialize($value);
 		} else {
-			$this->session[ $key ] = $value;
+			$this->session[$key] = $value;
 		}
 
-		if ( $this->use_php_sessions ) {
-			$_SESSION['wpi' . $this->prefix ] = $this->session;
+		if ($this->use_php_sessions) {
+			$_SESSION['wpi' . $this->prefix] = $this->session;
 		}
 
-		return $this->session[ $key ];
+		return $this->session[$key];
 	}
 	
 	/**
@@ -167,29 +167,29 @@ 
 	 * @param integer $value Session variable.
 	 * @return string Session variable.
 	 */
-	public function un_set( $key ) {
-		if ( empty( $key ) ) {
+	public function un_set($key) {
+		if (empty($key)) {
 			return false;
 		}
 		
-		if ( is_array( $key ) ) {
+		if (is_array($key)) {
 			foreach ($key as $index) {
-				$index = sanitize_key( $index );
+				$index = sanitize_key($index);
 			
-				if ( $index && isset( $this->session[ $index ] ) ) {
-					unset( $this->session[ $index ] );
+				if ($index && isset($this->session[$index])) {
+					unset($this->session[$index]);
 				}
 			}
 		} else {
-			$key = sanitize_key( $key );
+			$key = sanitize_key($key);
 			
-			if ( isset( $this->session[ $key ] ) ) {
-				unset( $this->session[ $key ] );
+			if (isset($this->session[$key])) {
+				unset($this->session[$key]);
 			}
 		}
 
-		if ( $this->use_php_sessions ) {
-			$_SESSION['wpi' . $this->prefix ] = $this->session;
+		if ($this->use_php_sessions) {
+			$_SESSION['wpi' . $this->prefix] = $this->session;
 		}
 
 		return true;
@@ -204,14 +204,14 @@ 
 	 * @param integer $value Session variable.
 	 * @return string Session variable.
 	 */
-	public function is_set( $key ) {
-		$key = sanitize_key( $key );
+	public function is_set($key) {
+		$key = sanitize_key($key);
 		
-		if ( empty( $key ) ) {
+		if (empty($key)) {
 			return false;
 		}
 
-		if ( isset( $this->session[ $key ] ) ) {
+		if (isset($this->session[$key])) {
 			return true;
 		}
 
@@ -226,8 +226,8 @@ 
 	 * @param int $exp Default expiration (1 hour)
 	 * @return int
 	 */
-	public function set_expiration_variant_time( $exp ) {
-		return ( 30 * 60 * 23 );
+	public function set_expiration_variant_time($exp) {
+		return (30 * 60 * 23);
 	}
 
 	/**
@@ -238,8 +238,8 @@ 
 	 * @param int $exp Default expiration (1 hour)
 	 * @return int
 	 */
-	public function set_expiration_time( $exp ) {
-		return ( 30 * 60 * 24 );
+	public function set_expiration_time($exp) {
+		return (30 * 60 * 24);
 	}
 
 	/**
@@ -257,18 +257,18 @@ 
 		$ret = false;
 
 		// If the database variable is already set, no need to run autodetection
-		$wpinv_use_php_sessions = (bool)get_option( 'wpinv_use_php_sessions' );
+		$wpinv_use_php_sessions = (bool)get_option('wpinv_use_php_sessions');
 
-		if (!$wpinv_use_php_sessions ) {
+		if (!$wpinv_use_php_sessions) {
 			// Attempt to detect if the server supports PHP sessions
-			if ( function_exists( 'session_start' ) && ! ini_get( 'safe_mode' ) ) {
-				$this->set( 'wpinv_use_php_sessions', 1 );
+			if (function_exists('session_start') && !ini_get('safe_mode')) {
+				$this->set('wpinv_use_php_sessions', 1);
 				
-				if ( $this->get( 'wpinv_use_php_sessions' ) ) {
+				if ($this->get('wpinv_use_php_sessions')) {
 					$ret = true;
 
 					// Set the database option
-					update_option( 'wpinv_use_php_sessions', true );
+					update_option('wpinv_use_php_sessions', true);
 				}
 			}
 		} else {
@@ -276,20 +276,20 @@ 
 		}
 
 		// Enable or disable PHP Sessions based on the WPINV_USE_PHP_SESSIONS constant
-		if ( defined( 'WPINV_USE_PHP_SESSIONS' ) && WPINV_USE_PHP_SESSIONS ) {
+		if (defined('WPINV_USE_PHP_SESSIONS') && WPINV_USE_PHP_SESSIONS) {
 			$ret = true;
-		} else if ( defined( 'WPINV_USE_PHP_SESSIONS' ) && ! WPINV_USE_PHP_SESSIONS ) {
+		} else if (defined('WPINV_USE_PHP_SESSIONS') && !WPINV_USE_PHP_SESSIONS) {
 			$ret = false;
 		}
 
-		return (bool) apply_filters( 'wpinv_use_php_sessions', $ret );
+		return (bool)apply_filters('wpinv_use_php_sessions', $ret);
 	}
 
 	/**
 	 * Starts a new session if one hasn't started yet.
 	 */
 	public function maybe_start_session() {
-		if ( !session_id() && !headers_sent() ) {
+		if (!session_id() && !headers_sent()) {
 			session_start();
 		}
 	}

Braces +3 added lines, -1 removed lines

@@ -1,6 +1,8 @@
 <?php
 // Exit if accessed directly.
-if (!defined( 'ABSPATH' ) ) exit;
+if (!defined( 'ABSPATH' ) ) {
+    exit;
+}
 
 /**
  * WPInv_Session Class.

includes/gateways/authorizenet/anet_php_sdk/lib/AuthorizeNetAIM.php

Doc Comments +6 added lines, -6 removed lines

@@ -101,7 +101,7 @@ 
      * @param string $card_num The credit card number
      * @param string $exp_date CC expiration date
      *
-     * @return AuthorizeNetAIM_Response
+     * @return AuthorizeNetARB_Response
      */
     public function authorizeAndCapture($amount = false, $card_num = false, $exp_date = false)
     {
@@ -122,7 +122,7 @@ 
      * @param string $trans_id Transaction id to charge
      * @param string $amount   Dollar amount to charge if lesser than auth
      *
-     * @return AuthorizeNetAIM_Response
+     * @return AuthorizeNetARB_Response
      */
     public function priorAuthCapture($trans_id = false, $amount = false)
     {
@@ -141,7 +141,7 @@ 
      * @param string $card_num The credit card number
      * @param string $exp_date CC expiration date
      *
-     * @return AuthorizeNetAIM_Response
+     * @return AuthorizeNetARB_Response
      */
     public function authorizeOnly($amount = false, $card_num = false, $exp_date = false)
     {
@@ -160,7 +160,7 @@ 
      *
      * @param string $trans_id Transaction id to void
      *
-     * @return AuthorizeNetAIM_Response
+     * @return AuthorizeNetARB_Response
      */
     public function void($trans_id = false)
     {
@@ -179,7 +179,7 @@ 
      * @param string $card_num  The last 4 of credit card number
      * @param string $exp_date  CC expiration date
      *
-     * @return AuthorizeNetAIM_Response
+     * @return AuthorizeNetARB_Response
      */
     public function captureOnly($auth_code = false, $amount = false, $card_num = false, $exp_date = false)
     {
@@ -200,7 +200,7 @@ 
      * @param string $amount   The dollar amount to credit
      * @param string $card_num The last 4 of credit card number
      *
-     * @return AuthorizeNetAIM_Response
+     * @return AuthorizeNetARB_Response
      */
     public function credit($trans_id = false, $amount = false, $card_num = false)
     {

Spacing +19 added lines, -19 removed lines

@@ -77,19 +77,19 @@ 
      * A list of all fields in the AIM API.
      * Used to warn user if they try to set a field not offered in the API.
      */
-    private $_all_aim_fields = array("address","allow_partial_auth","amount",
-        "auth_code","authentication_indicator", "bank_aba_code","bank_acct_name",
-        "bank_acct_num","bank_acct_type","bank_check_number","bank_name",
-        "card_code","card_num","cardholder_authentication_value","city","company",
-        "country","cust_id","customer_ip","delim_char","delim_data","description",
-        "duplicate_window","duty","echeck_type","email","email_customer",
-        "encap_char","exp_date","fax","first_name","footer_email_receipt",
-        "freight","header_email_receipt","invoice_num","last_name","line_item",
-        "login","method","phone","po_num","recurring_billing","relay_response",
-        "ship_to_address","ship_to_city","ship_to_company","ship_to_country",
-        "ship_to_first_name","ship_to_last_name","ship_to_state","ship_to_zip",
-        "split_tender_id","state","tax","tax_exempt","test_request","tran_key",
-        "trans_id","type","version","zip"
+    private $_all_aim_fields = array("address", "allow_partial_auth", "amount",
+        "auth_code", "authentication_indicator", "bank_aba_code", "bank_acct_name",
+        "bank_acct_num", "bank_acct_type", "bank_check_number", "bank_name",
+        "card_code", "card_num", "cardholder_authentication_value", "city", "company",
+        "country", "cust_id", "customer_ip", "delim_char", "delim_data", "description",
+        "duplicate_window", "duty", "echeck_type", "email", "email_customer",
+        "encap_char", "exp_date", "fax", "first_name", "footer_email_receipt",
+        "freight", "header_email_receipt", "invoice_num", "last_name", "line_item",
+        "login", "method", "phone", "po_num", "recurring_billing", "relay_response",
+        "ship_to_address", "ship_to_city", "ship_to_company", "ship_to_country",
+        "ship_to_first_name", "ship_to_last_name", "ship_to_state", "ship_to_zip",
+        "split_tender_id", "state", "tax", "tax_exempt", "test_request", "tran_key",
+        "trans_id", "type", "version", "zip"
         );
     
     /**
@@ -403,7 +403,7 @@ 
             // Split Array
             $this->response = $response;
             if ($encap_char) {
-                $this->_response_array = explode($encap_char.$delimiter.$encap_char, substr($response, 1, -1));
+                $this->_response_array = explode($encap_char . $delimiter . $encap_char, substr($response, 1, -1));
             } else {
                 $this->_response_array = explode($delimiter, $response);
             }
@@ -457,7 +457,7 @@ 
             $this->duty                 = $this->_response_array[33];
             $this->freight              = $this->_response_array[34];
             $this->tax_exempt           = $this->_response_array[35];
-            $this->purchase_order_number= $this->_response_array[36];
+            $this->purchase_order_number = $this->_response_array[36];
             $this->md5_hash             = $this->_response_array[37];
             $this->card_code_response   = $this->_response_array[38];
             $this->cavv_response        = $this->_response_array[39];
@@ -484,10 +484,10 @@ 
             
             if ($this->error) {
                 $this->error_message = "AuthorizeNet Error:
-                Response Code: ".$this->response_code."
-                Response Subcode: ".$this->response_subcode."
-                Response Reason Code: ".$this->response_reason_code."
-                Response Reason Text: ".$this->response_reason_text."
+                Response Code: ".$this->response_code . "
+                Response Subcode: ".$this->response_subcode . "
+                Response Reason Code: ".$this->response_reason_code . "
+                Response Reason Text: ".$this->response_reason_text . "
                 ";
             }
         } else {

includes/gateways/authorizenet/anet_php_sdk/lib/AuthorizeNetCIM.php

Doc Comments +16 added lines, -16 removed lines

@@ -51,7 +51,7 @@ 
      * @param AuthorizeNetCustomer $customerProfile
      * @param string               $validationMode
      *
-     * @return AuthorizeNetCIM_Response
+     * @return AuthorizeNetARB_Response
      */
     public function createCustomerProfile($customerProfile, $validationMode = "none")
     {
@@ -69,7 +69,7 @@ 
      * @param AuthorizeNetPaymentProfile $paymentProfile
      * @param string                     $validationMode
      *
-     * @return AuthorizeNetCIM_Response
+     * @return AuthorizeNetARB_Response
      */
     public function createCustomerPaymentProfile($customerProfileId, $paymentProfile, $validationMode = "none")
     {
@@ -87,7 +87,7 @@ 
      * @param int                        $customerProfileId
      * @param AuthorizeNetAddress        $shippingAddress
      *
-     * @return AuthorizeNetCIM_Response
+     * @return AuthorizeNetARB_Response
      */
     public function createCustomerShippingAddress($customerProfileId, $shippingAddress)
     {
@@ -105,7 +105,7 @@ 
      * @param AuthorizeNetTransaction    $transaction
      * @param string                     $extraOptionsString
      *
-     * @return AuthorizeNetCIM_Response
+     * @return AuthorizeNetARB_Response
      */
     public function createCustomerProfileTransaction($transactionType, $transaction, $extraOptionsString = "")
     {
@@ -122,7 +122,7 @@ 
      *
      * @param int $customerProfileId
      *
-     * @return AuthorizeNetCIM_Response
+     * @return AuthorizeNetARB_Response
      */
     public function deleteCustomerProfile($customerProfileId)
     {
@@ -137,7 +137,7 @@ 
      * @param int $customerProfileId
      * @param int $customerPaymentProfileId
      *
-     * @return AuthorizeNetCIM_Response
+     * @return AuthorizeNetARB_Response
      */
     public function deleteCustomerPaymentProfile($customerProfileId, $customerPaymentProfileId)
     {
@@ -153,7 +153,7 @@ 
      * @param int $customerProfileId
      * @param int $customerAddressId
      *
-     * @return AuthorizeNetCIM_Response
+     * @return AuthorizeNetARB_Response
      */
     public function deleteCustomerShippingAddress($customerProfileId, $customerAddressId)
     {
@@ -166,7 +166,7 @@ 
     /**
      * Get all customer profile ids.
      *
-     * @return AuthorizeNetCIM_Response
+     * @return AuthorizeNetARB_Response
      */
     public function getCustomerProfileIds()
     {
@@ -179,7 +179,7 @@ 
      *
      * @param int $customerProfileId
      *
-     * @return AuthorizeNetCIM_Response
+     * @return AuthorizeNetARB_Response
      */
     public function getCustomerProfile($customerProfileId)
     {
@@ -194,7 +194,7 @@ 
      * @param int $customerProfileId
      * @param int $customerPaymentProfileId
      *
-     * @return AuthorizeNetCIM_Response
+     * @return AuthorizeNetARB_Response
      */
     public function getCustomerPaymentProfile($customerProfileId, $customerPaymentProfileId)
     {
@@ -210,7 +210,7 @@ 
      * @param int $customerProfileId
      * @param int $customerAddressId
      *
-     * @return AuthorizeNetCIM_Response
+     * @return AuthorizeNetARB_Response
      */
     public function getCustomerShippingAddress($customerProfileId, $customerAddressId)
     {
@@ -226,7 +226,7 @@ 
      * @param int                        $customerProfileId
      * @param AuthorizeNetCustomer       $customerProfile
      *
-     * @return AuthorizeNetCIM_Response
+     * @return AuthorizeNetARB_Response
      */
     public function updateCustomerProfile($customerProfileId, $customerProfile)
     {
@@ -245,7 +245,7 @@ 
      * @param AuthorizeNetPaymentProfile $paymentProfile
      * @param string                     $validationMode
      *
-     * @return AuthorizeNetCIM_Response
+     * @return AuthorizeNetARB_Response
      */
     public function updateCustomerPaymentProfile($customerProfileId, $customerPaymentProfileId, $paymentProfile, $validationMode = "none")
     {
@@ -265,7 +265,7 @@ 
      * @param int                        $customerShippingAddressId
      * @param AuthorizeNetAddress        $shippingAddress
      *
-     * @return AuthorizeNetCIM_Response
+     * @return AuthorizeNetARB_Response
      */
     public function updateCustomerShippingAddress($customerProfileId, $customerShippingAddressId, $shippingAddress)
     {
@@ -284,7 +284,7 @@ 
      * @param int                        $splitTenderId
      * @param string                     $splitTenderStatus
      *
-     * @return AuthorizeNetCIM_Response
+     * @return AuthorizeNetARB_Response
      */
     public function updateSplitTenderGroup($splitTenderId, $splitTenderStatus)
     {
@@ -303,7 +303,7 @@ 
      * @param int                        $cardCode
      * @param string                     $validationMode
      *
-     * @return AuthorizeNetCIM_Response
+     * @return AuthorizeNetARB_Response
      */
     public function validateCustomerPaymentProfile($customerProfileId, $customerPaymentProfileId, $customerShippingAddressId, $cardCode, $validationMode = "testMode")
     {

Indentation +3 added lines, -3 removed lines

@@ -316,9 +316,9 @@
         return $this->_sendRequest();
     }
     
-     /**
-     * @return string
-     */
+        /**
+         * @return string
+         */
     protected function _getPostUrl()
     {
         return ($this->_sandbox ? self::SANDBOX_URL : self::LIVE_URL);

Spacing +11 added lines, -11 removed lines

@@ -309,10 +309,10 @@ 
     {
         $this->_validationMode = $validationMode;
         $this->_constructXml("validateCustomerPaymentProfileRequest");
-        $this->_xml->addChild("customerProfileId",$customerProfileId);
-        $this->_xml->addChild("customerPaymentProfileId",$customerPaymentProfileId);
-        $this->_xml->addChild("customerShippingAddressId",$customerShippingAddressId);
-        $this->_xml->addChild("cardCode",$cardCode);
+        $this->_xml->addChild("customerProfileId", $customerProfileId);
+        $this->_xml->addChild("customerPaymentProfileId", $customerPaymentProfileId);
+        $this->_xml->addChild("customerShippingAddressId", $customerShippingAddressId);
+        $this->_xml->addChild("cardCode", $cardCode);
         return $this->_sendRequest();
     }
     
@@ -341,13 +341,13 @@ 
      */
     protected function _setPostString()
     {
-        ($this->_validationMode != "none" ? $this->_xml->addChild('validationMode',$this->_validationMode) : "");
+        ($this->_validationMode != "none" ? $this->_xml->addChild('validationMode', $this->_validationMode) : "");
         $this->_post_string = $this->_xml->asXML();
         
         // Add extraOptions CDATA
         if ($this->_extraOptions) {
             $this->_xml->addChild("extraOptions");
-            $this->_post_string = str_replace("<extraOptions></extraOptions>",'<extraOptions><![CDATA[' . $this->_extraOptions . ']]></extraOptions>', $this->_xml->asXML());
+            $this->_post_string = str_replace("<extraOptions></extraOptions>", '<extraOptions><![CDATA[' . $this->_extraOptions . ']]></extraOptions>', $this->_xml->asXML());
             $this->_extraOptions = false;
         }
         // Blank out our validation mode, so that we don't include it in calls that
@@ -362,12 +362,12 @@ 
      */
     private function _constructXml($request_type)
     {
-        $string = '<?xml version="1.0" encoding="utf-8"?><'.$request_type.' xmlns="AnetApi/xml/v1/schema/AnetApiSchema.xsd"></'.$request_type.'>';
+        $string = '<?xml version="1.0" encoding="utf-8"?><' . $request_type . ' xmlns="AnetApi/xml/v1/schema/AnetApiSchema.xsd"></' . $request_type . '>';
         $this->_xml = @new SimpleXMLElement($string);
         $merchant = $this->_xml->addChild('merchantAuthentication');
-        $merchant->addChild('name',$this->_api_login);
-        $merchant->addChild('transactionKey',$this->_transaction_key);
-        ($this->_refId ? $this->_xml->addChild('refId',$this->_refId) : "");
+        $merchant->addChild('name', $this->_api_login);
+        $merchant->addChild('transactionKey', $this->_transaction_key);
+        ($this->_refId ? $this->_xml->addChild('refId', $this->_refId) : "");
     }
     
     /**
@@ -387,7 +387,7 @@ 
                         $this->_addObject($items, $item);
                     }
                 } else {
-                    $destination->addChild($key,$value);
+                    $destination->addChild($key, $value);
                 }
             } elseif (is_object($value) && self::_notEmpty($value)) {
                 $dest = $destination->addChild($key);

includes/gateways/authorizenet/anet_php_sdk/lib/AuthorizeNetCP.php

Doc Comments +1 added lines, -1 removed lines

@@ -79,7 +79,7 @@
      *
      * @param string $response
      * 
-     * @return AuthorizeNetAIM_Response
+     * @return AuthorizeNetCP_Response
      */
     protected function _handleResponse($response)
     {

Spacing +5 added lines, -5 removed lines

@@ -116,7 +116,7 @@ 
                 
                 $this->xml = @simplexml_load_string($response);
                 // Set all fields
-                $this->version              = array_pop(array_slice(explode('"', $response), 1,1));
+                $this->version              = array_pop(array_slice(explode('"', $response), 1, 1));
                 $this->response_code        = (string)$this->xml->ResponseCode;
                 
                 if ($this->response_code == 1) {
@@ -144,7 +144,7 @@ 
                 // Split Array
                 $this->response = $response;
                 if ($encap_char) {
-                    $this->_response_array = explode($encap_char.$delimiter.$encap_char, substr($response, 1, -1));
+                    $this->_response_array = explode($encap_char . $delimiter . $encap_char, substr($response, 1, -1));
                 } else {
                     $this->_response_array = explode($delimiter, $response);
                 }
@@ -190,9 +190,9 @@ 
             
             if ($this->error) {
                 $this->error_message = "AuthorizeNet Error:
-                Response Code: ".$this->response_code."
-                Response Reason Code: ".$this->response_reason_code."
-                Response Reason Text: ".$this->response_reason_text."
+                Response Code: ".$this->response_code . "
+                Response Reason Code: ".$this->response_reason_code . "
+                Response Reason Text: ".$this->response_reason_text . "
                 ";
             }
             

includes/gateways/authorizenet/anet_php_sdk/lib/shared/AuthorizeNetRequest.php

Doc Comments +1 added lines

@@ -28,6 +28,7 @@
     /**
      * Get the post url. We need this because until 5.3 you
      * you could not access child constants in a parent class.
+     * @return string|null
      */
     abstract protected function _getPostUrl();
     

Spacing +1 added lines, -1 removed lines

@@ -96,7 +96,7 @@
             curl_setopt($curl_request, CURLOPT_SSL_VERIFYPEER, false);
         }
         
-        if (preg_match('/xml/',$post_url)) {
+        if (preg_match('/xml/', $post_url)) {
             curl_setopt($curl_request, CURLOPT_HTTPHEADER, Array("Content-Type: text/xml"));
         }
         

includes/gateways/authorizenet/anet_php_sdk/lib/shared/AuthorizeNetXMLResponse.php

Doc Comments +1 added lines

@@ -110,6 +110,7 @@
      * Grabs the contents of a unique element.
      *
      * @param  string
+     * @param string $elementName
      * @return string
      */
     protected function _getElementContents($elementName) 

Spacing +4 added lines, -4 removed lines

@@ -29,7 +29,7 @@ 
             $this->xml = @simplexml_load_string($response);
             
             // Remove namespaces for use with XPath.
-            $this->xpath_xml = @simplexml_load_string(preg_replace('/ xmlns:xsi[^>]+/','',$response));
+            $this->xpath_xml = @simplexml_load_string(preg_replace('/ xmlns:xsi[^>]+/', '', $response));
         }
     }
     
@@ -116,12 +116,12 @@ 
     {
         $start = "<$elementName>";
         $end = "</$elementName>";
-        if (strpos($this->response,$start) === false || strpos($this->response,$end) === false) {
+        if (strpos($this->response, $start) === false || strpos($this->response, $end) === false) {
             return false;
         } else {
-            $start_position = strpos($this->response, $start)+strlen($start);
+            $start_position = strpos($this->response, $start) + strlen($start);
             $end_position = strpos($this->response, $end);
-            return substr($this->response, $start_position, $end_position-$start_position);
+            return substr($this->response, $start_position, $end_position - $start_position);
         }
     }
 

includes/gateways/authorizenet/Authorize.Net-XML/AuthnetXML.class.php

Doc Comments +4 added lines

@@ -69,6 +69,7 @@ 
      * remove XML response namespaces
      * without this php will spit out warinings
      * @see http://community.developer.authorize.net/t5/Integration-and-Testing/ARB-with-SimpleXML-PHP-Issue/m-p/7128#M5139
+     * @return string
      */
     private function removeResponseXMLNS($input)
     {
@@ -150,6 +151,9 @@ 
 		$this->process();
 	}
 
+	/**
+	 * @param SimpleXMLElement $xml
+	 */
 	private function setParameters($xml, $array)
 	{
 	    if (is_array($array))

Indentation +47 added lines, -47 removed lines

@@ -40,7 +40,7 @@ 
     const EXCEPTION_CURL = 10;
 
     private $ch;
-	private $login;
+    private $login;
     private $response;
     private $response_xml;
     private $results;
@@ -48,9 +48,9 @@ 
     private $url;
     private $xml;
 
-	public function __construct($login, $transkey, $test = self::USE_PRODUCTION_SERVER)
-	{
-		$login    = trim($login);
+    public function __construct($login, $transkey, $test = self::USE_PRODUCTION_SERVER)
+    {
+        $login    = trim($login);
         $transkey = trim($transkey);
         if (empty($login) || empty($transkey))
         {
@@ -63,7 +63,7 @@ 
 
         $subdomain = ($test) ? 'apitest' : 'api';
         $this->url = 'https://' . $subdomain . '.authorize.net/xml/v1/request.api';
-	}
+    }
 
     /**
      * remove XML response namespaces
@@ -78,9 +78,9 @@ 
         return str_replace(' xmlns:xsd="http://www.w3.org/2001/XMLSchema"','',$input);
     }
 
-	public function __toString()
-	{
-	    $output  = '';
+    public function __toString()
+    {
+        $output  = '';
         $output .= '<table summary="Authorize.Net Results" id="authnet">' . "\n";
         $output .= '<tr>' . "\n\t\t" . '<th colspan="2"><b>Class Parameters</b></th>' . "\n" . '</tr>' . "\n";
         $output .= '<tr>' . "\n\t\t" . '<td><b>API Login ID</b></td><td>' . $this->login . '</td>' . "\n" . '</tr>' . "\n";
@@ -117,9 +117,9 @@ 
         $output .= '</table>';
 
         return $output;
-	}
+    }
 
-	public function __destruct()
+    public function __destruct()
     {
         if (isset($this->ch))
         {
@@ -128,31 +128,31 @@ 
     }
 
     public function __get($var)
-	{
-	    return $this->response_xml->$var;
-	}
-
-	public function __set($key, $value)
-	{
-	    trigger_error('You cannot set parameters directly in ' . __CLASS__ . '.', E_USER_WARNING);
-	    return false;
-	}
-
-	public function __call($api_call, $args)
-	{
-	    $this->xml = new SimpleXMLElement('<' . $api_call . '></' . $api_call . '>');
+    {
+        return $this->response_xml->$var;
+    }
+
+    public function __set($key, $value)
+    {
+        trigger_error('You cannot set parameters directly in ' . __CLASS__ . '.', E_USER_WARNING);
+        return false;
+    }
+
+    public function __call($api_call, $args)
+    {
+        $this->xml = new SimpleXMLElement('<' . $api_call . '></' . $api_call . '>');
         $this->xml->addAttribute('xmlns', 'AnetApi/xml/v1/schema/AnetApiSchema.xsd');
-	    $merch_auth = $this->xml->addChild('merchantAuthentication');
+        $merch_auth = $this->xml->addChild('merchantAuthentication');
         $merch_auth->addChild('name', $this->login);
-	    $merch_auth->addChild('transactionKey', $this->transkey);
+        $merch_auth->addChild('transactionKey', $this->transkey);
 
-		$this->setParameters($this->xml, $args[0]);
-		$this->process();
-	}
+        $this->setParameters($this->xml, $args[0]);
+        $this->process();
+    }
 
-	private function setParameters($xml, $array)
-	{
-	    if (is_array($array))
+    private function setParameters($xml, $array)
+    {
+        if (is_array($array))
         {
             $first = true;
             foreach ($array as $key => $value)
@@ -184,34 +184,34 @@ 
                 }
             }
         }
-	}
+    }
 
-	private function process()
-	{
-		$this->xml = $this->xml->asXML();
+    private function process()
+    {
+        $this->xml = $this->xml->asXML();
 
-		$this->ch = curl_init();
+        $this->ch = curl_init();
         curl_setopt($this->ch, CURLOPT_URL, $this->url);
-    	curl_setopt($this->ch, CURLOPT_RETURNTRANSFER, 1);
-    	curl_setopt($this->ch, CURLOPT_HTTPHEADER, Array("Content-Type: text/xml"));
-    	curl_setopt($this->ch, CURLOPT_HEADER, 0);
-    	curl_setopt($this->ch, CURLOPT_POSTFIELDS, $this->xml);
-    	curl_setopt($this->ch, CURLOPT_POST, 1);
-    	curl_setopt($this->ch, CURLOPT_SSL_VERIFYHOST, 2);
-    	curl_setopt($this->ch, CURLOPT_CAINFO, dirname(__FILE__) . '/ssl/cert.pem');
+        curl_setopt($this->ch, CURLOPT_RETURNTRANSFER, 1);
+        curl_setopt($this->ch, CURLOPT_HTTPHEADER, Array("Content-Type: text/xml"));
+        curl_setopt($this->ch, CURLOPT_HEADER, 0);
+        curl_setopt($this->ch, CURLOPT_POSTFIELDS, $this->xml);
+        curl_setopt($this->ch, CURLOPT_POST, 1);
+        curl_setopt($this->ch, CURLOPT_SSL_VERIFYHOST, 2);
+        curl_setopt($this->ch, CURLOPT_CAINFO, dirname(__FILE__) . '/ssl/cert.pem');
 
         if(($this->response = curl_exec($this->ch)) !== false)
         {
             $this->response_xml = @new SimpleXMLElement($this->response);
 
-		    curl_close($this->ch);
+            curl_close($this->ch);
             unset($this->ch);
             return;
-		}
+        }
         throw new AuthnetXMLException('Connection error: ' . curl_error($this->ch) . ' (' . curl_errno($this->ch) . ')', self::EXCEPTION_CURL);
-	}
+    }
 
-	public function isSuccessful()
+    public function isSuccessful()
     {
         return $this->response_xml->messages->resultCode == 'Ok';
     }

Spacing +9 added lines, -9 removed lines

@@ -50,7 +50,7 @@ 
 
 	public function __construct($login, $transkey, $test = self::USE_PRODUCTION_SERVER)
 	{
-		$login    = trim($login);
+		$login = trim($login);
         $transkey = trim($transkey);
         if (empty($login) || empty($transkey))
         {
@@ -59,7 +59,7 @@ 
 
         $this->login    = trim($login);
         $this->transkey = trim($transkey);
-        $test           = (bool) $test;
+        $test           = (bool)$test;
 
         $subdomain = ($test) ? 'apitest' : 'api';
         $this->url = 'https://' . $subdomain . '.authorize.net/xml/v1/request.api';
@@ -73,14 +73,14 @@ 
     private function removeResponseXMLNS($input)
     {
         // why remove them one at a time? all three aren't consistantly used in the response
-        $input = str_replace(' xmlns="AnetApi/xml/v1/schema/AnetApiSchema.xsd"','',$input);
-        $input = str_replace(' xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"','',$input);
-        return str_replace(' xmlns:xsd="http://www.w3.org/2001/XMLSchema"','',$input);
+        $input = str_replace(' xmlns="AnetApi/xml/v1/schema/AnetApiSchema.xsd"', '', $input);
+        $input = str_replace(' xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"', '', $input);
+        return str_replace(' xmlns:xsd="http://www.w3.org/2001/XMLSchema"', '', $input);
     }
 
 	public function __toString()
 	{
-	    $output  = '';
+	    $output = '';
         $output .= '<table summary="Authorize.Net Results" id="authnet">' . "\n";
         $output .= '<tr>' . "\n\t\t" . '<th colspan="2"><b>Class Parameters</b></th>' . "\n" . '</tr>' . "\n";
         $output .= '<tr>' . "\n\t\t" . '<td><b>API Login ID</b></td><td>' . $this->login . '</td>' . "\n" . '</tr>' . "\n";
@@ -159,9 +159,9 @@ 
             {
                 if (is_array($value))
                 {
-                    if(is_numeric($key))
+                    if (is_numeric($key))
                     {
-                        if($first === true)
+                        if ($first === true)
                         {
                             $xmlx  = $xml;
                             $first = false;
@@ -200,7 +200,7 @@ 
     	curl_setopt($this->ch, CURLOPT_SSL_VERIFYHOST, 2);
     	curl_setopt($this->ch, CURLOPT_CAINFO, dirname(__FILE__) . '/ssl/cert.pem');
 
-        if(($this->response = curl_exec($this->ch)) !== false)
+        if (($this->response = curl_exec($this->ch)) !== false)
         {
             $this->response_xml = @new SimpleXMLElement($this->response);
 

Braces +3 added lines, -6 removed lines

@@ -165,20 +165,17 @@
                         {
                             $xmlx  = $xml;
                             $first = false;
-                        }
-                        else
+                        } else
                         {
                             $parent = $xml->xpath('parent::*');
                             $xmlx = $parent[0]->addChild($xml->getName());
                         }
-                    }
-                    else
+                    } else
                     {
                         $xmlx = $xml->addChild($key);
                     }
                     $this->setParameters($xmlx, $value);
-                }
-                else
+                } else
                 {
                     $xml->$key = $value;
                 }

includes/libraries/GeoIp2/WebService/Client.php

Doc Comments +5 added lines

@@ -189,6 +189,11 @@
         return $this->responseFor('insights', 'Insights', $ipAddress);
     }
 
+    /**
+     * @param string $endpoint
+     * @param string $class
+     * @param string $ipAddress
+     */
     private function responseFor($endpoint, $class, $ipAddress)
     {
         $path = implode('/', array(self::$basePath, $endpoint, $ipAddress));

Unused Use Statements -1 removed lines

@@ -9,7 +9,6 @@
 use GeoIp2\Exception\InvalidRequestException;
 use GeoIp2\Exception\OutOfQueriesException;
 use GeoIp2\ProviderInterface;
-use MaxMind\Exception\InvalidInputException;
 use MaxMind\WebService\Client as WsClient;
 
 /**

Spacing +1 added lines, -1 removed lines

@@ -76,7 +76,7 @@
         // This is for backwards compatibility. Do not remove except for a
         // major version bump.
         if (is_string($options)) {
-            $options = array( 'host' => $options );
+            $options = array('host' => $options);
         }
 
         if (!isset($options['host'])) {

includes/libraries/MaxMind/Db/Reader.php

Doc Comments +13 added lines

@@ -115,6 +115,9 @@ 
         return $this->resolveDataPointer($pointer);
     }
 
+    /**
+     * @param string $ipAddress
+     */
     private function findAddressInTree($ipAddress)
     {
         // XXX - could simplify. Done as a byte array to ease porting
@@ -146,6 +149,9 @@ 
     }
 
 
+    /**
+     * @param integer $length
+     */
     private function startNode($length)
     {
         // Check if we are looking up an IPv4 address in an IPv6 tree. If this
@@ -178,6 +184,9 @@ 
         return $node;
     }
 
+    /**
+     * @param integer $index
+     */
     private function readNode($nodeNumber, $index)
     {
         $baseOffset = $nodeNumber * $this->metadata->nodeByteSize;
@@ -230,6 +239,10 @@ 
      * are much faster algorithms (e.g., Boyer-Moore) for this if speed is ever
      * an issue, but I suspect it won't be.
      */
+
+    /**
+     * @param string $filename
+     */
     private function findMetadataStart($filename)
     {
         $handle = $this->fileHandle;