@@ -269,7 +269,9 @@ |
||
269 | 269 | self::$log[] = '<strong>Options:</strong> <pre>'.htmlspecialchars(print_r($options, true)).'</pre>'; |
270 | 270 | unset( $form_part_signed ); |
271 | 271 | foreach ($options as $option) { |
272 | - if( !isset($form_part_signed) ) $form_part_signed = $list[0]; |
|
272 | + if( !isset($form_part_signed) ) { |
|
273 | + $form_part_signed = $list[0]; |
|
274 | + } |
|
273 | 275 | $option_signed = preg_replace( |
274 | 276 | '%'.preg_quote($option[1]).preg_quote($option[2]).preg_quote($option[1]).'%', |
275 | 277 | $option[1].self::fc_hash_value($code, $list[2], $option[2], 'value', FALSE).$option[1], |
@@ -81,13 +81,17 @@ discard block |
||
81 | 81 | public static function getCustomer($Member = null) { |
82 | 82 | |
83 | 83 | // throw error if no $Member Object |
84 | - if (!isset($Member)) trigger_error('No Member set', E_USER_ERROR); |
|
84 | + if (!isset($Member)) { |
|
85 | + trigger_error('No Member set', E_USER_ERROR); |
|
86 | + } |
|
85 | 87 | |
86 | 88 | // grab customer record from API |
87 | 89 | |
88 | 90 | $foxyData = array(); |
89 | 91 | $foxyData["api_action"] = "customer_get"; |
90 | - if ($Member->Customer_ID) $foxyData["customer_id"] = $Member->Customer_ID; |
|
92 | + if ($Member->Customer_ID) { |
|
93 | + $foxyData["customer_id"] = $Member->Customer_ID; |
|
94 | + } |
|
91 | 95 | $foxyData["customer_email"] = $Member->Email; |
92 | 96 | |
93 | 97 | return self::getAPIRequest($foxyData); |
@@ -102,7 +106,9 @@ discard block |
||
102 | 106 | $foxyData = array(); |
103 | 107 | $foxyData["api_action"] = "customer_save"; |
104 | 108 | // customer_id will be 0 if created in SilverStripe. |
105 | - if ($Member->Customer_ID) $foxyData["customer_id"] = $Member->Customer_ID; |
|
109 | + if ($Member->Customer_ID) { |
|
110 | + $foxyData["customer_id"] = $Member->Customer_ID; |
|
111 | + } |
|
106 | 112 | $foxyData["customer_email"] = $Member->Email; |
107 | 113 | $foxyData["customer_password_hash"] = $Member->Password; |
108 | 114 | $foxyData["customer_password_salt"] = $Member->Salt; |
@@ -41,14 +41,18 @@ |
||
41 | 41 | |
42 | 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) { |
|
45 | + $versionsPostPublish[] = $versionRow; |
|
46 | + } |
|
45 | 47 | |
46 | 48 | $product2->delete(); |
47 | 49 | $this->assertTrue(!$product2->isPublished()); |
48 | 50 | |
49 | 51 | $versions = DB::query('Select * FROM "ProductPage_versions" WHERE "RecordID" = '. $productID); |
50 | 52 | $versionsPostDelete = array(); |
51 | - foreach($versions as $versionRow) $versionsPostDelete[] = $versionRow; |
|
53 | + foreach($versions as $versionRow) { |
|
54 | + $versionsPostDelete[] = $versionRow; |
|
55 | + } |
|
52 | 56 | |
53 | 57 | $this->assertTrue($versionsPostPublish == $versionsPostDelete); |
54 | 58 |