A hook needs to be created to allow other modules to implement delivery methods such as FTP. While we're at it, let's move the filesystem and e-mail delivery methods into their own sub-modules.

Cheers,

Antoine

CommentFileSizeAuthor
#3 uc_edi-996970_2.patch79.26 KBjantoine
#2 uc_edi-996970_1.patch77.05 KBjantoine
#1 uc_edi_996970.patch75.88 KBjantoine

Comments

jantoine’s picture

jantoine commented
Status: Active » Needs review
StatusFileSize
new uc_edi_996970.patch75.88 KB

I am attaching a patch that accomplishes the following:
- Wrapped export preview in check_plain() to encode html characters.
- Increment uc_edi_order_export_next_batch_id on each export.
- Organized information by object (order, customer, etc.), import/export and additional.
- Added a hook for delivery method modules to hook into.
- Added an export delivery methods settings form.
- Moved the filesystem delivery method into it's own Ubercart EDI filesystem module.
- Moved the e-mail delivery method into it's own Ubercart EDI e-mail module.
- Moved the Ubercart EDI and sub-modules into it's own Ubercart - EDI package.

Once this patch is comitted, the next step will be FTP!

Cheers,

Antoine

jantoine’s picture

jantoine commented
StatusFileSize
new uc_edi-996970_1.patch77.05 KB

I forgot to update the CHANGELOG.txt and I found a few minor issues. Attaching an updated patch.

Cheers,

Antoine

jantoine’s picture

jantoine commented
Status: Needs review » Fixed
StatusFileSize
new uc_edi-996970_2.patch79.26 KB

The attached patch has been committed.

Cheers,

Antoine

Status: Fixed » Closed (fixed)

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