Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(164)

Issue 287270043: Add support for third-party numbering plugins (WooCommerce PDF Invoices by WooThemes)

Can't Edit
Can't Publish+Mail
Start Review
Created:
1 year, 10 months ago by Reinhold
Modified:
1 year, 10 months ago
Reviewers:
Base URL:
ssh://open-tools.net/var/www/open-tools.net/private/git/Others/woothemes-pdf-invoice@master
Visibility:
Public.

Description

Add support for third-party numbering plugins -) If sequential numbers are set to No, then the order ID is first used, but then third-party plugins can override the number by implementing the woocommerce_invoice_number($default, $order_id) and/or woocommerce_create_invoice_number($default, $order) filters -) If sequential numbers are set to Yes, then third-party plugins will NOT be called. This implements the API proposed at http://open-tools.net/blog/63-proposal-for-a-general-api-for-third-party-woocommerce-invoice-numbering-plugins.html for the "WooCommerce PDF Invoices" plugin by Andrew Benbow / WooThemes (woocommerce-pdf-invoice) Also (Fix for order numbers): - Use generic function to get the order number rather than solutions that are hardcoded to one plugin

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+20 lines, -18 lines) Patch
M classes/class-pdf-functions-class.php View 1 chunk +6 lines, -1 line 0 comments Download
M classes/class-pdf-send-pdf-class.php View 2 chunks +10 lines, -13 lines 0 comments Download
M classes/class-pdf-settings-class.php View 3 chunks +4 lines, -3 lines 0 comments Download
D lib/dompdf/lib/fonts/log.htm View 1 chunk +0 lines, -1 line 0 comments Download

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 204d58d