From 42e69c0aabcfcc8891e55db7b43456314a3e41d1 Mon Sep 17 00:00:00 2001
From: Maxime Bourget <bmx007@gmail.com>
Date: Sun, 26 May 2013 18:32:45 +0100
Subject: [PATCH] Fix discount and parenthesis

Now parse line return the *raw* discount (instead of /100).
Meaning its treated exactly as the other.

Before formula where converted to integer (/100)
before being expanded or evaluated.
---
 includes/textcart_manager.inc        | 7 +++++--
 test/textcartManager/parseTest.php   | 4 ++--
 test/textcartManager/processTest.php | 9 +++++++++
 3 files changed, 16 insertions(+), 4 deletions(-)

diff --git a/includes/textcart_manager.inc b/includes/textcart_manager.inc
index c0f1bfb..45acd24 100644
--- a/includes/textcart_manager.inc
+++ b/includes/textcart_manager.inc
@@ -223,6 +223,9 @@ class TextCartManager {
 
       }
 
+			// convert the percentage to float.
+			if($discount) $discount /= 100;
+
       #echo "Stock Code : $stock_code</br><ul>";
       #echo "<li>mode : $mode</li>";
       #echo "<li>quantity : $quantity</li>"   ;
@@ -422,7 +425,7 @@ class TextCartManager {
       ,"stock_code" => $stock_code
       ,"quantity" => $quantity
       ,"price" => $price
-      ,"discount" => $discount !== null && $discount !== '' ? $discount/100.0 : null
+      ,"discount" => $discount
       ,"description" => $description
       ,"date" => $date
     );
@@ -438,7 +441,7 @@ class TextCartManager {
 
 	function get_default_discount($cart, $stock_code) {
 		if(empty($cart)) return null;
-		return $cart->default_discount;
+		return 100*$cart->default_discount;
 	}
 
 	function get_kit_description($cart, $stock_code) {
diff --git a/test/textcartManager/parseTest.php b/test/textcartManager/parseTest.php
index 45d2035..f1e5a2c 100644
--- a/test/textcartManager/parseTest.php
+++ b/test/textcartManager/parseTest.php
@@ -49,8 +49,8 @@ class ParseTest extends PHPUnit_Framework_TestCase {
 							,array("A ^2013/03/01 10", "A", '10', null, null, null, 'Date: 2013/3/1')
 							,array("A 2013/03/01 10", "A", '10', null, null, null, 'Date: 2013/3/1')
 							/*** Everything is passed ***/
-							,array("A 10 5.0 3% ^2013/03/01 | hello", "A", '10', '5.0', 0.03, "hello", 'Date: 2013/3/1')
-							,array("A 3% 10 5.0 ^2013/03/01 | hello", "A", '10', '5.0', 0.03, "hello", 'Date: 2013/3/1')
+							,array("A 10 5.0 3% ^2013/03/01 | hello", "A", '10', '5.0', 3, "hello", 'Date: 2013/3/1')
+							,array("A 3% 10 5.0 ^2013/03/01 | hello", "A", '10', '5.0', 3, "hello", 'Date: 2013/3/1')
 		);
 	}
 /**
diff --git a/test/textcartManager/processTest.php b/test/textcartManager/processTest.php
index 0d2a9fd..d200257 100644
--- a/test/textcartManager/processTest.php
+++ b/test/textcartManager/processTest.php
@@ -19,6 +19,7 @@ class ProcessTest extends PHPUnit_Framework_TestCase {
 
 	public function processNormalExamples() {
 		return array(
+/*
 							array("A", "A", 1, 5.2, 0.02)
 							,array("A 3", "A", 3, 5.2, 0.02)
 							,array("A $3", "A", 1, 3, 0.02)
@@ -28,6 +29,14 @@ class ProcessTest extends PHPUnit_Framework_TestCase {
 							,array(":# 5%\nA", "A", 1, 5.2, 0.05)
 							,array("A 0%", "A", 1, 5.2, 0)
 							,array("A 10%", "A", 1, 5.2, 0.1)
+
+							// overriding discount 
+							,array(":# 0%\nA 5%", "A", 1, 5.2, 0.05) // line win
+							,array(":# 10%\nA 0%", "A", 1, 5.2, 0)
+							,array(":# (0)%\nA 0.5%", "A", 1, 5.2, 0) // template win
+							,array(":# (10)%\nA 0%", "A", 1, 5.2, 0.1)
+							,*/array(":A (10)%\nA ", "A", 1, 5.2, 0.1)
+							,array(":# 10%\nA 35 ", "A", 1, 5.2, 0.1)
 							);
 	}
 
-- 
2.30.2