[Weberp-svn] SF.net SVN: weberp:[8401] trunk/BankAccounts.php

[<tim...@us...>]

Revision: 8401
          http://weberp.svn.sourceforge.net/weberp/?rev=8401&view=rev
Author:   tim_schofield
Date:     2011-12-27 13:17:35 +0000 (Tue, 27 Dec 2011)
Log Message:
-----------
Remove select * sql, only retrieve fields we need

Modified Paths:
--------------
    trunk/BankAccounts.php

Modified: trunk/BankAccounts.php
===================================================================
--- trunk/BankAccounts.php	2011-12-27 13:17:21 UTC (rev 8400)
+++ trunk/BankAccounts.php	2011-12-27 13:17:35 UTC (rev 8401)
@@ -80,7 +80,7 @@
 
 		/*Check if there are already transactions against this account - cant allow change currency if there are*/
 
-		$sql = "SELECT * FROM banktrans WHERE bankact=" . $SelectedBankAccount;
+		$sql = "SELECT banktransid FROM banktrans WHERE bankact=" . $SelectedBankAccount;
 		$BankTransResult = DB_query($sql,$db);
 		if (DB_num_rows($BankTransResult)>0) {
 			$sql = "UPDATE bankaccounts
@@ -91,7 +91,7 @@
 				invoice ='" . $_POST['DefAccount'] . "'
 			WHERE accountcode = '" . $SelectedBankAccount . "'";
 			prnMsg(_('Note that it is not possible to change the currency of the account once there are transactions against it'),'warn');
-	echo '<br />';
+			echo '<br />';
 		} else {
 			$sql = "UPDATE bankaccounts
 				SET bankaccountname='" . $_POST['BankAccountName'] . "',

This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.