OLD Super Orders 2.0 (See v3.0 thread instead)

The source of the error page displayed after I click on "confirm order" button in checkout says the following:

<br />
<b>Warning</b>:  str_repeat() [<a href='function.str-repeat'>function.str-repeat</a>]: Second argument has to be greater than or equal to 0. in <b>C:\xampp\htdocs\PAC\shop\includes\modules\payment\cc.php</b> on line <b>259</b><br />

<br /><br />Email Error: SMTP Error: Data not accepted.<br /><br />
```The section in the cc.php file which includes the offending line says:

line256 $len = strlen($_POST['cc_number']);
line257 $this->cc_middle = substr($_POST['cc_number'], 4, ($len-8));
line258 if ( (defined('MODULE_PAYMENT_CC_EMAIL')) && (zen_validate_email(MODULE_PAYMENT_CC_EMAIL)) ) {
line259 $order->info['cc_number'] = substr($_POST['cc_number'], 0, 4) . str_repeat('X', (strlen($_POST['cc_number']) - 8)) . substr($_POST['cc_number'], -4);
line260 }
line261 }

Hope this helps!

BlindSide, bravo to your great mod - I love it!

I have a question regarding modification of the Packing Slip - I would like to show the total shipped item number at the end of the listing of purchased items, something like e.g.:

Total Shipped: 5 ITEMS

In the event of spilt slip, it will show whatever total the shipment is.

I realized it could be just a line of code to insert in the super_packingslip.php but I am not good at coding. Hope you would help me here! Many thanks!!!

so that I don't have to read 63 pages

does this mod support spliting orders with a quantity option?

TIA

TBR, try out this one-line change in super_orders_functions.php. Find the first commented out line, where it is replaced with a line that seems to work better:

function zen_db_scrub_in($string, $strip_tags = false) {
  if ( $string == '""' || $string == "''" || strcasecmp($string, 'null') == 0 || strcasecmp($string, 'now()') == 0 ) {
    return $string;
  }
  elseif (is_string($string)) {
    $string = trim(stripslashes($string));
    // $string = nl2br($string);  // replaced line
    $string = str_replace(array("\r\n", "\r", "\n"), "<br />", $string);
    if ($strip_tags) {
      $string = strip_tags($string);
    }
    //$string = mysql_real_escape_string($string);
    $string = mysql_escape_string($string);
    return $string;
  }
  elseif (is_array($string)) {
    reset($string);
    while (list($key, $value) = each($string)) {
      if (!is_numeric($value)) $string[$key] = zen_db_scrub_in($value);
    }
    return $string;
  }
  else {
    return $string;
  }
}

Hope that works for you.

Two quick questions about Super Orders 2.0 (which is a great mod!)

1. When I print out packing slips, why do some of them include the image of the product, and some do not include the images? I can't figure out any rhyme or reason on what makes the difference?

2. In the Batch Status Update, the comment box is really small (like 1 character by 2 characters). Is there anything I can do about that, to make it so I can edit something that I've pasted there?

Thanks!

LW

Hi,

I have successfully installed the super order mod (looks great), but when i place a test order and go to: Admin--->Customers---->Super orders then look at the order placed (which was by purchase order) then add a PO to the order, then proceed to pay it by adding the full amount due, the page then updates saying that they still owe X amount even though they have paid the full amount, so I have tried to add the amount due as well and the amount changes but there is still an amount due. I am confused why its doing this as there always appears to be an amount due. I hope I have explained this ok, I would really appreciate your help.

Thanks,

Ian

Anyone know how to remove this mod from the database completely, it has caused nothing but chaos the client has no clue what he was doing and I am wondering if anyone has ever uninstalled successfully.

i keep getting this error when im trying to apply the sql patch:

1062 Duplicate entry 'CA' for key 2
in:
[INSERT INTO so_payment_types VALUES (NULL, 1, 'CA', 'Cash');]
If you were entering information, press the BACK button in your browser and re-check the information you had entered to be sure you left no blank fields.

i tried both with the file and with copy and past the query

anyine knows what to do....?

im installing the SO 2.0,, and i am in the 7th point,, --->

Open includes/classes/order.php, then find this line:

$insert_id = $db->Insert_ID();

Immediately after it, add the following code:

// BEGIN Super Orders edit

// add CC data as a line item to SO payment system

if (zen_not_null($this->info['cc_type']) || zen_not_null($this->info['cc_owner']) || zen_not_null($this->info['cc_number'])) {

require(DIR_WS_CLASSES . 'super_order.php');
$so = new super_order($insert_id);
$so->cc_line_item();
}
// END

Super Orders edit

The comments will ensure you know what this is for the next time you need to upgrade.

i can't find this line: $insert_id = $db->Insert_ID();

what to do with this now?

can anybody help?

regards

sorry i forgot to say that im using version 1.3.7

and one thing also i found with it is in:

Edit the order.php class (admin side)
The order class does a great job of collecting everything you need for an
order...almost.

The class completely lacks any ID numbers for the
information it represents, which is really useful information to have.

Super Orders uses the customers_id and products_id in a few places, so we
need to add them to the order class.

a. Open admin/includes/classes/order.php,

find the line that reads as follows:

    ...

all are done in the new version

regards

saher

I love super orders.

Is there a way to take off " Orders " so that I default to Super Orders all the time.
Or just make it so that the "Orders" tab is super orders.

Super Orders is fantastic - thanks so much for this mod. Maybe this is too obvious for me to figure out, but is there a way to have the order comments show up on the batch print? We occasionally have people make a last minute request to change the shipping address, or to ship 2 orders together. We note these in the comments section, but this is not printing in the batch print. Thanks!

BouncerFL:

find, in super_orders.php:

if ($order->totals[$i]['class'] == 'ot_shipping') {
$format_shipping = explode(" (", $order->totals[$i]['title'], 2);
$clean_shipping = rtrim($format_shipping[0], ":");
$display_title = $clean_shipping . ':';
}
else {
$display_title = $order->totals[$i]['title'];
} */

> ```php
$display_title = $order->totals[$i]['title'];

When I make these changes in any of the SO files (orders, invoice, packingslip) in order to get the shipping method to show (not just the carrier) the page just comes up blank. Has anyone else encountered this problem?

Please disregard the previous post. I had a mistake in commenting out of the section of code mentioned. The problem is resolved. Thanks for the help.

Has anyone had any problems when doing a batch status update for notifying the customer. I want to be able to send a batch message to all customers who's status has changed with a comment. When I do this, i get the success update at the top of the screen, but no email is sent out, and a red X is on notify customer box, any ideas?

Thanks
Samantha

Sam_uk:

Has anyone had any problems when doing a batch status update for notifying the customer. I want to be able to send a batch message to all customers who's status has changed with a comment. When I do this, i get the success update at the top of the screen, but no email is sent out, and a red X is on notify customer box, any ideas?

Known issue with no known resolution :(

Would love to know when/if anyone has ever figured it... this is a great feature and apparently it works for some and not for others. I would be willing to help collect data for comparison if anyone is able to help figure out what the difference is between mine, yours, and those that work... :smile:

redheads:

Known issue with no known resolution :(

Would love to know when/if anyone has ever figured it... this is a great feature and apparently it works for some and not for others. I would be willing to help collect data for comparison if anyone is able to help figure out what the difference is between mine, yours, and those that work... :smile:

I ain't no programmer, but I "think" this might get it working (old version):

super_batch_status.php

function batch_status($oID, $status, $comments, $notify = 0, $notify_comments = 0) {
        global $db, $messageStack;
        [B]global $orders_status_array; // (bug fix)[/B]

HTH :smile:

DogTags:

function batch_status($oID, $status, $comments, $notify = 0, $notify_comments = 0) {
global $db, $messageStack;
[B]global $orders_status_array; // (bug fix)[/B]

> 
> HTH :smile:

I am guessing this is something you used to repair the problem... Thanks for posting it, I know this has been an issue on this mod/thread for a while. I will give it a try and report back next time I need to do a batch-send! 
:-)

Thanks for the bug fix try, Dogtags...
no change here when applied, but it was certainly worth a shot and I appreciate the idea.

I add the message, tell it to update, screen updates with
'Status has been updated successfully!'
in the header for as many orders as I have selected. Looks good thus far... then... In the order, a red X where 'customer notified' and no email is actually generated.

Thanks for the try! If anyone has ever successfully gone from no emails generated to a work-around, please tell us here :smile:

Dogtags thanks for trying, redheads i know what you mean i am getting the same problem as you.
Its a great feature to use, if anyone has got this to work or a workaround, will be really greatful

Thanks
Samantha