From: <dai...@us...> - 2011-03-30 09:06:41
|
Revision: 4526 http://web-erp.svn.sourceforge.net/web-erp/?rev=4526&view=rev Author: daintree Date: 2011-03-30 09:06:33 +0000 (Wed, 30 Mar 2011) Log Message: ----------- SQL fixing quotes - not many, fixing gettext a few strings, Tims Reprint GRN Modified Paths: -------------- trunk/EDIMessageFormat.php trunk/EDISendInvoices.php trunk/FixedAssetCategories.php trunk/FixedAssetDepreciation.php trunk/FixedAssetItems.php trunk/GLAccountCSV.php trunk/GLBudgets.php trunk/GLCodesInquiry.php trunk/GLJournal.php trunk/PDFGrn.php trunk/doc/Change.log.html trunk/includes/ConnectDB.inc trunk/index.php trunk/sql/mysql/upgrade3.11.1-4.00.sql Modified: trunk/EDIMessageFormat.php =================================================================== --- trunk/EDIMessageFormat.php 2011-03-29 13:55:41 UTC (rev 4525) +++ trunk/EDIMessageFormat.php 2011-03-30 09:06:33 UTC (rev 4526) @@ -1,9 +1,7 @@ <?php -/* $Revision: 1.10 $ */ + /* $Id$*/ -//$PageSecurity = 10; - include('includes/session.inc'); $title = _('EDI Message Format'); include('includes/header.inc'); @@ -92,7 +90,7 @@ $sql="DELETE FROM edimessageformat WHERE id='" . $_GET['delete']."'"; $result = DB_query($sql,$db); - $msg='The selected message line has been deleted'; + $msg=_('The selected message line has been deleted'); } if (isset($msg)) { @@ -174,13 +172,13 @@ if (isset($SelectedMessageLine)) { //editing an existing message line - $sql = 'SELECT messagetype, + $sql = "SELECT messagetype, partnercode, section, sequenceno, linetext FROM edimessageformat - WHERE id=' . $SelectedMessageLine; + WHERE id='" . $SelectedMessageLine . "'"; $result = DB_query($sql, $db); $myrow = DB_fetch_array($result); @@ -206,25 +204,25 @@ echo '<tr><th colspan=2><font size=3>' . _('Definition of') . ' ' . $myrow['messagetype'] . ' ' . _('for') . ' ' . $myrow['partnercode'].'</font></th></tr>'; } -echo '<tr><td>Section:</td>'; +echo '<tr><td>' . _('Section') . ':</td>'; echo '<td>'; echo '<select tabindex=2 name="Section">'; if ($_POST['Section']=='Heading') { - echo "<option selected VALUE='Heading'>" . _('Heading'); + echo "<option selected VALUE='Heading'>" . _('Heading') . '</option>'; } else { - echo "<option value='Heading'>" . _('Heading'); + echo "<option value='Heading'>" . _('Heading') . '</option>'; } if (isset($_POST['Section']) and $_POST['Section']=='Detail') { - echo "<option selected VALUE='Detail'>" . _('Detail'); + echo "<option selected VALUE='Detail'>" . _('Detail') . '</option>'; } else { - echo "<option value='Detail'>" . _('Detail'); + echo "<option value='Detail'>" . _('Detail') . '</option>'; } if (isset($_POST['Section']) and $_POST['Section']=='Summary') { - echo "<option selected VALUE='Summary'>" . _('Summary'); + echo "<option selected VALUE='Summary'>" . _('Summary') . '</option>'; } else { - echo "<option value='Summary'>" . _('Summary'); + echo "<option value='Summary'>" . _('Summary') . '</option>'; } echo '</select>'; Modified: trunk/EDISendInvoices.php =================================================================== --- trunk/EDISendInvoices.php 2011-03-29 13:55:41 UTC (rev 4525) +++ trunk/EDISendInvoices.php 2011-03-30 09:06:33 UTC (rev 4526) @@ -1,10 +1,7 @@ <?php -/* $Revision: 1.11 $ */ /* $Id$*/ -//$PageSecurity =15; - include ('includes/session.inc'); include ('includes/header.inc'); include('includes/SQL_CommonFunctions.inc'); //need for EDITransNo @@ -100,7 +97,11 @@ then replace variable names with data write the output to a file one line at a time */ - $sql = "SELECT section, linetext FROM edimessageformat WHERE partnercode='" . $CustDetails['debtorno'] . "' AND messagetype='INVOIC' ORDER BY sequenceno"; + $sql = "SELECT section, + linetext + FROM edimessageformat + WHERE partnercode='" . $CustDetails['debtorno'] . "' + AND messagetype='INVOIC' ORDER BY sequenceno"; $ErrMsg = _('An error occurred in getting the EDI format template for') . ' ' . $CustDetails['debtorno'] . ' ' . _('because'); $MessageLinesResult = DB_query($sql, $db,$ErrMsg); @@ -145,7 +146,7 @@ stockmaster WHERE stockmoves.stockid = stockmaster.stockid AND stockmoves.type=10 - AND stockmoves.transno=" . $TransNo . " + AND stockmoves.transno='" . $TransNo . "' AND stockmoves.show_on_inv_crds=1"; } else { /* credit note */ @@ -160,7 +161,7 @@ FROM stockmoves, stockmaster WHERE stockmoves.stockid = stockmaster.stockid - AND stockmoves.type=11 and stockmoves.transno=" . $TransNo . " + AND stockmoves.type=11 and stockmoves.transno='" . $TransNo . "' AND stockmoves.show_on_inv_crds=1"; } $TransLinesResult = DB_query($sql,$db); @@ -218,10 +219,10 @@ $MessageSent = $mail->send(array($CustDetails['ediaddress'])); if ($MessageSent==True){ - echo '<br><br>'; + echo '<br /><br />'; prnMsg(_('EDI Message') . ' ' . $EDITransNo . ' ' . _('was successfully emailed'),'success'); } else { - echo '<br><br>'; + echo '<br /><br />'; prnMsg(_('EDI Message') . ' ' . $EDITransNo . _('could not be emailed to') . ' ' . $CustDetails['ediaddress'],'error'); } } else { /*it must be ftp transport */ @@ -230,16 +231,16 @@ $conn_id = ftp_connect($CustDetails['ediaddress']); // login with username and password $login_result = ftp_login($conn_id, $CustDetails['ediserveruser'], $CustDetails['ediserverpwd']); // check connection if ((!$conn_id) || (!$login_result)) { - prnMsg( _('Ftp connection has failed'). '<br>' . _('Attempted to connect to') . ' ' . $CustDetails['ediaddress'] . ' ' ._('for user') . ' ' . $CustDetails['ediserveruser'],'error'); + prnMsg( _('Ftp connection has failed'). '<br />' . _('Attempted to connect to') . ' ' . $CustDetails['ediaddress'] . ' ' ._('for user') . ' ' . $CustDetails['ediserveruser'],'error'); include('includes/footer.inc'); exit; } $MessageSent = ftp_put($conn_id, $_SESSION['EDI_MsgPending'] . '/EDI_INV_' . $EDITransNo, 'EDI_INV_' . $EDITransNo, FTP_ASCII); // check upload status if (!$MessageSent) { - echo '<br><br>'; + echo '<br /><br />'; prnMsg(_('EDI Message') . ' ' . $EDITransNo . ' ' . _('could not be sent via ftp to') .' ' . $CustDetails['ediaddress'],'error'); } else { - echo '<br><br>'; + echo '<br /><br />'; prnMsg( _('Successfully uploaded EDI_INV_') . $EDITransNo . ' ' . _('via ftp to') . ' ' . $CustDetails['ediaddress'],'success'); } // close the FTP stream ftp_quit($conn_id); Modified: trunk/FixedAssetCategories.php =================================================================== --- trunk/FixedAssetCategories.php 2011-03-29 13:55:41 UTC (rev 4525) +++ trunk/FixedAssetCategories.php 2011-03-30 09:06:33 UTC (rev 4526) @@ -1,7 +1,6 @@ <?php -/* $Revision: 1.1 $ */ -//$PageSecurity = 11; +/* $Id$*/ include('includes/session.inc'); @@ -93,18 +92,18 @@ } elseif ($InputError !=1) { $sql = "INSERT INTO fixedassetcategories (categoryid, - categorydescription, - costact, - depnact, - disposalact, - accumdepnact) - VALUES ( - '" . $_POST['CategoryID'] . "', - '" . $_POST['CategoryDescription'] . "', - '" . $_POST['CostAct'] . "', - '" . $_POST['DepnAct'] . "', - '" . $_POST['DisposalAct'] . "', - '" . $_POST['AccumDepnAct'] . "')"; + categorydescription, + costact, + depnact, + disposalact, + accumdepnact) + VALUES ( + '" . $_POST['CategoryID'] . "', + '" . $_POST['CategoryDescription'] . "', + '" . $_POST['CostAct'] . "', + '" . $_POST['DepnAct'] . "', + '" . $_POST['DisposalAct'] . "', + '" . $_POST['AccumDepnAct'] . "')"; $ErrMsg = _('Could not insert the new fixed asset category') . $_POST['CategoryDescription'] . _('because'); $result = DB_query($sql,$db,$ErrMsg); prnMsg(_('A new fixed asset category record has been added for') . ' ' . $_POST['CategoryDescription'],'success'); @@ -129,7 +128,7 @@ $myrow = DB_fetch_row($result); if ($myrow[0]>0) { prnMsg(_('Cannot delete this fixed asset category because fixed assets have been created using this category') . - '<br> ' . _('There are') . ' ' . $myrow[0] . ' ' . _('fixed assets referring to this category code'),'warn'); + '<br /> ' . _('There are') . ' ' . $myrow[0] . ' ' . _('fixed assets referring to this category code'),'warn'); } else { $sql="DELETE FROM fixedassetcategories WHERE categoryid='" . $SelectedCategory . "'"; @@ -147,15 +146,15 @@ or deletion of the records*/ $sql = 'SELECT categoryid, - categorydescription, - costact, - depnact, - disposalact, - accumdepnact - FROM fixedassetcategories'; + categorydescription, + costact, + depnact, + disposalact, + accumdepnact + FROM fixedassetcategories'; $result = DB_query($sql,$db); - echo "<br><table class=selection>\n"; + echo "<br /><table class=selection>\n"; echo '<tr><th>' . _('Cat Code') . '</th> <th>' . _('Description') . '</th> <th>' . _('Cost GL') . '</th> @@ -200,7 +199,7 @@ //end of ifs and buts! if (isset($SelectedCategory)) { - echo "<br><div class='centre'><a href='" . $_SERVER['PHP_SELF'] . '?' . SID . ">" ._('Show All Fixed Asset Categories') . "</a></div>"; + echo "<br /><div class='centre'><a href='" . $_SERVER['PHP_SELF'] . '?' . SID . ">" ._('Show All Fixed Asset Categories') . "</a></div>"; } echo '<form name="CategoryForm" method="post" action="' . $_SERVER['PHP_SELF'] . '?' . SID . '">'; @@ -240,23 +239,23 @@ } //SQL to poulate account selection boxes -$sql = "SELECT accountcode, +$sql = 'SELECT accountcode, accountname FROM chartmaster, accountgroups WHERE chartmaster.group_=accountgroups.groupname and accountgroups.pandl=0 - ORDER BY accountcode"; + ORDER BY accountcode'; $BSAccountsResult = DB_query($sql,$db); -$sql = "SELECT accountcode, +$sql = 'SELECT accountcode, accountname FROM chartmaster, accountgroups WHERE chartmaster.group_=accountgroups.groupname and accountgroups.pandl!=0 - ORDER BY accountcode"; + ORDER BY accountcode'; $PnLAccountsResult = DB_query($sql,$db); @@ -323,7 +322,7 @@ } //end while loop -echo '</select></td></tr></table><br>'; +echo '</select></td></tr></table><br />'; echo '<div class="centre"><input type="Submit" name="submit" value="' . _('Enter Information') . '"></div>'; Modified: trunk/FixedAssetDepreciation.php =================================================================== --- trunk/FixedAssetDepreciation.php 2011-03-29 13:55:41 UTC (rev 4525) +++ trunk/FixedAssetDepreciation.php 2011-03-30 09:06:33 UTC (rev 4526) @@ -1,8 +1,6 @@ <?php - /* $Id: FixedAssetDepreciation.php 4213 2010-12-22 14:33:20Z tim_schofield $*/ -//$PageSecurity = 10; Now from DB include('includes/session.inc'); $title = _('Depreciation Journal Entry'); @@ -12,12 +10,12 @@ /*Get the last period depreciation (depn is transtype =44) was posted for */ $result = DB_query('SELECT periods.lastdate_in_period, - max(fixedassettrans.periodno) - FROM fixedassettrans INNER JOIN periods - ON fixedassettrans.periodno=periods.periodno - WHERE transtype=44 - GROUP BY periods.lastdate_in_period - ORDER BY periods.lastdate_in_period DESC',$db); + max(fixedassettrans.periodno) + FROM fixedassettrans INNER JOIN periods + ON fixedassettrans.periodno=periods.periodno + WHERE transtype=44 + GROUP BY periods.lastdate_in_period + ORDER BY periods.lastdate_in_period DESC',$db); $LastDepnRun = DB_fetch_row($result); @@ -44,30 +42,30 @@ /* Get list of assets for journal */ $sql="SELECT fixedassets.assetid, - fixedassets.description, - fixedassets.depntype, - fixedassets.depnrate, - fixedassets.datepurchased, - fixedassetcategories.accumdepnact, - fixedassetcategories.depnact, - fixedassetcategories.categorydescription, - SUM(CASE WHEN fixedassettrans.fixedassettranstype='cost' THEN fixedassettrans.amount ELSE 0 END) AS costtotal, - SUM(CASE WHEN fixedassettrans.fixedassettranstype='depn' THEN fixedassettrans.amount ELSE 0 END) AS depnbfwd - FROM fixedassets - INNER JOIN fixedassetcategories - ON fixedassets.assetcategoryid=fixedassetcategories.categoryid - INNER JOIN fixedassettrans - ON fixedassets.assetid=fixedassettrans.assetid - WHERE fixedassettrans.transdate<='" . FormatDateForSQL($_POST['ProcessDate']) . "' - GROUP BY fixedassets.assetid, - fixedassets.description, - fixedassets.depntype, - fixedassets.depnrate, - fixedassets.datepurchased, - fixedassetcategories.accumdepnact, - fixedassetcategories.depnact, - fixedassetcategories.categorydescription - ORDER BY assetcategoryid, assetid"; + fixedassets.description, + fixedassets.depntype, + fixedassets.depnrate, + fixedassets.datepurchased, + fixedassetcategories.accumdepnact, + fixedassetcategories.depnact, + fixedassetcategories.categorydescription, + SUM(CASE WHEN fixedassettrans.fixedassettranstype='cost' THEN fixedassettrans.amount ELSE 0 END) AS costtotal, + SUM(CASE WHEN fixedassettrans.fixedassettranstype='depn' THEN fixedassettrans.amount ELSE 0 END) AS depnbfwd + FROM fixedassets + INNER JOIN fixedassetcategories + ON fixedassets.assetcategoryid=fixedassetcategories.categoryid + INNER JOIN fixedassettrans + ON fixedassets.assetid=fixedassettrans.assetid + WHERE fixedassettrans.transdate<='" . FormatDateForSQL($_POST['ProcessDate']) . "' + GROUP BY fixedassets.assetid, + fixedassets.description, + fixedassets.depntype, + fixedassets.depnrate, + fixedassets.datepurchased, + fixedassetcategories.accumdepnact, + fixedassetcategories.depnact, + fixedassetcategories.categorydescription +ORDER BY assetcategoryid, assetid"; $AssetsResult=DB_query($sql, $db); $InputError = false; //always hope for the best @@ -83,15 +81,15 @@ echo '<p></p><table>'; $Heading = '<tr><th>' . _('Asset ID') . '</th> - <th>' . _('Description') . '</th> - <th>' . _('Date Purchased') . '</th> - <th>' . _('Cost') . '</th> - <th>' . _('Accum Depn') . '</th> - <th>' . _('B/fwd Book Value') . '</th> - <th>'. _('Depn Type') . '</th> - <th>'. _('Depn Rate') . '</th> - <th>' . _('New Depn') . '</th> - </tr>'; + <th>' . _('Description') . '</th> + <th>' . _('Date Purchased') . '</th> + <th>' . _('Cost') . '</th> + <th>' . _('Accum Depn') . '</th> + <th>' . _('B/fwd Book Value') . '</th> + <th>'. _('Depn Type') . '</th> + <th>'. _('Depn Rate') . '</th> + <th>' . _('New Depn') . '</th> + </tr>'; echo $Heading; $AssetCategoryDescription ='0'; @@ -106,12 +104,12 @@ if ($AssetCategoryDescription != $AssetRow['categorydescription'] OR $AssetCategoryDescription =='0'){ if ($AssetCategoryDescription !='0'){ //then print totals echo '<tr><th colspan=3 align="right">' . _('Total for') . ' ' . $AssetCategoryDescription . ' </th> - <th class="number">' . number_format($TotalCategoryCost,2) . '</th> - <th class="number">' . number_format($TotalCategoryAccumDepn,2) . '</th> - <th class="number">' . number_format(($TotalCategoryCost-$TotalCategoryAccumDepn),2) . '</th> - <th colspan=2></th> - <th class="number">' . number_format($TotalCategoryDepn,2) . '</th> - </tr>'; + <th class="number">' . number_format($TotalCategoryCost,2) . '</th> + <th class="number">' . number_format($TotalCategoryAccumDepn,2) . '</th> + <th class="number">' . number_format(($TotalCategoryCost-$TotalCategoryAccumDepn),2) . '</th> + <th colspan=2></th> + <th class="number">' . number_format($TotalCategoryDepn,2) . '</th> + </tr>'; } echo '<tr><th colspan=9 align="left">' . $AssetRow['categorydescription'] . '</th></tr>'; $AssetCategoryDescription = $AssetRow['categorydescription']; @@ -147,15 +145,15 @@ $k++; } echo '<td>' . $AssetRow['assetid'] . '</td> - <td>' . $AssetRow['description'] . '</td> - <td>' . ConvertSQLDate($AssetRow['datepurchased']) . '</td> - <td class="number">' . number_format($AssetRow['costtotal'],2) . '</td> - <td class="number">' . number_format($AssetRow['depnbfwd'],2) . '</td> - <td class="number">' . number_format($AssetRow['costtotal']-$AssetRow['depnbfwd'],2) . '</td> - <td align="center">' . $DepreciationType . '</td> - <td class="number">' . $AssetRow['depnrate'] . '</td> - <td class="number">' . number_format($NewDepreciation ,2) . '</td> - </tr>'; + <td>' . $AssetRow['description'] . '</td> + <td>' . ConvertSQLDate($AssetRow['datepurchased']) . '</td> + <td class="number">' . number_format($AssetRow['costtotal'],2) . '</td> + <td class="number">' . number_format($AssetRow['depnbfwd'],2) . '</td> + <td class="number">' . number_format($AssetRow['costtotal']-$AssetRow['depnbfwd'],2) . '</td> + <td align="center">' . $DepreciationType . '</td> + <td class="number">' . $AssetRow['depnrate'] . '</td> + <td class="number">' . number_format($NewDepreciation ,2) . '</td> + </tr>'; $TotalCategoryCost +=$AssetRow['costtotal']; $TotalCategoryAccumDepn +=$AssetRow['depnbfwd']; $TotalCategoryDepn +=$NewDepreciation; @@ -167,62 +165,62 @@ //debit depreciation expense $SQL = "INSERT INTO gltrans (type, - typeno, - trandate, - periodno, - account, - narrative, - amount) - VALUES (44, - '" . $TransNo . "', - '" . FormatDateForSQL($_POST['ProcessDate']) . "', - '" . $PeriodNo . "', - '" . $AssetRow['depnact'] . "', - '" . $AssetRow['assetid'] . "', - '" . $NewDepreciation ."')"; + typeno, + trandate, + periodno, + account, + narrative, + amount) + VALUES (44, + '" . $TransNo . "', + '" . FormatDateForSQL($_POST['ProcessDate']) . "', + '" . $PeriodNo . "', + '" . $AssetRow['depnact'] . "', + '" . $AssetRow['assetid'] . "', + '" . $NewDepreciation ."')"; $ErrMsg = _('Cannot insert a depreciation GL entry for the depreciation because'); $DbgMsg = _('The SQL that failed to insert the GL Trans record was'); $result = DB_query($SQL,$db,$ErrMsg,$DbgMsg,true); $SQL = "INSERT INTO gltrans (type, - typeno, - trandate, - periodno, - account, - narrative, - amount) - VALUES (44, - '" . $TransNo . "', - '" . FormatDateForSQL($_POST['ProcessDate']) . "', - '" . $PeriodNo . "', - '" . $AssetRow['accumdepnact'] . "', - '" . $AssetRow['assetid'] . "', - '" . -$NewDepreciation ."')"; + typeno, + trandate, + periodno, + account, + narrative, + amount) + VALUES (44, + '" . $TransNo . "', + '" . FormatDateForSQL($_POST['ProcessDate']) . "', + '" . $PeriodNo . "', + '" . $AssetRow['accumdepnact'] . "', + '" . $AssetRow['assetid'] . "', + '" . -$NewDepreciation ."')"; $result = DB_query($SQL,$db,$ErrMsg,$DbgMsg,true); //insert the fixedassettrans record $SQL = "INSERT INTO fixedassettrans (assetid, - transtype, - transno, - transdate, - periodno, - inputdate, - fixedassettranstype, - amount) - VALUES ('" . $AssetRow['assetid'] . "', - '44', - '" . $TransNo . "', - '" . FormatDateForSQL($_POST['ProcessDate']) . "', - '" . $PeriodNo . "', - '" . Date('Y-m-d') . "', - 'depn', - '" . $NewDepreciation . "')"; + transtype, + transno, + transdate, + periodno, + inputdate, + fixedassettranstype, + amount) + VALUES ('" . $AssetRow['assetid'] . "', + '44', + '" . $TransNo . "', + '" . FormatDateForSQL($_POST['ProcessDate']) . "', + '" . $PeriodNo . "', + '" . Date('Y-m-d') . "', + 'depn', + '" . $NewDepreciation . "')"; $ErrMsg = _('Cannot insert a fixed asset transaction entry for the depreciation because'); $DbgMsg = _('The SQL that failed to insert the fixed asset transaction record was'); $result = DB_query($SQL,$db,$ErrMsg,$DbgMsg,true); /*now update the accum depn in fixedassets */ $SQL = "UPDATE fixedassets SET accumdepn = accumdepn + " . $NewDepreciation . " - WHERE assetid = '" . $AssetRow['assetid'] . "'"; + WHERE assetid = '" . $AssetRow['assetid'] . "'"; $ErrMsg = _('CRITICAL ERROR! NOTE DOWN THIS ERROR AND SEEK ASSISTANCE. The fixed asset accumulated depreciation could not be updated:'); $DbgMsg = _('The following SQL was used to attempt the update the accumulated depreciation of the asset was:'); $Result = DB_query($SQL,$db,$ErrMsg, $DbgMsg, true); @@ -249,7 +247,7 @@ $result = DB_Txn_Commit($db); prnMsg(_('Depreciation') . ' ' . $TransNo . ' ' . _('has been successfully entered'),'success'); unset($_POST['ProcessDate']); - echo '<br><a href="index.php' . '?' . SID . '">' ._('Return to main menu').'</a>'; + echo '<br /><a href="index.php' . '?' . SID . '">' ._('Return to main menu').'</a>'; /*And post the journal too */ include ('includes/GLPostings.inc'); } else { @@ -265,7 +263,7 @@ <td>' . $_POST['ProcessDate'] .'</td>'; } echo '<td><div class="centre"><input type="submit" name="CommitDepreciation" value="'._('Commit Depreciation').'"></div>'; - echo '</tr></table><br>'; + echo '</tr></table><br />'; echo '</form>'; } include('includes/footer.inc'); Modified: trunk/FixedAssetItems.php =================================================================== --- trunk/FixedAssetItems.php 2011-03-29 13:55:41 UTC (rev 4525) +++ trunk/FixedAssetItems.php 2011-03-30 09:06:33 UTC (rev 4526) @@ -1,16 +1,13 @@ <?php /* $Id$ */ -/* $Revision: 1.3 $ */ -//$PageSecurity = 11; - include('includes/session.inc'); $title = _('Fixed Assets'); include('includes/header.inc'); include('includes/SQL_CommonFunctions.inc'); -echo '<a href="' . $rootpath . '/SelectAsset.php?' . SID . '">' . _('Back to Select') . '</a><br>' . "\n"; +echo '<a href="' . $rootpath . '/SelectAsset.php">' . _('Back to Select') . '</a><br />' . "\n"; echo '<p class="page_title_text"><img src="'.$rootpath.'/css/'.$theme.'/images/money_add.png" title="' . _('Fixed Asset Items') . '" alt="" />' . ' ' . $title . '</p>'; @@ -132,7 +129,10 @@ $PeriodNo = GetPeriod(Date($_SESSION['DefaultDateFormat']),$db); /* Get the new account codes for the new asset category */ - $result = DB_query("SELECT costact, accumdepnact FROM fixedassetcategories WHERE categoryid='" . $_POST['AssetCategoryID'] . "'",$db); + $result = DB_query("SELECT costact, + accumdepnact + FROM fixedassetcategories + WHERE categoryid='" . $_POST['AssetCategoryID'] . "'",$db); $NewAccounts = DB_fetch_array($result); $TransNo = GetNextTransNo( 42, $db); /* transaction type is asset category change */ @@ -144,75 +144,75 @@ periodno, account, narrative, - amount) "; - $SQL= $SQL . "VALUES (42, - '" . $TransNo . "', - '" . Date('Y-m-d') . "', - '" . $PeriodNo . "', - '" . $OldDetails['costact'] . "', - '" . $AssetID . ' ' . _('change category') . ' ' . $OldDetails['assetcategoryid'] . ' - ' . $_POST['AssetCategoryID'] . "', - '" . -$OldDetails['cost']. "' - )"; + amount) + VALUES ('42', + '" . $TransNo . "', + '" . Date('Y-m-d') . "', + '" . $PeriodNo . "', + '" . $OldDetails['costact'] . "', + '" . $AssetID . ' ' . _('change category') . ' ' . $OldDetails['assetcategoryid'] . ' - ' . $_POST['AssetCategoryID'] . "', + '" . -$OldDetails['cost']. "' + )"; $ErrMsg = _('Cannot insert a GL entry for the change of asset category because'); $DbgMsg = _('The SQL that failed to insert the cost GL Trans record was'); $result = DB_query($SQL,$db,$ErrMsg,$DbgMsg,true); //debit cost for the new category $SQL = "INSERT INTO gltrans (type, - typeno, - trandate, - periodno, - account, - narrative, - amount) "; - $SQL= $SQL . "VALUES (42, - '" . $TransNo . "', - '" . Date('Y-m-d') . "', - '" . $PeriodNo . "', - '" . $NewAccounts['costact'] . "', - '" . $AssetID . ' ' . _('change category') . ' ' . $OldDetails['assetcategoryid'] . ' - ' . $_POST['AssetCategoryID'] . "', - '" . $OldDetails['cost']. "' - )"; + typeno, + trandate, + periodno, + account, + narrative, + amount) + VALUES ('42', + '" . $TransNo . "', + '" . Date('Y-m-d') . "', + '" . $PeriodNo . "', + '" . $NewAccounts['costact'] . "', + '" . $AssetID . ' ' . _('change category') . ' ' . $OldDetails['assetcategoryid'] . ' - ' . $_POST['AssetCategoryID'] . "', + '" . $OldDetails['cost']. "' + )"; $ErrMsg = _('Cannot insert a GL entry for the change of asset category because'); $DbgMsg = _('The SQL that failed to insert the cost GL Trans record was'); $result = DB_query($SQL,$db,$ErrMsg,$DbgMsg,true); if ($OldDetails['accumdepn']!=0) { //debit accumdepn for the old category $SQL = "INSERT INTO gltrans (type, - typeno, - trandate, - periodno, - account, - narrative, - amount) "; - $SQL= $SQL . "VALUES (42, - '" . $TransNo . "', - '" . Date('Y-m-d') . "', - '" . $PeriodNo . "', - '" . $OldDetails['accumdepnact'] . "', - '" . $AssetID . ' ' . _('change category') . ' ' . $OldDetails['assetcategoryid'] . ' - ' . $_POST['AssetCategoryID'] . "', - '" . $OldDetails['accumdepn']. "' - )"; + typeno, + trandate, + periodno, + account, + narrative, + amount) + VALUES ('42', + '" . $TransNo . "', + '" . Date('Y-m-d') . "', + '" . $PeriodNo . "', + '" . $OldDetails['accumdepnact'] . "', + '" . $AssetID . ' ' . _('change category') . ' ' . $OldDetails['assetcategoryid'] . ' - ' . $_POST['AssetCategoryID'] . "', + '" . $OldDetails['accumdepn']. "' + )"; $ErrMsg = _('Cannot insert a GL entry for the change of asset category because'); $DbgMsg = _('The SQL that failed to insert the cost GL Trans record was'); $result = DB_query($SQL,$db,$ErrMsg,$DbgMsg,true); //credit accum depn for the new category $SQL = "INSERT INTO gltrans (type, - typeno, - trandate, - periodno, - account, - narrative, - amount) "; - $SQL= $SQL . "VALUES (42, - '" . $TransNo . "', - '" . Date('Y-m-d') . "', - '" . $PeriodNo . "', - '" . $NewAccounts['accumdepnact'] . "', - '" . $AssetID . ' ' . _('change category') . ' ' . $OldDetails['assetcategoryid'] . ' - ' . $_POST['AssetCategoryID'] . "', - '" . $OldDetails['accumdepn']. "' - )"; + typeno, + trandate, + periodno, + account, + narrative, + amount) + VALUES ('42', + '" . $TransNo . "', + '" . Date('Y-m-d') . "', + '" . $PeriodNo . "', + '" . $NewAccounts['accumdepnact'] . "', + '" . $AssetID . ' ' . _('change category') . ' ' . $OldDetails['assetcategoryid'] . ' - ' . $_POST['AssetCategoryID'] . "', + '" . $OldDetails['accumdepn']. "' + )"; $ErrMsg = _('Cannot insert a GL entry for the change of asset category because'); $DbgMsg = _('The SQL that failed to insert the cost GL Trans record was'); $result = DB_query($SQL,$db,$ErrMsg,$DbgMsg,true); @@ -234,25 +234,25 @@ $result = DB_query($sql,$db,$ErrMsg,$DbgMsg); prnMsg( _('Asset') . ' ' . $AssetID . ' ' . _('has been updated'), 'success'); - echo '<br>'; + echo '<br />'; } else { //it is a NEW part $sql = "INSERT INTO fixedassets (description, - longdescription, - assetcategoryid, - assetlocation, - depntype, - depnrate, - barcode, - serialno) - VALUES ( - '" . $_POST['Description'] . "', - '" . $_POST['LongDescription'] . "', - '" . $_POST['AssetCategoryID'] . "', - '" . $_POST['AssetLocation'] . "', - '" . $_POST['DepnType'] . "', - '" . $_POST['DepnRate']. "', - '" . $_POST['BarCode'] . "', - '" . $_POST['SerialNo'] . "' )"; + longdescription, + assetcategoryid, + assetlocation, + depntype, + depnrate, + barcode, + serialno) + VALUES ( + '" . $_POST['Description'] . "', + '" . $_POST['LongDescription'] . "', + '" . $_POST['AssetCategoryID'] . "', + '" . $_POST['AssetLocation'] . "', + '" . $_POST['DepnType'] . "', + '" . $_POST['DepnRate']. "', + '" . $_POST['BarCode'] . "', + '" . $_POST['SerialNo'] . "' )"; $ErrMsg = _('The asset could not be added because'); $DbgMsg = _('The SQL that was used to add the asset failed was'); $result = DB_query($sql,$db, $ErrMsg, $DbgMsg); @@ -272,7 +272,7 @@ $result = DB_Txn_Commit($db); } } else { - echo '<br>'. "\n"; + echo '<br />'. "\n"; prnMsg( _('Validation failed, no updates or deletes took place'), 'error'); } @@ -282,12 +282,12 @@ $CancelDelete = 0; //what validation is required before allowing deletion of assets .... maybe there should be no deletion option? $result = DB_query("SELECT cost, - accumdepn, - accumdepnact, - costact - FROM fixedassets INNER JOIN fixedassetcategories - ON fixedassets.assetcategoryid=fixedassetcategories.categoryid - WHERE assetid='" . $AssetID . "'", $db); + accumdepn, + accumdepnact, + costact + FROM fixedassets INNER JOIN fixedassetcategories + ON fixedassets.assetcategoryid=fixedassetcategories.categoryid + WHERE assetid='" . $AssetID . "'", $db); $AssetRow = DB_fetch_array($result); $NBV = $AssetRow['cost'] -$AssetRow['accumdepn']; if ($NBV!=0) { @@ -313,40 +313,40 @@ if ($AssetRow['cost'] > 0){ //credit cost for the asset deleted $SQL = "INSERT INTO gltrans (type, - typeno, - trandate, - periodno, - account, - narrative, - amount) "; - $SQL= $SQL . "VALUES (43, - '" . $TransNo . "', - '" . Date('Y-m-d') . "', - '" . $PeriodNo . "', - '" . $AssetRow['costact'] . "', - '" . _('Delete asset') . ' ' . $AssetID . "', - '" . -$AssetRow['cost']. "' - )"; + typeno, + trandate, + periodno, + account, + narrative, + amount) + VALUES ('43', + '" . $TransNo . "', + '" . Date('Y-m-d') . "', + '" . $PeriodNo . "', + '" . $AssetRow['costact'] . "', + '" . _('Delete asset') . ' ' . $AssetID . "', + '" . -$AssetRow['cost']. "' + )"; $ErrMsg = _('Cannot insert a GL entry for the deletion of the asset because'); $DbgMsg = _('The SQL that failed to insert the cost GL Trans record was'); $result = DB_query($SQL,$db,$ErrMsg,$DbgMsg,true); //debit accumdepn for the depreciation removed on deletion of this asset $SQL = "INSERT INTO gltrans (type, - typeno, - trandate, - periodno, - account, - narrative, - amount) "; - $SQL= $SQL . "VALUES (43, - '" . $TransNo . "', - '" . Date('Y-m-d') . "', - '" . $PeriodNo . "', - '" . $AssetRow['accumdepnact'] . "', - '" . _('Delete asset') . ' ' . $AssetID . "', - '" . $Asset['accumdepn']. "' - )"; + typeno, + trandate, + periodno, + account, + narrative, + amount) + VALUES ('43', + '" . $TransNo . "', + '" . Date('Y-m-d') . "', + '" . $PeriodNo . "', + '" . $AssetRow['accumdepnact'] . "', + '" . _('Delete asset') . ' ' . $AssetID . "', + '" . $Asset['accumdepn']. "' + )"; $ErrMsg = _('Cannot insert a GL entry for the reversal of accumulated depreciation on deletion of the asset because'); $DbgMsg = _('The SQL that failed to insert the cost GL Trans record was'); $result = DB_query($SQL,$db,$ErrMsg,$DbgMsg,true); @@ -467,7 +467,7 @@ } if ($AssetImgLink!=_('No Image')) { - echo '<td>' . _('Image') . '<br>'.$AssetImgLink . '</td></tr>'; + echo '<td>' . _('Image') . '<br />'.$AssetImgLink . '</td></tr>'; } else { echo '</td></tr>'; } @@ -484,13 +484,13 @@ while ($myrow=DB_fetch_array($result)){ if (!isset($_POST['AssetCategoryID']) or $myrow['categoryid']==$_POST['AssetCategoryID']){ - echo '<option selected VALUE="'. $myrow['categoryid'] . '">' . $myrow['categorydescription']; + echo '<option selected VALUE="'. $myrow['categoryid'] . '">' . $myrow['categorydescription'] . '</option>'; } else { - echo '<option VALUE="'. $myrow['categoryid'] . '">' . $myrow['categorydescription']; + echo '<option VALUE="'. $myrow['categoryid'] . '">' . $myrow['categorydescription']. '</option>'; } $category=$myrow['categoryid']; } -echo '</select><a target="_blank" href="'. $rootpath . '/FixedAssetCategories.php?' . SID . '">'.' ' . _('Add or Modify Asset Categories') . '</a></td></tr>'; +echo '</select><a target="_blank" href="'. $rootpath . '/FixedAssetCategories.php">'.' ' . _('Add or Modify Asset Categories') . '</a></td></tr>'; if (!isset($_POST['AssetCategoryID'])) { $_POST['AssetCategoryID']=$category; } @@ -545,12 +545,12 @@ echo '<tr><td>' . _('Net Book Value') . ':</td><td class="number">' . number_format($AssetRow['cost']-$AssetRow['accumdepn'],2) . '</td></tr>'; $result = DB_query('SELECT periods.lastdate_in_period, - max(fixedassettrans.periodno) - FROM fixedassettrans INNER JOIN periods - ON fixedassettrans.periodno=periods.periodno - WHERE transtype=44 - GROUP BY periods.lastdate_in_period - ORDER BY periods.lastdate_in_period DESC',$db); + max(fixedassettrans.periodno) + FROM fixedassettrans INNER JOIN periods + ON fixedassettrans.periodno=periods.periodno + WHERE transtype=44 + GROUP BY periods.lastdate_in_period + ORDER BY periods.lastdate_in_period DESC',$db); $LastDepnRun = DB_fetch_row($result); if(DB_num_rows($result)==0){ @@ -563,13 +563,13 @@ if (isset($New)) { - echo '<div class=centre><br><input type="Submit" name="submit" value="' . _('Insert New Fixed Asset') . '">'; + echo '<div class=centre><br /><input type="Submit" name="submit" value="' . _('Insert New Fixed Asset') . '">'; } else { - echo '<br><div class=centre><input type="submit" name="submit" value="' . _('Update') . '"></div>'; + echo '<br /><div class=centre><input type="submit" name="submit" value="' . _('Update') . '"></div>'; prnMsg( _('Only click the Delete button if you are sure you wish to delete the asset. Only assets with a zero book value can be deleted'), 'warn', _('WARNING')); - echo '<br><div class=centre><input type="Submit" name="delete" value="' . _('Delete This Asset') . '" onclick="return confirm(\'' . _('Are You Sure? Only assets with a zero book value can be deleted.') . '\');"></div>'; + echo '<br /><div class=centre><input type="Submit" name="delete" value="' . _('Delete This Asset') . '" onclick="return confirm(\'' . _('Are You Sure? Only assets with a zero book value can be deleted.') . '\');"></div>'; } echo '</form></div>'; Modified: trunk/GLAccountCSV.php =================================================================== --- trunk/GLAccountCSV.php 2011-03-29 13:55:41 UTC (rev 4525) +++ trunk/GLAccountCSV.php 2011-03-30 09:06:33 UTC (rev 4526) @@ -1,9 +1,8 @@ <?php -/* $Id: $*/ -/* $Revision: 1.00 $ */ -//$PageSecurity = 8; +/* $Id: GLAccountCSV.php 4492 2011-02-18 09:56:52Z daintree $ */ + include ('includes/session.inc'); $title = _('General Ledger Account Report'); include('includes/header.inc'); @@ -19,7 +18,7 @@ echo '<div class="page_help_text">' . _('Use the keyboard Shift key to select multiple accounts and periods') . '</div><br>'; -echo "<form method='POST' action=" . $_SERVER['PHP_SELF'] . '?' . SID . '>'; +echo '<form method="POST" action="' . $_SERVER['PHP_SELF'] . '">'; echo '<input type="hidden" name="FormID" value="' . $_SESSION['FormID'] . '" />'; /*Dates in SQL format for the last day of last month*/ @@ -35,10 +34,10 @@ $i=0; while ($myrow=DB_fetch_array($AccountsResult,$db)){ if(isset($_POST['Account'][$i]) AND $myrow['accountcode'] == $_POST['Account'][$i]){ - echo '<option selected VALUE=' . $myrow['accountcode'] . '>' . $myrow['accountcode'] . ' ' . $myrow['accountname']; + echo '<option selected VALUE=' . $myrow['accountcode'] . '>' . $myrow['accountcode'] . ' ' . $myrow['accountname'] . '</option>'; $i++; } else { - echo '<option VALUE=' . $myrow['accountcode'] . '>' . $myrow['accountcode'] . ' ' . $myrow['accountname']; + echo '<option VALUE=' . $myrow['accountcode'] . '>' . $myrow['accountcode'] . ' ' . $myrow['accountname'] . '</option>'; } } echo '</select></td>'; @@ -51,10 +50,10 @@ while ($myrow=DB_fetch_array($Periods,$db)){ if (isset($SelectedPeriod[$id]) and $myrow['periodno'] == $SelectedPeriod[$id]){ - echo '<option selected VALUE=' . $myrow['periodno'] . '>' . _(MonthAndYearFromSQLDate($myrow['lastdate_in_period'])); + echo '<option selected VALUE=' . $myrow['periodno'] . '>' . _(MonthAndYearFromSQLDate($myrow['lastdate_in_period'])) . '</option>'; $id++; } else { - echo '<option VALUE=' . $myrow['periodno'] . '>' . _(MonthAndYearFromSQLDate($myrow['lastdate_in_period'])); + echo '<option VALUE=' . $myrow['periodno'] . '>' . _(MonthAndYearFromSQLDate($myrow['lastdate_in_period'])) . '</option>'; } } echo '</select></td></tr>'; @@ -71,9 +70,9 @@ echo '<option value=0>0 - '._('All tags'); while ($myrow=DB_fetch_array($result)){ if (isset($_POST['tag']) and $_POST['tag']==$myrow['tagref']){ - echo '<option selected value=' . $myrow['tagref'] . '>' . $myrow['tagref'].' - ' .$myrow['tagdescription']; + echo '<option selected value=' . $myrow['tagref'] . '>' . $myrow['tagref'].' - ' .$myrow['tagdescription'] . '</option>'; } else { - echo '<option value=' . $myrow['tagref'] . '>' . $myrow['tagref'].' - ' .$myrow['tagdescription']; + echo '<option value=' . $myrow['tagref'] . '>' . $myrow['tagref'].' - ' .$myrow['tagdescription'] . '</option>'; } } echo '</select></td></tr>'; @@ -95,7 +94,6 @@ prnMsg(_('An account or range of accounts must be selected from the list box'),'info'); include('includes/footer.inc'); exit; - } if (!file_exists($_SESSION['reports_dir'])){ @@ -115,10 +113,10 @@ foreach ($_POST['Account'] as $SelectedAccount){ /*Is the account a balance sheet or a profit and loss account */ $result = DB_query("SELECT chartmaster.accountname, - accountgroups.pandl - FROM accountgroups - INNER JOIN chartmaster ON accountgroups.groupname=chartmaster.group_ - WHERE chartmaster.accountcode=$SelectedAccount",$db); + accountgroups.pandl + FROM accountgroups + INNER JOIN chartmaster ON accountgroups.groupname=chartmaster.group_ + WHERE chartmaster.accountcode=$SelectedAccount",$db); $AccountDetailRow = DB_fetch_row($result); $AccountName = $AccountDetailRow[1]; if ($AccountDetailRow[1]==1){ @@ -140,11 +138,11 @@ gltrans.periodno, gltrans.tag FROM gltrans, systypes - WHERE gltrans.account = $SelectedAccount + WHERE gltrans.account = '$SelectedAccount' AND systypes.typeid=gltrans.type AND posted=1 - AND periodno>=$FirstPeriodSelected - AND periodno<=$LastPeriodSelected + AND periodno>='$FirstPeriodSelected' + AND periodno<='$LastPeriodSelected' ORDER BY periodno, gltrans.trandate, counterindex"; } else { @@ -157,11 +155,11 @@ gltrans.periodno, gltrans.tag FROM gltrans, systypes - WHERE gltrans.account = $SelectedAccount + WHERE gltrans.account = '$SelectedAccount' AND systypes.typeid=gltrans.type AND posted=1 - AND periodno>=$FirstPeriodSelected - AND periodno<=$LastPeriodSelected + AND periodno>='$FirstPeriodSelected' + AND periodno<='$LastPeriodSelected' AND tag='".$_POST['tag']."' ORDER BY periodno, gltrans.trandate, counterindex"; } @@ -177,8 +175,8 @@ actual, period FROM chartdetails - WHERE chartdetails.accountcode= $SelectedAccount - AND chartdetails.period=" . $FirstPeriodSelected; + WHERE chartdetails.accountcode= '$SelectedAccount' + AND chartdetails.period='" . $FirstPeriodSelected . "'"; $ErrMsg = _('The chart details for account') . ' ' . $SelectedAccount . ' ' . _('could not be retrieved'); $ChartDetailsResult = DB_query($sql,$db,$ErrMsg); @@ -206,8 +204,8 @@ actual, period FROM chartdetails - WHERE chartdetails.accountcode= $SelectedAccount - AND chartdetails.period=" . $PeriodNo; + WHERE chartdetails.accountcode= '$SelectedAccount' + AND chartdetails.period='" . $PeriodNo . "'"; $ErrMsg = _('The chart details for account') . ' ' . $SelectedAccount . ' ' . _('could not be retrieved'); $ChartDetailsResult = DB_query($sql,$db,$ErrMsg); @@ -225,7 +223,6 @@ $RunningTotal += $myrow['amount']; $PeriodTotal += $myrow['amount']; - $FormatedTranDate = ConvertSQLDate($myrow['trandate']); $tagsql="SELECT tagdescription FROM tags WHERE tagref='".$myrow['tag'] . "'"; @@ -264,4 +261,4 @@ } /* end of if CreateCSV button hit */ include('includes/footer.inc'); -?> +?> \ No newline at end of file Modified: trunk/GLBudgets.php =================================================================== --- trunk/GLBudgets.php 2011-03-29 13:55:41 UTC (rev 4525) +++ trunk/GLBudgets.php 2011-03-30 09:06:33 UTC (rev 4526) @@ -1,7 +1,5 @@ <?php - /* $Id$*/ -//$PageSecurity = 10; include('includes/session.inc'); include('includes/SQL_CommonFunctions.inc'); @@ -23,12 +21,12 @@ } if (isset($_POST['update'])) { - prnMsg('Budget updated successfully', 'success'); + prnMsg(_('Budget updated successfully'), 'success'); } //If an account hasn't been selected then select one here. echo '<p class="page_title_text"><img src="'.$rootpath.'/css/'.$theme.'/images/maintenance.png" title="' . _('Search') . '" alt="" />' . ' ' . $title.'</p>'; -echo '<form action=' . $_SERVER['PHP_SELF'] . '?' . SID . ' method=post name="selectaccount">'; +echo '<form action="' . $_SERVER['PHP_SELF'] . '" method="post" name="selectaccount">'; echo '<input type="hidden" name="FormID" value="' . $_SESSION['FormID'] . '" />'; echo '<table>'; @@ -36,9 +34,9 @@ onChange='ReloadForm(selectaccount.Select)'>"; $SQL = 'SELECT accountcode, - accountname - FROM chartmaster - ORDER BY accountcode'; + accountname + FROM chartmaster + ORDER BY accountcode'; $result=DB_query($SQL,$db); if (DB_num_rows($result)==0){ @@ -48,10 +46,10 @@ while ($myrow=DB_fetch_array($result)){ $account = $myrow['accountcode'] . ' - ' . $myrow['accountname']; if (isset($SelectedAccount) and isset($LastCode) and $SelectedAccount==$myrow['accountcode']){ - echo '<option selected value=' . $myrow['accountcode'] . '>' . $account; + echo '<option selected value=' . $myrow['accountcode'] . '>' . $account . '</option>'; $PrevCode=$LastCode; } else { - echo '<option value=' . $myrow['accountcode'] . '>' . $account; + echo '<option value=' . $myrow['accountcode'] . '>' . $account . '</option>'; if (isset($SelectedAccount) and isset($LastCode) and $SelectedAccount == $LastCode) { $NextCode=$myrow['accountcode']; } @@ -148,7 +146,7 @@ // Table Headers - echo '<form name="form" action=' . $_SERVER['PHP_SELF'] . '?' . SID . ' method=post>'; + echo '<form name="form" action="' . $_SERVER['PHP_SELF'] . '" method=post>'; echo '<input type="hidden" name="FormID" value="' . $_SESSION['FormID'] . '" />'; echo '<br /><table class=selection>'; echo '<tr><th colspan=3>'. _('Last Financial Year') .'</th>'; @@ -156,7 +154,7 @@ echo '<th colspan=3>'. _('Next Financial Year') .'</th></tr>'; echo '<tr><th colspan=3>'. _('Year ended').' - '. - Date($_SESSION['DefaultDateFormat'],YearEndDate($_SESSION['YearEnd'], -1)) .'</th>'; + Date($_SESSION['DefaultDateFormat'],YearEndDate($_SESSION['YearEnd'],-1)) .'</th>'; echo '<th colspan=3>'. _('Year ended').' - '. Date($_SESSION['DefaultDateFormat'],YearEndDate($_SESSION['YearEnd'],0)) .'</th>'; echo '<th colspan=3>'. _('Year ended').' - '. @@ -170,7 +168,6 @@ } echo '</tr>'; - // Main Table for ($i=1; $i<=12; $i++) { Modified: trunk/GLCodesInquiry.php =================================================================== --- trunk/GLCodesInquiry.php 2011-03-29 13:55:41 UTC (rev 4525) +++ trunk/GLCodesInquiry.php 2011-03-30 09:06:33 UTC (rev 4526) @@ -1,9 +1,7 @@ <?php /* $Id$*/ -/* $Revision: 1.9 $ */ -//$PageSecurity = 8; include ('includes/session.inc'); $title = _('GL Codes Inquiry'); @@ -13,20 +11,20 @@ $SQL = 'SELECT group_, accountcode , accountname - FROM chartmaster INNER JOIN accountgroups ON chartmaster.group_ = accountgroups.groupname - ORDER BY sequenceintb, - accountcode'; + FROM chartmaster INNER JOIN accountgroups ON chartmaster.group_ = accountgroups.groupname + ORDER BY sequenceintb, + accountcode'; $ErrMsg = _('No general ledger accounts were returned by the SQL because'); $AccountsResult = DB_query($SQL,$db,$ErrMsg); /*show a table of the orders returned by the SQL */ -echo "<table cellpadding=2 colspan=2> +echo '<table cellpadding=2 colspan=2> <tr> - <th>"._('Group')."</font></th> - <th>"._('Code')."</font></th> - <th>"._('Account Name').'</font></th> + <th>' . _('Group') . '</font></th> + <th>' . _('Code') . '</font></th> + <th>' . _('Account Name') . '</font></th> </tr>'; $j = 1; @@ -63,7 +61,5 @@ //end of while loop echo '</table>'; - include('includes/footer.inc'); - -?> +?> \ No newline at end of file Modified: trunk/GLJournal.php =================================================================== --- trunk/GLJournal.php 2011-03-29 13:55:41 UTC (rev 4525) +++ trunk/GLJournal.php 2011-03-30 09:06:33 UTC (rev 4526) @@ -1,11 +1,9 @@ <?php /* $Id$*/ -/* $Revision: 1.27 $ */ include('includes/DefineJournalClass.php'); -//$PageSecurity = 10; include('includes/session.inc'); $title = _('Journal Entry'); @@ -24,7 +22,7 @@ Journals cannot be entered against bank accounts GL postings involving bank accounts must be done using a receipt or a payment transaction to ensure a bank trans is available for matching off vs statements */ - $SQL = "SELECT accountcode FROM bankaccounts"; + $SQL = 'SELECT accountcode FROM bankaccounts'; $result = DB_query($SQL,$db); $i=0; while ($Act = DB_fetch_row($result)){ @@ -68,8 +66,8 @@ account, narrative, amount, - tag) "; - $SQL= $SQL . "VALUES (0, + tag) + VALUES ('0', '" . $TransNo . "', '" . FormatDateForSQL($_SESSION['JournalDetail']->JnlDate) . "', '" . $PeriodNo . "', @@ -90,8 +88,8 @@ account, narrative, amount, - tag) "; - $SQL= $SQL . "VALUES (0, + tag) + VALUES ('0', '" . $TransNo . "', '" . FormatDateForSQL($_SESSION['JournalDetail']->JnlDate) . "', '" . ($PeriodNo + 1) . "', @@ -104,7 +102,6 @@ $ErrMsg =_('Cannot insert a GL entry for the reversing journal because'); $DbgMsg = _('The SQL that failed to insert the GL Trans record was'); $result = DB_query($SQL,$db,$ErrMsg,$DbgMsg,true); - } } @@ -120,7 +117,7 @@ unset($_SESSION['JournalDetail']); /*Set up a newy in case user wishes to enter another */ - echo "<br /><a href='" . $_SERVER['PHP_SELF'] . '?' . SID . "&NewJournal=Yes'>"._('Enter Another General Ledger Journal').'</a>'; + echo '<br /><a href="' . $_SERVER['PHP_SELF'] . '?NewJournal=Yes">'._('Enter Another General Ledger Journal').'</a>'; /*And post the journal too */ include ('includes/GLPostings.inc'); include ('includes/footer.inc'); @@ -132,21 +129,17 @@ $_SESSION['JournalDetail']->Remove_GLEntry($_GET['Delete']); } elseif (isset($_POST['Process']) and $_POST['Process']==_('Accept')){ //user hit submit a new GL Analysis line into the journal - if($_POST['GLCode']!='') - { + if ($_POST['GLCode']!='') { $extract = explode(' - ',$_POST['GLCode']); $_POST['GLCode'] = $extract[0]; } - if($_POST['Debit']>0) - { + if ($_POST['Debit']>0) { $_POST['GLAmount'] = $_POST['Debit']; - } - elseif($_POST['Credit']>0) - { + } elseif ($_POST['Credit']>0) { $_POST['GLAmount'] = '-' . $_POST['Credit']; } if ($_POST['GLManualCode'] != '' AND is_numeric($_POST['GLManualCode'])){ - // If a manual code was entered need to check it exists and isnt a bank account + // If a manual code was entered need to check it exists and isnt a bank account $AllowThisPosting = true; //by default if ($_SESSION['ProhibitJournalsToControlAccounts'] == 1){ if ($_SESSION['CompanyRecord']['gllink_debtors'] == '1' AND $_POST['GLManualCode'] == $_SESSION['CompanyRecord']['debtorsact']){ @@ -228,18 +221,10 @@ unset($_POST['GLManualCode']); } -// set up the form whatever -/* -if (!isset($_SESSION['JournalDetail']->JnlDate)){ - $_POST['JournalProcessDate']= Date($_SESSION['DefaultDateFormat']); - $_SESSION['JournalDetail']->JnlDate = $_POST['JournalProcessDate']; -} -*/ -echo '<form action="' . $_SERVER['PHP_SELF'] . '?' . SID . '" method="post" name="form">'; +echo '<form action="' . $_SERVER['PHP_SELF'] . '" method="post" name="form">'; echo '<input type="hidden" name="FormID" value="' . $_SESSION['FormID'] . '" />'; - echo '<p class="page_title_text"><img src="'.$rootpath.'/css/'.$theme.'/images/maintenance.png" title="' . _('Search') . '" alt="" />' . ' ' . $title.'</p>'; // A new table in the first column of the main table @@ -287,8 +272,8 @@ $SQL = 'SELECT tagref, tagdescription - FROM tags - ORDER BY tagref'; + FROM tags + ORDER BY tagref'; $result=DB_query($SQL,$db); echo '<option value="0">0 - None</option>'; @@ -309,10 +294,10 @@ "'".'The account code '."'".'+ this.value+ '."'".' doesnt exist'."'".')"' . ' value="'. $_POST['GLManualCode'] .'" /></td>'; -$sql="SELECT accountcode, +$sql='SELECT accountcode, accountname FROM chartmaster - ORDER BY accountcode"; + ORDER BY accountcode'; $result=DB_query($sql, $db); echo '<td><select name="GLCode" onChange="return assignComboToInput(this,'.'GLManualCode'.')">'; @@ -336,15 +321,14 @@ $_POST['Debit'] = ''; } - echo '</tr><tr><th>' . _('Debit') . '</th>'.'<td><input type="text" class="number" Name = "Debit" ' . 'onChange="eitherOr(this, '.'Credit'.')"'. ' Maxlength="12" size="10" value="' . $_POST['Debit'] . '" /></td>'; -echo '</tr><tr><th>' . _('Credit') . "</th>".'<td><input type="text" class="number" Name = "Credit" ' . +echo '</tr><tr><th>' . _('Credit') . '</th><td><input type="text" class="number" Name = "Credit" ' . 'onChange="eitherOr(this, '.'Debit'.')"'. ' Maxlength="12" size="10" value="' . $_POST['Credit'] . '" /></td>'; echo '</tr><tr><td></td><td></td><th>'. _('Narrative'). '</th>'; -echo '</tr><tr><th></th><th>' . _('GL Narrative') . "</th>"; +echo '</tr><tr><th></th><th>' . _('GL Narrative') . '</th>'; echo '<td><input type="text" name="GLNarrative" maxlength="100" size="100" value="' . $_POST['GLNarrative'] . '" /></td>'; @@ -398,22 +382,22 @@ } echo '<td>' . $JournalItem->Narrative . "</td> - <td><a href='" . $_SERVER['PHP_SELF'] . '?' . SID . '&Delete=' . $JournalItem->ID . "'>"._('Delete').'</a></td> + <td><a href='" . $_SERVER['PHP_SELF'] . '?Delete=' . $JournalItem->ID . "'>"._('Delete').'</a></td> </tr>'; } echo '<tr class="EvenTableRows"><td></td> - <td class="number"><b> Total </b></td> + <td class="number"><b>' . _('Total') . '</b></td> <td class="number"><b>' . number_format($debittotal,2) . '</b></td> <td class="number"><b>' . number_format($credittotal,2) . '</b></td></tr>'; if ($debittotal!=$credittotal) { - echo '<td align="center" style="background-color: #fddbdb"><b>Required to balance - </b>' . + echo '<td align="center" style="background-color: #fddbdb"><b>' . _('Required to balance') .' - </b>' . number_format(abs($debittotal-$credittotal),2); } if ($debittotal>$credittotal) { - echo ' Credit</td></tr>'; + echo ' ' . _('Credit') . '</td></tr>'; } else if ($debittotal<$credittotal) { - echo ' Debit</td></tr>'; + echo ' ' . _('Debit') . '</td></tr>'; } echo '</table>'; Modified: trunk/PDFGrn.php =================================================================== --- trunk/PDFGrn.php 2011-03-29 13:55:41 UTC (rev 4525) +++ trunk/PDFGrn.php 2011-03-30 09:06:33 UTC (rev 4526) @@ -2,7 +2,6 @@ /* $Id$*/ -//$PageSecurity = 2; Now comes from DB - read in from session include('includes/session.inc'); if (isset($_GET['GRNNo'])) { Modified: trunk/doc/Change.log.html =================================================================== --- trunk/doc/Change.log.html 2011-03-29 13:55:41 UTC (rev 4525) +++ trunk/doc/Change.log.html 2011-03-30 09:06:33 UTC (rev 4526) @@ -1,5 +1,8 @@ <p><font SIZE=4 COLOR=BLUE><b>webERP Change Log</b></font></p> -<p> +<p /> +<p>30/3/11 Tim - New ReprintGRN.php script takes a purchase order and allows any line received to have GRN(s) reprinted</p> +<p>29/3/11 Phil - checking for unquoted SQL and for SQL where literals quoted with double quotes rather than single - double quotes are not ANSI compatible - so making the SQL more ANSI compatible by doing this as suggested by Tim. Many many scripts involved will take a week or so</p> +<p /> <p>27/3/11 - Version 4.03.5 Release <p> <p>27/3/11: Phil fixed SQL upgrade script to add the Z_ChangeSupplierCode.php script is added to the scripts table</p> Modified: trunk/includes/ConnectDB.inc =================================================================== --- trunk/includes/ConnectDB.inc 2011-03-29 13:55:41 UTC (rev 4525) +++ trunk/includes/ConnectDB.inc 2011-03-30 09:06:33 UTC (rev 4526) @@ -4,7 +4,7 @@ * this value is saved in the $_SESSION['Versionumber'] when includes/GetConfig.php is run * if VersionNumber is < $Version then the DB update script is run */ -$Version='4.03.5'; //must update manually every time there is a DB change +$Version='4.03.6'; //must update manually every time there is a DB change require_once ($PathPrefix .'includes/MiscFunctions.php'); Modified: trunk/index.php =================================================================== --- trunk/index.php 2011-03-29 13:55:41 UTC (rev 4525) +++ trunk/index.php 2011-03-30 09:06:33 UTC (rev 4526) @@ -605,17 +605,17 @@ <table width="100%" class="table_index"> <tr> <td class="menu_group_item"> - <?php echo '<p>• <a href="' . $rootpath . "/StockSerialItemResearch.php?" . SID . '">' . _('Serial Item Research Tool') . '</a></p>'; ?> + <?php echo '<p>• <a href="' . $rootpath . '/StockSerialItemResearch.php">' . _('Serial Item Research Tool') . '</a></p>'; ?> </td> </tr> <tr> <td class="menu_group_item"> - <?php echo '<p>• <a href="' . $rootpath . "/PDFPrintLabel.php?" . SID . '">' . _('Print Price Labels') . '</a></p>'; ?> + <?php echo '<p>• <a href="' . $rootpath . '/PDFPrintLabel.php">' . _('Print Price Labels') . '</a></p>'; ?> </td> </tr> <tr> <td class="menu_group_item"> - <?php echo '<p>• <a href="' . $rootpath . "/StockMovements.php?" . SID . '">' . _('Inventory Item Movements') . '</a></p>'; ?> + <?php echo '<p>• <a href="' . $rootpath . '/StockMovements.php">' . _('Inventory Item Movements') . '</a></p>'; ?> </td> </tr> <tr> @@ -705,6 +705,11 @@ </tr> <tr> <td class="menu_group_item"> + <?php echo '<p>• <a href="' . $rootpath . '/ReprintGRN.php">' . _('Reprint GRN') . '</a></p>'; ?> + </td> + </tr> + <tr> + <td class="menu_group_item"> <?php echo GetRptLinks('inv'); ?> </td> </tr> Modified: trunk/sql/mysql/upgrade3.11.1-4.00.sql =================================================================== --- trunk/sql/mysql/upgrade3.11.1-4.00.sql 2011-03-29 13:55:41 UTC (rev 4525) +++ trunk/sql/mysql/upgrade3.11.1-4.00.sql 2011-03-30 09:06:33 UTC (rev 4526) @@ -835,4 +835,6 @@ ALTER TABLE `locations` CHANGE `cashsalecustomer` `cashsalecustomer` VARCHAR( 10 ) DEFAULT ''; UPDATE config SET confvalue='4.03.3' WHERE confname='VersionNumber'; INSERT INTO `scripts` (`script`, `pagesecurity`, `description`) VALUES ('Z_ChangeSupplierCode.php', '15', 'Script to change a supplier code accross all tables necessary'); -UPDATE config SET confvalue='4.03.5' WHERE confname='VersionNumber'; \ No newline at end of file +UPDATE config SET confvalue='4.03.5' WHERE confname='VersionNumber'; +INSERT INTO `scripts` (`script` ,`pagesecurity` ,`description`) VALUES ( 'ReprintGRN.php', '11', 'Allows selection of a goods received batch for reprinting the goods received note given a purchase order number'); +UPDATE config SET confvalue='4.03.6' WHERE confname='VersionNumber'; This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site. |