Edit Orders v3.0 for ZC 1.3.9 [Support Thread]

[clydejones]

thank you for helping with this.

i did replace my admin > edit orders.php with the file in the zip and got a blank page...


I think I'll just sit tight until a formal update to this mod comes along.

I appreciate everyone's input...and just knowing how edit orders is meant to work so I don't have unreasonable expectations.

Try this:

edit_orders2.zip

[DivaVocals]

thank you for helping with this.

i did replace my admin > edit orders.php with the file in the zip and got a blank page...


I think I'll just sit tight until a formal update to this mod comes along.

I appreciate everyone's input...and just knowing how edit orders is meant to work so I don't have unreasonable expectations.

Well I suggest you don't do that.. I can only work on this in my SPARE time (this means no PAID client work and NO day job obligations)

It's MORE THAN likely that all I will do anyway is test, confirm, and submit Clyde's modifications posted above..

That said, one of the things I looked at doing one day when MY TIME permits, is to synchronize the Edit Orders comments code to look/act like the default orders.php. (for the record, this is ONE of the big changes we made that will be seen in the next Super Orders release -- AND BEFORE ANYONE ASKS -- when I get time to finalize it, I will release it -- I work on it a little bit at a time -- should be soon..)

It would be a CHALLENGE for me to do this since I am NOT really a coder, but I am not afraid!!! This is also why it won't happen tomorrow, but it is on my "to-do" list (or should I say my "to-try list")..

[athena]

Hi clyde

ok..so this did send an email copy to spec address in admin > configuration > email options > send copy to admin order status etc.

but it was just this text:

Subject line: [ORDERS STATUS] Order Update #6817

This email address was given to us by you or by one of our customers. If you
feel that you have received this email in error, please send an email to
[email protected]
This e-mail is sent in accordance with the US CAN-SPAM Law in effect
01/01/2004. Removal requests can be sent to this address and will be honored
and respected.

The email that came to the customer had just "order update" in the subject line.

The email looked correct in every way and is email_template_order_status.html

but only the comments section was filled in.

then below that:

EMAIL_TEXT_STATUS_UPDATED
EMAIL_TEXT_STATUS_LABEL
Please reply to this email if you have any questions.

[clydejones]

Hi clyde

ok..so this did send an email copy to spec address in admin > configuration > email options > send copy to admin order status etc.

but it was just this text:



The email that came to the customer had just "order update" in the subject line.

The email looked correct in every way and is email_template_order_status.html

but only the comments section was filled in.

then below that:

I'll check some more: looks like the language file may need to be updated as well.

[athena]

here is the code that is not cooperating:

there is a blank line below the comments

$html_msg['EMAIL_TEXT_STATUS_UPDATED'] = str_replace('\n','', EMAIL_TEXT_STATUS_UPDATED);
            $html_msg['EMAIL_TEXT_STATUS_LABEL'] = str_replace('\n','', sprintf(EMAIL_TEXT_STATUS_LABEL, $orders_status_array[$status] ));
            $html_msg['EMAIL_TEXT_NEW_STATUS'] = $orders_status_array[$status];

complete email without header:

Dear Tommy Tester Tester,

Order Number: 6817

Detailed Invoice

Date Ordered: Friday 22 January, 2010

Comments:
testing if this will send copy email to me



EMAIL_TEXT_STATUS_UPDATED

EMAIL_TEXT_STATUS_LABEL

Please reply to this email if you have any questions.

My Shop

[clydejones]

here is the code that is not cooperating:

there is a blank line below the comments

$html_msg['EMAIL_TEXT_STATUS_UPDATED'] = str_replace('\n','', EMAIL_TEXT_STATUS_UPDATED);
            $html_msg['EMAIL_TEXT_STATUS_LABEL'] = str_replace('\n','', sprintf(EMAIL_TEXT_STATUS_LABEL, $orders_status_array[$status] ));
            $html_msg['EMAIL_TEXT_NEW_STATUS'] = $orders_status_array[$status];

complete email without header:

Dear Tommy Tester Tester,

Order Number: 6817

Detailed Invoice

Date Ordered: Friday 22 January, 2010

Comments:
testing if this will send copy email to me



EMAIL_TEXT_STATUS_UPDATED

EMAIL_TEXT_STATUS_LABEL

Please reply to this email if you have any questions.

My Shop

Hope this solves it.

unzip the package - rename the YOUR_ADMIN folder to match the name of your admin folder - upload/FTP to your server.

edit_orders3.zip

[DivaVocals]

and let me add this.. I'm still not sure this is an actual "issue".. Though I've sorta followed all of this, I lost track of what is wrong or being fixed.. Sorry -- with all of the back to back to back posts, there's now THREE pages of stuff to sort through on this one "issue". So it's been difficult to get a real sense of "what's really WRONG here"..

On a high level, I do understand what Clyde did, but until I look at it in detail, I'm not completely sure that there is an actual issue here to be "fixed"..

All that to say I will still take a look at what has been done, test and update if necessary.. (Clyde I hope you don't mind that I may poke at ya with a few questions when I can take a look at this in depth)

Well I suggest you don't do that.. I can only work on this in my SPARE time (this means no PAID client work and NO day job obligations)

It's MORE THAN likely that all I will do anyway is test, confirm, and submit Clyde's modifications posted above..

That said, one of the things I looked at doing one day when MY TIME permits, is to synchronize the Edit Orders comments code to look/act like the default orders.php. (for the record, this is ONE of the big changes we made that will be seen in the next Super Orders release -- AND BEFORE ANYONE ASKS -- when I get time to finalize it, I will release it -- I work on it a little bit at a time -- should be soon..)

It would be a CHALLENGE for me to do this since I am NOT really a coder, but I am not afraid!!! This is also why it won't happen tomorrow, but it is on my "to-do" list (or should I say my "to-try list")..

[athena]

Clyde,

THis appears to be a success! Above link labeled--edit_orders3.zip--

admin > Edit_orders.php now appears to be not only cooperating with but also sending email_template_order_status.html when an order is updated and is successfully sending a copy of the email to the additional email address specified.

All the fields in email_template_order_status.html are filled out and the url link to the invoice has become a text link. All links work.

------------

Thank you so much for your no fuss response. I hope this helps others with this issue.

Thanks DivaVocals for encouraging me to hang in there and work with Clyde on this.


MY PROCESS:

Uploaded Clydes latest changes, namely "edit_orders3.zip" as directed to admin > edit_orders.php and admin > includes > languages > english > edit_orders.php

TEST:

*logon to admin
*send copy of admin orders status emails to - Status........."1"
*go to customers > orders > "Mr. test customer" > edit button to edit_orders.php > message posted with status "update"

RESULT:

*CUSTOMER EMAIL: edit_orders.php successfully sent customer the email_template_order_status.html rather than the email_template_default.html;

*all fields accounted for,

*text link in email works!

Admin copy - Text message complete:

My Shop
------------------------------------------------------
Order Number: 6817
Detailed Invoice: Click here for your Detailed Invoice
Date Ordered: Friday 22 January, 2010

Comments:
testing the April 14 Clyde changes, including
languages/english/edit_orders.php should send email to admin copy and should
send the email_template_order_status.html with all fields accounted for.


Your order has been updated to the following status:
New status: Updated

Please reply to this email if you have any questions.

This email address was given to us by you or by one of our customers. If you
feel that you have received this email in error, please send an email to
[email protected]
This e-mail is sent in accordance with the US CAN-SPAM Law in effect
01/01/2004. Removal requests can be sent to this address and will be honored
and respected.

[clydejones]

Clyde I hope you don't mind that I may poke at ya with a few questions when I can take a look at this in depth

Not at all - All I did was replace the email code in edit_orders with the email code from orders.php.

Clyde,
THis appears to be a success! Above link labeled--edit_orders3.zip--

Glad it worked out for you.

[DivaVocals]

Not at all - All I did was replace the email code in edit_orders with the email code from orders.php..

I'll take a look as my time permits this weekend.. Will probably bug you a tad.. :)