[Web-erp-svn] SF.net SVN: web-erp:[7941] trunk

[<tu...@us...>]

Revision: 7941
          http://sourceforge.net/p/web-erp/reponame/7941
Author:   turbopt
Date:     2018-02-07 21:26:15 +0000 (Wed, 07 Feb 2018)
Log Message:
-----------
Tim (PaulT commit) UserSettings.php: Fix the 'Maximum Number of Records to Display' from populating with the session default at page load instead of the user's setting. Applied Tim's improved handling. (Reported in forums by Paul Becker: http://www.weberp.org/forum/showthread.php?tid=8081)

Modified Paths:
--------------
    trunk/UserSettings.php
    trunk/doc/Change.log

Modified: trunk/UserSettings.php
===================================================================
--- trunk/UserSettings.php	2018-02-06 20:23:07 UTC (rev 7940)
+++ trunk/UserSettings.php	2018-02-07 21:26:15 UTC (rev 7941)
@@ -106,10 +106,6 @@
 echo '<div>';
 echo '<input type="hidden" name="FormID" value="' . $_SESSION['FormID'] . '" />';
 
-If (!isset($_POST['DisplayRecordsMax']) OR $_POST['DisplayRecordsMax']=='') {
-	$_POST['DisplayRecordsMax'] = $_SESSION['DefaultDisplayRecordsMax'];
-}
-
 echo '<table class="selection">
 		<tr>
 			<td>', _('User ID'), ':</td>
@@ -120,7 +116,7 @@
 			<td>', $_SESSION['UsersRealName'], '<input name="RealName" type="hidden" value="', $_SESSION['UsersRealName'], '" /></td></tr>
 		<tr>
 			<td>', _('Maximum Number of Records to Display'), ':</td>
-			<td><input class="integer" maxlength="3" name="DisplayRecordsMax" required="required" size="3" title="', _('The input must be positive integer'), '" type="text" value="', $_POST['DisplayRecordsMax'], '" /></td>
+			<td><input class="integer" maxlength="3" name="DisplayRecordsMax" required="required" size="3" title="', _('The input must be positive integer'), '" type="text" value="', $_SESSION['DisplayRecordsMax'], '" /></td>
 		</tr>';
 
 // Select language:
@@ -253,4 +249,4 @@
 	</form>';
 
 include('includes/footer.php');
-?>
+?>
\ No newline at end of file

Modified: trunk/doc/Change.log
===================================================================
--- trunk/doc/Change.log	2018-02-06 20:23:07 UTC (rev 7940)
+++ trunk/doc/Change.log	2018-02-07 21:26:15 UTC (rev 7941)
@@ -1,5 +1,6 @@
 webERP Change Log
 
+7/2/18   Tim (PaulT commit) UserSettings.php: Fix the 'Maximum Number of Records to Display' from populating with the session default at page load instead of the user's setting. Applied Tim's improved handling. (Reported in forums by Paul Becker: http://www.weberp.org/forum/showthread.php?tid=8081) 
 6/2/18   geo_displaymap_customers.php, geo_displaymap_suppliers.php: Fix a few PHP short-tags, and move some javascript from PHP output to fix 'missing tag' validation complaints.
 6/2/18   MRPPlannedPurchasekOrders.php, MRPPlannedWorkOrders.php: PaulT: Add missing table cell to work orders to match recent change to planned purchase orders and replace 'where clause joins' with table join in both files. Paul B/PaulT: Apply consistent code formatting between both files. (Some consistency matters reported in forums: http://www.weberp.org/forum/showthread.php?tid=8061)
 5/2/18   PaulT: SalesGraph.php: Rework previous 7908 implementation that caused graphing to break. (Reported broken in forums by Paul Becker: http://www.weberp.org/forum/showthread.php?tid=8071)