silverstripe / cwp-core

code/extensions/LoginAttemptNotifications.php

Indentation +63 added lines, -63 removed lines

@@ -19,78 +19,78 @@
 class LoginAttemptNotifications extends Extension
 {
 
-    /**
-     *
-     * @return mixed null
-     */
-    public function init()
-    {
+	/**
+	 *
+	 * @return mixed null
+	 */
+	public function init()
+	{
 
-        // Exclude default admin.
-        $member = Security::getCurrentUser();
-        if (!$member || !$member->ID) {
-            return;
-        }
+		// Exclude default admin.
+		$member = Security::getCurrentUser();
+		if (!$member || !$member->ID) {
+			return;
+		}
 
-        $message = null;
-        $session = $this->owner->getRequest()->getSession();
+		$message = null;
+		$session = $this->owner->getRequest()->getSession();
 
-        Requirements::javascript('cwp/cwp-core:javascript/LoginAttemptNotifications.js');
-        $sessionLastVisited = $session->get('LoginAttemptNotifications.SessionLastVisited');
-        if ($sessionLastVisited) {
-            // Session already in progress. Show all attempts since the session was last visited.
+		Requirements::javascript('cwp/cwp-core:javascript/LoginAttemptNotifications.js');
+		$sessionLastVisited = $session->get('LoginAttemptNotifications.SessionLastVisited');
+		if ($sessionLastVisited) {
+			// Session already in progress. Show all attempts since the session was last visited.
 
-            $meantimeLoginAttempts = LoginAttempt::get()->filter([
-                'MemberID' => $member->ID,
-                'Created:GreaterThan' => $sessionLastVisited
-            ]);
+			$meantimeLoginAttempts = LoginAttempt::get()->filter([
+				'MemberID' => $member->ID,
+				'Created:GreaterThan' => $sessionLastVisited
+			]);
 
-            $attempts = $meantimeLoginAttempts->count();
-            if ($attempts) {
-                $lastVisitedObj = DBDatetime::create();
-                $lastVisitedObj->setValue($sessionLastVisited);
-                $elapsed = $lastVisitedObj->TimeDiff();
-                $failures = $meantimeLoginAttempts->filter(['Status' => 'Failure'])->count();
-                $IPs = array_unique($meantimeLoginAttempts->column('IP'));
+			$attempts = $meantimeLoginAttempts->count();
+			if ($attempts) {
+				$lastVisitedObj = DBDatetime::create();
+				$lastVisitedObj->setValue($sessionLastVisited);
+				$elapsed = $lastVisitedObj->TimeDiff();
+				$failures = $meantimeLoginAttempts->filter(['Status' => 'Failure'])->count();
+				$IPs = array_unique($meantimeLoginAttempts->column('IP'));
 
-                if ($attempts == 1) {
-                    $statusString = $failures ? "a failed" : "a successful";
-                    $message = "In the last $elapsed $statusString login attempt to your account was "
-                        . "registered. The attempt was made from ${IPs[0]}. ";
-                } else {
-                    if ($failures == $attempts) {
-                        $statusString = $failures ? "failed" : "successful";
-                        $message = "In the last $elapsed $attempts $statusString login "
-                            . "attempts to your account were registered. ";
-                    } else {
-                        $message = "In the last $elapsed $attempts login attempts to your "
-                            . "account were registered, of which $failures failed. ";
-                    }
+				if ($attempts == 1) {
+					$statusString = $failures ? "a failed" : "a successful";
+					$message = "In the last $elapsed $statusString login attempt to your account was "
+						. "registered. The attempt was made from ${IPs[0]}. ";
+				} else {
+					if ($failures == $attempts) {
+						$statusString = $failures ? "failed" : "successful";
+						$message = "In the last $elapsed $attempts $statusString login "
+							. "attempts to your account were registered. ";
+					} else {
+						$message = "In the last $elapsed $attempts login attempts to your "
+							. "account were registered, of which $failures failed. ";
+					}
 
-                    $message .= "The attempts were from " . implode(', ', $IPs) . '. ';
+					$message .= "The attempts were from " . implode(', ', $IPs) . '. ';
 
-                    // TODO: add this call to action in a way that doesn't break out of the availabel space. Fix CSS?
-                    // $message .= "If you suspect somebody else might be trying to access
-                    //  . "your account, please contact support.";
-                }
-            }
-        } else {
-            // New session - show last login attempt.
-            // TODO: this currently does NOT surface to the frontend in any way.
-            $lastLoginAttempt = LoginAttempt::get()->filter([
-                        'MemberID' => $member->ID
-            ])->sort('Created DESC')->First();
+					// TODO: add this call to action in a way that doesn't break out of the availabel space. Fix CSS?
+					// $message .= "If you suspect somebody else might be trying to access
+					//  . "your account, please contact support.";
+				}
+			}
+		} else {
+			// New session - show last login attempt.
+			// TODO: this currently does NOT surface to the frontend in any way.
+			$lastLoginAttempt = LoginAttempt::get()->filter([
+						'MemberID' => $member->ID
+			])->sort('Created DESC')->First();
 
-            if ($lastLoginAttempt) {
-                $date = $lastLoginAttempt->Created;
-                $message = "Last login attempt to your account was on $lastLoginAttempt->Created "
-                    . "from $lastLoginAttempt->IP";
-                $message .= $lastLoginAttempt->Status == 'Failure' ? " and was successful." : "and has failed.";
-            }
-        }
+			if ($lastLoginAttempt) {
+				$date = $lastLoginAttempt->Created;
+				$message = "Last login attempt to your account was on $lastLoginAttempt->Created "
+					. "from $lastLoginAttempt->IP";
+				$message .= $lastLoginAttempt->Status == 'Failure' ? " and was successful." : "and has failed.";
+			}
+		}
 
-        $session->set('LoginAttemptNotifications.SessionLastVisited', DBDatetime::now()->Format('Y-m-d H:i:s'));
+		$session->set('LoginAttemptNotifications.SessionLastVisited', DBDatetime::now()->Format('Y-m-d H:i:s'));
 
-        $this->owner->getResponse()->addHeader('X-LoginAttemptNotifications', $message);
-    }
+		$this->owner->getResponse()->addHeader('X-LoginAttemptNotifications', $message);
+	}
 }

Spacing +3 added lines, -3 removed lines

@@ -53,12 +53,12 @@ 
                 $failures = $meantimeLoginAttempts->filter(['Status' => 'Failure'])->count();
                 $IPs = array_unique($meantimeLoginAttempts->column('IP'));
 
-                if ($attempts == 1) {
+                if ($attempts==1) {
                     $statusString = $failures ? "a failed" : "a successful";
                     $message = "In the last $elapsed $statusString login attempt to your account was "
                         . "registered. The attempt was made from ${IPs[0]}. ";
                 } else {
-                    if ($failures == $attempts) {
+                    if ($failures==$attempts) {
                         $statusString = $failures ? "failed" : "successful";
                         $message = "In the last $elapsed $attempts $statusString login "
                             . "attempts to your account were registered. ";
@@ -85,7 +85,7 @@ 
                 $date = $lastLoginAttempt->Created;
                 $message = "Last login attempt to your account was on $lastLoginAttempt->Created "
                     . "from $lastLoginAttempt->IP";
-                $message .= $lastLoginAttempt->Status == 'Failure' ? " and was successful." : "and has failed.";
+                $message .= $lastLoginAttempt->Status=='Failure' ? " and was successful." : "and has failed.";
             }
         }
 

code/extensions/RichLinksExtension.php

Indentation +44 added lines, -44 removed lines

@@ -16,48 +16,48 @@
 class RichLinksExtension extends Extension
 {
 
-    /**
-     * @var array
-     */
-    private static $casting = [
-        'RichLinks' => 'HTMLText'
-    ];
-
-    /**
-     * @return string
-     */
-    public function RichLinks()
-    {
-        // Note:
-        // Assume we can use Regexes because the link will always be formatted
-        // in the same way coming from the CMS.
-
-        $content = $this->owner->value;
-
-        // Find all file links for processing.
-        preg_match_all('/<a.*href="\[file_link,id=([0-9]+)\].*".*>.*<\/a>/U', $content, $matches);
-
-        // Attach the file type and size to each of the links.
-        for ($i = 0; $i < count($matches[0]); $i++) {
-            $file = DataObject::get_by_id('File', $matches[1][$i]);
-            if ($file) {
-                $size = $file->getSize();
-                $ext = strtoupper($file->getExtension());
-                // Replace the closing </a> tag with the size span (and reattach the closing tag).
-                $newLink = substr($matches[0][$i], 0, strlen($matches[0][$i]) - 4)
-                    . "<span class='fileExt'> [$ext, $size]</span></a>";
-                $content = str_replace($matches[0][$i], $newLink, $content);
-            }
-        }
-
-        // Inject extra attributes into the external links.
-        $pattern = '/(<a.*)(href=\"https?:\/\/[^\"]*\"[^>]*>.*)(<\/a>)/iU';
-        $replacement = sprintf(
-            '$1class="external" rel="external" title="%s" $2<span class="nonvisual-indicator">(external link)</span>$3',
-            _t('RichLinks.OpenLinkTitle', 'Open external link')
-        );
-        $content = preg_replace($pattern, $replacement, $content, -1);
-
-        return $content;
-    }
+	/**
+	 * @var array
+	 */
+	private static $casting = [
+		'RichLinks' => 'HTMLText'
+	];
+
+	/**
+	 * @return string
+	 */
+	public function RichLinks()
+	{
+		// Note:
+		// Assume we can use Regexes because the link will always be formatted
+		// in the same way coming from the CMS.
+
+		$content = $this->owner->value;
+
+		// Find all file links for processing.
+		preg_match_all('/<a.*href="\[file_link,id=([0-9]+)\].*".*>.*<\/a>/U', $content, $matches);
+
+		// Attach the file type and size to each of the links.
+		for ($i = 0; $i < count($matches[0]); $i++) {
+			$file = DataObject::get_by_id('File', $matches[1][$i]);
+			if ($file) {
+				$size = $file->getSize();
+				$ext = strtoupper($file->getExtension());
+				// Replace the closing </a> tag with the size span (and reattach the closing tag).
+				$newLink = substr($matches[0][$i], 0, strlen($matches[0][$i]) - 4)
+					. "<span class='fileExt'> [$ext, $size]</span></a>";
+				$content = str_replace($matches[0][$i], $newLink, $content);
+			}
+		}
+
+		// Inject extra attributes into the external links.
+		$pattern = '/(<a.*)(href=\"https?:\/\/[^\"]*\"[^>]*>.*)(<\/a>)/iU';
+		$replacement = sprintf(
+			'$1class="external" rel="external" title="%s" $2<span class="nonvisual-indicator">(external link)</span>$3',
+			_t('RichLinks.OpenLinkTitle', 'Open external link')
+		);
+		$content = preg_replace($pattern, $replacement, $content, -1);
+
+		return $content;
+	}
 }

code/extensions/CustomHtmlEditorField.php

Indentation +9 added lines, -9 removed lines

@@ -13,13 +13,13 @@
 class CustomHtmlEditorFieldToolbar extends Extension
 {
 
-    /**
-     * @param Form $form
-     * @return void
-     */
-    public function updateMediaForm(Form $form)
-    {
-        Requirements::add_i18n_javascript('cwp/cwp-core:javascript/lang');
-        Requirements::javascript('cwp/cwp-core:javascript/CustomHtmlEditorFieldToolbar.js');
-    }
+	/**
+	 * @param Form $form
+	 * @return void
+	 */
+	public function updateMediaForm(Form $form)
+	{
+		Requirements::add_i18n_javascript('cwp/cwp-core:javascript/lang');
+		Requirements::javascript('cwp/cwp-core:javascript/CustomHtmlEditorFieldToolbar.js');
+	}
 }

code/model/CwpSearchIndex.php

Indentation +74 added lines, -74 removed lines

@@ -18,86 +18,86 @@
 abstract class CwpSearchIndex extends SolrIndex
 {
 
-    /**
-     * Copy all fields into both search and spellcheck data source
-     *
-     * @var array
-     * @config
-     */
-    private static $copy_fields = array(
-        '_text',
-        '_spellcheckText'
-    );
+	/**
+	 * Copy all fields into both search and spellcheck data source
+	 *
+	 * @var array
+	 * @config
+	 */
+	private static $copy_fields = array(
+		'_text',
+		'_spellcheckText'
+	);
 
-    /**
-     * Default dictionary to use. This will overwrite the 'spellcheck.dictionary' option for searches given,
-     * unless set to empty.
-     *
-     * '_spellcheck' is a predefined by the cwp infrastructure, which is configured
-     * to be built from the '_spellcheckText' field. You can't rename this within CWP.
-     *
-     * @var string
-     * @config
-     */
-    private static $dictionary = '_spellcheck';
+	/**
+	 * Default dictionary to use. This will overwrite the 'spellcheck.dictionary' option for searches given,
+	 * unless set to empty.
+	 *
+	 * '_spellcheck' is a predefined by the cwp infrastructure, which is configured
+	 * to be built from the '_spellcheckText' field. You can't rename this within CWP.
+	 *
+	 * @var string
+	 * @config
+	 */
+	private static $dictionary = '_spellcheck';
 
-    public function init()
-    {
-        // Add optional boost
-        if (SiteTree::has_extension(CwpSearchBoostExtension::class)) {
-            $this->setFieldBoosting(SiteTree::class . '_SearchBoost', SiteTree::config()->get('search_boost'));
-        }
-    }
+	public function init()
+	{
+		// Add optional boost
+		if (SiteTree::has_extension(CwpSearchBoostExtension::class)) {
+			$this->setFieldBoosting(SiteTree::class . '_SearchBoost', SiteTree::config()->get('search_boost'));
+		}
+	}
 
-    /**
-     * Upload config for this index to the given store
-     *
-     * @param SolrConfigStore $store
-     */
-    public function uploadConfig($store)
-    {
-        parent::uploadConfig($store);
+	/**
+	 * Upload config for this index to the given store
+	 *
+	 * @param SolrConfigStore $store
+	 */
+	public function uploadConfig($store)
+	{
+		parent::uploadConfig($store);
 
-        // Upload configured synonyms {@see SynonymsSiteConfig}
-        $siteConfig = SiteConfig::current_site_config();
-        if ($siteConfig->SearchSynonyms) {
-            $store->uploadString(
-                $this->getIndexName(),
-                'synonyms.txt',
-                $siteConfig->SearchSynonyms
-            );
-        }
-    }
+		// Upload configured synonyms {@see SynonymsSiteConfig}
+		$siteConfig = SiteConfig::current_site_config();
+		if ($siteConfig->SearchSynonyms) {
+			$store->uploadString(
+				$this->getIndexName(),
+				'synonyms.txt',
+				$siteConfig->SearchSynonyms
+			);
+		}
+	}
 
-    /**
-     *
-     * @return string
-     */
-    public function getFieldDefinitions()
-    {
-        $xml = parent::getFieldDefinitions();
-        $xml .= "\n\n\t\t<!-- Additional custom fields for spell checking -->";
-        $xml .= "\n\t\t<field name='_spellcheckText' type='textSpellHtml' indexed='true' "
-            . "stored='false' multiValued='true' />";
+	/**
+	 *
+	 * @return string
+	 */
+	public function getFieldDefinitions()
+	{
+		$xml = parent::getFieldDefinitions();
+		$xml .= "\n\n\t\t<!-- Additional custom fields for spell checking -->";
+		$xml .= "\n\t\t<field name='_spellcheckText' type='textSpellHtml' indexed='true' "
+			. "stored='false' multiValued='true' />";
 
-        return $xml;
-    }
+		return $xml;
+	}
 
-    /**
-     *
-     * @param SearchQuery $query
-     * @param int $offset
-     * @param int $limit
-     * @param array $params
-     * @return ArrayData
-     */
-    public function search(SearchQuery $query, $offset = -1, $limit = -1, $params = [])
-    {
-        // Override dictionary if given
-        if ($dictionary = $this->config()->dictionary) {
-            $params["spellcheck.dictionary"] = $dictionary;
-        }
+	/**
+	 *
+	 * @param SearchQuery $query
+	 * @param int $offset
+	 * @param int $limit
+	 * @param array $params
+	 * @return ArrayData
+	 */
+	public function search(SearchQuery $query, $offset = -1, $limit = -1, $params = [])
+	{
+		// Override dictionary if given
+		if ($dictionary = $this->config()->dictionary) {
+			$params["spellcheck.dictionary"] = $dictionary;
+		}
 
-        return parent::search($query, $offset, $limit, $params);
-    }
+		return parent::search($query, $offset, $limit, $params);
+	}
 }

code/CwpAtomFeed.php

Indentation +49 added lines, -49 removed lines

@@ -18,57 +18,57 @@
 
 class CwpAtomFeed extends RSSFeed
 {
-    public function __construct(
-        SS_List $entries,
-        $link,
-        $title,
-        $description = null,
-        $titleField = "Title",
-        $descriptionField = "Content",
-        $authorField = null,
-        $lastModified = null,
-        $etag = null
-    ) {
-        parent::__construct(
-            $entries,
-            $link,
-            $title,
-            $description,
-            $titleField,
-            $descriptionField,
-            $authorField,
-            $lastModified
-        );
+	public function __construct(
+		SS_List $entries,
+		$link,
+		$title,
+		$description = null,
+		$titleField = "Title",
+		$descriptionField = "Content",
+		$authorField = null,
+		$lastModified = null,
+		$etag = null
+	) {
+		parent::__construct(
+			$entries,
+			$link,
+			$title,
+			$description,
+			$titleField,
+			$descriptionField,
+			$authorField,
+			$lastModified
+		);
 
-        $this->setTemplate(__CLASS__);
-    }
+		$this->setTemplate(__CLASS__);
+	}
 
-    /**
-     * Include an link to the feed
-     *
-     * @param string $url URL of the feed
-     * @param string $title Title to show
-     */
-    public static function linkToFeed($url, $title = null)
-    {
-        $title = Convert::raw2xml($title);
-        Requirements::insertHeadTags(
-            '<link rel="alternate" type="application/atom+xml" title="' . $title .
-            '" href="' . $url . '" />'
-        );
-    }
+	/**
+	 * Include an link to the feed
+	 *
+	 * @param string $url URL of the feed
+	 * @param string $title Title to show
+	 */
+	public static function linkToFeed($url, $title = null)
+	{
+		$title = Convert::raw2xml($title);
+		Requirements::insertHeadTags(
+			'<link rel="alternate" type="application/atom+xml" title="' . $title .
+			'" href="' . $url . '" />'
+		);
+	}
 
-    /**
-     * Output the feed to the browser
-     *
-     * @return DBHTMLText
-     */
-    public function outputToBrowser()
-    {
-        $output = parent::outputToBrowser();
-        $response = Controller::curr()->getResponse();
-        $response->addHeader("Content-Type", "application/atom+xml");
+	/**
+	 * Output the feed to the browser
+	 *
+	 * @return DBHTMLText
+	 */
+	public function outputToBrowser()
+	{
+		$output = parent::outputToBrowser();
+		$response = Controller::curr()->getResponse();
+		$response->addHeader("Content-Type", "application/atom+xml");
 
-        return $output;
-    }
+		return $output;
+	}
 }

code/CwpSolrConfigStore.php

Indentation +57 added lines, -57 removed lines

@@ -12,68 +12,68 @@
  */
 class CwpSolrConfigStore implements SolrConfigStore
 {
-    /**
-     * @var string
-     */
-    protected $remote = '';
+	/**
+	 * @var string
+	 */
+	protected $remote = '';
 
-    /**
-     * @var string
-     */
-    protected $url = '';
+	/**
+	 * @var string
+	 */
+	protected $url = '';
 
-    /**
-     * @param array $config
-     */
-    public function __construct($config)
-    {
-        $options = Solr::solr_options();
+	/**
+	 * @param array $config
+	 */
+	public function __construct($config)
+	{
+		$options = Solr::solr_options();
 
-        $this->url = implode('', [
-            'http://',
-            isset($config['auth']) ? $config['auth'] . '@' : '',
-            $options['host'] . ':' . $options['port'],
-            $config['path']
-        ]);
-        $this->remote = $config['remotepath'];
-    }
+		$this->url = implode('', [
+			'http://',
+			isset($config['auth']) ? $config['auth'] . '@' : '',
+			$options['host'] . ':' . $options['port'],
+			$config['path']
+		]);
+		$this->remote = $config['remotepath'];
+	}
 
-    /**
-     * @param string $index
-     * @param string $file
-     * @return void
-     */
-    public function uploadFile($index, $file)
-    {
-        $this->uploadString($index, basename($file), file_get_contents($file));
-    }
+	/**
+	 * @param string $index
+	 * @param string $file
+	 * @return void
+	 */
+	public function uploadFile($index, $file)
+	{
+		$this->uploadString($index, basename($file), file_get_contents($file));
+	}
 
-    /**
-     *
-     * @param string $index
-     * @param string $filename
-     * @param string $string
-     * @return void
-     */
-    public function uploadString($index, $filename, $string)
-    {
-        $targetDir = "{$this->url}/config/$index";
+	/**
+	 *
+	 * @param string $index
+	 * @param string $filename
+	 * @param string $string
+	 * @return void
+	 */
+	public function uploadString($index, $filename, $string)
+	{
+		$targetDir = "{$this->url}/config/$index";
 
-        file_get_contents($targetDir . '/' . $filename, false, stream_context_create([
-            'http' => [
-                'method' => 'POST',
-                'header' => 'Content-type: application/octet-stream',
-                'content' => (string) $string
-            ]
-        ]));
-    }
+		file_get_contents($targetDir . '/' . $filename, false, stream_context_create([
+			'http' => [
+				'method' => 'POST',
+				'header' => 'Content-type: application/octet-stream',
+				'content' => (string) $string
+			]
+		]));
+	}
 
-    /**
-     * @param string $index
-     * @return string
-     */
-    public function instanceDir($index)
-    {
-        return $this->remote ? "{$this->remote}/$index" : $index;
-    }
+	/**
+	 * @param string $index
+	 * @return string
+	 */
+	public function instanceDir($index)
+	{
+		return $this->remote ? "{$this->remote}/$index" : $index;
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
             'http' => [
                 'method' => 'POST',
                 'header' => 'Content-type: application/octet-stream',
-                'content' => (string) $string
+                'content' => (string)$string
             ]
         ]));
     }

code/CwpSolr.php

Indentation +105 added lines, -105 removed lines

@@ -14,119 +14,119 @@
  */
 class CwpSolr
 {
-    use Configurable;
+	use Configurable;
 
-    /**
-     *
-     * @var array
-     */
-    private static $options;
+	/**
+	 *
+	 * @var array
+	 */
+	private static $options;
 
-    /**
-     * Configure Solr.
-     *
-     * $options - An array consisting of:
-     *
-     * 'extraspath' - (String) Where to find Solr core configuartion files.
-     *     Defaults to '<BASE_PATH>/mysite/conf/extras'.
-     * 'version' - select the Solr configuration to use when in CWP. One of:
-     * * 'cwp-4': preferred version, uses secured 4.x service available on CWP
-     * * 'local-4': this can be use for development using silverstripe-localsolr package, 4.x branch
-     */
-    public static function configure()
-    {
-        if (!class_exists(Solr::class)) {
-            return;
-        }
+	/**
+	 * Configure Solr.
+	 *
+	 * $options - An array consisting of:
+	 *
+	 * 'extraspath' - (String) Where to find Solr core configuartion files.
+	 *     Defaults to '<BASE_PATH>/mysite/conf/extras'.
+	 * 'version' - select the Solr configuration to use when in CWP. One of:
+	 * * 'cwp-4': preferred version, uses secured 4.x service available on CWP
+	 * * 'local-4': this can be use for development using silverstripe-localsolr package, 4.x branch
+	 */
+	public static function configure()
+	{
+		if (!class_exists(Solr::class)) {
+			return;
+		}
 
-        // get options from configuration
-        $options = static::config()->get('options');
+		// get options from configuration
+		$options = static::config()->get('options');
 
-        // get version specific options
-        switch ($options['version']) {
-            case 'cwp-4':
-                $solrOptions = self::options_for_cwp($options);
-                break;
-            case 'local-4':
-                $solrOptions = self::options_for_local($options);
-                break;
-            default:
-                throw new InvalidArgumentException(sprintf(
-                    'Solr version "%s" is not supported on CWP. Please use "local-4" on local ' .
-                        'and "cwp-4" on production. For preferred configuration see ' .
-                        'https://www.cwp.govt.nz/developer-docs/.',
-                    $options['version']
-                ));
-                break;
-        }
+		// get version specific options
+		switch ($options['version']) {
+			case 'cwp-4':
+				$solrOptions = self::options_for_cwp($options);
+				break;
+			case 'local-4':
+				$solrOptions = self::options_for_local($options);
+				break;
+			default:
+				throw new InvalidArgumentException(sprintf(
+					'Solr version "%s" is not supported on CWP. Please use "local-4" on local ' .
+						'and "cwp-4" on production. For preferred configuration see ' .
+						'https://www.cwp.govt.nz/developer-docs/.',
+					$options['version']
+				));
+				break;
+		}
 
-        // Allow users to override extras path.
-        // CAUTION: CWP does not permit usage of customised solrconfig.xml.
-        if (isset($options['extraspath']) && file_exists($options['extraspath'])) {
-            $solrOptions['extraspath'] = $options['extraspath'];
-        } elseif (file_exists(BASE_PATH . '/mysite/conf/extras')) {
-            $solrOptions['extraspath'] = BASE_PATH . '/mysite/conf/extras';
-        }
+		// Allow users to override extras path.
+		// CAUTION: CWP does not permit usage of customised solrconfig.xml.
+		if (isset($options['extraspath']) && file_exists($options['extraspath'])) {
+			$solrOptions['extraspath'] = $options['extraspath'];
+		} elseif (file_exists(BASE_PATH . '/mysite/conf/extras')) {
+			$solrOptions['extraspath'] = BASE_PATH . '/mysite/conf/extras';
+		}
 
-        Solr::configure_server($solrOptions);
-    }
+		Solr::configure_server($solrOptions);
+	}
 
-    /**
-     * @throws Exception
-     */
-    public static function options_from_environment()
-    {
-        throw new Exception(
-            'CwpSolr::options_from_environment has been deprecated, in favour of implicit Solr ' .
-            'configuration provided by the CwpSolr class in the cwp-core module. For preferred configuration see ' .
-            'https://www.cwp.govt.nz/developer-docs/.'
-        );
-    }
+	/**
+	 * @throws Exception
+	 */
+	public static function options_from_environment()
+	{
+		throw new Exception(
+			'CwpSolr::options_from_environment has been deprecated, in favour of implicit Solr ' .
+			'configuration provided by the CwpSolr class in the cwp-core module. For preferred configuration see ' .
+			'https://www.cwp.govt.nz/developer-docs/.'
+		);
+	}
 
-    /**
-     * @param array $options
-     * @return array
-     */
-    public static function options_for_cwp($options)
-    {
-        $version = $options['version'];
+	/**
+	 * @param array $options
+	 * @return array
+	 */
+	public static function options_for_cwp($options)
+	{
+		$version = $options['version'];
 
-        return [
-            'host' => Environment::getEnv('SOLR_SERVER'),
-            'port' => Environment::getEnv('SOLR_PORT'),
-            'path' => '/v4/',
-            'version' => 4,
-            'indexstore' => [
-                'mode' => CwpSolrConfigStore::class,
-                'path' => '/v4',
-            ],
-        ];
-    }
+		return [
+			'host' => Environment::getEnv('SOLR_SERVER'),
+			'port' => Environment::getEnv('SOLR_PORT'),
+			'path' => '/v4/',
+			'version' => 4,
+			'indexstore' => [
+				'mode' => CwpSolrConfigStore::class,
+				'path' => '/v4',
+			],
+		];
+	}
 
-    /**
-     *
-     * @param array $options
-     * @return array
-     */
-    public static function options_for_local($options)
-    {
-        return [
-            'host' => Environment::getEnv('SOLR_SERVER') ? Environment::getEnv('SOLR_SERVER') : 'localhost',
-            'port' => Environment::getEnv('SOLR_PORT') ? Environment::getEnv('SOLR_PORT') : 8983,
-            'path' => Environment::getEnv('SOLR_PATH') ? Environment::getEnv('SOLR_PATH') : '/solr/',
-            'version' => 4,
-            'indexstore' => [
-                'mode' => Environment::getEnv('SOLR_MODE') ? Environment::getEnv('SOLR_MODE') : 'file',
-                'auth' => Environment::getEnv('SOLR_AUTH') ? Environment::getEnv('SOLR_AUTH') : null,
-                // Allow storing the solr index and config data in an arbitrary location,
-                // e.g. outside of the webroot
-                'path' => Environment::getEnv('SOLR_INDEXSTORE_PATH')
-                    ? Environment::getEnv('SOLR_INDEXSTORE_PATH')
-                    : BASE_PATH . '/.solr',
-                'remotepath' => Environment::getEnv('SOLR_REMOTE_PATH')
-                    ? Environment::getEnv('SOLR_REMOTE_PATH')
-                    : null
-            ]
-        ];
-    }
+	/**
+	 *
+	 * @param array $options
+	 * @return array
+	 */
+	public static function options_for_local($options)
+	{
+		return [
+			'host' => Environment::getEnv('SOLR_SERVER') ? Environment::getEnv('SOLR_SERVER') : 'localhost',
+			'port' => Environment::getEnv('SOLR_PORT') ? Environment::getEnv('SOLR_PORT') : 8983,
+			'path' => Environment::getEnv('SOLR_PATH') ? Environment::getEnv('SOLR_PATH') : '/solr/',
+			'version' => 4,
+			'indexstore' => [
+				'mode' => Environment::getEnv('SOLR_MODE') ? Environment::getEnv('SOLR_MODE') : 'file',
+				'auth' => Environment::getEnv('SOLR_AUTH') ? Environment::getEnv('SOLR_AUTH') : null,
+				// Allow storing the solr index and config data in an arbitrary location,
+				// e.g. outside of the webroot
+				'path' => Environment::getEnv('SOLR_INDEXSTORE_PATH')
+					? Environment::getEnv('SOLR_INDEXSTORE_PATH')
+					: BASE_PATH . '/.solr',
+				'remotepath' => Environment::getEnv('SOLR_REMOTE_PATH')
+					? Environment::getEnv('SOLR_REMOTE_PATH')
+					: null
+			]
+		];
+	}
 }

code/CwpStatsReport.php

Indentation +104 added lines, -104 removed lines

@@ -20,108 +20,108 @@
  */
 class CwpStatsReport extends Report
 {
-    public function title()
-    {
-        return _t(__CLASS__ . '.Title', 'Summary statistics');
-    }
-
-    public function description()
-    {
-        return _t(
-            __CLASS__ . '.Description',
-            'This report provides various statistics for this site. The "total live page count" is the number that ' .
-                'can be compared against the instance size specifications.'
-        );
-    }
-
-    public function columns()
-    {
-        return [
-            'Name' => _t(__CLASS__ . '.Name', 'Name'),
-            'Count' => _t(__CLASS__ . '.Count', 'Count'),
-        ];
-    }
-
-    /**
-     * Manually create source records for the report. Agreggates cannot be provided as a column of a DataQuery result.
-     *
-     * {@inheritDoc}
-     */
-    public function sourceRecords($params = [], $sort = null, $limit = null)
-    {
-        $records = [];
-
-        // Get the query to apply across all variants: looks at all subsites, translations, live stage only.
-        $crossVariant = (function ($dataQuery) {
-            $params = [
-                'Subsite.filter' => false,
-                'Versioned.mode' => 'stage',
-                'Versioned.stage' => Versioned::LIVE,
-            ];
-
-            return $dataQuery->setDataQueryParam($params);
-        });
-
-        // Total.
-        $records[] = [
-            'Name' => _t(
-                __CLASS__ . '.TotalPageCount',
-                'Total live page count, across all translations and subsites'
-            ),
-            'Count' => $crossVariant(SiteTree::get())->count(),
-        ];
-
-        if (class_exists(Subsite::class)) {
-            // Main site.
-            $records[] = [
-                'Name' => _t(__CLASS__ . '.PagesForMainSite', '- in the main site'),
-                'Count' => $crossVariant(SiteTree::get())
-                    ->filter(['SubsiteID' => 0])
-                    ->count(),
-            ];
-
-            // Per subsite.
-            $subsites = Subsite::get();
-            foreach ($subsites as $subsite) {
-                $records[] = [
-                    'Name' => _t(
-                        __CLASS__ . '.PagesForSubsite',
-                        "- in the subsite '{SubsiteTitle}'",
-                        ['SubsiteTitle' => $subsite->Title]
-                    ),
-                    'Count' => $crossVariant(SiteTree::get())
-                        ->filter(['SubsiteID' => $subsite->ID])
-                        ->count(),
-                ];
-            }
-        }
-
-        // Files.
-        $records[] = [
-            'Name' => _t('CwpStatsReport.FileCount', 'File count'),
-            'Count' => File::get()
-                ->setDataQueryParam('Subsite.filter', false)
-                ->filter(['ClassName:not' => Folder::class])
-                ->count(),
-        ];
-
-        return ArrayList::create($records);
-    }
-
-    /**
-     * @return GridField
-     */
-    public function getReportField()
-    {
-        /** @var GridField $gridField */
-        $gridField = parent::getReportField();
-
-        /** @var GridFieldConfig $gridConfig */
-        $gridConfig = $gridField->getConfig();
-        $gridConfig->removeComponentsByType(GridFieldPrintButton::class);
-        $gridConfig->removeComponentsByType(GridFieldExportButton::class);
-        $gridConfig->removeComponentsByType(GridFieldSortableHeader::class);
-
-        return $gridField;
-    }
+	public function title()
+	{
+		return _t(__CLASS__ . '.Title', 'Summary statistics');
+	}
+
+	public function description()
+	{
+		return _t(
+			__CLASS__ . '.Description',
+			'This report provides various statistics for this site. The "total live page count" is the number that ' .
+				'can be compared against the instance size specifications.'
+		);
+	}
+
+	public function columns()
+	{
+		return [
+			'Name' => _t(__CLASS__ . '.Name', 'Name'),
+			'Count' => _t(__CLASS__ . '.Count', 'Count'),
+		];
+	}
+
+	/**
+	 * Manually create source records for the report. Agreggates cannot be provided as a column of a DataQuery result.
+	 *
+	 * {@inheritDoc}
+	 */
+	public function sourceRecords($params = [], $sort = null, $limit = null)
+	{
+		$records = [];
+
+		// Get the query to apply across all variants: looks at all subsites, translations, live stage only.
+		$crossVariant = (function ($dataQuery) {
+			$params = [
+				'Subsite.filter' => false,
+				'Versioned.mode' => 'stage',
+				'Versioned.stage' => Versioned::LIVE,
+			];
+
+			return $dataQuery->setDataQueryParam($params);
+		});
+
+		// Total.
+		$records[] = [
+			'Name' => _t(
+				__CLASS__ . '.TotalPageCount',
+				'Total live page count, across all translations and subsites'
+			),
+			'Count' => $crossVariant(SiteTree::get())->count(),
+		];
+
+		if (class_exists(Subsite::class)) {
+			// Main site.
+			$records[] = [
+				'Name' => _t(__CLASS__ . '.PagesForMainSite', '- in the main site'),
+				'Count' => $crossVariant(SiteTree::get())
+					->filter(['SubsiteID' => 0])
+					->count(),
+			];
+
+			// Per subsite.
+			$subsites = Subsite::get();
+			foreach ($subsites as $subsite) {
+				$records[] = [
+					'Name' => _t(
+						__CLASS__ . '.PagesForSubsite',
+						"- in the subsite '{SubsiteTitle}'",
+						['SubsiteTitle' => $subsite->Title]
+					),
+					'Count' => $crossVariant(SiteTree::get())
+						->filter(['SubsiteID' => $subsite->ID])
+						->count(),
+				];
+			}
+		}
+
+		// Files.
+		$records[] = [
+			'Name' => _t('CwpStatsReport.FileCount', 'File count'),
+			'Count' => File::get()
+				->setDataQueryParam('Subsite.filter', false)
+				->filter(['ClassName:not' => Folder::class])
+				->count(),
+		];
+
+		return ArrayList::create($records);
+	}
+
+	/**
+	 * @return GridField
+	 */
+	public function getReportField()
+	{
+		/** @var GridField $gridField */
+		$gridField = parent::getReportField();
+
+		/** @var GridFieldConfig $gridConfig */
+		$gridConfig = $gridField->getConfig();
+		$gridConfig->removeComponentsByType(GridFieldPrintButton::class);
+		$gridConfig->removeComponentsByType(GridFieldExportButton::class);
+		$gridConfig->removeComponentsByType(GridFieldSortableHeader::class);
+
+		return $gridField;
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
         $records = [];
 
         // Get the query to apply across all variants: looks at all subsites, translations, live stage only.
-        $crossVariant = (function ($dataQuery) {
+        $crossVariant = (function($dataQuery) {
             $params = [
                 'Subsite.filter' => false,
                 'Versioned.mode' => 'stage',

code/CwpInitialisationFilter.php

Indentation +58 added lines, -58 removed lines

@@ -14,71 +14,71 @@
 class CwpInitialisationFilter implements RequestFilter
 {
 
-    /**
-     * @var boolean
-     *
-     * Enable egress proxy. This works on the principle of setting http(s)_proxy environment variables,
-     *  which will be automatically picked up by curl. This means RestfulService and raw curl
-     *  requests should work out of the box. Stream-based requests need extra manual configuration.
-     *  Refer to https://www.cwp.govt.nz/guides/core-technical-documentation/common-web-platform-core/en/how-tos/external_http_requests_with_proxy
-     *
-     * @config
-     * @var bool
-     */
-    private static $egress_proxy_default_enabled = true;
+	/**
+	 * @var boolean
+	 *
+	 * Enable egress proxy. This works on the principle of setting http(s)_proxy environment variables,
+	 *  which will be automatically picked up by curl. This means RestfulService and raw curl
+	 *  requests should work out of the box. Stream-based requests need extra manual configuration.
+	 *  Refer to https://www.cwp.govt.nz/guides/core-technical-documentation/common-web-platform-core/en/how-tos/external_http_requests_with_proxy
+	 *
+	 * @config
+	 * @var bool
+	 */
+	private static $egress_proxy_default_enabled = true;
 
-    /**
-     * @var array
-     *
-     * Configure the list of domains to bypass proxy by setting the NO_PROXY environment variable.
-     * 'services.cwp.govt.nz' needs to be present for Solr and Docvert internal CWP integration.
-     * 'localhost' is necessary for accessing services on the same instance such as tika-server for text extraction.
-     *
-     * @config
-     * @var string[]
-     */
-    private static $egress_proxy_exclude_domains = [
-        'services.cwp.govt.nz',
-        'localhost',
-    ];
+	/**
+	 * @var array
+	 *
+	 * Configure the list of domains to bypass proxy by setting the NO_PROXY environment variable.
+	 * 'services.cwp.govt.nz' needs to be present for Solr and Docvert internal CWP integration.
+	 * 'localhost' is necessary for accessing services on the same instance such as tika-server for text extraction.
+	 *
+	 * @config
+	 * @var string[]
+	 */
+	private static $egress_proxy_exclude_domains = [
+		'services.cwp.govt.nz',
+		'localhost',
+	];
 
-    /**
-     *
-     * @param  HTTPRequest $request
-     * @return boolean
-     */
-    public function preRequest(HTTPRequest $request)
-    {
-        if (Config::inst()->get(__CLASS__, 'egress_proxy_default_enabled')) {
-            if (Environment::getEnv('SS_OUTBOUND_PROXY') && Environment::getEnv('SS_OUTBOUND_PROXY_PORT')) {
-                $proxy = Environment::getEnv('SS_OUTBOUND_PROXY');
-                $proxyPort = Environment::getEnv('SS_OUTBOUND_PROXY_PORT');
+	/**
+	 *
+	 * @param  HTTPRequest $request
+	 * @return boolean
+	 */
+	public function preRequest(HTTPRequest $request)
+	{
+		if (Config::inst()->get(__CLASS__, 'egress_proxy_default_enabled')) {
+			if (Environment::getEnv('SS_OUTBOUND_PROXY') && Environment::getEnv('SS_OUTBOUND_PROXY_PORT')) {
+				$proxy = Environment::getEnv('SS_OUTBOUND_PROXY');
+				$proxyPort = Environment::getEnv('SS_OUTBOUND_PROXY_PORT');
 
-                Environment::setEnv('http_proxy', $proxy . ':' . $proxyPort);
-                Environment::setEnv('https_proxy', $proxy. ':' . $proxyPort);
-            }
-        }
+				Environment::setEnv('http_proxy', $proxy . ':' . $proxyPort);
+				Environment::setEnv('https_proxy', $proxy. ':' . $proxyPort);
+			}
+		}
 
-        $noProxy = Config::inst()->get(__CLASS__, 'egress_proxy_exclude_domains');
+		$noProxy = Config::inst()->get(__CLASS__, 'egress_proxy_exclude_domains');
 
-        if (!empty($noProxy)) {
-            if (!is_array($noProxy)) {
-                $noProxy = [$noProxy];
-            }
+		if (!empty($noProxy)) {
+			if (!is_array($noProxy)) {
+				$noProxy = [$noProxy];
+			}
 
-            // Merge with exsiting if needed.
-            if (Environment::getEnv('NO_PROXY')) {
-                $noProxy = array_merge(explode(',', Environment::getEnv('NO_PROXY')), $noProxy);
-            }
+			// Merge with exsiting if needed.
+			if (Environment::getEnv('NO_PROXY')) {
+				$noProxy = array_merge(explode(',', Environment::getEnv('NO_PROXY')), $noProxy);
+			}
 
-            Environment::setEnv('NO_PROXY', implode(',', array_unique($noProxy)));
-        }
+			Environment::setEnv('NO_PROXY', implode(',', array_unique($noProxy)));
+		}
 
-        return true;
-    }
+		return true;
+	}
 
-    public function postRequest(HTTPRequest $request, HTTPResponse $response)
-    {
-        return true;
-    }
+	public function postRequest(HTTPRequest $request, HTTPResponse $response)
+	{
+		return true;
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
                 $proxyPort = Environment::getEnv('SS_OUTBOUND_PROXY_PORT');
 
                 Environment::setEnv('http_proxy', $proxy . ':' . $proxyPort);
-                Environment::setEnv('https_proxy', $proxy. ':' . $proxyPort);
+                Environment::setEnv('https_proxy', $proxy . ':' . $proxyPort);
             }
         }