silverstripe / deploynaut

code/control/EmailMessagingService.php

Braces +5 added lines, -10 removed lines

@@ -4,8 +4,7 @@ 
  * @package deploynaut
  * @subpackage control
  */
-class EmailMessagingService implements ConfirmationMessagingService
-{
+class EmailMessagingService implements ConfirmationMessagingService {
 
     /**
      * @config
@@ -19,8 +18,7 @@ 
      */
     private static $default_subject = 'Deploynaut notification';
 
-    public function sendMessage($source, $message, $recipients, $arguments = array())
-    {
+    public function sendMessage($source, $message, $recipients, $arguments = array()) {
         $from = empty($arguments['from'])
             ? Config::inst()->get(get_class($this), 'default_from')
             : $arguments['from'];
@@ -42,8 +40,7 @@ 
      * @param string $subject
      * @return boolean True if success
      */
-    protected function sendIndividualMessages($source, $message, $recipients, $from, $subject)
-    {
+    protected function sendIndividualMessages($source, $message, $recipients, $from, $subject) {
         // Split recipients that are comma separated
         if (is_string($recipients) && stripos($recipients, ',')) {
             $recipients = explode(',', $recipients);
@@ -77,8 +74,7 @@ 
      * @param string $subject
      * @param string $body
      */
-    protected function sendMessageTo($source, $from, $to, $subject, $body)
-    {
+    protected function sendMessageTo($source, $from, $to, $subject, $body) {
         if ($to instanceof Member) {
             $to = $to->Email;
         }
@@ -94,8 +90,7 @@ 
      * @param string $subject
      * @param string $body
      */
-    protected function sendViaEmail($source, $from, $to, $subject, $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)");

code/control/PipelineController.php

Braces +6 added lines, -12 removed lines

@@ -1,7 +1,6 @@ 
 <?php
 
-class PipelineController extends Controller
-{
+class PipelineController extends Controller {
 
     private static $allowed_actions = array(
         'abort',
@@ -13,8 +12,7 @@ 
 
     protected $pipeline = null;
 
-    public function __construct($controller, Pipeline $pipeline)
-    {
+    public function __construct($controller, Pipeline $pipeline) {
         $this->controller = $controller;
         parent::__construct();
         $this->pipeline = $pipeline;
@@ -23,8 +21,7 @@ 
     /**
      * Shows status of this pipeline
      */
-    public function index()
-    {
+    public function index() {
         return $this->controller->customise(new ArrayData(array(
             'Pipeline' => $this->pipeline
         )))->renderWith('DNRoot_pipeline');
@@ -33,8 +30,7 @@ 
     /**
      * Get log for this pipeline
      */
-    public function log()
-    {
+    public function log() {
         $log = $this->pipeline->getLogger();
         if ($log->exists()) {
             $content = $log->content();
@@ -61,8 +57,7 @@ 
     /**
      * Aborts the current pipeline
      */
-    public function abort()
-    {
+    public function abort() {
         if ($this->pipeline->canAbort()) {
             $this->pipeline->markAborted();
         }
@@ -72,8 +67,7 @@ 
     /**
      * Perform an action on the current pipeline step
      */
-    public function step()
-    {
+    public function step() {
         $action = $this->request->param('ID');
         $step = $this->pipeline->CurrentStep();
 

code/dbtypes/GitSHA.php

Braces +4 added lines, -8 removed lines

@@ -1,14 +1,12 @@ 
 <?php
 
-class GitSHA extends Varchar
-{
+class GitSHA extends Varchar {
 
     /**
      * @param string|null $name
      * @param array $options
      */
-    public function __construct($name = null, $options = array())
-    {
+    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);
     }
@@ -16,16 +14,14 @@ 
     /**
      * @return string
      */
-    public function FullHash()
-    {
+    public function FullHash() {
         return $this->value;
     }
 
     /**
      * @return string
      */
-    public function ShortHash()
-    {
+    public function ShortHash() {
         return substr($this->value, 0, 7);
     }
 }

code/events/DeploynautEventInterface.php

Braces +1 added lines, -2 removed lines

@@ -1,7 +1,6 @@
 <?php
 
-interface DeploynautEventInterface extends \League\Event\EventInterface
-{
+interface DeploynautEventInterface extends \League\Event\EventInterface {
 
     /**
      * Should return the event name, e.g 'event.name'

code/events/GenericEvent.php

Braces +7 added lines, -14 removed lines

@@ -20,8 +20,7 @@ 
  *
  * See more on how to use the event system here: http://event.thephpleague.com/2.0/
  */
-class GenericEvent extends AbstractEvent implements DeploynautEventInterface
-{
+class GenericEvent extends AbstractEvent implements DeploynautEventInterface {
 
     /**
      * @var string
@@ -39,8 +38,7 @@ 
      *
      * @return GenericEvent
      */
-    public static function create($name, array $data)
-    {
+    public static function create($name, array $data) {
         return Injector::inst()->createWithArgs(get_called_class(), [$name, $data]);
     }
 
@@ -50,8 +48,7 @@ 
      * @param string $eventName - e.g. 'event.name'
      * @param array $data
      */
-    public function __construct($eventName, array $data)
-    {
+    public function __construct($eventName, array $data) {
         $this->name = $eventName;
         $this->data = $data;
     }
@@ -61,8 +58,7 @@ 
      *
      * @return string
      */
-    public function getName()
-    {
+    public function getName() {
         return $this->name;
     }
 
@@ -73,8 +69,7 @@ 
      *
      * @return string
      */
-    public function get($keyname)
-    {
+    public function get($keyname) {
         if (array_key_exists($keyname, $this->data)) {
             return $this->data[$keyname];
         }
@@ -86,8 +81,7 @@ 
      *
      * @return array
      */
-    public function asArray()
-    {
+    public function asArray() {
         $payload = $this->data;
         $payload['event'] = $this->getName();
         return $payload;
@@ -98,8 +92,7 @@ 
      *
      * @return string - returns the data as json
      */
-    public function asJSON()
-    {
+    public function asJSON() {
         return json_encode($this->asArray());
     }
 }

code/extensions/DeploynautFileExtension.php

Braces +2 added lines, -4 removed lines

@@ -1,14 +1,12 @@
 <?php
-class DeploynautFileExtension extends DataExtension
-{
+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()
-    {
+    public function canDownload() {
         $member = Member::currentUser();
         $file = $this->owner;
         $archive = DNDataArchive::get()->filter('ArchiveFileID', $file->ID)->First();

code/extensions/DeploynautSecurityExtension.php

Braces +2 added lines, -4 removed lines

@@ -1,10 +1,8 @@
 <?php
 
-class DeploynautSecurityExtension extends Extension
-{
+class DeploynautSecurityExtension extends Extension {
 
-    public function onAfterInit()
-    {
+    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.

code/extensions/FrontendLink.php

Braces +2 added lines, -4 removed lines

@@ -1,10 +1,8 @@
 <?php
 
-class FrontendLink extends DataExtension
-{
+class FrontendLink extends DataExtension {
 
-    public function updateItemEditForm($form)
-    {
+    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>',

code/extensions/ProjectMemberExtension.php

Braces +1 added lines, -2 removed lines

@@ -1,7 +1,6 @@
 <?php
 
-class ProjectMemberExtension extends DataExtension
-{
+class ProjectMemberExtension extends DataExtension {
 
     /**
      * @var array