gordonbanderson / weboftalent-adverts

code/AdvertLayoutType.php

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
     $fields->push(new TabSet('Root', $mainTab = new Tab('Main')));
         $mainTab->setTitle(_t('SiteTree.TABMAIN', 'Main'));
 
-        $fields->addFieldToTab('Root.Main',  new TextField('Title', 'The name of the category'));
+        $fields->addFieldToTab('Root.Main', new TextField('Title', 'The name of the category'));
         $fields->addFieldToTab('Root.Main', new NumericField('Width', 'The width of the advert in pixels'));
         $fields->addFieldToTab('Root.Main', new NumericField('Height', 'The height of the advert in pixels'));
 

code/AdvertCategory.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
     $fields->push(new TabSet('Root', $mainTab = new Tab('Main')));
         $mainTab->setTitle(_t('SiteTree.TABMAIN', 'Main'));
 
-        $fields->addFieldToTab('Root.Main',  new TextField('Title', 'The name of the category'));
+        $fields->addFieldToTab('Root.Main', new TextField('Title', 'The name of the category'));
         $fields->addFieldToTab('Root.Main', new CheckboxField('Enabled', 'Are adverts for this category enabled?'));
 
         return $fields;

code/AdvertController.php

Spacing +5 added lines, -5 removed lines

@@ -33,7 +33,7 @@ 
             $advert = Advert::get()->filter('DigitalSignature', $digsig)->first(); // should only be the one but make sure
 
             if (!$advert) {
-                $this->httpError(404, 'Advert "'.$digsig.'"" not found');
+                $this->httpError(404, 'Advert "' . $digsig . '"" not found');
             } else {
                 // record the click
                 $advert->Clickthroughs = $advert->Clickthroughs + 1;
@@ -51,7 +51,7 @@ 
 
             $advert = Advert::get()->filter('DigitalSignature', $digsig)->first(); // should only be the one but make sure
             if (!$advert) {
-                $this->httpError(404, 'Advert "'.$digsig.'"" not found');
+                $this->httpError(404, 'Advert "' . $digsig . '"" not found');
             } else {
                 // record the click
                 $advert->Impressions = $advert->Impressions + 1;
@@ -159,15 +159,15 @@ 
 
         // Normal operation:
         $mimeType = HTTP::get_mime_type($file_name);
-        header("Content-Type: {$mimeType}; name=\"".addslashes($file_name).'"');
-        header('Content-Disposition: attachment; filename='.addslashes($file_name));
+        header("Content-Type: {$mimeType}; name=\"" . addslashes($file_name) . '"');
+        header('Content-Disposition: attachment; filename=' . addslashes($file_name));
         header('Cache-Control: max-age=1, private');
         header("Content-Length: {$file_size}");
         header('Pragma: ');
 
         if (self::$use_x_sendfile) {
             session_write_close();
-            header('X-Sendfile: '.$file_path);
+            header('X-Sendfile: ' . $file_path);
             exit();
         } elseif ($filePointer = @fopen($file_path, 'rb')) {
             session_write_close();

code/Advert.php

Spacing +5 added lines, -5 removed lines

@@ -69,7 +69,7 @@ 
         $mainTab->setTitle(_t('SiteTree.TABMAIN', 'Main'));
 
     // human readable title
-    $fields->addFieldToTab('Root.Main',  new TextField('Title',
+    $fields->addFieldToTab('Root.Main', new TextField('Title',
                                 'Human readable title for the advert'));
 
     // a Javascript toggle on this field displays either the adbroker text field, or an image with URL
@@ -89,10 +89,10 @@ 
         }
 
     // quick tags, faster than the grid editor - these are processed prior to save to create/assign tags
-    $fields->addFieldToTab('Root.Main',  new TextField('WebsiteLink',
+    $fields->addFieldToTab('Root.Main', new TextField('WebsiteLink',
                                 'The URL that will be shown when the advert image is clicked'));
 
-        $fields->addFieldToTab('Root.Main',  new TextareaField('AdbrokerJavascript',
+        $fields->addFieldToTab('Root.Main', new TextareaField('AdbrokerJavascript',
                                 'JavaScript provided by the adbroker'));
 
         $fields->addFieldToTab('Root.Main', $sdf = new DateField('StartDate', 'The date the advert becomes active'));
@@ -130,8 +130,8 @@ 
     - last edited
     Otherwise the clickthrough will fail
     */
-    $data = $this->ID.'_'.$this->AdvertCategory()->Title.'_'.$this->AdvertLayoutType()->Title.'_'.$this->AdbrokerJavascript;
-      $data .= '_'.$this->StartDate.'_'.$this->FinishDate.'_'.$this->ClickThroughs.'_advert';
+    $data = $this->ID . '_' . $this->AdvertCategory()->Title . '_' . $this->AdvertLayoutType()->Title . '_' . $this->AdbrokerJavascript;
+      $data .= '_' . $this->StartDate . '_' . $this->FinishDate . '_' . $this->ClickThroughs . '_advert';
       $hashed = hash('sha512', $data);
     //error_log("HASH CREATED:".$hashed);
     return $hashed;

code/AdvertControllerExtension.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
             $where = "AdvertLayoutType.Title = '$adverttype'";
 
             if (isset($advertcategory)) {
-                $where .= ' AND Advert.AdvertCategoryID = '.$advertcategoryid;
+                $where .= ' AND Advert.AdvertCategoryID = ' . $advertcategoryid;
                 $where .= ' AND (StartDate IS NULL OR !StartDate OR StartDate < NOW()) AND (FinishDate IS NULL OR !FinishDate OR NOW() < FinishDate)';
 
                 if (count(self::$advertsalreadyserved) > 0) {

code/SecureFileController.php

Spacing +3 added lines, -3 removed lines

@@ -214,15 +214,15 @@
 
         // Normal operation:
         $mimeType = HTTP::getMimeType($file_name);
-        header("Content-Type: {$mimeType}; name=\"".addslashes($file_name).'"');
-        header('Content-Disposition: attachment; filename='.addslashes($file_name));
+        header("Content-Type: {$mimeType}; name=\"" . addslashes($file_name) . '"');
+        header('Content-Disposition: attachment; filename=' . addslashes($file_name));
         header('Cache-Control: max-age=1, private');
         header("Content-Length: {$file_size}");
         header('Pragma: ');
 
         if (self::$use_x_sendfile) {
             session_write_close();
-            header('X-Sendfile: '.$file_path);
+            header('X-Sendfile: ' . $file_path);
             exit();
         } elseif ($filePointer = @fopen($file_path, 'rb')) {
             session_write_close();