From 0d755c8562a074ae72a6541491def6ff0db350de Mon Sep 17 00:00:00 2001
From: Maxime Bourget <bmx007@gmail.com>
Date: Thu, 13 Jun 2013 20:22:32 +0100
Subject: [PATCH] picked function works.

---
 includes/picking.inc | 30 +++++++++++++++++-------------
 1 file changed, 17 insertions(+), 13 deletions(-)

diff --git a/includes/picking.inc b/includes/picking.inc
index c92bd01..4947352 100644
--- a/includes/picking.inc
+++ b/includes/picking.inc
@@ -16,35 +16,39 @@ display_warning("value $value");
 	global	$Ajax;
 
 	$picker = new Picker();
+	list($debtor_no, $branch_code) = explode('_', $id);
 
 	// insert or remove pick order form pick table initial_quantity 
-	if(!$value) {
-		$picker->insertAllForBranch($id);
+	if($picker->picked($debtor_no, $branch_code)) {
+		$picker->deleteAllForBranch($debtor_no, $branch_code);
 	}
 	else {
-		$picker->deleteAllForBranch($id);
+		$picker->insertAllForBranch($debtor_no, $branch_code);
 	}
 	$Ajax->activate('order_line_view');
 }
 
 
 class Picker {
-	function insertAllForBranch($id) {
-		list($debtor_no, $branch_code) = $this->branchForId($id);
-		display_warning("inserting for $id");
+	function picked($debtor_no, $branch_code) {
+		$sql = "SELECT *
+						FROM ".TB_PREF."sales_orders so
+						NATURAL JOIN ".TB_PREF."sales_order_details sod
+						JOIN ".TB_PREF."topick p on (sod.id = p.detail_id)
+						WHERE debtor_no = $debtor_no AND branch_code = $branch_code
+						";
 
-	}
+		return db_num_rows(db_query($sql)) > 0;
 
-	function deleteAllForBranch($id) {
-		display_warning("inserting for $id");
-		list($debtor_no, $branch_code) = $this->branchForId($id);
 	}
+	function insertAllForBranch($debtor_no, $branch_code) {
+		display_warning("inserting for $debtor_no $branch_code");
 
-	function branchForId($id) {
-		return explode('_', $id);
 	}
 
-
+	function deleteAllForBranch($debtor_no, $branch_code) {
+		display_warning("deleting for $debtor_no $branch_code");
+	}
 
 }
 
-- 
2.30.2