FraudLabs Pro Fraud Prevention (Support Thread)

[allmart]

I am using Zen Cart 1.5.5e the instructions included in the instructions of Fraudlabs Pro Version: 1.0.8 to modify admin/orders.php indicate to
Search for:
<div class="row noprint">
<div class="formArea">
<?php echo zen_draw_form('statusUpdate', FILENAME_ORDERS, zen_get_all_get_params(['action']) . 'action=update_order', 'post', 'class="form-horizontal"', true); ?>

and replace with:

<?php include(DIR_WS_MODULES . 'fraudlabspro/summary.php'); ?>
<div class="row noprint">
<div class="formArea">

but my file shows:

<!-- BEGIN TY TRACKER 3 - DISPLAY TRACKING ID IN COMMENTS TABLE ------------------------------->
<td class="dataTableHeadingContent smallText" valign="top" width="23%"><strong><?php echo TABLE_HEADING_TRACKING_ID; ?></strong></td>
<!-- END TY TRACKER 3 - DISPLAY TRACKING ID IN COMMENTS TABLE ------------------------------------------------------------>
<!-- Begin Ty Package Tracker Modification (Minor formatting changes) //-->
<td class="dataTableHeadingContent smallText" valign="top" width="40%"><strong><?php echo TABLE_HEADING_COMMENTS; ?></strong></td>
<!-- End Ty Package Tracker Modification (Minor formatting changes) //-->

not sure what to replace

[retched]

Suggested change for anyone running this on PHP 8.

1. Open /admin/includes/modules/fraudlabspro/summary.php
2. Delete the following line:
   

   $fraudlabspro_enabled = $db->Execute('select configuration_value FROM ' . TABLE_CONFIGURATION . " WHERE configuration_key = 'FRAUDLABSPRO_ENABLED'"); 

3. Find the following line (should be the next line):
   

   if ($fraudlabspro_enabled == 'true') { 

   and change it to:
   

   if (FRAUDLABSPRO_ENABLED == 'true') { 

Trying to pull this value using the $db-> results in an error (using ->Execute seems to generate the result as a MySQL Result field that cannot be assigned or read like a string). My fix to this? Use the raw DEFINE that is generated from all Configuration Keys in the database instead.

Hope this helps.

[retched]

Another bug fix. (Might create an update at this point...)

Open /admin/include/modules/fraudlabspro/summary.php


Change line 187 (or look for this block) to a </div>.

            echo '  <div class="form-group text-right">';
            echo '    <button id="btn-approve" class="btn btn-primary">Approve</button>';
            echo '    <button id="btn-reject" class="btn btn-primary">Reject</button>';
            echo '    <button id="btn-blacklist" class="btn btn-primary">Blacklist</button>';
            echo '  <div>';
            echo '</form>'; ?> 

and change that <div> to a </div> at the end.

[retched]

Another bug fix. (Might create an update at this point...)

Open /admin/include/modules/fraudlabspro/summary.php


Change line 187 (or look for this block) to a </div>.

            echo '  <div class="form-group text-right">';
            echo '    <button id="btn-approve" class="btn btn-primary">Approve</button>';
            echo '    <button id="btn-reject" class="btn btn-primary">Reject</button>';
            echo '    <button id="btn-blacklist" class="btn btn-primary">Blacklist</button>';
            echo '  <div>';
            echo '</form>'; ?> 

and change that <div> to a </div> at the end.

Submitted this change as version 1.4.1 to the Plugin database

[swguy]

Rather than

if (FRAUDLABSPRO_ENABLED == 'true') {


I would suggest

if (defined('FRAUDLABSPRO_ENABLED') && FRAUDLABSPRO_ENABLED == 'true') {

for PHP 8 use.

[retched]

Rather than

if (FRAUDLABSPRO_ENABLED == 'true') {


I would suggest

if (defined('FRAUDLABSPRO_ENABLED') && FRAUDLABSPRO_ENABLED == 'true') {

for PHP 8 use.

Noted that change and another bug I just found with the email domain variable too. I'm also working on a callback script but I don't want to submit it until I'm sure it works. Also working on a "fixed" (read: prettified) amount field to be properly format and not just a raw pull from the database complete with the float. (Started a thread in "Contribution guidelines" about it if anyone wants to help.)

[lat9]

Even better, IMO:

if (defined('FRAUDLABSPRO_ENABLED') && FRAUDLABSPRO_ENABLED === 'true') {

[retched]

Ready with those total changes but I think I have to wait for my other submission to be made before I can submit it.