dylangrech92 / seotoolbox

_config.php

Spacing +4 added lines, -4 removed lines

@@ -6,15 +6,15 @@
  *
  */
 
-define( 'SEOTOOLBOX_DIR', str_replace( Director::baseFolder().'/', '', __DIR__ ) );
+define('SEOTOOLBOX_DIR', str_replace(Director::baseFolder().'/', '', __DIR__));
 
 Config::inst()->update('SEOToolboxAdmin', 'menu_icon', SEOTOOLBOX_DIR.'/code/admin/icon/seotoolbox-icon.png');
-Config::inst()->update('LeftAndMain','extra_requirements_javascript', array(SEOTOOLBOX_DIR.'/js/seo_page_analyzer.js'));
-Config::inst()->update('LeftAndMain','extra_requirements_css', array(SEOTOOLBOX_DIR.'/css/seo_page_analyzer.css'));
+Config::inst()->update('LeftAndMain', 'extra_requirements_javascript', array(SEOTOOLBOX_DIR.'/js/seo_page_analyzer.js'));
+Config::inst()->update('LeftAndMain', 'extra_requirements_css', array(SEOTOOLBOX_DIR.'/css/seo_page_analyzer.css'));
 
 CMSMenu::add_link('seotest', 'Run Crawler', '/seotest', 9, array('target' => '_blank'));
 Requirements::customCSS('.icon.icon-16.icon-seotest{background:url('.SEOTOOLBOX_DIR.'/code/admin/icon/crawl-icon.png)}');
 
-if(Director::is_cli()){
+if (Director::is_cli()) {
     Config::inst()->update('Director', 'environment_type', 'dev');
 }

tests/AutomatedLinkTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -79,7 +79,7 @@
      * Renders the passed $page and returns all links found if any
      *
      * @param SiteTree $page
-     * @return DOMNameList
+     * @return DOMNodeList
      */
     public static function getLinksFromPage(SiteTree $page){
         $dom = self::getPageDOM($page);

Spacing +25 added lines, -25 removed lines

@@ -13,8 +13,8 @@ 
     /**
      * Test that links are included in the content field
      */
-    public function testInsertion(){
-        $this->objFromFixture('AutomatedLink','link');
+    public function testInsertion() {
+        $this->objFromFixture('AutomatedLink', 'link');
         $page   = $this->createPage('<p>Checking if phrase is replaced</p>');
         $link   = $this->getLinkFromPage($page);
         $this->assertTrue($link->nodeValue === 'phrase', 'Link wasn\'t created');
@@ -23,41 +23,41 @@ 
     /**
      * Test the individual properties of the automated link
      */
-    public function testProperties(){
-        $this->objFromFixture('AutomatedLink','link2');
+    public function testProperties() {
+        $this->objFromFixture('AutomatedLink', 'link2');
 
         $page   = self::createPage('<p>Checking if properties is with correct props</p>');
         $link   = self::getLinkFromPage($page);
 
-        $this->assertTrue( $link->nodeValue === 'properties', 'Phrase did not match' );
-        $this->assertTrue( $link->getAttribute('title') === 'test title', 'Title did not match' );
-        $this->assertTrue( $link->getAttribute('href') === '#test-anchor', 'Anchor did not match' );
-        $this->assertTrue( $link->getAttribute('target') === '_blank', 'Target did not match' );
-        $this->assertTrue( $link->getAttribute('rel') === 'nofollow', 'Rel did not match' );
+        $this->assertTrue($link->nodeValue === 'properties', 'Phrase did not match');
+        $this->assertTrue($link->getAttribute('title') === 'test title', 'Title did not match');
+        $this->assertTrue($link->getAttribute('href') === '#test-anchor', 'Anchor did not match');
+        $this->assertTrue($link->getAttribute('target') === '_blank', 'Target did not match');
+        $this->assertTrue($link->getAttribute('rel') === 'nofollow', 'Rel did not match');
     }
 
     /**
      * Test that the default properties match what we expect
      */
-    public function testDefaultProperties(){
-        $this->objFromFixture('AutomatedLink','link3');
+    public function testDefaultProperties() {
+        $this->objFromFixture('AutomatedLink', 'link3');
 
         $page   = self::createPage('<p>Checking if default is correct</p>');
         $link   = self::getLinkFromPage($page);
         $auto   = AutomatedLink::get()->find('Phrase', 'default');
 
-        $this->assertTrue( $link->nodeValue === 'default', 'Phrase did not match' );
-        $this->assertTrue( $link->getAttribute('title') === '', 'Title did not match' );
-        $this->assertTrue( $link->getAttribute('href') === '#', 'href did not match' );
-        $this->assertTrue( $link->getAttribute('target') === '', 'Target did not match' );
-        $this->assertTrue( $link->getAttribute('rel') === '', 'Rel did not match' );
-        $this->assertTrue( (int) $link->getAttribute('data-id') === $auto->ID, 'data-id did not match' );
+        $this->assertTrue($link->nodeValue === 'default', 'Phrase did not match');
+        $this->assertTrue($link->getAttribute('title') === '', 'Title did not match');
+        $this->assertTrue($link->getAttribute('href') === '#', 'href did not match');
+        $this->assertTrue($link->getAttribute('target') === '', 'Target did not match');
+        $this->assertTrue($link->getAttribute('rel') === '', 'Rel did not match');
+        $this->assertTrue((int) $link->getAttribute('data-id') === $auto->ID, 'data-id did not match');
     }
 
     /**
      * Test that the default properties match what we expect
      */
-    public function testLimitFilter(){
+    public function testLimitFilter() {
         $page   = self::createPage('<p>Checking if default is created only twice. default default</p>');
         $dom    = self::getPageDOM($page);
         $links  = $dom->getElementsByTagName('a');
@@ -67,7 +67,7 @@ 
     /**
      * Test that the default properties match what we expect
      */
-    public function testDisableFilter(){
+    public function testDisableFilter() {
         GlobalAutoLinkSettings::$enabled = false;
         $page   = self::createPage('<p>Checking if default is created at all when automated links are disabled.</p>');
         $links  = self::getLinksFromPage($page);
@@ -81,7 +81,7 @@ 
      * @param SiteTree $page
      * @return DOMNameList
      */
-    public static function getLinksFromPage(SiteTree $page){
+    public static function getLinksFromPage(SiteTree $page) {
         $dom = self::getPageDOM($page);
         return $dom->getElementsByTagName('a');
     }
@@ -94,9 +94,9 @@ 
      * @param SiteTree $page
      * @return DOMDocument
      */
-    public static function getLinkFromPage(SiteTree $page){
+    public static function getLinkFromPage(SiteTree $page) {
         $links = self::getLinksFromPage($page);
-        return ( $links->length != 1 ) ? new DOMElement() : $links->item(0);
+        return ($links->length != 1) ? new DOMElement() : $links->item(0);
     }
 
     /**
@@ -105,8 +105,8 @@ 
      * @param string|null $content
      * @return Page
      */
-    public static function createPage($content=null){
-        $page = Page::create(array( 'Content' => $content ));
+    public static function createPage($content = null) {
+        $page = Page::create(array('Content' => $content));
         $page->write();
         return $page;
     }
@@ -117,7 +117,7 @@ 
      * @param Page $page
      * @return DOMDocument
      */
-    public static function getPageDOM(Page $page){
+    public static function getPageDOM(Page $page) {
         $controller = Page_Controller::create($page);
         $controller->invokeWithExtensions('addAutomatedLinks');
         return AutomatedLink::constructDOMDocument($controller->Content);

tests/GlobalAutoLinkSettingsTest.php

Spacing +7 added lines, -7 removed lines

@@ -13,10 +13,10 @@ 
     /**
      * Test Max Links per page
      */
-    public function testMaxLinksPerPage(){
-        $this->objFromFixture('AutomatedLink','link');
-        $this->objFromFixture('AutomatedLink','link2');
-        $this->objFromFixture('AutomatedLink','link3');
+    public function testMaxLinksPerPage() {
+        $this->objFromFixture('AutomatedLink', 'link');
+        $this->objFromFixture('AutomatedLink', 'link2');
+        $this->objFromFixture('AutomatedLink', 'link3');
 
         $settings = GlobalAutoLinkSettings::get()->first();
         $settings->MaxLinksPerPage = 1;
@@ -30,7 +30,7 @@ 
     /**
      * Test that links are not created inside of exclude tags
      */
-    public function testExcludeTags(){
+    public function testExcludeTags() {
         $settings = GlobalAutoLinkSettings::get()->first();
         $settings->MaxLinksPerPage = 0;
         $settings->write();
@@ -45,7 +45,7 @@ 
     /**
      * Test that links are included in all the specified fields
      */
-    public function testIncludeIn(){
+    public function testIncludeIn() {
         $settings = GlobalAutoLinkSettings::get()->first();
         $settings->IncludeIn = 'Content, Introduction';
         $settings->write();
@@ -82,7 +82,7 @@ 
 /**
  * Simple test page to test settings
  */
-class TestPage extends Page implements TestOnly{
+class TestPage extends Page implements TestOnly {
     private static $db = array(
         'Introduction' => 'HTMLText'
     );