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

Thanks a lot man! i thought what it means by tracking orders is like a search order. :D

dale88:

Thanks a lot man! i thought what it means by tracking orders is like a search order. :DYou're welcome.. BTW, NOT a man, but a WOman..

DivaVocals,
Thanks for all the Work and Support you have done and still do to patch up Super Orders 2.0 along with everyone else that has contributed. I think this is a Great Mod by BlindSide.
I noticed that starting in REV 48, you took out ( I think it was you) the modify button and left these comments in it's place.

<!-- The following line has been commented out and replaced with the line below it. The modify/update function is not working so I removed the "Modify" button. proper accounting rules should not really allow "modifying" of Payments/refunds/POs. An adjustment is the correct way to "modify" a payment/refund/PO. I left the original code in place in case anyone wants to fix and reactivate this feature. I suggest that if you fix this that you should also share the fix in the Super Orders support thread.-->

<!--td align="right"><?php $so->button_update('payment', $so->payment[$a]['index']); $so->button_delete('payment', $so->payment[$a]['index']);?></td--> <td align="right"><?php $so->button_delete('payment', $so->payment[$a]['index']);?></td>

So, here is a fix for those that are interested.

The reason MODIFY does not work is because of a Security Patch ```
admin/includes/init_includes/init_security_patch_v138

It removes the ability to use the word **update** in a **$_GET** statement.

I agree, adjustments are the way to go, but what if you entered a check number incorrectly or something similar, I myself would rather Modify the payment.

That being said, There is a way to make Modify work again. You will need to edit two files. They are:

admin/includes/classes/super_order.php
Look around Line # 358 for:
// Displays a button that will open a popup window to update an existing payment entry
// This code assumes you have the popupWindow() function defined in your header!
// Valid $payment_mode entries are: 'payment', 'purchase_order', 'refund'
function button_update($payment_mode, $index) {
echo ' <a href="javascript:popupWindow('' .
zen_href_link(FILENAME_SUPER_PAYMENTS, 'oID=' . $this->oID . '&payment_mode=' . $payment_mode . '&index=' . $index . '&action=[B]update[/B]', 'NONSSL') . '', 'scrollbars=yes,resizable=yes,width=400,height=300,screenX=150,screenY=100,top=100,left=150')">' .
zen_image_button('btn_modify.gif', sprintf(ALT_TEXT_UPDATE, str_replace('_', ' ', $payment_mode))) . '</a>';
}
and
/admin/super_payments
Look around Line# 450 for:
case '[B]update[/B]':
$index = $_GET['index'];

    echo '  ' . zen_draw_form('update', FILENAME_SUPER_PAYMENTS, '', 'get', '', true) . NL;
    echo '    ' . zen_draw_hidden_field('action', $action) . NL;
    echo '    ' . zen_draw_hidden_field('process', 1) . NL;
    echo '    ' . zen_draw_hidden_field('payment_mode', $payment_mode) . NL;
    echo '    ' . zen_draw_hidden_field('oID', $so->oID) . NL;

Change the word **update** in both files to anything you want to use as long as you use the same word in both files. 
Remember to change the lines back in

admin/super_orders.php

starting around line #'s  477, 495, 539, 564, 582 and 637. (This is in REV 48a of Super Orders 2.0 and Zen-Cart 138a)
Copy your updated files to your store and Modify will now work.

REMEMBER: Always have a BACKUP of ANY file that you change.

I did indeed comment out the modify function.. IMO there was no point in leaving in a feature which did not work. I left the comments about why I disabled the feature, and I'm so glad you followed my suggestion and shared the fix with the rest of the class.. :clap:

I'll try and get some time soon to test, and if all goes well with testing, I'll bundle this up with and submit it to the downloads section..

JTheed:

DivaVocals,
Thanks for all the Work and Support you have done and still do to patch up Super Orders 2.0 along with everyone else that has contributed. I think this is a Great Mod by BlindSide.
I noticed that starting in REV 48, you took out ( I think it was you) the modify button and left these comments in it's place.

So, here is a fix for those that are interested.

The reason MODIFY does not work is because of a Security Patch ```
admin/includes/init_includes/init_security_patch_v138

> 
> I agree, adjustments are the way to go, but what if you entered a check number incorrectly or something similar, I myself would rather Modify the payment.
> 
> That being said, There is a way to make Modify work again. You will need to edit two files. They are:
> ```
admin/includes/classes/super_order.php
Look around Line # 358 for:
  // Displays a button that will open a popup window to update an existing payment entry
  // This code assumes you have the popupWindow() function defined in your header!
  // Valid $payment_mode entries are: 'payment', 'purchase_order', 'refund'
  function button_update($payment_mode, $index) {
    echo ' <a href="javascript:popupWindow(\'' .
    zen_href_link(FILENAME_SUPER_PAYMENTS, 'oID=' . $this->oID . '&payment_mode=' . $payment_mode . '&index=' . $index . '&action=[B]update[/B]', 'NONSSL') . '\', \'scrollbars=yes,resizable=yes,width=400,height=300,screenX=150,screenY=100,top=100,left=150\')">' .
    zen_image_button('btn_modify.gif', sprintf(ALT_TEXT_UPDATE, str_replace('_', ' ', $payment_mode))) . '</a>';
  }
and 
/admin/super_payments
Look around Line# 450 for:
      case '[B]update[/B]':
        $index = $_GET['index'];

        echo '  ' . zen_draw_form('update', FILENAME_SUPER_PAYMENTS, '', 'get', '', true) . NL;
        echo '    ' . zen_draw_hidden_field('action', $action) . NL;
        echo '    ' . zen_draw_hidden_field('process', 1) . NL;
        echo '    ' . zen_draw_hidden_field('payment_mode', $payment_mode) . NL;
        echo '    ' . zen_draw_hidden_field('oID', $so->oID) . NL;
```Change the word **update** in both files to anything you want to use as long as you use the same word in both files. 
> Remember to change the lines back in
> ```
 admin/super_orders.php 
```starting around line #'s  477, 495, 539, 564, 582 and 637. (This is in REV 48a of Super Orders 2.0 and Zen-Cart 138a)
> Copy your updated files to your store and Modify will now work.
> 
> REMEMBER: Always have a BACKUP of ANY file that you change.

Batch Shipping Charges

Anyway to add a column to see the Shipping Charges when using the Batch functions?

Either add a column or create a function to see just the shipping charges of the orders. This would be good to use to forecast shipping trends or simple accounting.

I am able to import the batch data to the application Numbers and to keep my accounting up to date on a monthly basis.

I'm getting an HTML validation error on the shipping label page. The error message is:

Line 4, Column 73: end tag for "meta" omitted, but OMITTAG NO was specified

<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">

You may have neglected to close an element, or perhaps you meant to "self-close" an element, that is, ending it with "/>" instead of ">".

The offending line seems to be:

<meta http-equiv="Content-Type" content="text/html; charset=<?php echo CHARSET; ?>" />

Originally it didn't have the "/" before the ">" so I added it, but am getting the same error. Any ideas?

Thanks.

mzimmers:

I'm getting an HTML validation error on the shipping label page. The error message is:

The offending line seems to be:

It could be the ending quotes on content="text/html; <-- "

Try it this way

 content="text/html"; charset="<?php echo CHARSET; ?>" 

I assume you didn't mean to imply I should remove the "/>" at the end.

Thanks for the suggestion, but it didn't seem to fix it.

Yes, you are correct, I didn't mean to have omit anything, just add the quotes.

What are you using to Validate the code?

I'm using whatever happens to be built into developer...I believe it's http://validator.w3.org.

Do you suspect the validator might be errant?

I just want to duplicate what you are doing is all.

You probably can't duplicate it...there's an implicit login to displaying the page in question.

I also just validated another page with the EXACT same meta line, and it passed. I suspect there's something wrong in one of the php files being required above this line in the file. So, don't expend too much energy trying to debug that line.

I'll post back if I find something.

I found this to be a missing /> in the login.php file. Look on line 56 and add the /> and the error will go away.

JT

Very good sleuthing, JTheed. There are also a couple of input statements in there that are also missing the "/" on their closing.

Odd how the validator reported this (or didn't report it, depending on your point of view).

In any event, it now validates. Thanks again for the help.

DivaVocals:

The other issue is that the "Back" button on the edit_orders.php page only takes you back to the the Super Orders order list page (super_orders.php). It should take you back to the detail page of the order you were editing.. It's minor and I can live with it..

Gonna keep messing around and try to figure out how to correct this, but for now the major edits to join these two apps seems to work.. Thought I'd share.. Contemplating including this as an optional edit for Super Orders.. (if I can get the back button to work correctly) I'm also contemplating an optional edit for merging Admin Comments with Super Orders as well..
To fix this Back button issue:

In edit_orders.php on line 659, find:

            <td class="pageHeading" align="right"><?php echo '<a href="' . zen_href_link(FILENAME_ORDERS, zen_get_all_get_params(array('action'))) . '">' . zen_image_button('button_back.gif', IMAGE_BACK) . '</a>'; 

and replace with:

            <td class="pageHeading" align="right"><?php echo '<a href="' . zen_href_link(FILENAME_SUPER_ORDERS, 'page=1&oID=' . $oID . '&action=edit', 'NONSSL') . '">' . zen_image_button('button_back.gif', IMAGE_BACK) . '</a>'; 

virtualomega:

To fix this Back button issue:

In edit_orders.php on line 659, find:

        <td class="pageHeading" align="right"><?php echo '<a href="' . zen_href_link(FILENAME_ORDERS, zen_get_all_get_params(array('action'))) . '">' . zen_image_button('button_back.gif', IMAGE_BACK) . '</a>'; 

> 
> and replace with:
> ```
            <td class="pageHeading" align="right"><?php echo '<a href="' . zen_href_link(FILENAME_SUPER_ORDERS, 'page=1&oID=' . $oID . '&action=edit', 'NONSSL') . '">' . zen_image_button('button_back.gif', IMAGE_BACK) . '</a>'; 

Thanks for the fix. Remember not to overwrite the ?></td> at the end of the line.

JT

Wow!! Thanks soooooooooo much.... Absolutely AWESOME!!!!!:clap:

BTW, I will be putting together a new update for Super Orders with this and other foxes that have been posted or sent to me.. I'm a little tight on time right now but it will be posted in the next week or two.. Thanks to all who have posted fixes/updates.. You all ROCK!!!

virtualomega:

To fix this Back button issue:

In edit_orders.php on line 659, find:

        <td class="pageHeading" align="right"><?php echo '<a href="' . zen_href_link(FILENAME_ORDERS, zen_get_all_get_params(array('action'))) . '">' . zen_image_button('button_back.gif', IMAGE_BACK) . '</a>'; 

> ```
            <td class="pageHeading" align="right"><?php echo '<a href="' . zen_href_link(FILENAME_SUPER_ORDERS, 'page=1&oID=' . $oID . '&action=edit', 'NONSSL') . '">' . zen_image_button('button_back.gif', IMAGE_BACK) . '</a>'; 

In \admin\edit_orders.php, I have the above code in two places - Lines 1048 and 1608.

Do I have the right file? As the original reference says Line 659, my \admin\edit_orders.php must be substantially different.

Nevertheless, do I apply the changes in both places.

Best Regards

This is what I get for doing drive-by postings before heading off to work in the mornings.. This is not a SUPER ORDERS fix, but an EDIT ORDERS fix..

I will be happy to cross post this on the Edit Orders support thread if it hasn't been done already, but let me be clear that this code will NOT be included in the next Super Orders update.. However, what I will include in the next Super Orders update is an update to the README file with a link to the Edit Orders fix posted above.

DivaVocals:

Wow!! Thanks soooooooooo much.... Absolutely AWESOME!!!!!:clap:

BTW, I will be putting together a new update for Super Orders with this and other foxes that have been posted or sent to me.. I'm a little tight on time right now but it will be posted in the next week or two.. Thanks to all who have posted fixes/updates.. You all ROCK!!!

While cross posting the fix here is absolutely cool for those of us using both mods together.. We really should (myself included) take any FURTHER discussion of this fix over to the Edit Orders support thread as it really is an EDIT ORDERS fix..

dw08gm:

In \admin\edit_orders.php, I have the above code in two places - Lines 1048 and 1608.

Do I have the right file? As the original reference says Line 659, my \admin\edit_orders.php must be substantially different.

Nevertheless, do I apply the changes in both places.

Best Regards

Ma foi.

I notice an Edit Orders support thread has recently been setup here commencing 27 January 2010. !!!

http://www.zen-cart.com/forum/showthread.php?t=146652

If you post elsewhere, could you please inform.

TIA