Find match from text-file and mark it as found for all matches in table.

[masimies]

Hi,
Tried so much combinations with this issue, that my eyes are pulling out....

So, i like to check, if there is match for some values in table against uploaded text-file. Actually checking if customer was paid his invoices.
This code below is working generally, but mark only first row what match. Not continuing to next row.
Yes, preg_match and stripos works like so, and stops to first match. I tried some foreach-functions, but after results didn't got the working resolution. Just got echoed values, without getting them inside update query.

Need for some help with this. If you have some good proposals for get this working.

Here is the code, what should be fixed to correct way:

if(isset($_POST['scrstr'])) {
$scrstr=$_POST['scrstr'];
$file=file_get_contents('upload/'.$scrstr.'');
$filename='upload/'.$scrstr.'';
$fetch = mysqli_query($db,"SELECT * FROM $invoicingtable WHERE paid<>'1'");
while($row = mysqli_fetch_array($fetch)) {
$id=$row['id'];
$invoicenr=$row['invoicenr'];
$custname=$row['custname'];
$custname = preg_replace('/\s+/', '', $custname);
$totalsum=$row['totalsum'];
$totalsum = preg_replace('/\./', '', $totalsum);
if(preg_match('/'.$invoicenr.'/', $file)) { $val1res=1; $updfoundinv = mysqli_query($db,"UPDATE $invoicingtable SET paidstat='$filename' WHERE id='$id'"); }
if(preg_match('/'.$custname.'/', $file)) { $val2res=1; }
if(preg_match('/'.$totalsum.'/', $file)) { $val3res=1; }
if($val1res==1 AND $val2res==1 AND $val3res==1) { $updpaidstat = mysqli_query($db,"UPDATE $invoicingtable SET paid='1', paidstat='$filename' WHERE id='$id'"); }
header('Location:invoicedopen.php');
mysqli_close($db); } }

[NogDog]

If you supply a variable as the optional 3rd parameter to preg_match(), it will be populated with an array of all matches found. Perhaps that's what you're looking for?

PS: I reformatted it in my IDE so I had some chance of following along:

if(isset($_POST['scrstr'])) {
    $scrstr=$_POST['scrstr'];
    $file=file_get_contents('upload/'.$scrstr.'');
    $filename='upload/'.$scrstr.'';
    $fetch = mysqli_query($db,"SELECT * FROM $invoicingtable WHERE paid<>'1'");
    while($row = mysqli_fetch_array($fetch)) {
        $id        = $row['id'];
        $invoicenr = $row['invoicenr'];
        $custname  = $row['custname'];
        $custname  = preg_replace('/\s+/', '', $custname);
        $totalsum  = $row['totalsum'];
        $totalsum  = preg_replace('/\./', '', $totalsum);
        if(preg_match('/'.$invoicenr.'/', $file)) {
            $val1res=1; $updfoundinv = mysqli_query($db,"UPDATE $invoicingtable SET paidstat='$filename' WHERE id='$id'");
        }
        if(preg_match('/'.$custname.'/', $file)) {
            $val2res=1;
        }
        if(preg_match('/'.$totalsum.'/', $file)) {
            $val3res=1;
        }
        if($val1res==1 AND $val2res==1 AND $val3res==1) {
            $updpaidstat = mysqli_query($db,"UPDATE $invoicingtable SET paid='1', paidstat='$filename' WHERE id='$id'");
        }
        header('Location:invoicedopen.php');
        mysqli_close($db);
    }
}  

[masimies]

Yes, but these array syntaxes are quite hard to me. I allready got lost when looking at several different methods to produce those.
This method was closest to work, but didn't manage to apply it to my context:

$file = file_get_contents("filename.txt");
$searchstrings = 'lol, hello, ****, text';
$breakstrings = explode(',',$searchstrings);

foreach ($breakstrings as $values){
if(!strpos($file, $values)) {
echo $values." string not found!\n";
} else {
echo $values." string Found!\n";
}
}

Actuly, there is need only for 2-variables. $invoicenr and $totalsum, because it is possible, that someone else pay the bill. So invoicenr and totalsum are more unique for searching.

[NogDog]

Wait a second: could it simply be because your header() and mysqli_close() are inside the fetch loop? That means you closing it down after the first result row is processed. (That's where consistent indenting can help. 😉 )

[masimies]

Hahahaa! You hawk-eye!
That was the reason for not looping. Added closing to wrong position when removed one else-row.
This is not new for me. I just cannot see the simpliest error sometimes when tired for coding hours.

But, there is still problem. Now code finds too well invoicenr. Even the number isn't in text-file. So the problem has to be in preg_match syntax:
if(preg_match('/'.$invoicenr.'/', $file))
The data in text-file is formatted like this. There is number-rows without spaces or other characters:
161234510000009810000
16-digits for invoicenumber and rest of for sum. There could be more digits around them or between.
So preg_match should be found excat numbers by variable values given.

[masimies]

Yep, the solution so far is, to use only invoicenr. Code finds well these numbers individually, but trying to do anything else,results go in incorrect state. Even numbers what are not found in text-file, are handled as found. This is working as it should, but it is leaving behind manual work, for open manually text-file, and find the totalsum. And after that, confirm invoice as paid...

if(isset($POST['scrstr'])) {
$scrstr=$POST['scrstr'];
$file=file_get_contents('upload/'.$scrstr.'');
$filename='upload/'.$scrstr.'';
$fetch = mysqli_query($db,"SELECT * FROM $invoicingtable WHERE paid<>'1'");
while($row = mysqli_fetch_array($fetch)) {
$id=$row['id'];
$invoicenr=$row['invoicenr'];
if(preg_match('/'.$invoicenr.'/', $file)) { $updfoundinv = mysqli_query($db,"UPDATE $invoicingtable SET paidstat='$filename' WHERE id='$id'"); } }
header('Location:invoicedopen.php');
mysqli_close($db); }

I have to investigate more, how the totalsum can be found also in one loop, together with invoicenumber.

[masimies]

This, reading the text-file directly did not work. So i change this totally.
I insert first the text-file to table, row by row. In same time, script marks open invoices, what found in text-file as found in this file-import. Now the invoice-number, paid amount and paid-date are in one row in table. After this, it is easier to check the invoice with invoice-number, but have to do it invoice by invoice. Not in loop, all at once.

Grabbing date from text was one new problem, but solved with:
http://board.phpbuilder.com/showthread.php?10395065-Get-the-date-from-txt-line&p=11057229