silverstripe / deploynaut

code/control/EmailMessagingService.php

Indentation +87 added lines, -87 removed lines

@@ -7,101 +7,101 @@
 class EmailMessagingService implements ConfirmationMessagingService
 {
 
-    /**
-     * @config
-     * @var string
-     */
-    private static $default_from = 'deploynaut@silverstripe.com';
+	/**
+	 * @config
+	 * @var string
+	 */
+	private static $default_from = 'deploynaut@silverstripe.com';
 
-    /**
-     * @config
-     * @var string
-     */
-    private static $default_subject = 'Deploynaut notification';
+	/**
+	 * @config
+	 * @var string
+	 */
+	private static $default_subject = 'Deploynaut notification';
 
-    public function sendMessage($source, $message, $recipients, $arguments = array())
-    {
-        $from = empty($arguments['from'])
-            ? Config::inst()->get(get_class($this), 'default_from')
-            : $arguments['from'];
-        $subject = empty($arguments['subject'])
-            ? Config::inst()->get(get_class($this), 'default_subject')
-            : $arguments['subject'];
+	public function sendMessage($source, $message, $recipients, $arguments = array())
+	{
+		$from = empty($arguments['from'])
+			? Config::inst()->get(get_class($this), 'default_from')
+			: $arguments['from'];
+		$subject = empty($arguments['subject'])
+			? Config::inst()->get(get_class($this), 'default_subject')
+			: $arguments['subject'];
 
-        // Split users and send individually
-        $this->sendIndividualMessages($source, $message, $recipients, $from, $subject);
-    }
+		// Split users and send individually
+		$this->sendIndividualMessages($source, $message, $recipients, $from, $subject);
+	}
 
-    /**
-     * Separates recipients into individual users
-     *
-     * @param PipelineStep $source Source client step
-     * @param string $message Plain text message
-     * @param mixed $recipients Either a Member object, string, or array of strings or Member objects
-     * @param string $from
-     * @param string $subject
-     * @return boolean True if success
-     */
-    protected function sendIndividualMessages($source, $message, $recipients, $from, $subject)
-    {
-        // Split recipients that are comma separated
-        if (is_string($recipients) && stripos($recipients, ',')) {
-            $recipients = explode(',', $recipients);
-            return $this->sendIndividualMessages($source, $message, $recipients, $from, $subject);
-        }
+	/**
+	 * Separates recipients into individual users
+	 *
+	 * @param PipelineStep $source Source client step
+	 * @param string $message Plain text message
+	 * @param mixed $recipients Either a Member object, string, or array of strings or Member objects
+	 * @param string $from
+	 * @param string $subject
+	 * @return boolean True if success
+	 */
+	protected function sendIndividualMessages($source, $message, $recipients, $from, $subject)
+	{
+		// Split recipients that are comma separated
+		if (is_string($recipients) && stripos($recipients, ',')) {
+			$recipients = explode(',', $recipients);
+			return $this->sendIndividualMessages($source, $message, $recipients, $from, $subject);
+		}
 
-        // Iterate through arrays
-        if (is_array($recipients) || $recipients instanceof SS_List) {
-            foreach ($recipients as $recipient) {
-                $this->sendIndividualMessages($source, $message, $recipient, $from, $subject);
-            }
-            return true;
-        }
+		// Iterate through arrays
+		if (is_array($recipients) || $recipients instanceof SS_List) {
+			foreach ($recipients as $recipient) {
+				$this->sendIndividualMessages($source, $message, $recipient, $from, $subject);
+			}
+			return true;
+		}
 
-        if ($recipients) {
-            $this->sendMessageTo($source, $from, $recipients, $subject, $message);
-            return true;
-        }
+		if ($recipients) {
+			$this->sendMessageTo($source, $from, $recipients, $subject, $message);
+			return true;
+		}
 
-        // Can't send to empty recipient
-        return false;
-    }
+		// Can't send to empty recipient
+		return false;
+	}
 
 
-    /**
-     * Send an message to a single recipient
-     *
-     * @param PipelineStep $source Client step
-     * @param string $from
-     * @param string|Member $to
-     * @param string $subject
-     * @param string $body
-     */
-    protected function sendMessageTo($source, $from, $to, $subject, $body)
-    {
-        if ($to instanceof Member) {
-            $to = $to->Email;
-        }
-        $this->sendViaEmail($source, $from, $to, $subject, $body);
-    }
+	/**
+	 * Send an message to a single recipient
+	 *
+	 * @param PipelineStep $source Client step
+	 * @param string $from
+	 * @param string|Member $to
+	 * @param string $subject
+	 * @param string $body
+	 */
+	protected function sendMessageTo($source, $from, $to, $subject, $body)
+	{
+		if ($to instanceof Member) {
+			$to = $to->Email;
+		}
+		$this->sendViaEmail($source, $from, $to, $subject, $body);
+	}
 
-    /**
-     * Send an email to a single recipient
-     *
-     * @param PipelineStep $source Client step
-     * @param string $from
-     * @param string $to
-     * @param string $subject
-     * @param string $body
-     */
-    protected function sendViaEmail($source, $from, $to, $subject, $body)
-    {
-        $email = new Email($from, $to, $subject, $body);
-        if ($source->getDryRun()) {
-            $source->log("[Skipped] Sent message to $to (subject: $subject)");
-        } else {
-            $email->sendPlain();
-            $source->log("Sent message to $to (subject: $subject)");
-        }
-    }
+	/**
+	 * Send an email to a single recipient
+	 *
+	 * @param PipelineStep $source Client step
+	 * @param string $from
+	 * @param string $to
+	 * @param string $subject
+	 * @param string $body
+	 */
+	protected function sendViaEmail($source, $from, $to, $subject, $body)
+	{
+		$email = new Email($from, $to, $subject, $body);
+		if ($source->getDryRun()) {
+			$source->log("[Skipped] Sent message to $to (subject: $subject)");
+		} else {
+			$email->sendPlain();
+			$source->log("Sent message to $to (subject: $subject)");
+		}
+	}
 }

code/control/PipelineController.php

Indentation +71 added lines, -71 removed lines

@@ -3,85 +3,85 @@
 class PipelineController extends Controller
 {
 
-    private static $allowed_actions = array(
-        'abort',
-        'log',
-        'step'
-    );
+	private static $allowed_actions = array(
+		'abort',
+		'log',
+		'step'
+	);
 
-    protected $controller = null;
+	protected $controller = null;
 
-    protected $pipeline = null;
+	protected $pipeline = null;
 
-    public function __construct($controller, Pipeline $pipeline)
-    {
-        $this->controller = $controller;
-        parent::__construct();
-        $this->pipeline = $pipeline;
-    }
+	public function __construct($controller, Pipeline $pipeline)
+	{
+		$this->controller = $controller;
+		parent::__construct();
+		$this->pipeline = $pipeline;
+	}
 
-    /**
-     * Shows status of this pipeline
-     */
-    public function index()
-    {
-        return $this->controller->customise(new ArrayData(array(
-            'Pipeline' => $this->pipeline
-        )))->renderWith('DNRoot_pipeline');
-    }
+	/**
+	 * Shows status of this pipeline
+	 */
+	public function index()
+	{
+		return $this->controller->customise(new ArrayData(array(
+			'Pipeline' => $this->pipeline
+		)))->renderWith('DNRoot_pipeline');
+	}
 
-    /**
-     * Get log for this pipeline
-     */
-    public function log()
-    {
-        $log = $this->pipeline->getLogger();
-        if ($log->exists()) {
-            $content = $log->content();
-        } else {
-            $content = 'Waiting for action to start';
-        }
+	/**
+	 * Get log for this pipeline
+	 */
+	public function log()
+	{
+		$log = $this->pipeline->getLogger();
+		if ($log->exists()) {
+			$content = $log->content();
+		} else {
+			$content = 'Waiting for action to start';
+		}
 
-        $sendJSON = (strpos($this->request->getHeader('Accept'), 'application/json') !== false)
-            || $this->request->getExtension() == 'json';
+		$sendJSON = (strpos($this->request->getHeader('Accept'), 'application/json') !== false)
+			|| $this->request->getExtension() == 'json';
 
-        $content = preg_replace('/(?:(?:\r\n|\r|\n)\s*){2}/s', "\n", $content);
-        if ($sendJSON) {
-            $this->response->addHeader("Content-type", "application/json");
-            return json_encode(array(
-                'status' => $this->pipeline->Status,
-                'content' => $content,
-            ));
-        } else {
-            $this->response->addHeader("Content-type", "text/plain");
-            return $content;
-        }
-    }
+		$content = preg_replace('/(?:(?:\r\n|\r|\n)\s*){2}/s', "\n", $content);
+		if ($sendJSON) {
+			$this->response->addHeader("Content-type", "application/json");
+			return json_encode(array(
+				'status' => $this->pipeline->Status,
+				'content' => $content,
+			));
+		} else {
+			$this->response->addHeader("Content-type", "text/plain");
+			return $content;
+		}
+	}
 
-    /**
-     * Aborts the current pipeline
-     */
-    public function abort()
-    {
-        if ($this->pipeline->canAbort()) {
-            $this->pipeline->markAborted();
-        }
-        return $this->redirect($this->pipeline->Environment()->Link());
-    }
+	/**
+	 * Aborts the current pipeline
+	 */
+	public function abort()
+	{
+		if ($this->pipeline->canAbort()) {
+			$this->pipeline->markAborted();
+		}
+		return $this->redirect($this->pipeline->Environment()->Link());
+	}
 
-    /**
-     * Perform an action on the current pipeline step
-     */
-    public function step()
-    {
-        $action = $this->request->param('ID');
-        $step = $this->pipeline->CurrentStep();
+	/**
+	 * Perform an action on the current pipeline step
+	 */
+	public function step()
+	{
+		$action = $this->request->param('ID');
+		$step = $this->pipeline->CurrentStep();
 
-        // Check if the action is available on this step
-        if ($step && ($actions = $step->allowedActions()) && isset($actions[$action])) {
-            // Execute this action, allowing it to override the httpresponse given
-            $step->$action();
-        }
-        return $this->redirect($this->pipeline->Environment()->Link());
-    }
+		// Check if the action is available on this step
+		if ($step && ($actions = $step->allowedActions()) && isset($actions[$action])) {
+			// Execute this action, allowing it to override the httpresponse given
+			$step->$action();
+		}
+		return $this->redirect($this->pipeline->Environment()->Link());
+	}
 }

code/dbtypes/GitSHA.php

Indentation +23 added lines, -23 removed lines

@@ -3,29 +3,29 @@
 class GitSHA extends Varchar
 {
 
-    /**
-     * @param string|null $name
-     * @param array $options
-     */
-    public function __construct($name = null, $options = array())
-    {
-        // Size should probably be 40, but to avoid schema change leave for now
-        parent::__construct($name, 255, $options);
-    }
+	/**
+	 * @param string|null $name
+	 * @param array $options
+	 */
+	public function __construct($name = null, $options = array())
+	{
+		// Size should probably be 40, but to avoid schema change leave for now
+		parent::__construct($name, 255, $options);
+	}
 
-    /**
-     * @return string
-     */
-    public function FullHash()
-    {
-        return $this->value;
-    }
+	/**
+	 * @return string
+	 */
+	public function FullHash()
+	{
+		return $this->value;
+	}
 
-    /**
-     * @return string
-     */
-    public function ShortHash()
-    {
-        return substr($this->value, 0, 7);
-    }
+	/**
+	 * @return string
+	 */
+	public function ShortHash()
+	{
+		return substr($this->value, 0, 7);
+	}
 }

code/events/DeploynautEventInterface.php

Indentation +22 added lines, -22 removed lines

@@ -3,29 +3,29 @@
 interface DeploynautEventInterface extends \League\Event\EventInterface
 {
 
-    /**
-     * Should return the event name, e.g 'event.name'
-     *
-     * @return string
-     */
-    public function getName();
+	/**
+	 * Should return the event name, e.g 'event.name'
+	 *
+	 * @return string
+	 */
+	public function getName();
 
-    /**
-     * Get the value for the $keyname or empty string
-     *
-     * @param string $keyname
-     *
-     * @return string
-     */
-    public function get($keyname);
+	/**
+	 * Get the value for the $keyname or empty string
+	 *
+	 * @param string $keyname
+	 *
+	 * @return string
+	 */
+	public function get($keyname);
 
-    /**
-     * @return array
-     */
-    public function asArray();
+	/**
+	 * @return array
+	 */
+	public function asArray();
 
-    /**
-     * @return string - returns the data as json
-     */
-    public function asJSON();
+	/**
+	 * @return string - returns the data as json
+	 */
+	public function asJSON();
 }

code/events/GenericEvent.php

Indentation +72 added lines, -72 removed lines

@@ -23,83 +23,83 @@
 class GenericEvent extends AbstractEvent implements DeploynautEventInterface
 {
 
-    /**
-     * @var string
-     */
-    protected $name = '';
+	/**
+	 * @var string
+	 */
+	protected $name = '';
 
-    /**
-     * @var array
-     */
-    protected $data = [];
+	/**
+	 * @var array
+	 */
+	protected $data = [];
 
-    /**
-     * @param string $name - event name
-     * @param array $data
-     *
-     * @return GenericEvent
-     */
-    public static function create($name, array $data)
-    {
-        return Injector::inst()->createWithArgs(get_called_class(), [$name, $data]);
-    }
+	/**
+	 * @param string $name - event name
+	 * @param array $data
+	 *
+	 * @return GenericEvent
+	 */
+	public static function create($name, array $data)
+	{
+		return Injector::inst()->createWithArgs(get_called_class(), [$name, $data]);
+	}
 
-    /**
-     * Event constructor
-     *
-     * @param string $eventName - e.g. 'event.name'
-     * @param array $data
-     */
-    public function __construct($eventName, array $data)
-    {
-        $this->name = $eventName;
-        $this->data = $data;
-    }
+	/**
+	 * Event constructor
+	 *
+	 * @param string $eventName - e.g. 'event.name'
+	 * @param array $data
+	 */
+	public function __construct($eventName, array $data)
+	{
+		$this->name = $eventName;
+		$this->data = $data;
+	}
 
-    /**
-     * get the event name, e.g 'event.name'
-     *
-     * @return string
-     */
-    public function getName()
-    {
-        return $this->name;
-    }
+	/**
+	 * get the event name, e.g 'event.name'
+	 *
+	 * @return string
+	 */
+	public function getName()
+	{
+		return $this->name;
+	}
 
-    /**
-     * Get the value for the $keyname or empty string
-     *
-     * @param string $keyname
-     *
-     * @return string
-     */
-    public function get($keyname)
-    {
-        if (array_key_exists($keyname, $this->data)) {
-            return $this->data[$keyname];
-        }
-        return '';
-    }
+	/**
+	 * Get the value for the $keyname or empty string
+	 *
+	 * @param string $keyname
+	 *
+	 * @return string
+	 */
+	public function get($keyname)
+	{
+		if (array_key_exists($keyname, $this->data)) {
+			return $this->data[$keyname];
+		}
+		return '';
+	}
 
-    /**
-     * return all the data as an array, including the event name as 'event';
-     *
-     * @return array
-     */
-    public function asArray()
-    {
-        $payload = $this->data;
-        $payload['event'] = $this->getName();
-        return $payload;
-    }
+	/**
+	 * return all the data as an array, including the event name as 'event';
+	 *
+	 * @return array
+	 */
+	public function asArray()
+	{
+		$payload = $this->data;
+		$payload['event'] = $this->getName();
+		return $payload;
+	}
 
-    /**
-     * return all the data as a JSON string, including the event name as 'event';
-     *
-     * @return string - returns the data as json
-     */
-    public function asJSON()
-    {
-        return json_encode($this->asArray());
-    }
+	/**
+	 * return all the data as a JSON string, including the event name as 'event';
+	 *
+	 * @return string - returns the data as json
+	 */
+	public function asJSON()
+	{
+		return json_encode($this->asArray());
+	}
 }

code/extensions/DeploynautFileExtension.php

Indentation +16 added lines, -16 removed lines

@@ -1,22 +1,22 @@
 <?php
 class DeploynautFileExtension extends DataExtension
 {
-    /**
-     * If this file is attached to a {@link DNDataArchive}, then we need to check security permissions, to ensure the
-     * currently logged in {@link Member} can download the file.
-     *
-     * This uses the secureassets module to provide the security of these assets.
-     */
-    public function canDownload()
-    {
-        $member = Member::currentUser();
-        $file = $this->owner;
-        $archive = DNDataArchive::get()->filter('ArchiveFileID', $file->ID)->First();
+	/**
+	 * If this file is attached to a {@link DNDataArchive}, then we need to check security permissions, to ensure the
+	 * currently logged in {@link Member} can download the file.
+	 *
+	 * This uses the secureassets module to provide the security of these assets.
+	 */
+	public function canDownload()
+	{
+		$member = Member::currentUser();
+		$file = $this->owner;
+		$archive = DNDataArchive::get()->filter('ArchiveFileID', $file->ID)->First();
 
-        if ($archive) {
-            return $archive->canDownload($member);
-        }
+		if ($archive) {
+			return $archive->canDownload($member);
+		}
 
-        return true; // By default, files can be downloaded from assets/ normally
-    }
+		return true; // By default, files can be downloaded from assets/ normally
+	}
 }

code/extensions/DeploynautSecurityExtension.php

Indentation +9 added lines, -9 removed lines

@@ -3,13 +3,13 @@
 class DeploynautSecurityExtension extends Extension
 {
 
-    public function onAfterInit()
-    {
-        // the 'ping' action is called via AJAX from the /admin and will cause
-        // the admin section to 'grow' over time. We only need the css and js
-        // for login, reset, logout and so on.
-        if (!$this->owner->getRequest()->isAjax()) {
-            DNRoot::include_requirements();
-        }
-    }
+	public function onAfterInit()
+	{
+		// the 'ping' action is called via AJAX from the /admin and will cause
+		// the admin section to 'grow' over time. We only need the css and js
+		// for login, reset, logout and so on.
+		if (!$this->owner->getRequest()->isAjax()) {
+			DNRoot::include_requirements();
+		}
+	}
 }

code/extensions/FrontendLink.php

Indentation +11 added lines, -11 removed lines

@@ -3,15 +3,15 @@
 class FrontendLink extends DataExtension
 {
 
-    public function updateItemEditForm($form)
-    {
-        if ($this->owner->record->hasMethod('Link')) {
-            $link = sprintf(
-                '<a style="margin: 0.5em" target="deploynaut-frontend" href="%s">Preview &raquo;</a>',
-                $this->owner->record->Link()
-            );
-            $actions = $form->Actions();
-            $actions->push(LiteralField::create('FrontendLink', $link));
-        }
-    }
+	public function updateItemEditForm($form)
+	{
+		if ($this->owner->record->hasMethod('Link')) {
+			$link = sprintf(
+				'<a style="margin: 0.5em" target="deploynaut-frontend" href="%s">Preview &raquo;</a>',
+				$this->owner->record->Link()
+			);
+			$actions = $form->Actions();
+			$actions->push(LiteralField::create('FrontendLink', $link));
+		}
+	}
 }

code/extensions/ProjectMemberExtension.php

Indentation +6 added lines, -6 removed lines

@@ -3,10 +3,10 @@
 class ProjectMemberExtension extends DataExtension
 {
 
-    /**
-     * @var array
-     */
-    private static $belongs_many_many = array(
-        'StarredProjects' => 'DNProject'
-    );
+	/**
+	 * @var array
+	 */
+	private static $belongs_many_many = array(
+		'StarredProjects' => 'DNProject'
+	);
 }