My Checkout Without Account Mod

If I like this I will port it to Japanese and upload it.

JMS_Music, you said "I accept that The Queens English is the only true language...". It's always been my contention that if you only left children alone, they'd all grow up speaking English. Well, it makes sense to me. :P

KGZotU, I am almost bouncing up and down on my chair waiting for this. I suggest you take Friday off and have a loooong weekend. :D

I was very interested in craftzombie's mod, but decided, since I had to choose one way or the other, that I needed the accounts. But this removes the need to choose. Let the customer decide. That's great!

Thanks, and get to work.

Rob

Alright, well for those "bouncing up and down" it will probably be tomorrow evening. (; I'm engaged Monday-Thursday, and I try to leave the heavy stuff for Friday-Sunday.

I really appreciate all of the interest, not to mention the translation.

--Joe

HALLELUJAH

:clap:

Eh, thanks. (;

Alright, I have the SQL version done. I've tried it on a couple of fresh installs and it seems ready to go.

If you've already installed a previous version then you can do the following:

1. Install the SQL patch.
2. Overwrite all of the new files. That is, those files in /install/includes. The files which are not overwrites or overrides of stock zen files. (If you've already started to customize these files, let me know. I can tell you specifically what changes need to be made).
3. Overwrite, override, or edit the following files:
   /admin/orders.php
   /includes/modules/pages/password_forgotten/header.php
   /includes/modules/YOUR_TEMPLATE/create_account.php

Also, check your old download of this mod, if you don't have the mods of:
/includes/languages/english/YOUR_TEMPLATE/gv_redeem.php
and
/includes/modules/pages/gv_redeem/header.php

You'll want them if you're going to deal with gift certificates.

Let me know if that's at all confusing or you need any help. (;

Barring any bugs, I think this is pretty much done. Please let me know of any problems. In particular, I'm hoping we've eliminated all references of accounts for COWOA customers.

Thanks for the help,
Joe

Is there a place where we can download the new version? Thanks for working on this.

Scott

Sure, just visit the wiki link in my sig.

Thanks for letting me know that isn't clear, I'll make a note down there. (;

--Joe

Been on it all day, its not doin it :-( But the good news is that the Translations are nearly finished .-)

Files Uploaded
SQL Uploaded
File Names double checked
test purchase

BOOM

Same error

am comparing the files (new to old) at the mo but not finding anything that will throw this confirmation Email out.....

Any ideas???

Could it be Server Side???

RIchie

Richie,

Taking a look at your store, it appears that you still have the code that is responsible for making the '.' at the end of the e-mail address. I verify this by beginning the COWOA process, and then going to the 'Contact Us' page.

The code that does this is one of the mod specific files, /includes/modules/YOUR_TEMPLATE/no_account.php.

The line:
$email_address .= ".";

I went ahead and verified that this is not in the new package. Perhaps there was an overwrite error? Or maybe the old file is in the override folder, but the new one got placed in the default folder? If you try overwriting again and it doesn't solve the problem, do a developer's toolkit search for that line of code above.

I hope this helps,
Joe

Latest SQL version works perfectly - no problems, as far as I can see.

Just two minor points though:

Now there is no '.' at the end of the email address it's impossible to tell which are account and which are not - just be useful to know, really, and not that vital.

Order status add-on. If you check the status of an order there is a repeat of the email and order number input boxes at the bottom of the order status page. If you press the 'continue' button it takes you back to the previous email and order number input page with a logical error message about no input having been made. I assumed first time round that the 'Continue' button would take me back to the home page or something similar.

One final thing - we use PDFOrder Centre (Oops! Richie, I'm a Brit too!) which exhibits the same email problems as Super Orders - just thought you'd like to know.

Apart from that - Fantastic.

TPHoare,

This will partially address your post:

INSERT INTO query_builder ( query_id , query_category , query_name , query_description , query_string ) VALUES ( '', 'email,newsletters', 'Permanent Account Holders Only', 'Send email only to permanent account holders ', 'select customers_email_address, customers_firstname, customers_lastname from TABLE_CUSTOMERS where COWOA_account != true order by customers_lastname, customers_firstname, customers_email_address');

This will create a new category in your drop down, 'choose the group to e-mail this to' menu. I'll come out with an edit/overwrite for admin/customers.php that displays whether a customer is COWOA or permanent. These seem pretty essential, thanks for pointing them out. (;

I'm going to take a step back from template changes for the moment. In the meantime, however, if anybody has any changes that they feel make this a little more usable or such feel free to post them. And feel free to post requests as well. I'll get to them a bit later. (;

Thanks much,
Joe

PS: PDFOrder Center and Super Orders both seem like common mods. Now that I've got the SQL version running, I'll go ahead and post edits for these. I'll also post general edits for anything else that sends out mail.

Alright, I don't have Super Orders or PDF Order Center to test these on, but I've given the following a good eye and I think they should work. If you're using these, could you confirm that they work as they should or alternatively tell me if they don't?

Thanks much.

Super Orders
File: admin/includes/functions/extra_functions/super_orders_functions/

Find:

$customer_info = $db->Execute("SELECT customers_name, customers_email_address, date_purchased
                                 FROM " . TABLE_ORDERS . "
                                 WHERE orders_id = '" . $oID . "'");

Replace with:

$customer_info = $db->Execute("SELECT customers_name, customers_email_address, date_purchased, COWOA_account
                                 FROM " . TABLE_ORDERS . "
                                 WHERE orders_id = '" . $oID . "'");

Find:

  // send email to customer
  $message = STORE_NAME . "\n" . EMAIL_SEPARATOR . "\n" .
  EMAIL_TEXT_ORDER_NUMBER . ' ' . $oID . "\n\n" .
  EMAIL_TEXT_INVOICE_URL . ' ' . zen_catalog_href_link(FILENAME_CATALOG_ACCOUNT_HISTORY_INFO, 'order_id=' . $oID, 'SSL') . "\n\n" .
  EMAIL_TEXT_DATE_ORDERED . ' ' . zen_date_long($customer_info->fields['date_purchased']) . "\n\n" .
  strip_tags($notify_comments) .
  EMAIL_TEXT_STATUS_UPDATED . sprintf(EMAIL_TEXT_STATUS_LABEL, $orders_status_array[$status] ) .
  EMAIL_TEXT_STATUS_PLEASE_REPLY;

  $html_msg['EMAIL_CUSTOMERS_NAME']    = $customer_info->fields['customers_name'];
  $html_msg['EMAIL_TEXT_ORDER_NUMBER'] = EMAIL_TEXT_ORDER_NUMBER . ' ' . $oID;
  $html_msg['EMAIL_TEXT_INVOICE_URL']  = '<a href="' . zen_catalog_href_link(FILENAME_CATALOG_ACCOUNT_HISTORY_INFO, 'order_id=' . $oID, 'SSL') .'">'.str_replace(':','',EMAIL_TEXT_INVOICE_URL).'</a>';
  $html_msg['EMAIL_TEXT_DATE_ORDERED'] = EMAIL_TEXT_DATE_ORDERED . ' ' . zen_date_long($customer_info->fields['date_purchased']);
  $html_msg['EMAIL_TEXT_STATUS_COMMENTS'] = $notify_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];
  $html_msg['EMAIL_TEXT_STATUS_PLEASE_REPLY'] = str_replace('\n','', EMAIL_TEXT_STATUS_PLEASE_REPLY);

Replace With:

// send email to customer
  $message = STORE_NAME . "\n" . EMAIL_SEPARATOR . "\n" .
  EMAIL_TEXT_ORDER_NUMBER . ' ' . $oID . "\n\n";
  if(!$customer_info->fields['COWOA_account'])
    $message .= EMAIL_TEXT_INVOICE_URL . ' ' . zen_catalog_href_link(FILENAME_CATALOG_ACCOUNT_HISTORY_INFO, 'order_id=' . $oID, 'SSL') . "\n\n";
  $message .= EMAIL_TEXT_DATE_ORDERED . ' ' . zen_date_long($customer_info->fields['date_purchased']) . "\n\n" .
  strip_tags($notify_comments) .
  EMAIL_TEXT_STATUS_UPDATED . sprintf(EMAIL_TEXT_STATUS_LABEL, $orders_status_array[$status] ) .
  EMAIL_TEXT_STATUS_PLEASE_REPLY;

  $html_msg['EMAIL_CUSTOMERS_NAME']    = $customer_info->fields['customers_name'];
  $html_msg['EMAIL_TEXT_ORDER_NUMBER'] = EMAIL_TEXT_ORDER_NUMBER . ' ' . $oID;
  if(!$customer_info->fields['COWOA_account'])
    $html_msg['EMAIL_TEXT_INVOICE_URL']  = '<a href="' . zen_catalog_href_link(FILENAME_CATALOG_ACCOUNT_HISTORY_INFO, 'order_id=' . $oID, 'SSL') .'">'.str_replace(':','',EMAIL_TEXT_INVOICE_URL).'</a>';
  else
    $html_msg['EMAIL_TEXT_INVOICE_URL']  = '';   
  $html_msg['EMAIL_TEXT_DATE_ORDERED'] = EMAIL_TEXT_DATE_ORDERED . ' ' . zen_date_long($customer_info->fields['date_purchased']);
  $html_msg['EMAIL_TEXT_STATUS_COMMENTS'] = $notify_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];
  $html_msg['EMAIL_TEXT_STATUS_PLEASE_REPLY'] = str_replace('\n','', EMAIL_TEXT_STATUS_PLEASE_REPLY);

PDF Order Center

File: admin/pdfoc.php

Find:

$email .= "\n\n" .
                        $email_text_subject . ".\n\n" .  PDFOC_EMAIL_SEPARATOR .  "\n\n" .
                        PDFOC_EMAIL_TEXT_INVOICE_URL . ' ' . zen_catalog_href_link(FILENAME_CATALOG_ACCOUNT_HISTORY_INFO, 'order_id=' . $orders->fields['orders_id'], 'SSL') . "\n" .
                        PDFOC_EMAIL_TEXT_DATE_ORDERED . ' ' . zen_date_long($order->info['date_purchased']) . "\n\n" . $notify_comments .  PDFOC_EMAIL_SEPARATOR . "\n\n" .
                        PDFOC_EMAIL_IF_QUESTIONS . PDFOC_EMAIL_SIGNOFF . STORE_NAME . "\n\n" . PDFOC_EMAIL_SEPARATOR . "\n\n" .
                        EMAIL_DISCLAIMER;

               $email_html['EMAIL_SUBJECT'] = "\n\n";
               $email_html['EMAIL_MESSAGE_HTML'] = sprintf(PDFOC_EMAIL_SALUTATION,$order->customer['name']) . "\n\n" .
                        $email_text_subject . ".\n\n" .  PDFOC_EMAIL_SEPARATOR .  "\n\n" .
                        PDFOC_EMAIL_TEXT_INVOICE_URL . ' <a href="' . zen_catalog_href_link(FILENAME_CATALOG_ACCOUNT_HISTORY_INFO, 'order_id=' . $orders->fields['orders_id'], 'SSL') . '">' . zen_catalog_href_link(FILENAME_CATALOG_ACCOUNT_HISTORY_INFO, 'order_id=' . $orders->fields['orders_id'], 'SSL') . '</a>' . "\n" .
                        PDFOC_EMAIL_TEXT_DATE_ORDERED . ' ' . zen_date_long($order->info['date_purchased']) . "\n\n" . $notify_comments .  PDFOC_EMAIL_SEPARATOR . "\n\n" .
                        PDFOC_EMAIL_IF_QUESTIONS . PDFOC_EMAIL_SIGNOFF . STORE_NAME . "\n\n" . PDFOC_EMAIL_SEPARATOR . "\n\n";

Replace With:

$customer_info = $db->Execute("SELECT COWOA_account
                                 FROM " . TABLE_ORDERS . "
                                 WHERE orders_id = '" . $oID . "'");

               $email .= "\n\n" .
                        $email_text_subject . ".\n\n" .  PDFOC_EMAIL_SEPARATOR .  "\n\n";
                        if(!$customer_info->fields['COWOA_account'])
                          $email .= PDFOC_EMAIL_TEXT_INVOICE_URL . ' ' . zen_catalog_href_link(FILENAME_CATALOG_ACCOUNT_HISTORY_INFO, 'order_id=' . $orders->fields['orders_id'], 'SSL') . "\n";
                        email .= PDFOC_EMAIL_TEXT_DATE_ORDERED . ' ' . zen_date_long($order->info['date_purchased']) . "\n\n" . $notify_comments .  PDFOC_EMAIL_SEPARATOR . "\n\n" .
                        PDFOC_EMAIL_IF_QUESTIONS . PDFOC_EMAIL_SIGNOFF . STORE_NAME . "\n\n" . PDFOC_EMAIL_SEPARATOR . "\n\n" .
                        EMAIL_DISCLAIMER;

               $email_html['EMAIL_SUBJECT'] = "\n\n";
               $email_html['EMAIL_MESSAGE_HTML'] = sprintf(PDFOC_EMAIL_SALUTATION,$order->customer['name']) . "\n\n" .
                        $email_text_subject . ".\n\n" .  PDFOC_EMAIL_SEPARATOR .  "\n\n";
                        if(!$customer_info->fields['COWOA_account'])
                          $email_html['EMAIL_MESSAGE_HTML'] .= PDFOC_EMAIL_TEXT_INVOICE_URL . ' <a href="' . zen_catalog_href_link(FILENAME_CATALOG_ACCOUNT_HISTORY_INFO, 'order_id=' . $orders->fields['orders_id'], 'SSL') . '">' . zen_catalog_href_link(FILENAME_CATALOG_ACCOUNT_HISTORY_INFO, 'order_id=' . $orders->fields['orders_id'], 'SSL') . '</a>' . "\n";
                        $email_html['EMAIL_MESSAGE_HTML'] .= PDFOC_EMAIL_TEXT_DATE_ORDERED . ' ' . zen_date_long($order->info['date_purchased']) . "\n\n" . $notify_comments .  PDFOC_EMAIL_SEPARATOR . "\n\n" .
                        PDFOC_EMAIL_IF_QUESTIONS . PDFOC_EMAIL_SIGNOFF . STORE_NAME . "\n\n" . PDFOC_EMAIL_SEPARATOR . "\n\n";

Fraid I get this error PDFOrder Center pdfoc.php:

Parse error: syntax error, unexpected T_CONCAT_EQUAL in line 392

Line 392 is:

               email .= PDFOC_EMAIL_TEXT_DATE_ORDERED . ' ' . zen_date_long($order->info['date_purchased']) . "\n\n" . $notify_comments .  PDFOC_EMAIL_SEPARATOR . "\n\n" .

I am hopeless with php so am not much help I am afraid

Ammendment to code above

For PDF Order Center. Replace With block should read:

$customer_info = $db->Execute("SELECT COWOA_account
                                 FROM " . TABLE_ORDERS . "
                                 WHERE orders_id = '" . $oID . "'");

               $email .= "\n\n" .
                        $email_text_subject . ".\n\n" .  PDFOC_EMAIL_SEPARATOR .  "\n\n";
                        if(!$customer_info->fields['COWOA_account'])
                          $email .= PDFOC_EMAIL_TEXT_INVOICE_URL . ' ' . zen_catalog_href_link(FILENAME_CATALOG_ACCOUNT_HISTORY_INFO, 'order_id=' . $orders->fields['orders_id'], 'SSL') . "\n";
                        $email .= PDFOC_EMAIL_TEXT_DATE_ORDERED . ' ' . zen_date_long($order->info['date_purchased']) . "\n\n" . $notify_comments .  PDFOC_EMAIL_SEPARATOR . "\n\n" .
                        PDFOC_EMAIL_IF_QUESTIONS . PDFOC_EMAIL_SIGNOFF . STORE_NAME . "\n\n" . PDFOC_EMAIL_SEPARATOR . "\n\n" .
                        EMAIL_DISCLAIMER;

               $email_html['EMAIL_SUBJECT'] = "\n\n";
               $email_html['EMAIL_MESSAGE_HTML'] = sprintf(PDFOC_EMAIL_SALUTATION,$order->customer['name']) . "\n\n" .
                        $email_text_subject . ".\n\n" .  PDFOC_EMAIL_SEPARATOR .  "\n\n";
                        if(!$customer_info->fields['COWOA_account'])
                          $email_html['EMAIL_MESSAGE_HTML'] .= PDFOC_EMAIL_TEXT_INVOICE_URL . ' <a href="' . zen_catalog_href_link(FILENAME_CATALOG_ACCOUNT_HISTORY_INFO, 'order_id=' . $orders->fields['orders_id'], 'SSL') . '">' . zen_catalog_href_link(FILENAME_CATALOG_ACCOUNT_HISTORY_INFO, 'order_id=' . $orders->fields['orders_id'], 'SSL') . '</a>' . "\n";
                        $email_html['EMAIL_MESSAGE_HTML'] .= PDFOC_EMAIL_TEXT_DATE_ORDERED . ' ' . zen_date_long($order->info['date_purchased']) . "\n\n" . $notify_comments .  PDFOC_EMAIL_SEPARATOR . "\n\n" .
                        PDFOC_EMAIL_IF_QUESTIONS . PDFOC_EMAIL_SIGNOFF . STORE_NAME . "\n\n" . PDFOC_EMAIL_SEPARATOR . "\n\n";

TPHoare:

Thanks for catching this. I left out a '$'. On the line you pointed out, place a dollar sign just before 'email'.

--Joe

Hi Joe

Now I am getting this:

1054 Unknown column 'COWOA_account' in 'field list'
in:
[SELECT COWOA_account FROM orders WHERE orders_id = '49068']
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.

Cheers

Tim

You've got a PM. Rather than incrementally solve this on the board I should figure it out first. (;

--Joe

Hi Joe again - Have sorted out the error above - missing ' around 'COWOA_account' I think. Emails are being sent but they don't reflect the change of status - so status could be 'On Order' but email just has no subject and no content.

By the way, I'm personally not worried about sending emails from PDFOrder Center, because we very rarely, if ever, send bulk emails and much prefer to update status from the normal ZenCart order admin module. So, don't burn the midnight oil just on my account.

Hi Joe,

I downloaded the new version but haven't been able to take a look at it yet. I am wondering, for those of us who are using the old version... is it better to switch to the new version? I have had several orders placed using the old version and everything seems to be working great. In the long run, are there any problems you can think of if I just leave it the way it is? Or did you foresee future problems (perhaps with the period at the end of the email addresses) and that is why you made a new version? Much thanks again!

TPHoare:

Hmm...can you tell me if there's the same problem with orders made from permanent accounts?

craftzombie:

It was mainly an issue of a minority of servers not liking the set up. I think that the only feature that I didn't have implemented before the SQL version was the extra drop e-mail, newsletter, coupon, gv, etc. drop down option by which you can elect to send only to permanent account holders.

If you were to move to the SQL version, the only issues you would face with the old COWOA accounts would be as you update their order status.

Hope this helps,
Joe