silverstripe / silverstripe-siteconfig

tests/behat/features/bootstrap/SilverStripe/SiteConfig/Test/Behaviour/ThemeContext.php

Doc Comments +6 added lines

@@ -54,6 +54,9 @@ 
         $this->requireFile(BASE_PATH . '/themes/' . $theme . '/templates/' . $template, $content);
     }
 
+    /**
+     * @param string $filename
+     */
     protected function requireFile($filename, $content)
     {
         // Already exists
@@ -71,6 +74,9 @@ 
         }
     }
 
+    /**
+     * @param string $dirname
+     */
     protected function requireDir($dirname)
     {
         // Directory doesn't exist, create it and mark it for deletion

Unused Use Statements -8 removed lines

@@ -2,15 +2,7 @@
 
 namespace SilverStripe\Siteconfig\Test\Behaviour;
 
-use Behat\Behat\Context\ClosuredContextInterface;
-use Behat\Behat\Context\TranslatedContextInterface;
 use Behat\Behat\Context\BehatContext;
-use Behat\Behat\Context\Step;
-use Behat\Behat\Event\StepEvent;
-use Behat\Behat\Exception\PendingException;
-use Behat\Mink\Driver\Selenium2Driver;
-use Behat\Gherkin\Node\PyStringNode;
-use Behat\Gherkin\Node\TableNode;
 
 /**
  * Context used to create fixtures in the SilverStripe ORM.

Indentation +86 added lines, -86 removed lines

@@ -17,100 +17,100 @@
  */
 class ThemeContext extends BehatContext
 {
-    protected $restoreFiles = array();
-    protected $restoreDirectories = array();
+	protected $restoreFiles = array();
+	protected $restoreDirectories = array();
 
-    /**
-     * Create a test theme
-     *
-     * @Given /^a theme "(?<theme>[^"]+)"/
-     */
-    public function stepCreateTheme($theme)
-    {
-        if (!preg_match('/^[0-9a-zA-Z_-]+$/', $theme)) {
-            throw new \InvalidArgumentException("Bad theme '$theme'");
-        }
+	/**
+	 * Create a test theme
+	 *
+	 * @Given /^a theme "(?<theme>[^"]+)"/
+	 */
+	public function stepCreateTheme($theme)
+	{
+		if (!preg_match('/^[0-9a-zA-Z_-]+$/', $theme)) {
+			throw new \InvalidArgumentException("Bad theme '$theme'");
+		}
 
-        $this->requireDir(BASE_PATH . '/themes');
-        $this->requireDir(BASE_PATH . '/themes/' . $theme);
-        $this->requireDir(BASE_PATH . '/themes/' . $theme . '/templates');
-    }
+		$this->requireDir(BASE_PATH . '/themes');
+		$this->requireDir(BASE_PATH . '/themes/' . $theme);
+		$this->requireDir(BASE_PATH . '/themes/' . $theme . '/templates');
+	}
 
-    /**
-     * Create a template within a test theme
-     *
-     * @Given /^a template "(?<template>[^"]+)" in theme "(?<theme>[^"]+)" with content "(?<content>[^"]+)"/
-     */
-    public function stepCreateTemplate($template, $theme, $content)
-    {
-        if (!preg_match('/^[0-9a-zA-Z_-]+$/', $theme)) {
-            throw new \InvalidArgumentException("Bad theme '$theme'");
-        }
-        if (!preg_match('/^(Layout\/)?[0-9a-zA-Z_-]+\.ss$/', $template)) {
-            throw new \InvalidArgumentException("Bad template '$template'");
-        }
+	/**
+	 * Create a template within a test theme
+	 *
+	 * @Given /^a template "(?<template>[^"]+)" in theme "(?<theme>[^"]+)" with content "(?<content>[^"]+)"/
+	 */
+	public function stepCreateTemplate($template, $theme, $content)
+	{
+		if (!preg_match('/^[0-9a-zA-Z_-]+$/', $theme)) {
+			throw new \InvalidArgumentException("Bad theme '$theme'");
+		}
+		if (!preg_match('/^(Layout\/)?[0-9a-zA-Z_-]+\.ss$/', $template)) {
+			throw new \InvalidArgumentException("Bad template '$template'");
+		}
 
-        $this->stepCreateTheme($theme);
-        $this->requireFile(BASE_PATH . '/themes/' . $theme . '/templates/' . $template, $content);
-    }
+		$this->stepCreateTheme($theme);
+		$this->requireFile(BASE_PATH . '/themes/' . $theme . '/templates/' . $template, $content);
+	}
 
-    protected function requireFile($filename, $content)
-    {
-        // Already exists
-        if (file_exists($filename)) {
-            // If the content is different, remember old content for restoration
-            $origContent = file_get_contents($filename);
-            if ($origContent != $content) {
-                file_put_contents($filename, $content);
-                $this->restoreFiles[$filename] = $origContent;
-            }
-        // Doesn't exist, mark it for deletion after test
-        } else {
-            file_put_contents($filename, $content);
-            $this->restoreFiles[$filename] = null;
-        }
-    }
+	protected function requireFile($filename, $content)
+	{
+		// Already exists
+		if (file_exists($filename)) {
+			// If the content is different, remember old content for restoration
+			$origContent = file_get_contents($filename);
+			if ($origContent != $content) {
+				file_put_contents($filename, $content);
+				$this->restoreFiles[$filename] = $origContent;
+			}
+		// Doesn't exist, mark it for deletion after test
+		} else {
+			file_put_contents($filename, $content);
+			$this->restoreFiles[$filename] = null;
+		}
+	}
 
-    protected function requireDir($dirname)
-    {
-        // Directory doesn't exist, create it and mark it for deletion
-        if (!file_exists($dirname)) {
-            mkdir($dirname);
-            $this->restoreDirectories[] = $dirname;
-        }
-    }
+	protected function requireDir($dirname)
+	{
+		// Directory doesn't exist, create it and mark it for deletion
+		if (!file_exists($dirname)) {
+			mkdir($dirname);
+			$this->restoreDirectories[] = $dirname;
+		}
+	}
 
-    /**
-     * Clean up any theme manipulation
-     *
-     * @AfterScenario
-     */
-    public function cleanThemesAfterScenario()
-    {
-        // Restore any created/modified files.
-        //  - If modified, revert then to original contnet
-        //  - If created, delete them
-        if ($this->restoreFiles) {
-            foreach ($this->restoreFiles as $file => $origContent) {
-                if ($origContent === null) {
-                    unlink($file);
-                } else {
-                    file_put_contents($file, $origContent);
-                }
-            }
+	/**
+	 * Clean up any theme manipulation
+	 *
+	 * @AfterScenario
+	 */
+	public function cleanThemesAfterScenario()
+	{
+		// Restore any created/modified files.
+		//  - If modified, revert then to original contnet
+		//  - If created, delete them
+		if ($this->restoreFiles) {
+			foreach ($this->restoreFiles as $file => $origContent) {
+				if ($origContent === null) {
+					unlink($file);
+				} else {
+					file_put_contents($file, $origContent);
+				}
+			}
 
-            $this->restoreFiles = array();
-        }
+			$this->restoreFiles = array();
+		}
 
-        // Restore any created directories: that is, delete them
-        if ($this->restoreDirectories) {
-            // Flip the order so that nested direcotires are unlinked() first
-            $this->restoreDirectories = array_reverse($this->restoreDirectories);
-            foreach ($this->restoreDirectories as $dir) {
-                rmdir($dir);
-            }
+		// Restore any created directories: that is, delete them
+		if ($this->restoreDirectories) {
+			// Flip the order so that nested direcotires are unlinked() first
+			$this->restoreDirectories = array_reverse($this->restoreDirectories);
+			foreach ($this->restoreDirectories as $dir) {
+				rmdir($dir);
+			}
 
-            $this->restoreDirectories = array();
-        }
-    }
+			$this->restoreDirectories = array();
+		}
+	}
 }

Spacing +4 added lines, -4 removed lines

@@ -31,9 +31,9 @@ 
             throw new \InvalidArgumentException("Bad theme '$theme'");
         }
 
-        $this->requireDir(BASE_PATH . '/themes');
-        $this->requireDir(BASE_PATH . '/themes/' . $theme);
-        $this->requireDir(BASE_PATH . '/themes/' . $theme . '/templates');
+        $this->requireDir(BASE_PATH.'/themes');
+        $this->requireDir(BASE_PATH.'/themes/'.$theme);
+        $this->requireDir(BASE_PATH.'/themes/'.$theme.'/templates');
     }
 
     /**
@@ -51,7 +51,7 @@ 
         }
 
         $this->stepCreateTheme($theme);
-        $this->requireFile(BASE_PATH . '/themes/' . $theme . '/templates/' . $template, $content);
+        $this->requireFile(BASE_PATH.'/themes/'.$theme.'/templates/'.$template, $content);
     }
 
     protected function requireFile($filename, $content)

code/SiteConfig.php

Doc Comments +1 added lines, -1 removed lines

@@ -242,7 +242,7 @@
      * Can a user view this SiteConfig instance?
      *
      * @param Member $member
-     * @return boolean
+     * @return boolean|string
      */
     public function canView($member = null)
     {

Indentation +395 added lines, -395 removed lines

@@ -34,399 +34,399 @@
  */
 class SiteConfig extends DataObject implements PermissionProvider, TemplateGlobalProvider
 {
-    private static $db = array(
-        "Title" => "Varchar(255)",
-        "Tagline" => "Varchar(255)",
-        "CanViewType" => "Enum('Anyone, LoggedInUsers, OnlyTheseUsers', 'Anyone')",
-        "CanEditType" => "Enum('LoggedInUsers, OnlyTheseUsers', 'LoggedInUsers')",
-        "CanCreateTopLevelType" => "Enum('LoggedInUsers, OnlyTheseUsers', 'LoggedInUsers')",
-    );
-
-    private static $many_many = array(
-        "ViewerGroups" => "SilverStripe\\Security\\Group",
-        "EditorGroups" => "SilverStripe\\Security\\Group",
-        "CreateTopLevelGroups" => "SilverStripe\\Security\\Group"
-    );
-
-    private static $defaults = array(
-        "CanViewType" => "Anyone",
-        "CanEditType" => "LoggedInUsers",
-        "CanCreateTopLevelType" => "LoggedInUsers",
-    );
-
-    private static $table_name = 'SiteConfig';
-
-    /**
-     * Default permission to check for 'LoggedInUsers' to create or edit pages
-     *
-     * @var array
-     * @config
-     */
-    private static $required_permission = array('CMS_ACCESS_CMSMain', 'CMS_ACCESS_LeftAndMain');
-
-
-    public function populateDefaults()
-    {
-        $this->Title = _t('SiteConfig.SITENAMEDEFAULT', "Your Site Name");
-        $this->Tagline = _t('SiteConfig.TAGLINEDEFAULT', "your tagline here");
-
-        // Allow these defaults to be overridden
-        parent::populateDefaults();
-    }
-
-    /**
-     * Get the fields that are sent to the CMS.
-     *
-     * In your extensions: updateCMSFields($fields).
-     *
-     * @return FieldList
-     */
-    public function getCMSFields()
-    {
-        $groupsMap = array();
-
-        foreach (Group::get() as $group) {
-            // Listboxfield values are escaped, use ASCII char instead of »
-            $groupsMap[$group->ID] = $group->getBreadcrumbs(' > ');
-        }
-        asort($groupsMap);
-
-        $fields = new FieldList(
-            new TabSet("Root",
-                $tabMain = new Tab('Main',
-                    $titleField = new TextField("Title", _t('SiteConfig.SITETITLE', "Site title")),
-                    $taglineField = new TextField("Tagline", _t('SiteConfig.SITETAGLINE', "Site Tagline/Slogan"))
-                ),
-                $tabAccess = new Tab('Access',
-                    $viewersOptionsField = new OptionsetField("CanViewType", _t('SiteConfig.VIEWHEADER', "Who can view pages on this site?")),
-                    $viewerGroupsField = ListboxField::create("ViewerGroups", _t('SiteTree.VIEWERGROUPS', "Viewer Groups"))
-                        ->setSource($groupsMap)
-                        ->setAttribute(
-                            'data-placeholder',
-                            _t('SiteTree.GroupPlaceholder', 'Click to select group')
-                        ),
-                    $editorsOptionsField = new OptionsetField("CanEditType", _t('SiteConfig.EDITHEADER', "Who can edit pages on this site?")),
-                    $editorGroupsField = ListboxField::create("EditorGroups", _t('SiteTree.EDITORGROUPS', "Editor Groups"))
-                        ->setSource($groupsMap)
-                        ->setAttribute(
-                            'data-placeholder',
-                            _t('SiteTree.GroupPlaceholder', 'Click to select group')
-                        ),
-                    $topLevelCreatorsOptionsField = new OptionsetField("CanCreateTopLevelType", _t('SiteConfig.TOPLEVELCREATE', "Who can create pages in the root of the site?")),
-                    $topLevelCreatorsGroupsField = ListboxField::create("CreateTopLevelGroups", _t('SiteTree.TOPLEVELCREATORGROUPS', "Top level creators"))
-                        ->setSource($groupsMap)
-                        ->setAttribute(
-                            'data-placeholder',
-                            _t('SiteTree.GroupPlaceholder', 'Click to select group')
-                        )
-                )
-            ),
-            new HiddenField('ID')
-        );
-
-        $viewersOptionsSource = array();
-        $viewersOptionsSource["Anyone"] = _t('SiteTree.ACCESSANYONE', "Anyone");
-        $viewersOptionsSource["LoggedInUsers"] = _t('SiteTree.ACCESSLOGGEDIN', "Logged-in users");
-        $viewersOptionsSource["OnlyTheseUsers"] = _t('SiteTree.ACCESSONLYTHESE', "Only these people (choose from list)");
-        $viewersOptionsField->setSource($viewersOptionsSource);
-
-        $editorsOptionsSource = array();
-        $editorsOptionsSource["LoggedInUsers"] = _t('SiteTree.EDITANYONE', "Anyone who can log-in to the CMS");
-        $editorsOptionsSource["OnlyTheseUsers"] = _t('SiteTree.EDITONLYTHESE', "Only these people (choose from list)");
-        $editorsOptionsField->setSource($editorsOptionsSource);
-
-        $topLevelCreatorsOptionsField->setSource($editorsOptionsSource);
-
-        if (!Permission::check('EDIT_SITECONFIG')) {
-            $fields->makeFieldReadonly($viewersOptionsField);
-            $fields->makeFieldReadonly($viewerGroupsField);
-            $fields->makeFieldReadonly($editorsOptionsField);
-            $fields->makeFieldReadonly($editorGroupsField);
-            $fields->makeFieldReadonly($topLevelCreatorsOptionsField);
-            $fields->makeFieldReadonly($topLevelCreatorsGroupsField);
-            $fields->makeFieldReadonly($taglineField);
-            $fields->makeFieldReadonly($titleField);
-        }
-
-        if (file_exists(BASE_PATH . '/install.php')) {
-            $fields->addFieldToTab("Root.Main", new LiteralField("InstallWarningHeader",
-                "<p class=\"message warning\">" . _t("SiteTree.REMOVE_INSTALL_WARNING",
-                "Warning: You should remove install.php from this SilverStripe install for security reasons.")
-                . "</p>"), "Title");
-        }
-
-        $tabMain->setTitle(_t('SiteConfig.TABMAIN', "Main"));
-        $tabAccess->setTitle(_t('SiteConfig.TABACCESS', "Access"));
-        $this->extend('updateCMSFields', $fields);
-
-        return $fields;
-    }
-
-    /**
-     * Get the actions that are sent to the CMS.
-     *
-     * In your extensions: updateEditFormActions($actions)
-     *
-     * @return FieldList
-     */
-    public function getCMSActions()
-    {
-        if (Permission::check('ADMIN') || Permission::check('EDIT_SITECONFIG')) {
-            $actions = new FieldList(
-                FormAction::create('save_siteconfig',
-                    _t('CMSMain.SAVE', 'Save')
-                )->addExtraClass('btn-primary font-icon-save')
-            );
-        } else {
-            $actions = new FieldList();
-        }
-
-        $this->extend('updateCMSActions', $actions);
-
-        return $actions;
-    }
-
-    /**
-     * @return string
-     */
-    public function CMSEditLink()
-    {
-        return SiteConfigLeftAndMain::singleton()->Link();
-    }
-
-    /**
-     * Get the current sites SiteConfig, and creates a new one through
-     * {@link make_site_config()} if none is found.
-     *
-     * @return SiteConfig
-     */
-    public static function current_site_config()
-    {
-        if ($siteConfig = DataObject::get_one('SilverStripe\\SiteConfig\\SiteConfig')) {
-            return $siteConfig;
-        }
-
-        return self::make_site_config();
-    }
-
-    /**
-     * Setup a default SiteConfig record if none exists.
-     */
-    public function requireDefaultRecords()
-    {
-        parent::requireDefaultRecords();
-
-        $config = DataObject::get_one('SilverStripe\\SiteConfig\\SiteConfig');
-
-        if (!$config) {
-            self::make_site_config();
-
-            DB::alteration_message("Added default site config", "created");
-        }
-    }
-
-    /**
-     * Create SiteConfig with defaults from language file.
-     *
-     * @return SiteConfig
-     */
-    public static function make_site_config()
-    {
-        $config = SiteConfig::create();
-        $config->write();
-
-        return $config;
-    }
-
-    /**
-     * Can a user view this SiteConfig instance?
-     *
-     * @param Member $member
-     * @return boolean
-     */
-    public function canView($member = null)
-    {
-        if (!$member) {
-            $member = Member::currentUserID();
-        }
-        if ($member && is_numeric($member)) {
-            $member = DataObject::get_by_id('SilverStripe\\Security\\Member', $member);
-        }
-
-        $extended = $this->extendedCan('canView', $member);
-        if ($extended !== null) {
-            return $extended;
-        }
-
-        // Assuming all that can edit this object can also view it
-        return $this->canEdit($member);
-    }
-
-    /**
-     * Can a user view pages on this site? This method is only
-     * called if a page is set to Inherit, but there is nothing
-     * to inherit from.
-     *
-     * @param Member $member
-     * @return boolean
-     */
-    public function canViewPages($member = null)
-    {
-        if (!$member) {
-            $member = Member::currentUserID();
-        }
-        if ($member && is_numeric($member)) {
-            $member = DataObject::get_by_id('SilverStripe\\Security\\Member', $member);
-        }
-
-        if ($member && Permission::checkMember($member, "ADMIN")) {
-            return true;
-        }
-
-        $extended = $this->extendedCan('canViewPages', $member);
-        if ($extended !== null) {
-            return $extended;
-        }
-
-        if (!$this->CanViewType || $this->CanViewType == 'Anyone') {
-            return true;
-        }
-
-        // check for any logged-in users
-        if ($this->CanViewType === 'LoggedInUsers' && $member) {
-            return true;
-        }
-
-        // check for specific groups
-        if ($this->CanViewType === 'OnlyTheseUsers' && $member && $member->inGroups($this->ViewerGroups())) {
-            return true;
-        }
-
-        return false;
-    }
-
-    /**
-     * Can a user edit pages on this site? This method is only
-     * called if a page is set to Inherit, but there is nothing
-     * to inherit from, or on new records without a parent.
-     *
-     * @param Member $member
-     * @return boolean
-     */
-    public function canEditPages($member = null)
-    {
-        if (!$member) {
-            $member = Member::currentUserID();
-        }
-        if ($member && is_numeric($member)) {
-            $member = DataObject::get_by_id('SilverStripe\\Security\\Member', $member);
-        }
-
-        if ($member && Permission::checkMember($member, "ADMIN")) {
-            return true;
-        }
-
-        $extended = $this->extendedCan('canEditPages', $member);
-        if ($extended !== null) {
-            return $extended;
-        }
-
-        // check for any logged-in users with CMS access
-        if ($this->CanEditType === 'LoggedInUsers'
-            && Permission::checkMember($member, $this->config()->required_permission)
-        ) {
-            return true;
-        }
-
-            // check for specific groups
-        if ($this->CanEditType === 'OnlyTheseUsers' && $member && $member->inGroups($this->EditorGroups())) {
-            return true;
-        }
-
-        return false;
-    }
-
-    public function canEdit($member = null)
-    {
-        if (!$member) {
-            $member = Member::currentUserID();
-        }
-        if ($member && is_numeric($member)) {
-            $member = DataObject::get_by_id('SilverStripe\\Security\\Member', $member);
-        }
-
-        $extended = $this->extendedCan('canEdit', $member);
-        if ($extended !== null) {
-            return $extended;
-        }
-
-        return Permission::checkMember($member, "EDIT_SITECONFIG");
-    }
-
-    /**
-     * @return array
-     */
-    public function providePermissions()
-    {
-        return array(
-            'EDIT_SITECONFIG' => array(
-                'name' => _t('SiteConfig.EDIT_PERMISSION', 'Manage site configuration'),
-                'category' => _t('Permissions.PERMISSIONS_CATEGORY', 'Roles and access permissions'),
-                'help' => _t('SiteConfig.EDIT_PERMISSION_HELP', 'Ability to edit global access settings/top-level page permissions.'),
-                'sort' => 400
-            )
-        );
-    }
-
-    /**
-     * Can a user create pages in the root of this site?
-     *
-     * @param Member $member
-     * @return boolean
-     */
-    public function canCreateTopLevel($member = null)
-    {
-        if (!$member) {
-            $member = Member::currentUserID();
-        }
-        if ($member && is_numeric($member)) {
-            $member = DataObject::get_by_id('SilverStripe\\Security\\Member', $member);
-        }
-
-        if ($member && Permission::checkMember($member, "ADMIN")) {
-            return true;
-        }
-
-        $extended = $this->extendedCan('canCreateTopLevel', $member);
-        if ($extended !== null) {
-            return $extended;
-        }
-
-        // check for any logged-in users with CMS permission
-        if ($this->CanCreateTopLevelType === 'LoggedInUsers'
-            && Permission::checkMember($member, $this->config()->required_permission)
-        ) {
-            return true;
-        }
-
-        // check for specific groups
-        if ($this->CanCreateTopLevelType === 'OnlyTheseUsers'
-            && $member
-            && $member->inGroups($this->CreateTopLevelGroups())
-        ) {
-            return true;
-        }
-
-        return false;
-    }
-
-    /**
-     * Add $SiteConfig to all SSViewers
-     */
-    public static function get_template_global_variables()
-    {
-        return array(
-            'SiteConfig' => 'current_site_config',
-        );
-    }
+	private static $db = array(
+		"Title" => "Varchar(255)",
+		"Tagline" => "Varchar(255)",
+		"CanViewType" => "Enum('Anyone, LoggedInUsers, OnlyTheseUsers', 'Anyone')",
+		"CanEditType" => "Enum('LoggedInUsers, OnlyTheseUsers', 'LoggedInUsers')",
+		"CanCreateTopLevelType" => "Enum('LoggedInUsers, OnlyTheseUsers', 'LoggedInUsers')",
+	);
+
+	private static $many_many = array(
+		"ViewerGroups" => "SilverStripe\\Security\\Group",
+		"EditorGroups" => "SilverStripe\\Security\\Group",
+		"CreateTopLevelGroups" => "SilverStripe\\Security\\Group"
+	);
+
+	private static $defaults = array(
+		"CanViewType" => "Anyone",
+		"CanEditType" => "LoggedInUsers",
+		"CanCreateTopLevelType" => "LoggedInUsers",
+	);
+
+	private static $table_name = 'SiteConfig';
+
+	/**
+	 * Default permission to check for 'LoggedInUsers' to create or edit pages
+	 *
+	 * @var array
+	 * @config
+	 */
+	private static $required_permission = array('CMS_ACCESS_CMSMain', 'CMS_ACCESS_LeftAndMain');
+
+
+	public function populateDefaults()
+	{
+		$this->Title = _t('SiteConfig.SITENAMEDEFAULT', "Your Site Name");
+		$this->Tagline = _t('SiteConfig.TAGLINEDEFAULT', "your tagline here");
+
+		// Allow these defaults to be overridden
+		parent::populateDefaults();
+	}
+
+	/**
+	 * Get the fields that are sent to the CMS.
+	 *
+	 * In your extensions: updateCMSFields($fields).
+	 *
+	 * @return FieldList
+	 */
+	public function getCMSFields()
+	{
+		$groupsMap = array();
+
+		foreach (Group::get() as $group) {
+			// Listboxfield values are escaped, use ASCII char instead of &raquo;
+			$groupsMap[$group->ID] = $group->getBreadcrumbs(' > ');
+		}
+		asort($groupsMap);
+
+		$fields = new FieldList(
+			new TabSet("Root",
+				$tabMain = new Tab('Main',
+					$titleField = new TextField("Title", _t('SiteConfig.SITETITLE', "Site title")),
+					$taglineField = new TextField("Tagline", _t('SiteConfig.SITETAGLINE', "Site Tagline/Slogan"))
+				),
+				$tabAccess = new Tab('Access',
+					$viewersOptionsField = new OptionsetField("CanViewType", _t('SiteConfig.VIEWHEADER', "Who can view pages on this site?")),
+					$viewerGroupsField = ListboxField::create("ViewerGroups", _t('SiteTree.VIEWERGROUPS', "Viewer Groups"))
+						->setSource($groupsMap)
+						->setAttribute(
+							'data-placeholder',
+							_t('SiteTree.GroupPlaceholder', 'Click to select group')
+						),
+					$editorsOptionsField = new OptionsetField("CanEditType", _t('SiteConfig.EDITHEADER', "Who can edit pages on this site?")),
+					$editorGroupsField = ListboxField::create("EditorGroups", _t('SiteTree.EDITORGROUPS', "Editor Groups"))
+						->setSource($groupsMap)
+						->setAttribute(
+							'data-placeholder',
+							_t('SiteTree.GroupPlaceholder', 'Click to select group')
+						),
+					$topLevelCreatorsOptionsField = new OptionsetField("CanCreateTopLevelType", _t('SiteConfig.TOPLEVELCREATE', "Who can create pages in the root of the site?")),
+					$topLevelCreatorsGroupsField = ListboxField::create("CreateTopLevelGroups", _t('SiteTree.TOPLEVELCREATORGROUPS', "Top level creators"))
+						->setSource($groupsMap)
+						->setAttribute(
+							'data-placeholder',
+							_t('SiteTree.GroupPlaceholder', 'Click to select group')
+						)
+				)
+			),
+			new HiddenField('ID')
+		);
+
+		$viewersOptionsSource = array();
+		$viewersOptionsSource["Anyone"] = _t('SiteTree.ACCESSANYONE', "Anyone");
+		$viewersOptionsSource["LoggedInUsers"] = _t('SiteTree.ACCESSLOGGEDIN', "Logged-in users");
+		$viewersOptionsSource["OnlyTheseUsers"] = _t('SiteTree.ACCESSONLYTHESE', "Only these people (choose from list)");
+		$viewersOptionsField->setSource($viewersOptionsSource);
+
+		$editorsOptionsSource = array();
+		$editorsOptionsSource["LoggedInUsers"] = _t('SiteTree.EDITANYONE', "Anyone who can log-in to the CMS");
+		$editorsOptionsSource["OnlyTheseUsers"] = _t('SiteTree.EDITONLYTHESE', "Only these people (choose from list)");
+		$editorsOptionsField->setSource($editorsOptionsSource);
+
+		$topLevelCreatorsOptionsField->setSource($editorsOptionsSource);
+
+		if (!Permission::check('EDIT_SITECONFIG')) {
+			$fields->makeFieldReadonly($viewersOptionsField);
+			$fields->makeFieldReadonly($viewerGroupsField);
+			$fields->makeFieldReadonly($editorsOptionsField);
+			$fields->makeFieldReadonly($editorGroupsField);
+			$fields->makeFieldReadonly($topLevelCreatorsOptionsField);
+			$fields->makeFieldReadonly($topLevelCreatorsGroupsField);
+			$fields->makeFieldReadonly($taglineField);
+			$fields->makeFieldReadonly($titleField);
+		}
+
+		if (file_exists(BASE_PATH . '/install.php')) {
+			$fields->addFieldToTab("Root.Main", new LiteralField("InstallWarningHeader",
+				"<p class=\"message warning\">" . _t("SiteTree.REMOVE_INSTALL_WARNING",
+				"Warning: You should remove install.php from this SilverStripe install for security reasons.")
+				. "</p>"), "Title");
+		}
+
+		$tabMain->setTitle(_t('SiteConfig.TABMAIN', "Main"));
+		$tabAccess->setTitle(_t('SiteConfig.TABACCESS', "Access"));
+		$this->extend('updateCMSFields', $fields);
+
+		return $fields;
+	}
+
+	/**
+	 * Get the actions that are sent to the CMS.
+	 *
+	 * In your extensions: updateEditFormActions($actions)
+	 *
+	 * @return FieldList
+	 */
+	public function getCMSActions()
+	{
+		if (Permission::check('ADMIN') || Permission::check('EDIT_SITECONFIG')) {
+			$actions = new FieldList(
+				FormAction::create('save_siteconfig',
+					_t('CMSMain.SAVE', 'Save')
+				)->addExtraClass('btn-primary font-icon-save')
+			);
+		} else {
+			$actions = new FieldList();
+		}
+
+		$this->extend('updateCMSActions', $actions);
+
+		return $actions;
+	}
+
+	/**
+	 * @return string
+	 */
+	public function CMSEditLink()
+	{
+		return SiteConfigLeftAndMain::singleton()->Link();
+	}
+
+	/**
+	 * Get the current sites SiteConfig, and creates a new one through
+	 * {@link make_site_config()} if none is found.
+	 *
+	 * @return SiteConfig
+	 */
+	public static function current_site_config()
+	{
+		if ($siteConfig = DataObject::get_one('SilverStripe\\SiteConfig\\SiteConfig')) {
+			return $siteConfig;
+		}
+
+		return self::make_site_config();
+	}
+
+	/**
+	 * Setup a default SiteConfig record if none exists.
+	 */
+	public function requireDefaultRecords()
+	{
+		parent::requireDefaultRecords();
+
+		$config = DataObject::get_one('SilverStripe\\SiteConfig\\SiteConfig');
+
+		if (!$config) {
+			self::make_site_config();
+
+			DB::alteration_message("Added default site config", "created");
+		}
+	}
+
+	/**
+	 * Create SiteConfig with defaults from language file.
+	 *
+	 * @return SiteConfig
+	 */
+	public static function make_site_config()
+	{
+		$config = SiteConfig::create();
+		$config->write();
+
+		return $config;
+	}
+
+	/**
+	 * Can a user view this SiteConfig instance?
+	 *
+	 * @param Member $member
+	 * @return boolean
+	 */
+	public function canView($member = null)
+	{
+		if (!$member) {
+			$member = Member::currentUserID();
+		}
+		if ($member && is_numeric($member)) {
+			$member = DataObject::get_by_id('SilverStripe\\Security\\Member', $member);
+		}
+
+		$extended = $this->extendedCan('canView', $member);
+		if ($extended !== null) {
+			return $extended;
+		}
+
+		// Assuming all that can edit this object can also view it
+		return $this->canEdit($member);
+	}
+
+	/**
+	 * Can a user view pages on this site? This method is only
+	 * called if a page is set to Inherit, but there is nothing
+	 * to inherit from.
+	 *
+	 * @param Member $member
+	 * @return boolean
+	 */
+	public function canViewPages($member = null)
+	{
+		if (!$member) {
+			$member = Member::currentUserID();
+		}
+		if ($member && is_numeric($member)) {
+			$member = DataObject::get_by_id('SilverStripe\\Security\\Member', $member);
+		}
+
+		if ($member && Permission::checkMember($member, "ADMIN")) {
+			return true;
+		}
+
+		$extended = $this->extendedCan('canViewPages', $member);
+		if ($extended !== null) {
+			return $extended;
+		}
+
+		if (!$this->CanViewType || $this->CanViewType == 'Anyone') {
+			return true;
+		}
+
+		// check for any logged-in users
+		if ($this->CanViewType === 'LoggedInUsers' && $member) {
+			return true;
+		}
+
+		// check for specific groups
+		if ($this->CanViewType === 'OnlyTheseUsers' && $member && $member->inGroups($this->ViewerGroups())) {
+			return true;
+		}
+
+		return false;
+	}
+
+	/**
+	 * Can a user edit pages on this site? This method is only
+	 * called if a page is set to Inherit, but there is nothing
+	 * to inherit from, or on new records without a parent.
+	 *
+	 * @param Member $member
+	 * @return boolean
+	 */
+	public function canEditPages($member = null)
+	{
+		if (!$member) {
+			$member = Member::currentUserID();
+		}
+		if ($member && is_numeric($member)) {
+			$member = DataObject::get_by_id('SilverStripe\\Security\\Member', $member);
+		}
+
+		if ($member && Permission::checkMember($member, "ADMIN")) {
+			return true;
+		}
+
+		$extended = $this->extendedCan('canEditPages', $member);
+		if ($extended !== null) {
+			return $extended;
+		}
+
+		// check for any logged-in users with CMS access
+		if ($this->CanEditType === 'LoggedInUsers'
+			&& Permission::checkMember($member, $this->config()->required_permission)
+		) {
+			return true;
+		}
+
+			// check for specific groups
+		if ($this->CanEditType === 'OnlyTheseUsers' && $member && $member->inGroups($this->EditorGroups())) {
+			return true;
+		}
+
+		return false;
+	}
+
+	public function canEdit($member = null)
+	{
+		if (!$member) {
+			$member = Member::currentUserID();
+		}
+		if ($member && is_numeric($member)) {
+			$member = DataObject::get_by_id('SilverStripe\\Security\\Member', $member);
+		}
+
+		$extended = $this->extendedCan('canEdit', $member);
+		if ($extended !== null) {
+			return $extended;
+		}
+
+		return Permission::checkMember($member, "EDIT_SITECONFIG");
+	}
+
+	/**
+	 * @return array
+	 */
+	public function providePermissions()
+	{
+		return array(
+			'EDIT_SITECONFIG' => array(
+				'name' => _t('SiteConfig.EDIT_PERMISSION', 'Manage site configuration'),
+				'category' => _t('Permissions.PERMISSIONS_CATEGORY', 'Roles and access permissions'),
+				'help' => _t('SiteConfig.EDIT_PERMISSION_HELP', 'Ability to edit global access settings/top-level page permissions.'),
+				'sort' => 400
+			)
+		);
+	}
+
+	/**
+	 * Can a user create pages in the root of this site?
+	 *
+	 * @param Member $member
+	 * @return boolean
+	 */
+	public function canCreateTopLevel($member = null)
+	{
+		if (!$member) {
+			$member = Member::currentUserID();
+		}
+		if ($member && is_numeric($member)) {
+			$member = DataObject::get_by_id('SilverStripe\\Security\\Member', $member);
+		}
+
+		if ($member && Permission::checkMember($member, "ADMIN")) {
+			return true;
+		}
+
+		$extended = $this->extendedCan('canCreateTopLevel', $member);
+		if ($extended !== null) {
+			return $extended;
+		}
+
+		// check for any logged-in users with CMS permission
+		if ($this->CanCreateTopLevelType === 'LoggedInUsers'
+			&& Permission::checkMember($member, $this->config()->required_permission)
+		) {
+			return true;
+		}
+
+		// check for specific groups
+		if ($this->CanCreateTopLevelType === 'OnlyTheseUsers'
+			&& $member
+			&& $member->inGroups($this->CreateTopLevelGroups())
+		) {
+			return true;
+		}
+
+		return false;
+	}
+
+	/**
+	 * Add $SiteConfig to all SSViewers
+	 */
+	public static function get_template_global_variables()
+	{
+		return array(
+			'SiteConfig' => 'current_site_config',
+		);
+	}
 }

Spacing +2 added lines, -2 removed lines

@@ -148,9 +148,9 @@
             $fields->makeFieldReadonly($titleField);
         }
 
-        if (file_exists(BASE_PATH . '/install.php')) {
+        if (file_exists(BASE_PATH.'/install.php')) {
             $fields->addFieldToTab("Root.Main", new LiteralField("InstallWarningHeader",
-                "<p class=\"message warning\">" . _t("SiteTree.REMOVE_INSTALL_WARNING",
+                "<p class=\"message warning\">"._t("SiteTree.REMOVE_INSTALL_WARNING",
                 "Warning: You should remove install.php from this SilverStripe install for security reasons.")
                 . "</p>"), "Title");
         }

code/SiteConfigLeftAndMain.php

Doc Comments +1 added lines, -1 removed lines

@@ -134,7 +134,7 @@
 	 *
 	 * @param array $data
 	 * @param Form $form
-	 * @return String
+	 * @return \SilverStripe\ORM\FieldType\DBHTMLText
 	 */
     public function save_siteconfig($data, $form)
     {

Unused Use Statements -1 removed lines

@@ -8,7 +8,6 @@
 use SilverStripe\Forms\HiddenField;
 use SilverStripe\Forms\LiteralField;
 use SilverStripe\Forms\Form;
-use SilverStripe\ORM\ValidationException;
 use SilverStripe\ORM\ArrayList;
 use SilverStripe\ORM\ValidationResult;
 use SilverStripe\View\ArrayData;

Indentation +19 added lines, -19 removed lines

@@ -56,9 +56,9 @@ 
 	 */
 	public function init() {
 		parent::init();
-        if (defined('CMS_DIR')) {
-            Requirements::javascript(CMS_DIR . '/client/dist/js/bundle.js');
-        }
+		if (defined('CMS_DIR')) {
+			Requirements::javascript(CMS_DIR . '/client/dist/js/bundle.js');
+		}
 	}
 
 	/**
@@ -67,8 +67,8 @@ 
 	 *
 	 * @return Form
 	 */
-    public function getEditForm($id = null, $fields = null)
-    {
+	public function getEditForm($id = null, $fields = null)
+	{
 		$siteConfig = SiteConfig::current_site_config();
 		$fields = $siteConfig->getCMSFields();
 
@@ -77,7 +77,7 @@ 
 		$fields->push(new HiddenField('PreviewURL', 'Preview URL', $home));
 
 		// Added in-line to the form, but plucked into different view by LeftAndMain.Preview.js upon load
-        /** @skipUpgrade */
+		/** @skipUpgrade */
 		$fields->push($navField = new LiteralField('SilverStripeNavigator', $this->getSilverStripeNavigator()));
 		$navField->setAllowHTML(true);
 
@@ -108,21 +108,21 @@ 
 		$form->addExtraClass('flexbox-area-grow fill-height cms-content cms-edit-form');
 		$form->setAttribute('data-pjax-fragment', 'CurrentForm');
 
-        if ($form->Fields()->hasTabSet()) {
-            $form->Fields()->findOrMakeTab('Root')->setTemplate('SilverStripe\\Forms\\CMSTabSet');
-        }
+		if ($form->Fields()->hasTabSet()) {
+			$form->Fields()->findOrMakeTab('Root')->setTemplate('SilverStripe\\Forms\\CMSTabSet');
+		}
 		$form->setHTMLID('Form_EditForm');
 		$form->loadDataFrom($siteConfig);
 		$form->setTemplate($this->getTemplatesWithSuffix('_EditForm'));
 
 		// Use <button> to allow full jQuery UI styling
 		$actions = $actions->dataFields();
-        if ($actions) {
-            /** @var FormAction $action */
-            foreach ($actions as $action) {
-                $action->setUseButtonTag(true);
-            }
-        }
+		if ($actions) {
+			/** @var FormAction $action */
+			foreach ($actions as $action) {
+				$action->setUseButtonTag(true);
+			}
+		}
 
 		$this->extend('updateEditForm', $form);
 
@@ -136,8 +136,8 @@ 
 	 * @param Form $form
 	 * @return String
 	 */
-    public function save_siteconfig($data, $form)
-    {
+	public function save_siteconfig($data, $form)
+	{
 		$siteConfig = SiteConfig::current_site_config();
 		$form->saveInto($siteConfig);
 		$siteConfig->write();
@@ -146,8 +146,8 @@ 
 	}
 
 
-    public function Breadcrumbs($unlinked = false)
-    {
+	public function Breadcrumbs($unlinked = false)
+	{
 		return new ArrayList(array(
 			new ArrayData(array(
 				'Title' => static::menu_title(),

Spacing +2 added lines, -2 removed lines

@@ -57,7 +57,7 @@ 
 	public function init() {
 		parent::init();
         if (defined('CMS_DIR')) {
-            Requirements::javascript(CMS_DIR . '/client/dist/js/bundle.js');
+            Requirements::javascript(CMS_DIR.'/client/dist/js/bundle.js');
         }
 	}
 
@@ -96,7 +96,7 @@ 
 		)->setHTMLID('Form_EditForm');
 		$form->setValidationResponseCallback(function(ValidationResult $errors) use ($negotiator, $form) {
 			$request = $this->getRequest();
-			if($request->isAjax() && $negotiator) {
+			if ($request->isAjax() && $negotiator) {
 				$result = $form->forTemplate();
 				return $negotiator->respond($request, array(
 					'CurrentForm' => function() use($result) {

tests/behat/features/bootstrap/Context/FeatureContext.php

Unused Use Statements -3 removed lines

@@ -2,9 +2,6 @@
 
 namespace SilverStripe\Siteconfig\Test\Behaviour;
 
-use SilverStripe\BehatExtension\Context\SilverStripeContext;
-use SilverStripe\BehatExtension\Context\BasicContext;
-use SilverStripe\BehatExtension\Context\LoginContext;
 use SilverStripe\BehatExtension\Context\FixtureContext;
 use SilverStripe\Cms\Test\Behaviour;
 use SilverStripe\Core\ClassInfo;

Indentation +28 added lines, -28 removed lines

@@ -19,35 +19,35 @@
  */
 class FeatureContext extends \SilverStripe\Framework\Test\Behaviour\FeatureContext
 {
-    /**
-     * Initializes context.
-     * Every scenario gets it's own context object.
-     *
-     * @param   array   $parameters     context parameters (set them up through behat.yml)
-     */
-    public function __construct(array $parameters)
-    {
-        parent::__construct($parameters);
+	/**
+	 * Initializes context.
+	 * Every scenario gets it's own context object.
+	 *
+	 * @param   array   $parameters     context parameters (set them up through behat.yml)
+	 */
+	public function __construct(array $parameters)
+	{
+		parent::__construct($parameters);
 
-        // Override existing fixture context with more specific one
-        $fixtureContext = new \SilverStripe\Siteconfig\Test\Behaviour\FixtureContext($parameters);
-        $fixtureContext->setFixtureFactory($this->getFixtureFactory());
-        $this->useContext('FixtureContext', $fixtureContext);
+		// Override existing fixture context with more specific one
+		$fixtureContext = new \SilverStripe\Siteconfig\Test\Behaviour\FixtureContext($parameters);
+		$fixtureContext->setFixtureFactory($this->getFixtureFactory());
+		$this->useContext('FixtureContext', $fixtureContext);
 
-        // Add extra contexts with more steps
-        $this->useContext('ThemeContext', new \SilverStripe\Siteconfig\Test\Behaviour\ThemeContext($parameters));
-        if(!class_exists('SilverStripe\\CMS\\Model\\SiteTree')) {
-            return;
-        }
+		// Add extra contexts with more steps
+		$this->useContext('ThemeContext', new \SilverStripe\Siteconfig\Test\Behaviour\ThemeContext($parameters));
+		if(!class_exists('SilverStripe\\CMS\\Model\\SiteTree')) {
+			return;
+		}
 
-        // Use blueprints which auto-publish all subclasses of SiteTree
-        $factory = $fixtureContext->getFixtureFactory();
-        foreach (ClassInfo::subclassesFor('SilverStripe\\CMS\\Model\\SiteTree') as $id => $class) {
-            $blueprint = Injector::inst()->create('SilverStripe\\Dev\\FixtureBlueprint', $class);
-            $blueprint->addCallback('afterCreate', function ($obj, $identifier, &$data, &$fixtures) {
-                $obj->publish('Stage', 'Live');
-            });
-            $factory->define($class, $blueprint);
-        }
-    }
+		// Use blueprints which auto-publish all subclasses of SiteTree
+		$factory = $fixtureContext->getFixtureFactory();
+		foreach (ClassInfo::subclassesFor('SilverStripe\\CMS\\Model\\SiteTree') as $id => $class) {
+			$blueprint = Injector::inst()->create('SilverStripe\\Dev\\FixtureBlueprint', $class);
+			$blueprint->addCallback('afterCreate', function ($obj, $identifier, &$data, &$fixtures) {
+				$obj->publish('Stage', 'Live');
+			});
+			$factory->define($class, $blueprint);
+		}
+	}
 }

Spacing +2 added lines, -2 removed lines

@@ -36,7 +36,7 @@ 
 
         // Add extra contexts with more steps
         $this->useContext('ThemeContext', new \SilverStripe\Siteconfig\Test\Behaviour\ThemeContext($parameters));
-        if(!class_exists('SilverStripe\\CMS\\Model\\SiteTree')) {
+        if (!class_exists('SilverStripe\\CMS\\Model\\SiteTree')) {
             return;
         }
 
@@ -44,7 +44,7 @@ 
         $factory = $fixtureContext->getFixtureFactory();
         foreach (ClassInfo::subclassesFor('SilverStripe\\CMS\\Model\\SiteTree') as $id => $class) {
             $blueprint = Injector::inst()->create('SilverStripe\\Dev\\FixtureBlueprint', $class);
-            $blueprint->addCallback('afterCreate', function ($obj, $identifier, &$data, &$fixtures) {
+            $blueprint->addCallback('afterCreate', function($obj, $identifier, &$data, &$fixtures) {
                 $obj->publish('Stage', 'Live');
             });
             $factory->define($class, $blueprint);

tests/SiteConfigTest.php

Indentation +49 added lines, -49 removed lines

@@ -9,66 +9,66 @@
  */
 class SiteConfigTest extends SapphireTest
 {
-    protected static $fixture_file = 'SiteConfigTest.yml';
+	protected static $fixture_file = 'SiteConfigTest.yml';
 
-    protected $illegalExtensions = array(
-        'SilverStripe\\CMS\\Model\\SiteTree' => array('SiteTreeSubsites')
-    );
+	protected $illegalExtensions = array(
+		'SilverStripe\\CMS\\Model\\SiteTree' => array('SiteTreeSubsites')
+	);
 
-    public function setUpOnce()
-    {
-        // Fix issue with tests failing without CMS module
-        if (!class_exists('SilverStripe\\CMS\\Model\\SiteTree')) {
-            unset($this->illegalExtensions['SilverStripe\\CMS\\Model\\SiteTree']);
-        }
+	public function setUpOnce()
+	{
+		// Fix issue with tests failing without CMS module
+		if (!class_exists('SilverStripe\\CMS\\Model\\SiteTree')) {
+			unset($this->illegalExtensions['SilverStripe\\CMS\\Model\\SiteTree']);
+		}
 
-        parent::setUpOnce(); // TODO: Change the autogenerated stub
-    }
+		parent::setUpOnce(); // TODO: Change the autogenerated stub
+	}
 
-    public function testCanCreateRootPages()
-    {
-        $config = $this->objFromFixture('SilverStripe\\SiteConfig\\SiteConfig', 'default');
+	public function testCanCreateRootPages()
+	{
+		$config = $this->objFromFixture('SilverStripe\\SiteConfig\\SiteConfig', 'default');
 
-        // Log in without pages admin access
-        $this->logInWithPermission('CMS_ACCESS_AssetAdmin');
-        $this->assertFalse($config->canCreateTopLevel());
+		// Log in without pages admin access
+		$this->logInWithPermission('CMS_ACCESS_AssetAdmin');
+		$this->assertFalse($config->canCreateTopLevel());
 
-        // Login with necessary edit permission
-        $perms = SiteConfig::config()->required_permission;
-        $this->logInWithPermission(reset($perms));
-        $this->assertTrue($config->canCreateTopLevel());
-    }
+		// Login with necessary edit permission
+		$perms = SiteConfig::config()->required_permission;
+		$this->logInWithPermission(reset($perms));
+		$this->assertTrue($config->canCreateTopLevel());
+	}
 
-    public function testCanViewPages()
-    {
-        $config = $this->objFromFixture('SilverStripe\\SiteConfig\\SiteConfig', 'default');
-        $this->assertTrue($config->canViewPages());
-    }
+	public function testCanViewPages()
+	{
+		$config = $this->objFromFixture('SilverStripe\\SiteConfig\\SiteConfig', 'default');
+		$this->assertTrue($config->canViewPages());
+	}
 
-    public function testCanEdit()
-    {
-        $config = $this->objFromFixture('SilverStripe\\SiteConfig\\SiteConfig', 'default');
+	public function testCanEdit()
+	{
+		$config = $this->objFromFixture('SilverStripe\\SiteConfig\\SiteConfig', 'default');
 
-        // Unrelated permissions don't allow siteconfig
-        $this->logInWithPermission('CMS_ACCESS_AssetAdmin');
-        $this->assertFalse($config->canEdit());
+		// Unrelated permissions don't allow siteconfig
+		$this->logInWithPermission('CMS_ACCESS_AssetAdmin');
+		$this->assertFalse($config->canEdit());
 
-        // Only those with edit permission can do this
-        $this->logInWithPermission('EDIT_SITECONFIG');
-        $this->assertTrue($config->canEdit());
-    }
+		// Only those with edit permission can do this
+		$this->logInWithPermission('EDIT_SITECONFIG');
+		$this->assertTrue($config->canEdit());
+	}
 
-    public function testCanEditPages()
-    {
-        $config = $this->objFromFixture('SilverStripe\\SiteConfig\\SiteConfig', 'default');
+	public function testCanEditPages()
+	{
+		$config = $this->objFromFixture('SilverStripe\\SiteConfig\\SiteConfig', 'default');
 
-        // Log in without pages admin access
-        $this->logInWithPermission('CMS_ACCESS_AssetAdmin');
-        $this->assertFalse($config->canEditPages());
+		// Log in without pages admin access
+		$this->logInWithPermission('CMS_ACCESS_AssetAdmin');
+		$this->assertFalse($config->canEditPages());
 
-        // Login with necessary edit permission
-        $perms = SiteConfig::config()->required_permission;
-        $this->logInWithPermission(reset($perms));
-        $this->assertTrue($config->canEditPages());
-    }
+		// Login with necessary edit permission
+		$perms = SiteConfig::config()->required_permission;
+		$this->logInWithPermission(reset($perms));
+		$this->assertTrue($config->canEditPages());
+	}
 }