Merge pull request #8 from PrestaShop/dev

Deploy v2.1.0 of vatnumber module

Author: Quetzacoalt91

VATNumberTaxManager.php

@@ -28,9 +28,37 @@ class VATNumberTaxManager implements TaxManagerInterface
 {
 	public static function isAvailableForThisAddress(Address $address)
 	{
-		return (!empty($address->vat_number)
-			&& $address->id_country != Configuration::get('VATNUMBER_COUNTRY')
-			&& Configuration::get('VATNUMBER_MANAGEMENT')
+		/*
+		HOTFIX
+		
+		For some reason, this check is called 6 times (?)
+
+		1 w. the real address
+		2 w.o. the real address
+
+		1 w. the real address
+		2 w.o. the real address
+
+		=> [1 0 0 1 0 0]
+
+		So we need to filter out the weird calls...
+
+		We do this by caching the correct calls between calls;
+		by creating a static variable, which we save the address to,
+		if it does not contain NULL in some of the other fields.
+		*/
+
+		static $cached_address = NULL;
+
+		if ($address->id_customer != NULL) {
+			$cached_address = $address;
+		}
+
+		// Now, check on the cached address object
+		return (!empty($cached_address->vat_number)
+		    && !empty($cached_address->id_country)
+		    && $cached_address->id_country != Configuration::get('VATNUMBER_COUNTRY')
+		    && Configuration::get('VATNUMBER_MANAGEMENT')
 		);
 	}
 

config.xml

@@ -2,7 +2,7 @@
 <module>
     <name>vatnumber</name>
     <displayName><![CDATA[European VAT number]]></displayName>
-    <version><![CDATA[2.0.0]]></version>
+    <version><![CDATA[2.1.0]]></version>
     <description><![CDATA[Enables you to enter the intra-community VAT number when creating the address. You must fill in the company field to allow entering the VAT number.]]></description>
     <author><![CDATA[PrestaShop]]></author>
     <tab><![CDATA[billing_invoicing]]></tab>

vatnumber.php

@@ -33,7 +33,7 @@ public function __construct()
 	{
 		$this->name = 'vatnumber';
 		$this->tab = 'billing_invoicing';
-		$this->version = '2.0.0';
+		$this->version = '2.1.0';
 		$this->author = 'PrestaShop';
 		$this->need_instance = 0;