dynamic / foxystripe

code/model/foxycart.cart_validation.php

Doc Comments +3 added lines

@@ -43,6 +43,9 @@
 		self::setSecret(FoxyCart::getStoreKey());
 	}
 
+	/**
+	 * @return string
+	 */
 	public static function getSecret(){
 		return FoxyCart::getStoreKey();
 	}

Spacing +13 added lines, -13 removed lines

@@ -30,20 +30,20 @@ 
 	// protected static $cart_url = 'https://yourdomain.foxycart.com/cart';
 	protected static $cart_url;
 
-	public static function setCartURL($storeName = null){
+	public static function setCartURL($storeName = null) {
 		self::$cart_url = 'https://'.$storeName.'.faxycart.com/cart';
 	}
 
-	public static function setSecret($secret = null){
+	public static function setSecret($secret = null) {
 		self::$secret = $secret;
 	}
 
-	public function __construct(){
+	public function __construct() {
 		self::setCartURL(FoxyCart::getFoxyCartStoreName());
 		self::setSecret(FoxyCart::getStoreKey());
 	}
 
-	public static function getSecret(){
+	public static function getSecret() {
 		return FoxyCart::getStoreKey();
 	}
 
@@ -107,7 +107,7 @@ 
 				$codes[$pair['prefix']] = $pair['value'];
 			}
 		}
-		if ( ! count($codes)) {
+		if (!count($codes)) {
 			self::$log[] = '<strong style="color:#600;">No code found</strong> for the above link.';
 			return $fail;
 		}
@@ -131,7 +131,7 @@ 
 
 		if ($output) {
 			echo self::$cart_url.'?'.$qs;
-		} else {
+		}else {
 			return self::$cart_url.'?'.$qs;
 		}
 	}
@@ -149,18 +149,18 @@ 
 		if ($option_value == '--OPEN--') {
 			$hash = hash_hmac('sha256', $product_code.$option_name.$option_value, self::getSecret());
 			$value = ($urlencode) ? urlencode($option_name).'||'.$hash.'||open' : $option_name.'||'.$hash.'||open';
-		} else {
+		}else {
 			$hash = hash_hmac('sha256', $product_code.$option_name.$option_value, self::getSecret());
 			if ($method == 'name') {
 				$value = ($urlencode) ? urlencode($option_name).'||'.$hash : $option_name.'||'.$hash;
-			} else {
+			}else {
 				$value = ($urlencode) ? urlencode($option_value).'||'.$hash : $option_value.'||'.$hash;
 			}
 		}
 
 		if ($output) {
 			echo $value;
-		} else {
+		}else {
 			return $value;
 		}
 	}
@@ -248,7 +248,7 @@ 
 						$value[2] = ($value[2] == '') ? '--OPEN--' : $value[2];
 						if ($type[2] == 'radio') {
 							$input_signed = preg_replace('%([\'"])'.preg_quote($value[2]).'\1%', '${1}'.self::fc_hash_value($code, $name[2], $value[2], 'value', FALSE)."$1", $input);
-						} else {
+						}else {
 							$input_signed = preg_replace('%([\'"])'.$prefix.preg_quote($name[2]).'\1%', '${1}'.$prefix.self::fc_hash_value($code, $name[2], $value[2], 'name', FALSE)."$1", $input);
 						}
 						self::$log[] = '<strong>INPUT:</strong> Code: <strong>'.htmlspecialchars($prefix.$code).
@@ -267,14 +267,14 @@ 
 					$count['lists']++;
 					preg_match_all('%<option [^>]*value=([\'"])(.+?)\1[^>]*>(?:.*?)</option>%i', $list[0], $options, PREG_SET_ORDER);
 					self::$log[] = '<strong>Options:</strong> <pre>'.htmlspecialchars(print_r($options, true)).'</pre>';
-					unset( $form_part_signed );
+					unset($form_part_signed);
 					foreach ($options as $option) {
-						if( !isset($form_part_signed) ) $form_part_signed = $list[0];
+						if (!isset($form_part_signed)) $form_part_signed = $list[0];
 						$option_signed = preg_replace(
 							'%'.preg_quote($option[1]).preg_quote($option[2]).preg_quote($option[1]).'%',
 							$option[1].self::fc_hash_value($code, $list[2], $option[2], 'value', FALSE).$option[1],
 							$option[0]);
-						$form_part_signed = str_replace($option[0], $option_signed, $form_part_signed );
+						$form_part_signed = str_replace($option[0], $option_signed, $form_part_signed);
 						self::$log[] = '<strong>OPTION:</strong> Code: <strong>'.htmlspecialchars($prefix.$code).
 						               '</strong> :: Name: <strong>'.htmlspecialchars($prefix.$list[2]).
 						               '</strong> :: Value: <strong>'.htmlspecialchars($option[2]).

code/model/FoxyCart.php

Spacing +15 added lines, -15 removed lines

@@ -9,43 +9,43 @@ 
 
 	private static $keyPrefix = 'dYnm1c';
 
-	public static function setStoreKey($length = 54, $count = 0){
+	public static function setStoreKey($length = 54, $count = 0) {
 		$charset = 'ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789'.strtotime('now');
 		$strLength = strlen($charset);
 		$str = '';
-		while($count < $length){
-			$str .= $charset[mt_rand(0, $strLength-1)];
+		while ($count < $length) {
+			$str .= $charset[mt_rand(0, $strLength - 1)];
 			$count++;
 		}
-		return self::getKeyPrefix().substr(base64_encode($str),0,$length);
+		return self::getKeyPrefix().substr(base64_encode($str), 0, $length);
 	}
 
-	public static function getStoreKey(){
+	public static function getStoreKey() {
 		$config = SiteConfig::current_site_config();
-		if($config->StoreKey){
+		if ($config->StoreKey) {
 			return $config->StoreKey;
 		}
 		return null;
 	}
 
-	public static function store_name_warning(){
+	public static function store_name_warning() {
 		$warning = null;
-		if(self::getFoxyCartStoreName()===null){
+		if (self::getFoxyCartStoreName() === null) {
 			$warning = 'Must define FoxyCart Store Name in your site settings in the cms';
 		}
 		return $warning;
 	}
 
-	public static function getFoxyCartStoreName(){
+	public static function getFoxyCartStoreName() {
 		$config = SiteConfig::current_site_config();
-		if($config->StoreName){
+		if ($config->StoreName) {
 			return $config->StoreName;
 		}
 		return null;
 	}
 
 	public static function FormActionURL() {
-		return sprintf('https://%s.foxycart.com/cart', self::getFoxyCartStoreName() );
+		return sprintf('https://%s.foxycart.com/cart', self::getFoxyCartStoreName());
 	}
 
     /**
@@ -59,7 +59,7 @@ 
         $foxyData["api_token"] = FoxyCart::getStoreKey();
 
         $ch = curl_init();
-        curl_setopt($ch, CURLOPT_URL, "https://" . $foxy_domain . "/api");
+        curl_setopt($ch, CURLOPT_URL, "https://".$foxy_domain."/api");
         curl_setopt($ch, CURLOPT_POSTFIELDS, $foxyData);
         curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1);
         curl_setopt($ch, CURLOPT_CONNECTTIMEOUT, 5);
@@ -71,7 +71,7 @@ 
         // The following if block will print any CURL errors you might have
         if ($response == false) {
             //trigger_error("Could not connect to FoxyCart API", E_USER_ERROR);
-            SS_Log::log("Could not connect to FoxyCart API: " . $response, SS_Log::ERR);
+            SS_Log::log("Could not connect to FoxyCart API: ".$response, SS_Log::ERR);
         }
         curl_close($ch);
 
@@ -96,7 +96,7 @@ 
 
     public static function putCustomer($Member = null) {
         // throw error if no $Member Object
-        if (!isset($Member)) ;//trigger_error('No Member set', E_USER_ERROR);
+        if (!isset($Member)); //trigger_error('No Member set', E_USER_ERROR);
 
         // send updated customer record from API
         $foxyData = array();
@@ -112,7 +112,7 @@ 
         return self::getAPIRequest($foxyData);
     }
 
-	public static function getKeyPrefix(){
+	public static function getKeyPrefix() {
 		return self::$keyPrefix;
 	}
 

Doc Comments +3 added lines

@@ -94,6 +94,9 @@
 
     }
 
+    /**
+     * @param SS_Object $Member
+     */
     public static function putCustomer($Member = null) {
         // throw error if no $Member Object
         if (!isset($Member)) ;//trigger_error('No Member set', E_USER_ERROR);

code/objects/ProductImage.php

Spacing +2 added lines, -2 removed lines

@@ -5,7 +5,7 @@ 
  *
  */
 
-class ProductImage extends DataObject{
+class ProductImage extends DataObject {
 
 	private static $db = array(
 		'Title' => 'Text',
@@ -24,7 +24,7 @@ 
 		'Title' => 'Caption'
 	);
 
-	public function getCMSFields(){
+	public function getCMSFields() {
 		$fields = FieldList::create(
             TextField::create('Title')
                 ->setTitle(_t('ProductImage.Title', 'Product Image Title')),

code/objects/ProductCategory.php

Spacing +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
 	public function requireDefaultRecords() {
 		parent::requireDefaultRecords();
 		$allCats = DataObject::get('ProductCategory');
-		if(!$allCats->count()){
+		if (!$allCats->count()) {
 			$cat = new ProductCategory();
 			$cat->Title = 'Default';
 			$cat->Code = 'DEFAULT';

code/pages/OrderHistoryPage.php

Spacing +4 added lines, -4 removed lines

@@ -11,7 +11,7 @@ 
     private static $plural_name = 'Order History Pages';
     private static $description = 'Show a customers past orders. Requires authentication';
 
-	public function getCMSFields(){
+	public function getCMSFields() {
 		$fields = parent::getCMSFields();
 
 
@@ -41,10 +41,10 @@ 
     );
 
     public function checkMember() {
-        if(Member::currentUser()) {
+        if (Member::currentUser()) {
             return true;
-        } else {
-            return Security::permissionFailure ($this, _t (
+        }else {
+            return Security::permissionFailure($this, _t(
                 'AccountPage.CANNOTCONFIRMLOGGEDIN',
                 'Please login to view this page.'
             ));

code/extensions/FoxyStripeSiteConfig.php

Spacing +6 added lines, -6 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-class FoxyStripeSiteConfig extends DataExtension{
+class FoxyStripeSiteConfig extends DataExtension {
 
 	private static $db = array(
 		'StoreName' => 'Varchar(255)',
@@ -16,7 +16,7 @@ 
         'ProductLimit' => 10
     );
 
-	public function updateCMSFields(FieldList $fields){
+	public function updateCMSFields(FieldList $fields) {
 
         // set TabSet names to avoid spaces from camel case
         $fields->addFieldToTab('Root', new TabSet('FoxyStripe', 'FoxyStripe'));
@@ -55,7 +55,7 @@ 
         ));
 
         // configuration warning
-		if(FoxyCart::store_name_warning()!==null){
+		if (FoxyCart::store_name_warning() !== null) {
 			$fields->insertBefore(LiteralField::create(
                 "StoreSubDomainHeaderWarning",
                 _t(
@@ -137,14 +137,14 @@ 
 
         $siteConfig = SiteConfig::current_site_config();
 
-        if(!$siteConfig->StoreKey) {
+        if (!$siteConfig->StoreKey) {
             $key = FoxyCart::setStoreKey();
-            while(!ctype_alnum($key)){
+            while (!ctype_alnum($key)) {
                 $key = FoxyCart::setStoreKey();
             }
             $siteConfig->StoreKey = $key;
             $siteConfig->write();
-            DB::alteration_message($siteConfig->ClassName.": created FoxyCart Store Key " . $key, 'created');
+            DB::alteration_message($siteConfig->ClassName.": created FoxyCart Store Key ".$key, 'created');
         }
     }
 

code/extensions/FoxyStripePageExtension.php

Spacing +1 added lines, -1 removed lines

@@ -4,7 +4,7 @@
 
     // get FoxyCart Store Name for JS call
     public function getCartScript() {
-        return '<script src="https://cdn.foxycart.com/' . FoxyCart::getFoxyCartStoreName() . '/loader.js" async defer></script>';
+        return '<script src="https://cdn.foxycart.com/'.FoxyCart::getFoxyCartStoreName().'/loader.js" async defer></script>';
     }
 
 }
\ No newline at end of file

code/controllers/FoxyStripe_Controller.php

Doc Comments +15 added lines

@@ -34,6 +34,9 @@ 
 		}
 	}
 
+    /**
+     * @param string $encrypted
+     */
     public function handleDataFeed($encrypted, $decrypted){
         //handle encrypted & decrypted data
         $orders = new SimpleXMLElement($decrypted);
@@ -79,6 +82,10 @@ 
         }
     }
 
+    /**
+     * @param SimpleXMLElement $orders
+     * @param DataObject $transaction
+     */
     public function parseOrderInfo($orders, $transaction) {
 
         foreach ($orders->transactions->transaction as $order) {
@@ -95,6 +102,10 @@ 
         }
     }
 
+    /**
+     * @param SimpleXMLElement $orders
+     * @param DataObject $transaction
+     */
     public function parseOrderCustomer($orders, $transaction) {
 
         foreach ($orders->transactions->transaction as $order) {
@@ -133,6 +144,10 @@ 
         }
     }
 
+    /**
+     * @param SimpleXMLElement $orders
+     * @param DataObject $transaction
+     */
     public function parseOrderDetails($orders, $transaction) {
 
         // remove previous OrderDetails so we don't end up with duplicates

Spacing +13 added lines, -15 removed lines

@@ -17,9 +17,8 @@ 
 	    // handle POST from FoxyCart API transaction
 		if ((isset($_POST["FoxyData"]) OR isset($_POST['FoxySubscriptionData']))) {
 			$FoxyData_encrypted = (isset($_POST["FoxyData"])) ?
-                urldecode($_POST["FoxyData"]) :
-                urldecode($_POST["FoxySubscriptionData"]);
-			$FoxyData_decrypted = rc4crypt::decrypt(FoxyCart::getStoreKey(),$FoxyData_encrypted);
+                urldecode($_POST["FoxyData"]) : urldecode($_POST["FoxySubscriptionData"]);
+			$FoxyData_decrypted = rc4crypt::decrypt(FoxyCart::getStoreKey(), $FoxyData_encrypted);
 			self::handleDataFeed($FoxyData_encrypted, $FoxyData_decrypted);
 			
 			// extend to allow for additional integrations with Datafeed
@@ -27,14 +26,14 @@ 
 			
 			return 'foxy';
 			
-		} else {
+		}else {
 			
 			return "No FoxyData or FoxySubscriptionData received.";
 			
 		}
 	}
 
-    public function handleDataFeed($encrypted, $decrypted){
+    public function handleDataFeed($encrypted, $decrypted) {
         //handle encrypted & decrypted data
         $orders = new SimpleXMLElement($decrypted);
 
@@ -43,12 +42,11 @@ 
 
             if (isset($order->id)) {
                 ($transaction = Order::get()->filter('Order_ID', $order->id)->First()) ?
-                    $transaction :
-                    $transaction = Order::create();
+                    $transaction : $transaction = Order::create();
             }
 
             // save base order info
-            $transaction->Order_ID = (int) $order->id;
+            $transaction->Order_ID = (int)$order->id;
             $transaction->Response = $decrypted;
 
             // record transaction as order
@@ -103,7 +101,7 @@ 
             if (isset($order->customer_email) && $order->is_anonymous == 0) {
 
                 // if Customer is existing member, associate with current order
-                if(Member::get()->filter('Email', $order->customer_email)->First()) {
+                if (Member::get()->filter('Email', $order->customer_email)->First()) {
 
                     $customer = Member::get()->filter('Email', $order->customer_email)->First();
 
@@ -116,7 +114,7 @@ 
                     $customer->write();
                     */
 
-                } else {
+                }else {
 
                     // set PasswordEncryption to 'none' so imported, encrypted password is not encrypted again
                     Config::inst()->update('Security', 'password_encryption_algorithm', 'none');
@@ -227,17 +225,17 @@ 
         // get current member if logged in. If not, create a 'fake' user with Customer_ID = 0
         // fake user will redirect to FC checkout, ask customer to log in
         // to do: consider a login/registration form here if not logged in
-        if($Member = Member::currentUser()) {
+        if ($Member = Member::currentUser()) {
             $Member = Member::currentUser();
-        } else {
+        }else {
             $Member = new Member();
             $Member->Customer_ID = 0;
         }
 
-        $auth_token = sha1($Member->Customer_ID . '|' . $timestampNew . '|' . FoxyCart::getStoreKey());
+        $auth_token = sha1($Member->Customer_ID.'|'.$timestampNew.'|'.FoxyCart::getStoreKey());
 
-        $redirect_complete = 'https://' . FoxyCart::getFoxyCartStoreName() . '.foxycart.com/checkout?fc_auth_token=' . $auth_token .
-            '&fcsid=' . $fcsid . '&fc_customer_id=' . $Member->Customer_ID . '&timestamp=' . $timestampNew;
+        $redirect_complete = 'https://'.FoxyCart::getFoxyCartStoreName().'.foxycart.com/checkout?fc_auth_token='.$auth_token.
+            '&fcsid='.$fcsid.'&fc_customer_id='.$Member->Customer_ID.'&timestamp='.$timestampNew;
 	
 	    $this->redirect($redirect_complete);
 

code/forms/FoxyStripePurchaseForm.php

Doc Comments +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
     private $product;
 
     /**
-     * @param $siteConfig
+     * @param SiteConfig|null $siteConfig
      * @return $this
      */
     public function setSiteConfig($siteConfig)

Spacing +4 added lines, -4 removed lines

@@ -120,7 +120,7 @@ 
             $fields->push(HiddenField::create(ProductPage::getGeneratedValue($code, 'product_id',
                 $this->product->ID))->setValue($this->product->ID));
             $fields->push(HiddenField::create(ProductPage::getGeneratedValue($code, 'price',
-                $this->product->Price))->setValue($this->product->Price));//can't override id
+                $this->product->Price))->setValue($this->product->Price)); //can't override id
             $fields->push(HiddenField::create(ProductPage::getGeneratedValue($code, 'weight',
                 $this->product->Weight))->setValue($this->product->Weight));
 
@@ -148,11 +148,11 @@ 
 
             $fields->push(DropdownField::create('quantity', 'Quantity', $quantity));
 
-            $fields->push(HeaderField::create('submitPrice', '$' . $this->product->Price, 4)->addExtraClass('submit-price'));
+            $fields->push(HeaderField::create('submitPrice', '$'.$this->product->Price, 4)->addExtraClass('submit-price'));
             $fields->push(HeaderField::create('unavailableText', 'Selection unavailable', 4)->addExtraClass('hidden unavailable-text'));
 
             $this->extend('updatePurchaseFormFields', $fields);
-        } else {
+        }else {
             $fields->push(HeaderField::create('submitPrice', 'Currently Out of Stock'), 4);
         }
 
@@ -189,7 +189,7 @@ 
     protected function getProductOptionSet()
     {
 
-        $assignAvailable = function ($self) {
+        $assignAvailable = function($self) {
             $this->extend('updateFoxyStripePurchaseForm', $form);
             $self->Available = ($self->getAvailability()) ? true : false;
         };