CustomerAllocations.php: Minor code shuffle to fix view page source m…

…essage "Start tag 'div' seen in 'table'" reported in Firefox

CustomerAllocations.php

@@ -544,6 +544,11 @@
 
 	$result = DB_query($SQL);
 	$NoOfUnallocatedTrans = DB_num_rows($result);
+
+	if ($NoOfUnallocatedTrans == 0) {
+		prnMsg(_('There are no allocations to be done'),'info');
+	}
+	else {
 	$CurrentTransaction = 1;
 	$CurrentDebtor = '';
 	echo '<table class="selection">';
@@ -598,17 +603,16 @@
 	if (!isset($Balance)) {
 		$Balance=0;
 	}
-	if ($NoOfUnallocatedTrans == 0) {
-		prnMsg(_('There are no allocations to be done'),'info');
-	} else {
+
 		echo '<tr class="striped_row">
 				<td colspan="7" class="number"><b>' . locale_number_format($Balance,$CurrDecimalPlaces)  . '</b></td>
 				<td><b>' . $CurrCode . '</b></td>
 				<td><b>' . _('Balance') . '</b></td>
-			</tr>';
+			</tr>
+		</table>';
 	}
-	echo '</table>
-		<br />';
+
+	echo '<br />';
 }
 
 include('includes/footer.php');

doc/Change.log

@@ -1,5 +1,6 @@
 webERP Change Log
 
+17/2/18  PaulT: CustomerAllocations.php: Minor code shuffle to fix view page source message "Start tag 'div' seen in 'table'" reported in Firefox.
 17/2/18  PaulT: Customers.php, ShopParameters.php: Fix view page source message "No space between attributes" reported in Firefox. 
 17/2/18  PaulT: Labels.php: Remove extra closing </td></tr> tag pair.
 17/2/18  PaulT: FixedAssetLocations.php: Move closing condition brace to cover entire table output to avoid a stray closing table tag output if the condition is not met. Also, replace some style attributes with equivalent CSS.