webERP Forum
bug: Internal stock request - Printable Version

+- webERP Forum (http://www.weberp.org/forum)
+-- Forum: webERP Discussion (http://www.weberp.org/forum/forumdisplay.php?fid=1)
+--- Forum: Problems / Bugs? (http://www.weberp.org/forum/forumdisplay.php?fid=8)
+--- Thread: bug: Internal stock request (/showthread.php?tid=8392)



bug: Internal stock request - ge_zhiyuan - 04-08-2020

file InternalStockRequest.php, line 209 and line 237:

variable $myrow while in the while-loop this variable name is $MyRow.

cannot upload png-file to show this bug.

PHP Code:
while ($myrow=DB_fetch_array($result)){
    if (isset(
$_SESSION['Request']->Location) AND $_SESSION['Request']->Location==$myrow['loccode']){
        echo 
'<option selected value="'$MyRow['loccode'], '">'$MyRow['loccode'], ' - 'htmlspecialchars($MyRow['locationname'], ENT_QUOTES,'UTF-8'), '</option>';
    } else {
        echo 
'<option value="'$MyRow['loccode'], '">'$MyRow['loccode'], ' - 'htmlspecialchars($MyRow['locationname'], ENT_QUOTES,'UTF-8'),  '</option>';
    }


should be revised to:
PHP Code:
while ($MyRow=DB_fetch_array($result)){
    if (isset(
$_SESSION['Request']->Location) AND $_SESSION['Request']->Location==$myrow['loccode']){
        echo 
'<option selected value="'$MyRow['loccode'], '">'$MyRow['loccode'], ' - 'htmlspecialchars($MyRow['locationname'], ENT_QUOTES,'UTF-8'), '</option>';
    } else {
        echo 
'<option value="'$MyRow['loccode'], '">'$MyRow['loccode'], ' - 'htmlspecialchars($MyRow['locationname'], ENT_QUOTES,'UTF-8'),  '</option>';
    }




RE: bug: Internal stock request - TurboPT - 04-09-2020

Thanks for the report on that, ge_zhiyuan!

Will get that change applied later today.


RE: bug: Internal stock request - TurboPT - 04-09-2020

Hello, ge_zhiyuan!

Checking this file tonight, it looks like that was fixed back in September, along with some other variable name issues.
Thanks for the report, was worth checking!

GitHub file reference

GitHub commit diff reference