X-Git-Url: https://delta.frontaccounting.com/gitweb/?a=blobdiff_plain;f=includes%2Freferences.inc;h=6a8c7af2d0f87044206bc2fb0f243697e90b1051;hb=53d942f2a0d20cce5e9c409c6485867ce0869e4d;hp=671db9346e2a3b4267a71041ed958d798863a301;hpb=c895b045c679a5295f40881f918114cfbd3ccf3f;p=fa-stable.git

diff --git a/includes/references.inc b/includes/references.inc
index 671db934..6a8c7af2 100644
--- a/includes/references.inc
+++ b/includes/references.inc
@@ -10,71 +10,74 @@
     See the License here <http://www.gnu.org/licenses/gpl-3.0.html>.
 ***********************************************************************/
 include_once($path_to_root . "/includes/db/references_db.inc");
+//---------------------------------------------------------------------------------------------
+//
+// For now (2.3) the references system has somewhat inconsistent design scheme.
+// Most transactions store references in respective table, but this is not the case
+// for journal entries. All references regardless of type are stored also in refs table. 
+// Reference uniquness now can be checked with is_new_reference() for all transactions. 
+// In near future this should be fixed either with removing reference fields 
+// in transaction tables, or adding ref in bank transaction/journal and removing refs table.
+//
 
 class references 
 {
-	
-	function save($type, $id, $reference) 
-	{
-		add_reference($type, $id, $reference);
-		if ($reference != 'auto')
-			$this->save_last($reference, $type);
-	}
-	
+	//
+	//	Get reference from refs table for given transaction.
+	//	Used for transactions which do not hold references (journal and bank).
+	//
 	function get($type, $id) 
 	{
 		return get_reference($type, $id);
-	}	
-	
-	function delete($type, $id) 
-	{
-		delete_reference($type, $id);
-	}	
-	
-	function update($type, $id, $reference) 
-	{
-	    update_reference($type, $id, $reference);
-		if ($reference != 'auto')
-			$this->save_last($reference, $type);
 	}
-	// check if reference is used for any non voided transaction (used for ST_JOURNALENTRY type)
+	//
+	// Check if reference is used for any non voided transaction (used for ST_JOURNALENTRY type)
+	//
 	function exists($type, $reference) 
 	{
 		return (find_reference($type, $reference) != null);
 	}
-	
-	function save_last($reference, $type) 
-	{
-		$next = $this->increment($reference);
-		save_next_reference($type, $next);
-	}
-	
+	//
+	// Get default reference on new transaction creation.
+	//
 	function get_next($type) 
 	{
 		return get_next_reference($type);
 	}
 	//
-	//	Restore previous reference if voided trans ref was the last one.
+	// Check reference is valid before add/update transaction.
 	//
-	function restore_last($type, $id) 
+	function is_valid($reference) 
 	{
-		$reference = $this->get($type, $id);
-		$last = $this->increment($this->get_next($type), true); //decrement
-		if ($reference==$last) {
-			save_next_reference($type, $last);
+		return strlen(trim($reference)) > 0;
+	}
+	//
+	//	Save reference (and prepare next) on write transaction.
+	//
+	function save($type, $id, $reference) 
+	{
+		update_reference($type, $id, $reference); // store in refs table
+		if ($reference == $this->get_next($type)) { // if reference was bigger or not changed from default
+			$next = $this->_increment($reference);	// increment default
+			save_next_reference($type, $next);
 		}
 	}
-	
-	//------------------------------------
-
-	function is_valid($reference) 
+	//
+	// Restore previous reference (if possible) after voiding transaction.
+	//
+	function restore_last($type, $id) 
 	{
-		return strlen(trim($reference)) > 0;
+		$reference = get_reference($type, $id);
+		$prev = $this->_increment($this->get_next($type), true); //decrement
+		if ($reference==$prev) {
+			save_next_reference($type, $prev);
+		}
 	}
+	//-----------------------------------------------------------------------
 	//
 	//	Increments (or decrements if $back==true) reference template
 	//
-	function increment($reference, $back=false) 
+	function _increment($reference, $back=false) 
 	{
 		// New method done by Pete. So f.i. WA036 will increment to WA037 and so on.
        	// If $reference contains at least one group of digits,
@@ -95,31 +98,48 @@ class references
         else 
             return $reference;
 	}
-	//------------------------------------
 }
 
 //----------------------------------------------------------------------------
-
-function is_new_reference($ref, $type)
+//
+//	Check if reference was not used so far (for other transaction than $trans_no)
+//
+function is_new_reference($ref, $type, $trans_no=0)
 {
 	$db_info = get_systype_db_info($type);
 	$db_name = $db_info[0];
 	$db_type = $db_info[1];
+	$db_trans = $db_info[2];
 	$db_ref = $db_info[3];
 	
-	if ($db_ref != null) 
-	{
-		$sql = "SELECT $db_ref FROM $db_name WHERE $db_ref='$ref'";
-		if ($db_type != null)
-			$sql .= " AND $db_type=$type";
-			 
-		$result = db_query($sql, "could not test for unique reference");
-		
-		return (db_num_rows($result) == 0);
-	}
+	$ref = db_escape(trim($ref));
+	$type = db_escape($type);
 	
-	// it's a type that doesn't use references - shouldn't be calling here, but say yes anyways
-	return true;
+	if ($db_ref == null) { // journal or bank trans store references in refs table
+		$db_name = TB_PREF."refs";
+		$db_type = 'type';
+		$db_trans = 'id';
+		$db_ref = 'reference';
+	}
+
+	if ($db_type != null) {
+		$sql = "SELECT $db_ref FROM $db_name 
+			LEFT JOIN ".TB_PREF."voided v ON 
+				$db_name.$db_type=v.type AND $db_name.$db_trans=v.id
+			WHERE $db_name.$db_ref=$ref AND ISNULL(v.id)
+				AND $db_name.$db_type=$type";
+	} else {
+		$sql = "SELECT $db_ref FROM $db_name 
+			LEFT JOIN ".TB_PREF."voided v ON 
+				v.type=$type AND $db_name.$db_trans=v.id
+			WHERE $db_ref=$ref AND ISNULL(v.id)";
+	}
+
+	$result = db_query($sql, "could not test for unique reference");
+
+	return (db_num_rows($result) == 0);
+
 }
 
+
 ?>
\ No newline at end of file