-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
OneClick TNT module integration (#191) #266
base: master
Are you sure you want to change the base?
Conversation
$formattedAddress = array( | ||
'email' => $this->apiData['user']['email'], | ||
'firstname' => isset($customerAddress['first_name']) ? $customerAddress['first_name'] : '', | ||
'lastname' => isset($customerAddress['last_name']) ? $customerAddress['last_name'] : '', | ||
'telephone' => $this->apiData['user']['phone'], | ||
'street' => $street, | ||
'postcode' => isset($customerAddress['postcode']) ? $customerAddress['postcode'] : '', | ||
'city' => isset($customerAddress['city']) ? $customerAddress['city'] : '', | ||
'city' => $city ? $city : isset($customerAddress['city']) ? $customerAddress['city'] : '', | ||
//'city' => isset($customerAddress['city']) ? $customerAddress['city'] : '', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
delete the comment
@@ -534,4 +540,23 @@ private function clearQuoteItemsCache() | |||
$address->unsetData('cached_items_nonominal'); | |||
} | |||
} | |||
|
|||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove this empty line
$formattedAddress = array( | ||
'email' => $this->apiData['user']['email'], | ||
'firstname' => isset($customerAddress['first_name']) ? $customerAddress['first_name'] : '', | ||
'lastname' => isset($customerAddress['last_name']) ? $customerAddress['last_name'] : '', | ||
'telephone' => $this->apiData['user']['phone'], | ||
'street' => $street, | ||
'postcode' => isset($customerAddress['postcode']) ? $customerAddress['postcode'] : '', | ||
'city' => isset($customerAddress['city']) ? $customerAddress['city'] : '', | ||
'city' => $city ? $city : isset($customerAddress['city']) ? $customerAddress['city'] : '', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'city' => $city,
|
||
|
||
/** | ||
* Get city name from the zipCode |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Get city name from the zipCode ; Specific for TNT module
8063589
to
99b64a3
Compare
*/ | ||
class Oyst_OneClick_Helper_ZipMap | ||
{ | ||
const ZIP_MAP = array( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In PHP 5.3 https://travis-ci.org/oystparis/oyst-1click-magento/jobs/345208028#L963
So could you use a property.
466b2b2
to
49bd268
Compare
if (isset($customerAddress['postcode'])) { | ||
$city = $this->mapCityZipCode($customerAddress['postcode']); | ||
|
||
if (is_null($city)) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this if block have to be outside of if (isset($customerAddress['postcode'])) {
7ca7699
to
cdbabae
Compare
cdbabae
to
bdd4f2f
Compare
bdd4f2f
to
7cb7452
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Intégration pour le module TNT de GFI
7cb7452
to
ad5e752
Compare
ad5e752
to
ccafeaf
Compare
ccafeaf
to
842c9a9
Compare
842c9a9
to
8d8871d
Compare
8d8871d
to
f6cca78
Compare
No description provided.