Passed
Push — develop ( d768fa...da82e5 )
by Dylan
02:43
created
code/dataobjects/AutomatedLink.php 1 patch
Spacing   +29 added lines, -29 removed lines patch added patch discarded remove patch
@@ -43,9 +43,9 @@  discard block
 block discarded – undo
43 43
         'Page' => 'SiteTree'
44 44
     );
45 45
 
46
-    private static $summary_fields    = array( 'Phrase', 'PointsTo' );
47
-    private static $searchable_fields = array( 'Phrase' );
48
-    private static $singular_name	  = 'Automated Link';
46
+    private static $summary_fields    = array('Phrase', 'PointsTo');
47
+    private static $searchable_fields = array('Phrase');
48
+    private static $singular_name = 'Automated Link';
49 49
     private static $plural_name	      = 'Automated Links';
50 50
     private static $parsableFields    = array();
51 51
 
@@ -54,7 +54,7 @@  discard block
 block discarded – undo
54 54
      *
55 55
      * @return string
56 56
      */
57
-    public function PointsTo(){
57
+    public function PointsTo() {
58 58
         return $this->Page()->Link();
59 59
     }
60 60
 
@@ -63,7 +63,7 @@  discard block
 block discarded – undo
63 63
      *
64 64
      * @return string
65 65
      */
66
-    public function Title(){
66
+    public function Title() {
67 67
         return $this->Phrase;
68 68
     }
69 69
 
@@ -72,23 +72,23 @@  discard block
 block discarded – undo
72 72
      *
73 73
      * @return String
74 74
      */
75
-    public function forTemplate(){
75
+    public function forTemplate() {
76 76
         return $this->getHTML();
77 77
     }
78 78
 
79
-    public function canView( $member = false ){
79
+    public function canView($member = false) {
80 80
         return Permission::check('AUTOMATEDLINK_VIEW');
81 81
     }
82 82
 
83
-    public function canEdit( $member = false ){
83
+    public function canEdit($member = false) {
84 84
         return Permission::check('AUTOMATEDLINK_EDIT');
85 85
     }
86 86
 
87
-    public function canDelete( $member = false ){
87
+    public function canDelete($member = false) {
88 88
         return Permission::check('AUTOMATEDLINK_DELETE');
89 89
     }
90 90
 
91
-    public function canCreate( $member = false ){
91
+    public function canCreate($member = false) {
92 92
         return Permission::check('AUTOMATEDLINK_CREATE');
93 93
     }
94 94
 
@@ -101,12 +101,12 @@  discard block
 block discarded – undo
101 101
        );
102 102
      }
103 103
 
104
-	public function requireDefaultRecords(){
104
+	public function requireDefaultRecords() {
105 105
 		parent::requireDefaultRecords();
106 106
 
107 107
 		// Update all links to redirector pages during dev/build
108
-		foreach( self::get() as $link ) {
109
-		    $link->CheckAndUpdateDestination( true );
108
+		foreach (self::get() as $link) {
109
+		    $link->CheckAndUpdateDestination(true);
110 110
 		}
111 111
 	}
112 112
 
@@ -207,15 +207,15 @@  discard block
 block discarded – undo
207 207
 	 * @Boolean $write - Write the changes if any
208 208
 	 * @return void
209 209
 	 */
210
-	public function CheckAndUpdateDestination( $write = false ){
210
+	public function CheckAndUpdateDestination($write = false) {
211 211
 		$this->extend('beforeCheckAndUpdateDestination', $write);
212 212
 
213
-		if( $this->PageID && $this->Page() &&
213
+		if ($this->PageID && $this->Page() &&
214 214
 			$this->Page()->ClassName == 'RedirectorPage' &&
215
-			$this->Page()->LinkToID && $this->Page()->RedirectionType == 'Internal' )
215
+			$this->Page()->LinkToID && $this->Page()->RedirectionType == 'Internal')
216 216
 		{
217 217
 			$this->PageID = $this->Page()->LinkToID;
218
-			if( $write ) {
218
+			if ($write) {
219 219
 			    $this->write();
220 220
 			}
221 221
 		}
@@ -245,8 +245,8 @@  discard block
 block discarded – undo
245 245
      * @param ContentController $controller
246 246
      * @return Boolean
247 247
      */
248
-    public function canBeAdded( ContentController $controller ){
249
-        return ( $this->SelfLinking || $controller->ID != $this->PageID );
248
+    public function canBeAdded(ContentController $controller) {
249
+        return ($this->SelfLinking || $controller->ID != $this->PageID);
250 250
     }
251 251
 
252 252
     /**
@@ -255,20 +255,20 @@  discard block
 block discarded – undo
255 255
      * @param string $html
256 256
      * @return DOMDocument
257 257
      */
258
-    public static function constructDOMDocument($html){
259
-        if( class_exists( 'HTML5_Parser' ) ){
260
-            $html5 = HTML5_Parser::parse( $html );
261
-            if($html5 instanceof DOMNodeList){
258
+    public static function constructDOMDocument($html) {
259
+        if (class_exists('HTML5_Parser')) {
260
+            $html5 = HTML5_Parser::parse($html);
261
+            if ($html5 instanceof DOMNodeList) {
262 262
                 $dom = new DOMDocument();
263
-                while($html5->length > 0) {
263
+                while ($html5->length > 0) {
264 264
                     $dom->appendChild($html5->item(0));
265 265
                 }
266
-            }else{
266
+            }else {
267 267
                 $dom = $html5;
268 268
             }
269
-        } else{
269
+        }else {
270 270
             $dom = new DOMDocument();
271
-            $dom->loadHTML( $html );
271
+            $dom->loadHTML($html);
272 272
         }
273 273
 
274 274
         return $dom;
@@ -280,9 +280,9 @@  discard block
 block discarded – undo
280 280
      * @param string $class
281 281
      * @return array
282 282
      */
283
-    public static function getAllDatabaseFields($class){
283
+    public static function getAllDatabaseFields($class) {
284 284
         $fields = array();
285
-        foreach (ClassInfo::ancestry($class, true) as $ancestor){
285
+        foreach (ClassInfo::ancestry($class, true) as $ancestor) {
286 286
             $fields = array_merge($fields, (array) DataObject::database_fields($ancestor));
287 287
         }
288 288
         return $fields;
Please login to merge, or discard this patch.
code/reports/AutomatedLinkReport.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -7,7 +7,7 @@  discard block
 block discarded – undo
7 7
  * Automated Link Report is a report that lists all automated
8 8
  * links and how they affect the website
9 9
  */
10
-class AutomatedLinkReport extends SS_Report{
10
+class AutomatedLinkReport extends SS_Report {
11 11
 
12 12
     protected $title       = 'Automated Link Report';
13 13
     protected $description = 'Shows a list of all automated links and how they affect the site';
@@ -33,8 +33,8 @@  discard block
 block discarded – undo
33 33
         );
34 34
     }
35 35
 
36
-    public function sourceRecords(){
37
-        if( !Config::inst()->get( $this->class, 'run_in_realtime' ) ) {
36
+    public function sourceRecords() {
37
+        if (!Config::inst()->get($this->class, 'run_in_realtime')) {
38 38
             return AutomatedLinkPageResult::get();
39 39
         }
40 40
 
Please login to merge, or discard this patch.