silverstripe / silverstripe-sharedraftcontent

code/controllers/ShareDraftController.php

Doc Comments +1 added lines, -1 removed lines

@@ -101,7 +101,7 @@
 	}
 
 	/**
-	 * @param mixed $page
+	 * @param DataObject|null $page
 	 *
 	 * @return mixed
 	 */

Spacing +3 added lines, -3 removed lines

@@ -38,7 +38,7 @@ 
 		 */
 		$shareToken = ShareToken::get()->filter('token', $token)->first();
 
-		if(!$shareToken) {
+		if (!$shareToken) {
 			return $this->errorPage();
 		}
 
@@ -52,7 +52,7 @@ 
 
 		$controller = $this->getControllerFor($page);
 
-		if(!$shareToken->isExpired() && $page->generateKey($shareToken->Token) === $key) {
+		if (!$shareToken->isExpired() && $page->generateKey($shareToken->Token) === $key) {
 			Requirements::css(SHAREDRAFTCONTENT_DIR . '/css/top-bar.css');
 
 			// Temporarily un-secure the draft site and switch to draft
@@ -79,7 +79,7 @@ 
 					'Latest' => $latest,
 				));
 				$include = (string) $data->renderWith('Includes/TopBar');
-			} catch(Exception $ex) {
+			} catch (Exception $ex) {
 				$restore();
 				throw $ex;
 			}

code/extensions/ShareDraftContentControllerExtension.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
 	 */
 	public function MakeShareDraftLink() {
 		if ($member = Member::currentUser()) {
-			if($this->owner->hasMethod('CurrentPage') && $this->owner->CurrentPage()->canEdit($member)) {
+			if ($this->owner->hasMethod('CurrentPage') && $this->owner->CurrentPage()->canEdit($member)) {
 				return $this->owner->CurrentPage()->ShareTokenLink();
 			} elseif ($this->owner->hasMethod('canEdit') && $this->owner->canEdit($member)) {
 				return $this->owner->ShareTokenLink();

code/extensions/ShareDraftContentSiteTreeExtension.php

Spacing +2 added lines, -2 removed lines

@@ -47,7 +47,7 @@ 
 	 * @return ShareToken
 	 */
 	protected function getNewShareToken() {
-		if(!$this->owner->ShareTokenSalt) {
+		if (!$this->owner->ShareTokenSalt) {
 			$this->owner->ShareTokenSalt = $this->getNewToken();
 			$this->owner->write();
 		}
@@ -57,7 +57,7 @@ 
 		$tries = 1;
 		$limit = 5;
 
-		while(!$found && ($tries++ < $limit)) {
+		while (!$found && ($tries++ < $limit)) {
 			$token = $this->getNewToken();
 
 			$found = ShareToken::get()->filter(array(

code/tasks/RemoveExpiredShareTokensTask.php

Indentation +6 added lines, -6 removed lines

@@ -1,11 +1,11 @@
 <?php
 /**
- * Remove expired share tokens.
- *
- * Action to remove all expired ShareTokens from the database.
- *
- * To run this action the user needs admin rights.
- */
+	 * Remove expired share tokens.
+	 *
+	 * Action to remove all expired ShareTokens from the database.
+	 *
+	 * To run this action the user needs admin rights.
+	 */
 class RemoveExpiredShareTokens extends BuildTask {
 
 	protected $title = 'Remove expired share tokens';

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
 	public function init() {
 		parent::init();
 
-		if(!Permission::check('ADMIN')) {
+		if (!Permission::check('ADMIN')) {
 			return Security::permissionFailure($this);
 		}
 	}