mdaniels5757 / waca

includes/AutoLoader.php

Indentation +29 added lines, -29 removed lines

@@ -13,37 +13,37 @@
  */
 class AutoLoader
 {
-    public static function load($class)
-    {
-        // handle namespaces sensibly
-        if (strpos($class, "Waca") !== false) {
-            // strip off the initial namespace
-            $class = str_replace("Waca\\", "", $class);
+	public static function load($class)
+	{
+		// handle namespaces sensibly
+		if (strpos($class, "Waca") !== false) {
+			// strip off the initial namespace
+			$class = str_replace("Waca\\", "", $class);
 
-            // swap backslashes for forward slashes to map to directory names
-            $class = str_replace("\\", "/", $class);
-        }
+			// swap backslashes for forward slashes to map to directory names
+			$class = str_replace("\\", "/", $class);
+		}
 
-        $paths = array(
-            __DIR__ . '/' . $class . ".php",
-            __DIR__ . '/DataObjects/' . $class . ".php",
-            __DIR__ . '/Providers/' . $class . ".php",
-            __DIR__ . '/Providers/Interfaces/' . $class . ".php",
-            __DIR__ . '/Validation/' . $class . ".php",
-            __DIR__ . '/Helpers/' . $class . ".php",
-            __DIR__ . '/Helpers/Interfaces/' . $class . ".php",
-            __DIR__ . '/' . $class . ".php",
-        );
+		$paths = array(
+			__DIR__ . '/' . $class . ".php",
+			__DIR__ . '/DataObjects/' . $class . ".php",
+			__DIR__ . '/Providers/' . $class . ".php",
+			__DIR__ . '/Providers/Interfaces/' . $class . ".php",
+			__DIR__ . '/Validation/' . $class . ".php",
+			__DIR__ . '/Helpers/' . $class . ".php",
+			__DIR__ . '/Helpers/Interfaces/' . $class . ".php",
+			__DIR__ . '/' . $class . ".php",
+		);
 
-        foreach ($paths as $file) {
-            if (file_exists($file)) {
-                /** @noinspection PhpIncludeInspection */
-                require_once($file);
-            }
+		foreach ($paths as $file) {
+			if (file_exists($file)) {
+				/** @noinspection PhpIncludeInspection */
+				require_once($file);
+			}
 
-            if (class_exists($class)) {
-                return;
-            }
-        }
-    }
+			if (class_exists($class)) {
+				return;
+			}
+		}
+	}
 }

includes/Fragments/RequestListData.php

Braces +2 added lines, -1 removed lines

@@ -38,7 +38,8 @@
         $requestList->requests = $requests;
 
         $userIds = array_map(
-            function(Request $entry) {
+            function(Request $entry)
+            {
                 return $entry->getReserved();
             },
             $requests

Indentation +73 added lines, -73 removed lines

@@ -19,77 +19,77 @@
 
 trait RequestListData
 {
-    // function imports from InternalPageBase etc.
-    protected abstract function getDatabase();
-
-    protected abstract function getXffTrustProvider();
-
-    /** @return SiteConfiguration */
-    protected abstract function getSiteConfiguration();
-
-    protected abstract function barrierTest($action, User $user, $pageName = null);
-
-    /**
-     * @param Request[] $requests
-     *
-     * @return RequestList
-     */
-    protected function prepareRequestData(array $requests) : RequestList
-    {
-        $requestList = new RequestList();
-        $requestList->requests = $requests;
-
-        $userIds = array_map(
-            function(Request $entry) {
-                return $entry->getReserved();
-            },
-            $requests
-        );
-
-        $requestList->userList = UserSearchHelper::get($this->getDatabase())->inIds($userIds)->fetchMap('username');
-
-        $requestList->requestTrustedIp = [];
-        $requestList->relatedIpRequests = [];
-        $requestList->relatedEmailRequests = [];
-
-        foreach ($requests as $request) {
-            $trustedIp = $this->getXffTrustProvider()->getTrustedClientIp(
-                $request->getIp(),
-                $request->getForwardedIp()
-            );
-
-            // TODO: Do we really want to return results from other domains?
-            RequestSearchHelper::get($this->getDatabase(), null)
-                ->byIp($trustedIp)
-                ->withConfirmedEmail()
-                ->excludingPurgedData($this->getSiteConfiguration())
-                ->excludingRequest($request->getId())
-                ->getRecordCount($ipCount);
-
-            RequestSearchHelper::get($this->getDatabase(), null)
-                ->byEmailAddress($request->getEmail())
-                ->withConfirmedEmail()
-                ->excludingPurgedData($this->getSiteConfiguration())
-                ->excludingRequest($request->getId())
-                ->getRecordCount($emailCount);
-
-            $requestList->requestTrustedIp[$request->getId()] = $trustedIp;
-            $requestList->relatedEmailRequests[$request->getId()] = $emailCount;
-            $requestList->relatedIpRequests[$request->getId()] = $ipCount;
-
-            $emailDomain = explode("@", $request->getEmail())[1];
-            $requestList->commonEmail[$request->getId()] = in_array(strtolower($emailDomain), $this->getSiteConfiguration()->getCommonEmailDomains())
-               || $request->getEmail() === $this->getSiteConfiguration()->getDataClearEmail();
-        }
-
-        $currentUser = User::getCurrent($this->getDatabase());
-
-        $requestList->canBan = $this->barrierTest('set', $currentUser, PageBan::class);
-        $requestList->canBreakReservation = $this->barrierTest('force', $currentUser, PageBreakReservation::class);
-        $requestList->showPrivateData = $this->barrierTest('alwaysSeePrivateData', $currentUser, 'RequestData');
-        $requestList->dataClearEmail = $this->getSiteConfiguration()->getDataClearEmail();
-        $requestList->dataClearIp = $this->getSiteConfiguration()->getDataClearIp();
-
-        return $requestList;
-    }
+	// function imports from InternalPageBase etc.
+	protected abstract function getDatabase();
+
+	protected abstract function getXffTrustProvider();
+
+	/** @return SiteConfiguration */
+	protected abstract function getSiteConfiguration();
+
+	protected abstract function barrierTest($action, User $user, $pageName = null);
+
+	/**
+	 * @param Request[] $requests
+	 *
+	 * @return RequestList
+	 */
+	protected function prepareRequestData(array $requests) : RequestList
+	{
+		$requestList = new RequestList();
+		$requestList->requests = $requests;
+
+		$userIds = array_map(
+			function(Request $entry) {
+				return $entry->getReserved();
+			},
+			$requests
+		);
+
+		$requestList->userList = UserSearchHelper::get($this->getDatabase())->inIds($userIds)->fetchMap('username');
+
+		$requestList->requestTrustedIp = [];
+		$requestList->relatedIpRequests = [];
+		$requestList->relatedEmailRequests = [];
+
+		foreach ($requests as $request) {
+			$trustedIp = $this->getXffTrustProvider()->getTrustedClientIp(
+				$request->getIp(),
+				$request->getForwardedIp()
+			);
+
+			// TODO: Do we really want to return results from other domains?
+			RequestSearchHelper::get($this->getDatabase(), null)
+				->byIp($trustedIp)
+				->withConfirmedEmail()
+				->excludingPurgedData($this->getSiteConfiguration())
+				->excludingRequest($request->getId())
+				->getRecordCount($ipCount);
+
+			RequestSearchHelper::get($this->getDatabase(), null)
+				->byEmailAddress($request->getEmail())
+				->withConfirmedEmail()
+				->excludingPurgedData($this->getSiteConfiguration())
+				->excludingRequest($request->getId())
+				->getRecordCount($emailCount);
+
+			$requestList->requestTrustedIp[$request->getId()] = $trustedIp;
+			$requestList->relatedEmailRequests[$request->getId()] = $emailCount;
+			$requestList->relatedIpRequests[$request->getId()] = $ipCount;
+
+			$emailDomain = explode("@", $request->getEmail())[1];
+			$requestList->commonEmail[$request->getId()] = in_array(strtolower($emailDomain), $this->getSiteConfiguration()->getCommonEmailDomains())
+			   || $request->getEmail() === $this->getSiteConfiguration()->getDataClearEmail();
+		}
+
+		$currentUser = User::getCurrent($this->getDatabase());
+
+		$requestList->canBan = $this->barrierTest('set', $currentUser, PageBan::class);
+		$requestList->canBreakReservation = $this->barrierTest('force', $currentUser, PageBreakReservation::class);
+		$requestList->showPrivateData = $this->barrierTest('alwaysSeePrivateData', $currentUser, 'RequestData');
+		$requestList->dataClearEmail = $this->getSiteConfiguration()->getDataClearEmail();
+		$requestList->dataClearIp = $this->getSiteConfiguration()->getDataClearIp();
+
+		return $requestList;
+	}
 }

includes/Helpers/TypeAheadHelper.php

Indentation +31 added lines, -31 removed lines

@@ -12,21 +12,21 @@ 
 
 class TypeAheadHelper implements ITypeAheadHelper
 {
-    private $definedClasses = array();
+	private $definedClasses = array();
 
-    /**
-     * @param string   $class     CSS class to apply this typeahead to.
-     * @param callable $generator Generator function taking no arguments to return an array of strings.
-     */
-    public function defineTypeAheadSource($class, callable $generator)
-    {
-        $dataList = '';
-        foreach ($generator() as $dataItem) {
-            $dataList .= '"' . htmlentities($dataItem) . '", ';
-        }
-        $dataList = "[" . rtrim($dataList, ", ") . "]";
+	/**
+	 * @param string   $class     CSS class to apply this typeahead to.
+	 * @param callable $generator Generator function taking no arguments to return an array of strings.
+	 */
+	public function defineTypeAheadSource($class, callable $generator)
+	{
+		$dataList = '';
+		foreach ($generator() as $dataItem) {
+			$dataList .= '"' . htmlentities($dataItem) . '", ';
+		}
+		$dataList = "[" . rtrim($dataList, ", ") . "]";
 
-        $script = <<<JS
+		$script = <<<JS
 
 $('.{$class}').typeahead({
         hint: true,
@@ -39,32 +39,32 @@ 
 })
 ;
 JS;
-        $this->definedClasses[$class] = $script;
-    }
+		$this->definedClasses[$class] = $script;
+	}
 
-    /**
-     * @return string HTML fragment containing a JS block for typeaheads.
-     */
-    public function getTypeAheadScriptBlock()
-    {
-        $jsBlocks = '';
+	/**
+	 * @return string HTML fragment containing a JS block for typeaheads.
+	 */
+	public function getTypeAheadScriptBlock()
+	{
+		$jsBlocks = '';
 
-        if (count($this->definedClasses) === 0) {
-            return '';
-        }
+		if (count($this->definedClasses) === 0) {
+			return '';
+		}
 
-        foreach ($this->definedClasses as $class => $js) {
-            $jsBlocks = $js . "\r\n\r\n";
-        }
+		foreach ($this->definedClasses as $class => $js) {
+			$jsBlocks = $js . "\r\n\r\n";
+		}
 
-        $data = <<<HTML
+		$data = <<<HTML
 <script type="text/javascript">
 	{$jsBlocks}
 </script>
 HTML;
 
-        $this->definedClasses = array();
+		$this->definedClasses = array();
 
-        return $data;
-    }
+		return $data;
+	}
 }

includes/Helpers/SearchHelpers/UserSearchHelper.php

Indentation +89 added lines, -89 removed lines

@@ -15,73 +15,73 @@ 
 
 class UserSearchHelper extends SearchHelperBase
 {
-    /**
-     * UserSearchHelper constructor.
-     *
-     * @param PdoDatabase $database
-     */
-    public function __construct(PdoDatabase $database)
-    {
-        parent::__construct($database, 'user', User::class);
-    }
-
-    /**
-     * Initiates a search for requests
-     *
-     * @param PdoDatabase $database
-     *
-     * @return UserSearchHelper
-     */
-    public static function get(PdoDatabase $database)
-    {
-        $helper = new UserSearchHelper($database);
-
-        return $helper;
-    }
-
-    /**
-     * @param string $status
-     *
-     * @return $this
-     */
-    public function byStatus($status)
-    {
-        $this->whereClause .= ' AND status = ?';
-        $this->parameterList[] = $status;
-
-        return $this;
-    }
-
-    public function statusIn($statuses)
-    {
-        $this->inClause('status', $statuses);
-
-        return $this;
-    }
-
-    /**
-     * @param string $role
-     *
-     * @return $this
-     */
-    public function byRole($role)
-    {
-        $this->joinClause .= ' INNER JOIN userrole r on origin.id = r.user';
-        $this->whereClause .= ' AND r.role = ?';
-        $this->parameterList[] = $role;
-
-        return $this;
-    }
-
-    /**
-     * @param DateTime $instant
-     *
-     * @return $this
-     */
-    public function lastActiveBefore(DateTime $instant)
-    {
-        $this->whereClause .= ' AND origin.lastactive < ? AND approvaldate.timestamp < ?';
-        $this->joinClause .= <<<'SQLFRAG'
+	/**
+	 * UserSearchHelper constructor.
+	 *
+	 * @param PdoDatabase $database
+	 */
+	public function __construct(PdoDatabase $database)
+	{
+		parent::__construct($database, 'user', User::class);
+	}
+
+	/**
+	 * Initiates a search for requests
+	 *
+	 * @param PdoDatabase $database
+	 *
+	 * @return UserSearchHelper
+	 */
+	public static function get(PdoDatabase $database)
+	{
+		$helper = new UserSearchHelper($database);
+
+		return $helper;
+	}
+
+	/**
+	 * @param string $status
+	 *
+	 * @return $this
+	 */
+	public function byStatus($status)
+	{
+		$this->whereClause .= ' AND status = ?';
+		$this->parameterList[] = $status;
+
+		return $this;
+	}
+
+	public function statusIn($statuses)
+	{
+		$this->inClause('status', $statuses);
+
+		return $this;
+	}
+
+	/**
+	 * @param string $role
+	 *
+	 * @return $this
+	 */
+	public function byRole($role)
+	{
+		$this->joinClause .= ' INNER JOIN userrole r on origin.id = r.user';
+		$this->whereClause .= ' AND r.role = ?';
+		$this->parameterList[] = $role;
+
+		return $this;
+	}
+
+	/**
+	 * @param DateTime $instant
+	 *
+	 * @return $this
+	 */
+	public function lastActiveBefore(DateTime $instant)
+	{
+		$this->whereClause .= ' AND origin.lastactive < ? AND approvaldate.timestamp < ?';
+		$this->joinClause .= <<<'SQLFRAG'
  LEFT JOIN (
     SELECT objectid, MAX(timestamp) timestamp 
     FROM log
@@ -89,16 +89,16 @@ 
     GROUP BY objectid
     ) approvaldate ON approvaldate.objectid = origin.id
 SQLFRAG;
-        $formattedDate = $instant->format("Y-m-d H:i:s");
-        $this->parameterList[] = $formattedDate;
-        $this->parameterList[] = $formattedDate;
+		$formattedDate = $instant->format("Y-m-d H:i:s");
+		$this->parameterList[] = $formattedDate;
+		$this->parameterList[] = $formattedDate;
 
-        return $this;
-    }
+		return $this;
+	}
 
-    public function getRoleMap(&$roleMap)
-    {
-        $query = <<<SQL
+	public function getRoleMap(&$roleMap)
+	{
+		$query = <<<SQL
             SELECT /* UserSearchHelper/roleMap */ 
                   r.user user
                 , group_concat(r.role SEPARATOR ', ') roles 
@@ -107,22 +107,22 @@ 
             GROUP BY r.user
 SQL;
 
-        $statement = $this->database->prepare($query);
-        $statement->execute($this->parameterList);
+		$statement = $this->database->prepare($query);
+		$statement->execute($this->parameterList);
 
-        $roleMap = array();
-        foreach ($statement->fetchAll(PDO::FETCH_ASSOC) as $row) {
-            $roleMap[$row['user']] = $row['roles'];
-        }
+		$roleMap = array();
+		foreach ($statement->fetchAll(PDO::FETCH_ASSOC) as $row) {
+			$roleMap[$row['user']] = $row['roles'];
+		}
 
-        return $this;
-    }
+		return $this;
+	}
 
-    public function withReservedRequest()
-    {
-        $this->joinClause = ' INNER JOIN request req ON req.reserved = origin.id';
-        $this->groupByClause = ' GROUP BY origin.id, origin.username';
+	public function withReservedRequest()
+	{
+		$this->joinClause = ' INNER JOIN request req ON req.reserved = origin.id';
+		$this->groupByClause = ' GROUP BY origin.id, origin.username';
 
-        return $this->fetchMap('username');
-    }
+		return $this->fetchMap('username');
+	}
 }

includes/Helpers/Interfaces/IMediaWikiClient.php

Indentation +1 added lines, -1 removed lines

@@ -10,5 +10,5 @@
 
 interface IMediaWikiClient
 {
-    function doApiCall($params, $method);
+	function doApiCall($params, $method);
 }
\ No newline at end of file

includes/Tasks/ApiPageBase.php

Indentation +18 added lines, -18 removed lines

@@ -12,24 +12,24 @@
 
 abstract class ApiPageBase extends TaskBase implements IRoutedTask, IApiAction
 {
-    final public function execute()
-    {
-        $this->main();
-    }
+	final public function execute()
+	{
+		$this->main();
+	}
 
-    /**
-     * @param string $routeName
-     */
-    public function setRoute($routeName)
-    {
-        // no-op
-    }
+	/**
+	 * @param string $routeName
+	 */
+	public function setRoute($routeName)
+	{
+		// no-op
+	}
 
-    /**
-     * @return string
-     */
-    public function getRouteName()
-    {
-        return 'main';
-    }
+	/**
+	 * @return string
+	 */
+	public function getRouteName()
+	{
+		return 'main';
+	}
 }

includes/Tasks/XmlApiPageBase.php

Indentation +73 added lines, -73 removed lines

@@ -17,77 +17,77 @@
 
 abstract class XmlApiPageBase extends ApiPageBase implements IXmlApiAction
 {
-    /**
-     * API result document
-     * @var DOMDocument
-     */
-    protected $document;
-
-    public function __construct()
-    {
-        $this->document = new DOMDocument('1.0');
-    }
-
-    /**
-     * Main function for this page, when no specific actions are called.
-     *
-     * @throws ApiException
-     * @return void
-     */
-    final protected function main()
-    {
-        if (headers_sent()) {
-            throw new ApiException('Headers have already been sent - this indicates a bug in the application!');
-        }
-
-        header("Content-Type: text/xml");
-
-        // javascript access control
-        $httpOrigin = WebRequest::origin();
-
-        if ($httpOrigin !== null) {
-            $CORSallowed = $this->getSiteConfiguration()->getCrossOriginResourceSharingHosts();
-
-            if (in_array($httpOrigin, $CORSallowed)) {
-                header("Access-Control-Allow-Origin: " . $httpOrigin);
-            }
-        }
-
-        $responseData = $this->runApiPage();
-
-        ob_end_clean();
-        print($responseData);
-        ob_start();
-    }
-
-    /**
-     * Method that runs API action
-     *
-     * @param DOMElement $apiDocument
-     *
-     * @return DOMElement
-     */
-    abstract public function executeApiAction(DOMElement $apiDocument);
-
-    /**
-     * @return string
-     */
-    final public function runApiPage()
-    {
-        $apiDocument = $this->document->createElement("api");
-
-        try {
-            $apiDocument = $this->executeApiAction($apiDocument);
-        }
-            /** @noinspection PhpRedundantCatchClauseInspection */
-        catch (ApiException $ex) {
-            $exception = $this->document->createElement("error");
-            $exception->setAttribute("message", $ex->getMessage());
-            $apiDocument->appendChild($exception);
-        }
-
-        $this->document->appendChild($apiDocument);
-
-        return $this->document->saveXML();
-    }
+	/**
+	 * API result document
+	 * @var DOMDocument
+	 */
+	protected $document;
+
+	public function __construct()
+	{
+		$this->document = new DOMDocument('1.0');
+	}
+
+	/**
+	 * Main function for this page, when no specific actions are called.
+	 *
+	 * @throws ApiException
+	 * @return void
+	 */
+	final protected function main()
+	{
+		if (headers_sent()) {
+			throw new ApiException('Headers have already been sent - this indicates a bug in the application!');
+		}
+
+		header("Content-Type: text/xml");
+
+		// javascript access control
+		$httpOrigin = WebRequest::origin();
+
+		if ($httpOrigin !== null) {
+			$CORSallowed = $this->getSiteConfiguration()->getCrossOriginResourceSharingHosts();
+
+			if (in_array($httpOrigin, $CORSallowed)) {
+				header("Access-Control-Allow-Origin: " . $httpOrigin);
+			}
+		}
+
+		$responseData = $this->runApiPage();
+
+		ob_end_clean();
+		print($responseData);
+		ob_start();
+	}
+
+	/**
+	 * Method that runs API action
+	 *
+	 * @param DOMElement $apiDocument
+	 *
+	 * @return DOMElement
+	 */
+	abstract public function executeApiAction(DOMElement $apiDocument);
+
+	/**
+	 * @return string
+	 */
+	final public function runApiPage()
+	{
+		$apiDocument = $this->document->createElement("api");
+
+		try {
+			$apiDocument = $this->executeApiAction($apiDocument);
+		}
+			/** @noinspection PhpRedundantCatchClauseInspection */
+		catch (ApiException $ex) {
+			$exception = $this->document->createElement("error");
+			$exception->setAttribute("message", $ex->getMessage());
+			$apiDocument->appendChild($exception);
+		}
+
+		$this->document->appendChild($apiDocument);
+
+		return $this->document->saveXML();
+	}
 }

includes/ConsoleTasks/PrecacheGeolocationTask.php

Indentation +53 added lines, -53 removed lines

@@ -14,30 +14,30 @@ 
 
 class PrecacheGeolocationTask extends ConsoleTaskBase
 {
-    public function execute()
-    {
-        $database = $this->getDatabase();
-        $locationProvider = $this->getLocationProvider();
+	public function execute()
+	{
+		$database = $this->getDatabase();
+		$locationProvider = $this->getLocationProvider();
 
-        while (true) {
-            echo "Beginning txn\n";
-            $database->beginTransaction();
+		while (true) {
+			echo "Beginning txn\n";
+			$database->beginTransaction();
 
-            try {
-                echo ". Fetching data...\n";
+			try {
+				echo ". Fetching data...\n";
 
-                // fetch a bunch of un-geolocated IPs from the database.
-                // Note we have to parse the forwardedip field in the database so we can test against the geolocation
-                // table.
-                //
-                // This guarantees we get ten unlocated IPs back, unless there actually aren't 10 available.
-                //
-                // Alternatives include downloading a small set of forwarded IPs, splitting it in PHP, constructing an
-                // IN() clause dynamically, sending that back to the database to check if there are geolocation entries,
-                // then repeating until we have 10 to process - and the fact that we'd have to potentially retrieve all
-                // IPs from the database before we find any at all. This way keeps all of that legwork in the database,
-                // at the cost of a more complex query.
-                $statement = $database->query(<<<SQL
+				// fetch a bunch of un-geolocated IPs from the database.
+				// Note we have to parse the forwardedip field in the database so we can test against the geolocation
+				// table.
+				//
+				// This guarantees we get ten unlocated IPs back, unless there actually aren't 10 available.
+				//
+				// Alternatives include downloading a small set of forwarded IPs, splitting it in PHP, constructing an
+				// IN() clause dynamically, sending that back to the database to check if there are geolocation entries,
+				// then repeating until we have 10 to process - and the fact that we'd have to potentially retrieve all
+				// IPs from the database before we find any at all. This way keeps all of that legwork in the database,
+				// at the cost of a more complex query.
+				$statement = $database->query(<<<SQL
                     SELECT /* PrecacheGeolocationTask */ p.prox
                     FROM (
                       SELECT trim(substring_index(substring_index(r.forwardedip, ',', n.n), ',', -1)) prox
@@ -54,42 +54,42 @@ 
                     WHERE NOT EXISTS (SELECT 1 FROM geolocation g WHERE g.address = p.prox FOR UPDATE)
                     LIMIT 10;
 SQL
-                );
+				);
 
-                $missingIps = $statement->fetchAll(PDO::FETCH_COLUMN);
+				$missingIps = $statement->fetchAll(PDO::FETCH_COLUMN);
 
-                $count = count($missingIps);
-                if ($count === 0) {
-                    echo ". Found nothing to do.\n";
-                    break;
-                }
+				$count = count($missingIps);
+				if ($count === 0) {
+					echo ". Found nothing to do.\n";
+					break;
+				}
 
-                echo ". Picked {$count} IP addresses\n";
+				echo ". Picked {$count} IP addresses\n";
 
-                foreach ($missingIps as $ip) {
-                    echo ". . Getting location for {$ip}...\n";
-                    $data = json_encode($locationProvider->getIpLocation($ip));
-                    echo ". . . {$data}\n";
-                }
+				foreach ($missingIps as $ip) {
+					echo ". . Getting location for {$ip}...\n";
+					$data = json_encode($locationProvider->getIpLocation($ip));
+					echo ". . . {$data}\n";
+				}
 
-                echo ". IP location fetch complete.\n";
-                $database->commit();
-                echo ". Committed txn.\n";
-            }
-            catch (Exception $ex) {
-                echo ". Encountered exception: " . $ex->getMessage() . "\n";
-                $database->rollBack();
-                echo ". Rolled back txn\n";
-                throw $ex;
-            }
-            finally {
-                if ($database->hasActiveTransaction()) {
-                    $database->rollBack();
-                    echo ". Rolled back txn\n";
-                }
-            }
-        }
+				echo ". IP location fetch complete.\n";
+				$database->commit();
+				echo ". Committed txn.\n";
+			}
+			catch (Exception $ex) {
+				echo ". Encountered exception: " . $ex->getMessage() . "\n";
+				$database->rollBack();
+				echo ". Rolled back txn\n";
+				throw $ex;
+			}
+			finally {
+				if ($database->hasActiveTransaction()) {
+					$database->rollBack();
+					echo ". Rolled back txn\n";
+				}
+			}
+		}
 
-        echo "Done.\n";
-    }
+		echo "Done.\n";
+	}
 }
\ No newline at end of file

includes/Helpers/Interfaces/IOAuthProtocolHelper.php

Indentation +45 added lines, -45 removed lines

@@ -15,53 +15,53 @@
 
 interface IOAuthProtocolHelper
 {
-    /**
-     * @return stdClass
-     *
-     * @throws Exception
-     * @throws CurlException
-     */
-    public function getRequestToken();
+	/**
+	 * @return stdClass
+	 *
+	 * @throws Exception
+	 * @throws CurlException
+	 */
+	public function getRequestToken();
 
-    /**
-     * @param string $requestToken
-     *
-     * @return string
-     */
-    public function getAuthoriseUrl($requestToken);
+	/**
+	 * @param string $requestToken
+	 *
+	 * @return string
+	 */
+	public function getAuthoriseUrl($requestToken);
 
-    /**
-     * @param string $oauthRequestToken
-     * @param string $oauthRequestSecret
-     * @param string $oauthVerifier
-     *
-     * @return stdClass
-     * @throws CurlException
-     * @throws Exception
-     */
-    public function callbackCompleted($oauthRequestToken, $oauthRequestSecret, $oauthVerifier);
+	/**
+	 * @param string $oauthRequestToken
+	 * @param string $oauthRequestSecret
+	 * @param string $oauthVerifier
+	 *
+	 * @return stdClass
+	 * @throws CurlException
+	 * @throws Exception
+	 */
+	public function callbackCompleted($oauthRequestToken, $oauthRequestSecret, $oauthVerifier);
 
-    /**
-     * @param string $oauthAccessToken
-     * @param string $oauthAccessSecret
-     *
-     * @return stdClass
-     * @throws CurlException
-     * @throws Exception
-     * @throws \MediaWiki\OAuthClient\Exception
-     */
-    public function getIdentityTicket($oauthAccessToken, $oauthAccessSecret);
+	/**
+	 * @param string $oauthAccessToken
+	 * @param string $oauthAccessSecret
+	 *
+	 * @return stdClass
+	 * @throws CurlException
+	 * @throws Exception
+	 * @throws \MediaWiki\OAuthClient\Exception
+	 */
+	public function getIdentityTicket($oauthAccessToken, $oauthAccessSecret);
 
-    /**
-     * @param array  $apiParams    array of parameters to send to the API
-     * @param string $accessToken  user's access token
-     * @param string $accessSecret user's secret
-     * @param string $method       HTTP method
-     *
-     * @return stdClass
-     * @throws ApplicationLogicException
-     * @throws CurlException
-     * @throws Exception
-     */
-    public function apiCall($apiParams, $accessToken, $accessSecret, $method = 'GET');
+	/**
+	 * @param array  $apiParams    array of parameters to send to the API
+	 * @param string $accessToken  user's access token
+	 * @param string $accessSecret user's secret
+	 * @param string $method       HTTP method
+	 *
+	 * @return stdClass
+	 * @throws ApplicationLogicException
+	 * @throws CurlException
+	 * @throws Exception
+	 */
+	public function apiCall($apiParams, $accessToken, $accessSecret, $method = 'GET');
 }
\ No newline at end of file