When trying to resend invoice e-mail I get.

Warning: Missing argument 3 for commerce_billy_mail_resend_form_submit(), called in C:\wamp\www\invoic\includes\form.inc on line 1513 and defined in commerce_billy_mail_resend_form_submit() (line 231 of C:\wamp\www\invoic\sites\all\modules\commerce_billy_mail\commerce_billy_mail.module).
Unable to send e-mail. Contact the site administrator if the problem persists.
Unable to send e-mail. Contact the site administrator if the problem persists.
The PDF could not be resent.
CommentFileSizeAuthor
#3 issue-2314467.patch3.85 KBlsolesen
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

lsolesen’s picture

lsolesen CreditAttribution: lsolesen commented

Seems that I need to apply a patch from #2093643: Allow PDF's filename configuration and add token support before this works.

After applying this, I still get:

Warning: Missing argument 3 for commerce_billy_mail_resend_form_submit(), called in /var/www/site/docroot/elevforeningen/includes/form.inc on line 1517 and defined in commerce_billy_mail_resend_form_submit() (line 231 of /var/www/site/docroot/elevforeningen/sites/all/modules/contrib/commerce_billy_mail/commerce_billy_mail.module).
Warning: realpath() expects parameter 1 to be a valid path, string given in drupal_realpath() (line 2297 of /var/www/site/docroot/elevforeningen/includes/file.inc).
Warning: is_file() expects parameter 1 to be a valid path, string given in mimemail_multipart_body() (line 325 of /var/www/site/docroot/elevforeningen/sites/all/modules/contrib/mimemail/mimemail.inc).
lsolesen’s picture

lsolesen CreditAttribution: lsolesen commented

The first error can be removed just by removing $order parameter from commerce_billy_resend_form_submit(), but the others are still a problem.

lsolesen’s picture

lsolesen CreditAttribution: lsolesen commented
Status: Active » Needs review
FileSize
issue-2314467.patch3.85 KB

I've attached a patch which solves problem 1.

Apfel007’s picture

Apfel007 CreditAttribution: Apfel007 commented

Are you sure? Where do you define "commerce_billy_pdf_gen_filename" ?

This patch is not working for me.

OK - I See, I need in any case the other patch too.

lsolesen’s picture

lsolesen CreditAttribution: lsolesen commented

Yeah. Both patches is needed.

panhwerwaseem queued 3: issue-2314467.patch for re-testing.

webjazznl’s picture

webjazznl CreditAttribution: webjazznl commented

Glad you are working on this, however i do not understand the priority Normal for this issue.

Because it basically breaks the checkout process of the customer on my site. The error pops up in stead of the "Thank you for purchase" screen upon confirmation of the order. This really justifies a new release of the module as a whole, for people like me who are uncomfortable with applying patches or adjusting code manually.

Thanks in advance for your help in this urgent matter.

panhwerwaseem’s picture

panhwerwaseem CreditAttribution: panhwerwaseem commented
Issue summary: View changes

I'm working on localhost drupal website when I click on resend invoice email it give me this error 

Warning: Missing argument 3 for commerce_billy_mail_resend_form_submit(), called in C:\wamp\www\invoic\includes\form.inc on line 1513 and defined in commerce_billy_mail_resend_form_submit() (line 231 of C:\wamp\www\invoic\sites\all\modules\commerce_billy_mail\commerce_billy_mail.module).
Unable to send e-mail. Contact the site administrator if the problem persists.
Unable to send e-mail. Contact the site administrator if the problem persists.
The PDF could not be resent.

please help me how i can solve it, because I'm new on drupal also don't know to how apply patches and every thing, please help me thanks

  • Anybody committed d5b5c13 on 7.x-1.x 
    Issue #2314467 by [Isolesen]: Adds comments to functions and removes an...
Anybody’s picture

Anybody
Primary language German
Location Porta Westfalica
CreditAttribution: Anybody commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.