| @@ -85,8 +85,8 @@ | ||
| 85 | 85 | |
| 86 | 86 | public function getReceiptLink() | 
| 87 | 87 |      { | 
| 88 | - $obj= HTMLVarchar::create(); | |
| 89 | -        $obj->setValue('<a href="' . $this->ReceiptURL . '" target="_blank" class="cms-panel-link action external-link">view</a>'); | |
| 88 | + $obj = HTMLVarchar::create(); | |
| 89 | +        $obj->setValue('<a href="'.$this->ReceiptURL.'" target="_blank" class="cms-panel-link action external-link">view</a>'); | |
| 90 | 90 | return $obj; | 
| 91 | 91 | } | 
| 92 | 92 | |
| @@ -23,11 +23,11 @@ | ||
| 23 | 23 |      { | 
| 24 | 24 |          $object = $this->objFromFixture('OptionGroup', 'size'); | 
| 25 | 25 | $orig = $object->Title; | 
| 26 | - $object->Title = '2' . $orig; | |
| 26 | + $object->Title = '2'.$orig; | |
| 27 | 27 |          $this->setExpectedException('ValidationException'); | 
| 28 | 28 | $object->write(); | 
| 29 | 29 | |
| 30 | - $object->Title = $orig . '&'; | |
| 30 | + $object->Title = $orig.'&'; | |
| 31 | 31 |          $this->setExpectedException('ValidationException'); | 
| 32 | 32 | $object->write(); | 
| 33 | 33 | } | 
| @@ -120,7 +120,7 @@ discard block | ||
| 120 | 120 | $fields->push(HiddenField::create(ProductPage::getGeneratedValue($code, 'product_id', | 
| 121 | 121 | $this->product->ID))->setValue($this->product->ID)); | 
| 122 | 122 | $fields->push(HiddenField::create(ProductPage::getGeneratedValue($code, 'price', | 
| 123 | - $this->product->Price))->setValue($this->product->Price));//can't override id | |
| 123 | + $this->product->Price))->setValue($this->product->Price)); //can't override id | |
| 124 | 124 | $fields->push(HiddenField::create(ProductPage::getGeneratedValue($code, 'weight', | 
| 125 | 125 | $this->product->Weight))->setValue($this->product->Weight)); | 
| 126 | 126 | |
| @@ -148,11 +148,11 @@ discard block | ||
| 148 | 148 | |
| 149 | 149 |              $fields->push(DropdownField::create('quantity', 'Quantity', $quantity)); | 
| 150 | 150 | |
| 151 | -            $fields->push(HeaderField::create('submitPrice', '$' . $this->product->Price, 4)->addExtraClass('submit-price')); | |
| 151 | +            $fields->push(HeaderField::create('submitPrice', '$'.$this->product->Price, 4)->addExtraClass('submit-price')); | |
| 152 | 152 |              $fields->push(HeaderField::create('unavailableText', 'Selection unavailable', 4)->addExtraClass('hidden unavailable-text')); | 
| 153 | 153 | |
| 154 | 154 |              $this->extend('updatePurchaseFormFields', $fields); | 
| 155 | -        } else { | |
| 155 | +        }else { | |
| 156 | 156 |              $fields->push(HeaderField::create('submitPrice', 'Currently Out of Stock'), 4); | 
| 157 | 157 | } | 
| 158 | 158 | |
| @@ -189,7 +189,7 @@ discard block | ||
| 189 | 189 | protected function getProductOptionSet() | 
| 190 | 190 |      { | 
| 191 | 191 | |
| 192 | -        $assignAvailable = function ($self) { | |
| 192 | +        $assignAvailable = function($self) { | |
| 193 | 193 |              $this->extend('updateFoxyStripePurchaseForm', $form); | 
| 194 | 194 | $self->Available = ($self->getAvailability()) ? true : false; | 
| 195 | 195 | }; | 
| @@ -95,7 +95,7 @@ discard block | ||
| 95 | 95 | 'Image used throughout site to represent this product'); | 
| 96 | 96 | |
| 97 | 97 | // Cateogry Dropdown field w/ add new | 
| 98 | -        $source = function () { | |
| 98 | +        $source = function() { | |
| 99 | 99 | return ProductCategory::get()->map()->toArray(); | 
| 100 | 100 | }; | 
| 101 | 101 |          $catField = DropdownField::create('CategoryID', _t('ProductPage.Category', 'FoxyCart Category'), $source()) | 
| @@ -139,7 +139,7 @@ discard block | ||
| 139 | 139 |          if (class_exists('GridFieldSortableRows')) { | 
| 140 | 140 |              $config->addComponent(new GridFieldSortableRows('SortOrder')); | 
| 141 | 141 |              $products = $this->ProductOptions()->sort('SortOrder'); | 
| 142 | -        } else { | |
| 142 | +        }else { | |
| 143 | 143 | $products = $this->ProductOptions(); | 
| 144 | 144 | } | 
| 145 | 145 |          $config->removeComponentsByType('GridFieldAddExistingAutocompleter'); | 
| @@ -195,7 +195,7 @@ discard block | ||
| 195 | 195 |                  ->setFolderName('Uploads/Products') | 
| 196 | 196 |                  ->setAllowedExtensions(array('jpg', 'jpeg', 'gif', 'png')) | 
| 197 | 197 | ->setAllowedMaxFileNumber(1), | 
| 198 | -            HeaderField::create('ProductImagesHD', _t('ProductPage.ProductImagesHD' . 'Product Image Gallery'), 2), | |
| 198 | +            HeaderField::create('ProductImagesHD', _t('ProductPage.ProductImagesHD'.'Product Image Gallery'), 2), | |
| 199 | 199 | $prodImagesField | 
| 200 | 200 | ->setDescription(_t( | 
| 201 | 201 | 'ProductPage.ProductImagesDescription', | 
| @@ -241,7 +241,7 @@ discard block | ||
| 241 | 241 | $product = ProductPage::get()->byID($this->ID); | 
| 242 | 242 |              if (isset($product->ParentID)) { | 
| 243 | 243 | $origParent = $product->ParentID; | 
| 244 | -            } else { | |
| 244 | +            }else { | |
| 245 | 245 | $origParent = null; | 
| 246 | 246 | } | 
| 247 | 247 | $currentParent = $this->ParentID; | 
| @@ -322,14 +322,13 @@ discard block | ||
| 322 | 322 |      ) { | 
| 323 | 323 |          $optionName = ($optionName !== null) ? preg_replace('/\s/', '_', $optionName) : $optionName; | 
| 324 | 324 | return (SiteConfig::current_site_config()->CartValidation) | 
| 325 | - ? FoxyCart_Helper::fc_hash_value($productCode, $optionName, $optionValue, $method, $output, $urlEncode) : | |
| 326 | - $optionValue; | |
| 325 | + ? FoxyCart_Helper::fc_hash_value($productCode, $optionName, $optionValue, $method, $output, $urlEncode) : $optionValue; | |
| 327 | 326 | } | 
| 328 | 327 | |
| 329 | 328 | // get FoxyCart Store Name for JS call | 
| 330 | 329 | public function getCartScript() | 
| 331 | 330 |      { | 
| 332 | - return '<script src="https://cdn.foxycart.com/' . FoxyCart::getFoxyCartStoreName() . '/loader.js" async defer></script>'; | |
| 331 | + return '<script src="https://cdn.foxycart.com/'.FoxyCart::getFoxyCartStoreName().'/loader.js" async defer></script>'; | |
| 333 | 332 | } | 
| 334 | 333 | |
| 335 | 334 | /** | 
| @@ -90,7 +90,7 @@ discard block | ||
| 90 | 90 | $parentCode = $product->Code; | 
| 91 | 91 | |
| 92 | 92 | // ProductOptionGroup Dropdown field w/ add new | 
| 93 | -        $groups = function () { | |
| 93 | +        $groups = function() { | |
| 94 | 94 | return OptionGroup::get()->map()->toArray(); | 
| 95 | 95 | }; | 
| 96 | 96 |          $groupFields = singleton('OptionGroup')->getCMSFields(); | 
| @@ -224,7 +224,7 @@ discard block | ||
| 224 | 224 | * @param bool $returnWithOnlyPlusMinus | 
| 225 | 225 | * @return string | 
| 226 | 226 | */ | 
| 227 | - public static function getOptionModifierActionSymbol($oma, $returnWithOnlyPlusMinus=false) | |
| 227 | + public static function getOptionModifierActionSymbol($oma, $returnWithOnlyPlusMinus = false) | |
| 228 | 228 |      { | 
| 229 | 229 |          switch ($oma) { | 
| 230 | 230 | case 'Subtract': | 
| @@ -291,7 +291,7 @@ discard block | ||
| 291 | 291 |      { | 
| 292 | 292 | $modPrice = ($this->PriceModifier) ? (string)$this->PriceModifier : '0'; | 
| 293 | 293 | $title = $this->Title; | 
| 294 | -        $title .= ($this->PriceModifier != 0) ? ': ('.OptionItem::getOptionModifierActionSymbol($this->PriceModifierAction, $returnWithOnlyPlusMinus=true).'$'.$modPrice.')' : ''; | |
| 294 | +        $title .= ($this->PriceModifier != 0) ? ': ('.OptionItem::getOptionModifierActionSymbol($this->PriceModifierAction, $returnWithOnlyPlusMinus = true).'$'.$modPrice.')' : ''; | |
| 295 | 295 | return $title; | 
| 296 | 296 | } | 
| 297 | 297 | |
| @@ -300,7 +300,7 @@ discard block | ||
| 300 | 300 | */ | 
| 301 | 301 | public function getAvailability() | 
| 302 | 302 |      { | 
| 303 | - $available = ($this->Available == 1) ? true : false ; | |
| 303 | + $available = ($this->Available == 1) ? true : false; | |
| 304 | 304 | |
| 305 | 305 |          $this->extend('updateOptionAvailability', $available); | 
| 306 | 306 | |
| @@ -1,6 +1,6 @@ discard block | ||
| 1 | 1 | <?php | 
| 2 | 2 | |
| 3 | -class CustomerExtension extends DataExtension{ | |
| 3 | +class CustomerExtension extends DataExtension { | |
| 4 | 4 | |
| 5 | 5 | private static $db = array( | 
| 6 | 6 | 'Customer_ID' => 'Int' | 
| @@ -25,9 +25,9 @@ discard block | ||
| 25 | 25 | $response = FoxyCart::putCustomer($this->owner); | 
| 26 | 26 | |
| 27 | 27 | // Grab customer_id record from FoxyCart response, store in Member | 
| 28 | -		if($response){ | |
| 28 | +		if ($response) { | |
| 29 | 29 | $foxyResponse = new SimpleXMLElement($response); | 
| 30 | - $this->owner->Customer_ID = (int) $foxyResponse->customer_id; | |
| 30 | + $this->owner->Customer_ID = (int)$foxyResponse->customer_id; | |
| 31 | 31 | } | 
| 32 | 32 | } | 
| 33 | 33 | |
| @@ -17,9 +17,8 @@ discard block | ||
| 17 | 17 | // handle POST from FoxyCart API transaction | 
| 18 | 18 |  		if ((isset($_POST["FoxyData"]) OR isset($_POST['FoxySubscriptionData']))) { | 
| 19 | 19 | $FoxyData_encrypted = (isset($_POST["FoxyData"])) ? | 
| 20 | - urldecode($_POST["FoxyData"]) : | |
| 21 | - urldecode($_POST["FoxySubscriptionData"]); | |
| 22 | - $FoxyData_decrypted = rc4crypt::decrypt(FoxyCart::getStoreKey(),$FoxyData_encrypted); | |
| 20 | + urldecode($_POST["FoxyData"]) : urldecode($_POST["FoxySubscriptionData"]); | |
| 21 | + $FoxyData_decrypted = rc4crypt::decrypt(FoxyCart::getStoreKey(), $FoxyData_encrypted); | |
| 23 | 22 | self::handleDataFeed($FoxyData_encrypted, $FoxyData_decrypted); | 
| 24 | 23 | |
| 25 | 24 | // extend to allow for additional integrations with Datafeed | 
| @@ -27,14 +26,14 @@ discard block | ||
| 27 | 26 | |
| 28 | 27 | return 'foxy'; | 
| 29 | 28 | |
| 30 | -		} else { | |
| 29 | +		}else { | |
| 31 | 30 | |
| 32 | 31 | return "No FoxyData or FoxySubscriptionData received."; | 
| 33 | 32 | |
| 34 | 33 | } | 
| 35 | 34 | } | 
| 36 | 35 | |
| 37 | -    public function handleDataFeed($encrypted, $decrypted){ | |
| 36 | +    public function handleDataFeed($encrypted, $decrypted) { | |
| 38 | 37 | //handle encrypted & decrypted data | 
| 39 | 38 | $orders = new SimpleXMLElement($decrypted); | 
| 40 | 39 | |
| @@ -43,12 +42,11 @@ discard block | ||
| 43 | 42 | |
| 44 | 43 |              if (isset($order->id)) { | 
| 45 | 44 |                  ($transaction = Order::get()->filter('Order_ID', $order->id)->First()) ? | 
| 46 | - $transaction : | |
| 47 | - $transaction = Order::create(); | |
| 45 | + $transaction : $transaction = Order::create(); | |
| 48 | 46 | } | 
| 49 | 47 | |
| 50 | 48 | // save base order info | 
| 51 | - $transaction->Order_ID = (int) $order->id; | |
| 49 | + $transaction->Order_ID = (int)$order->id; | |
| 52 | 50 | $transaction->Response = $decrypted; | 
| 53 | 51 | |
| 54 | 52 | // record transaction as order | 
| @@ -103,7 +101,7 @@ discard block | ||
| 103 | 101 |              if (isset($order->customer_email) && $order->is_anonymous == 0) { | 
| 104 | 102 | |
| 105 | 103 | // if Customer is existing member, associate with current order | 
| 106 | -                if(Member::get()->filter('Email', $order->customer_email)->First()) { | |
| 104 | +                if (Member::get()->filter('Email', $order->customer_email)->First()) { | |
| 107 | 105 | |
| 108 | 106 |                      $customer = Member::get()->filter('Email', $order->customer_email)->First(); | 
| 109 | 107 | |
| @@ -116,7 +114,7 @@ discard block | ||
| 116 | 114 | $customer->write(); | 
| 117 | 115 | */ | 
| 118 | 116 | |
| 119 | -                } else { | |
| 117 | +                }else { | |
| 120 | 118 | |
| 121 | 119 | // set PasswordEncryption to 'none' so imported, encrypted password is not encrypted again | 
| 122 | 120 |                      Config::inst()->update('Security', 'password_encryption_algorithm', 'none'); | 
| @@ -227,17 +225,17 @@ discard block | ||
| 227 | 225 | // get current member if logged in. If not, create a 'fake' user with Customer_ID = 0 | 
| 228 | 226 | // fake user will redirect to FC checkout, ask customer to log in | 
| 229 | 227 | // to do: consider a login/registration form here if not logged in | 
| 230 | -        if($Member = Member::currentUser()) { | |
| 228 | +        if ($Member = Member::currentUser()) { | |
| 231 | 229 | $Member = Member::currentUser(); | 
| 232 | -        } else { | |
| 230 | +        }else { | |
| 233 | 231 | $Member = new Member(); | 
| 234 | 232 | $Member->Customer_ID = 0; | 
| 235 | 233 | } | 
| 236 | 234 | |
| 237 | - $auth_token = sha1($Member->Customer_ID . '|' . $timestampNew . '|' . FoxyCart::getStoreKey()); | |
| 235 | + $auth_token = sha1($Member->Customer_ID.'|'.$timestampNew.'|'.FoxyCart::getStoreKey()); | |
| 238 | 236 | |
| 239 | - $redirect_complete = 'https://' . FoxyCart::getFoxyCartStoreName() . '.foxycart.com/checkout?fc_auth_token=' . $auth_token . | |
| 240 | - '&fcsid=' . $fcsid . '&fc_customer_id=' . $Member->Customer_ID . '×tamp=' . $timestampNew; | |
| 237 | + $redirect_complete = 'https://'.FoxyCart::getFoxyCartStoreName().'.foxycart.com/checkout?fc_auth_token='.$auth_token. | |
| 238 | + '&fcsid='.$fcsid.'&fc_customer_id='.$Member->Customer_ID.'×tamp='.$timestampNew; | |
| 241 | 239 | |
| 242 | 240 | $this->redirect($redirect_complete); | 
| 243 | 241 | |
| @@ -6,7 +6,7 @@ discard block | ||
| 6 | 6 | protected static $disable_themes = true; | 
| 7 | 7 | protected static $use_draft_site = false; | 
| 8 | 8 | |
| 9 | -    public function setUp(){ | |
| 9 | +    public function setUp() { | |
| 10 | 10 | parent::setUp(); | 
| 11 | 11 | |
| 12 | 12 |          ini_set('display_errors', 1); | 
| @@ -14,10 +14,10 @@ discard block | ||
| 14 | 14 | error_reporting(E_ALL & ~E_NOTICE & ~E_DEPRECATED); | 
| 15 | 15 | } | 
| 16 | 16 | |
| 17 | -    public function logOut(){ | |
| 17 | +    public function logOut() { | |
| 18 | 18 |          $this->session()->clear('loggedInAs'); | 
| 19 | 19 | } | 
| 20 | 20 | |
| 21 | -    public function testProductPage(){} | |
| 21 | +    public function testProductPage() {} | |
| 22 | 22 | |
| 23 | 23 | } | 
| @@ -1,10 +1,10 @@ discard block | ||
| 1 | 1 | <?php | 
| 2 | 2 | |
| 3 | -class ProductPageTest extends FS_Test{ | |
| 3 | +class ProductPageTest extends FS_Test { | |
| 4 | 4 | |
| 5 | 5 | protected static $use_draft_site = true; | 
| 6 | 6 | |
| 7 | -	function setUp(){ | |
| 7 | +	function setUp() { | |
| 8 | 8 | parent::setUp(); | 
| 9 | 9 | |
| 10 | 10 | $groupForItem = OptionGroup::create(); | 
| @@ -16,7 +16,7 @@ discard block | ||
| 16 | 16 | $productHolder->write();*/ | 
| 17 | 17 | } | 
| 18 | 18 | |
| 19 | -	function testProductCreation(){ | |
| 19 | +	function testProductCreation() { | |
| 20 | 20 | |
| 21 | 21 |  		$this->logInWithPermission('Product_CANCRUD'); | 
| 22 | 22 |  		$default = $this->objFromFixture('ProductCategory', 'default'); | 
| @@ -28,7 +28,7 @@ discard block | ||
| 28 | 28 | |
| 29 | 29 | } | 
| 30 | 30 | |
| 31 | -	function testProductDeletion(){ | |
| 31 | +	function testProductDeletion() { | |
| 32 | 32 | |
| 33 | 33 |  		$this->logInWithPermission('Product_CANCRUD'); | 
| 34 | 34 |  		$holder = $this->objFromFixture('ProductHolder', 'default'); | 
| @@ -39,22 +39,22 @@ discard block | ||
| 39 | 39 | $product2->doPublish(); | 
| 40 | 40 | $this->assertTrue($product2->isPublished()); | 
| 41 | 41 | |
| 42 | -		$versions = DB::query('Select * FROM "ProductPage_versions" WHERE "RecordID" = '. $productID); | |
| 42 | +		$versions = DB::query('Select * FROM "ProductPage_versions" WHERE "RecordID" = '.$productID); | |
| 43 | 43 | $versionsPostPublish = array(); | 
| 44 | - foreach($versions as $versionRow) $versionsPostPublish[] = $versionRow; | |
| 44 | + foreach ($versions as $versionRow) $versionsPostPublish[] = $versionRow; | |
| 45 | 45 | |
| 46 | 46 | $product2->delete(); | 
| 47 | 47 | $this->assertTrue(!$product2->isPublished()); | 
| 48 | 48 | |
| 49 | -		$versions = DB::query('Select * FROM "ProductPage_versions" WHERE "RecordID" = '. $productID); | |
| 49 | +		$versions = DB::query('Select * FROM "ProductPage_versions" WHERE "RecordID" = '.$productID); | |
| 50 | 50 | $versionsPostDelete = array(); | 
| 51 | - foreach($versions as $versionRow) $versionsPostDelete[] = $versionRow; | |
| 51 | + foreach ($versions as $versionRow) $versionsPostDelete[] = $versionRow; | |
| 52 | 52 | |
| 53 | 53 | $this->assertTrue($versionsPostPublish == $versionsPostDelete); | 
| 54 | 54 | |
| 55 | 55 | } | 
| 56 | 56 | |
| 57 | -	function testProductTitleLeadingWhiteSpace(){ | |
| 57 | +	function testProductTitleLeadingWhiteSpace() { | |
| 58 | 58 | |
| 59 | 59 |  		$this->logInWithPermission('ADMIN'); | 
| 60 | 60 | |
| @@ -69,7 +69,7 @@ discard block | ||
| 69 | 69 | |
| 70 | 70 | } | 
| 71 | 71 | |
| 72 | -	function testProductTitleTrailingWhiteSpace(){ | |
| 72 | +	function testProductTitleTrailingWhiteSpace() { | |
| 73 | 73 | |
| 74 | 74 |  		$this->logInWithPermission('ADMIN'); | 
| 75 | 75 | |
| @@ -84,7 +84,7 @@ discard block | ||
| 84 | 84 | |
| 85 | 85 | } | 
| 86 | 86 | |
| 87 | -	function testProductCategoryCreation(){ | |
| 87 | +	function testProductCategoryCreation() { | |
| 88 | 88 | |
| 89 | 89 |  		$this->logInWithPermission('Product_CANCRUD'); | 
| 90 | 90 |  		$category = $this->objFromFixture('ProductCategory', 'apparel'); | 
| @@ -97,7 +97,7 @@ discard block | ||
| 97 | 97 | |
| 98 | 98 | } | 
| 99 | 99 | |
| 100 | -	function testProductCategoryDeletion(){ | |
| 100 | +	function testProductCategoryDeletion() { | |
| 101 | 101 | |
| 102 | 102 |  		$this->logInWithPermission('Product_CANCRUD'); | 
| 103 | 103 | |
| @@ -124,11 +124,11 @@ discard block | ||
| 124 | 124 | |
| 125 | 125 | $category2->delete(); | 
| 126 | 126 | |
| 127 | -		$this->assertFalse(in_array($category2ID,ProductCategory::get()->column('ID'))); | |
| 127 | +		$this->assertFalse(in_array($category2ID, ProductCategory::get()->column('ID'))); | |
| 128 | 128 | |
| 129 | 129 | } | 
| 130 | 130 | |
| 131 | -	function testOptionGroupCreation(){ | |
| 131 | +	function testOptionGroupCreation() { | |
| 132 | 132 | |
| 133 | 133 |  		$this->logInWithPermission('Product_CANCRUD'); | 
| 134 | 134 | |
| @@ -139,7 +139,7 @@ discard block | ||
| 139 | 139 | |
| 140 | 140 | } | 
| 141 | 141 | |
| 142 | -	function testOptionGroupDeletion(){ | |
| 142 | +	function testOptionGroupDeletion() { | |
| 143 | 143 | |
| 144 | 144 |  		$this->logInWithPermission('ADMIN'); | 
| 145 | 145 |  		$group = $this->objFromFixture('OptionGroup', 'color'); | 
| @@ -158,7 +158,7 @@ discard block | ||
| 158 | 158 | |
| 159 | 159 | } | 
| 160 | 160 | |
| 161 | -	function testOptionItemCreation(){ | |
| 161 | +	function testOptionItemCreation() { | |
| 162 | 162 | |
| 163 | 163 |  		$this->logInWithPermission('Product_CANCRUD'); | 
| 164 | 164 | |
| @@ -176,7 +176,7 @@ discard block | ||
| 176 | 176 | |
| 177 | 177 | } | 
| 178 | 178 | |
| 179 | -	function testOptionItemDeletion(){ | |
| 179 | +	function testOptionItemDeletion() { | |
| 180 | 180 | |
| 181 | 181 |  		$this->logInWithPermission('ADMIN'); | 
| 182 | 182 | |
| @@ -200,44 +200,44 @@ discard block | ||
| 200 | 200 | |
| 201 | 201 | } | 
| 202 | 202 | |
| 203 | -	public function testProductDraftOptionDeletion(){ | |
| 203 | +	public function testProductDraftOptionDeletion() { | |
| 204 | 204 | |
| 205 | - self::$use_draft_site = false;//make sure we can publish | |
| 205 | + self::$use_draft_site = false; //make sure we can publish | |
| 206 | 206 | |
| 207 | 207 |  		$this->logInWithPermission('ADMIN'); | 
| 208 | 208 | |
| 209 | -		$holder = $this->objFromFixture('ProductHolder', 'default');//build holder page, ProductPage can't be on root level | |
| 209 | +		$holder = $this->objFromFixture('ProductHolder', 'default'); //build holder page, ProductPage can't be on root level | |
| 210 | 210 | $holder->doPublish(); | 
| 211 | 211 | |
| 212 | -		$product = $this->objFromFixture('ProductPage', 'product1');//build product page | |
| 212 | +		$product = $this->objFromFixture('ProductPage', 'product1'); //build product page | |
| 213 | 213 | $product->doPublish(); | 
| 214 | 214 | |
| 215 | 215 | $productID = $product->ID; | 
| 216 | 216 | |
| 217 | 217 | |
| 218 | -		$optionGroup = $this->objFromFixture('OptionGroup', 'size');//build the group for the options | |
| 218 | +		$optionGroup = $this->objFromFixture('OptionGroup', 'size'); //build the group for the options | |
| 219 | 219 | $optionGroup->write(); | 
| 220 | -		$option = $this->objFromFixture('OptionItem', 'small');//build first option | |
| 220 | +		$option = $this->objFromFixture('OptionItem', 'small'); //build first option | |
| 221 | 221 | $option->write(); | 
| 222 | -		$option2 = $this->objFromFixture('OptionItem', 'large');//build second option | |
| 222 | +		$option2 = $this->objFromFixture('OptionItem', 'large'); //build second option | |
| 223 | 223 | $option2->write(); | 
| 224 | 224 | |
| 225 | - $this->assertTrue($product->isPublished());//check that product is published | |
| 225 | + $this->assertTrue($product->isPublished()); //check that product is published | |
| 226 | 226 | |
| 227 | -		$product->deleteFromStage('Stage');//remove product from draft site | |
| 227 | +		$product->deleteFromStage('Stage'); //remove product from draft site | |
| 228 | 228 | |
| 229 | - $this->assertTrue($product->isPublished());//check product is still published | |
| 229 | + $this->assertTrue($product->isPublished()); //check product is still published | |
| 230 | 230 | |
| 231 | 231 |  		$testOption = $this->objFromFixture('OptionItem', 'large'); | 
| 232 | 232 | |
| 233 | - $this->assertThat($testOption->ID, $this->logicalNot($this->equalTo(0)));//make sure the first option still exists | |
| 233 | + $this->assertThat($testOption->ID, $this->logicalNot($this->equalTo(0))); //make sure the first option still exists | |
| 234 | 234 | |
| 235 | - $product->doRestoreToStage();//restore page to draft site | |
| 236 | - $product->doUnpublish();//unpublish page | |
| 237 | -		$product->deleteFromStage('Stage');//remove product from draft site | |
| 235 | + $product->doRestoreToStage(); //restore page to draft site | |
| 236 | + $product->doUnpublish(); //unpublish page | |
| 237 | +		$product->deleteFromStage('Stage'); //remove product from draft site | |
| 238 | 238 | |
| 239 | -		$checkDeleted = OptionItem::get()->filter(array('Title' => 'Large', 'ProductID' => $productID))->first();//query same option as above | |
| 239 | +		$checkDeleted = OptionItem::get()->filter(array('Title' => 'Large', 'ProductID' => $productID))->first(); //query same option as above | |
| 240 | 240 | |
| 241 | - $this->assertEquals($checkDeleted->ID, 0);//check that the ID is 0 (empty object/non-existent) | |
| 241 | + $this->assertEquals($checkDeleted->ID, 0); //check that the ID is 0 (empty object/non-existent) | |
| 242 | 242 | } | 
| 243 | 243 | } | 
| 244 | 244 | \ No newline at end of file |