Now that a hook exists that other modules can leverage to add delivery methods, an FTP delivery method would be really nice.

Cheers,

Antoine

CommentFileSizeAuthor
#7 uc_edi-997258_1.patch10.46 KBjantoine
#1 uc_edi-997258.patch9.69 KBjantoine

Comments

jantoine’s picture

jantoine commented
StatusFileSize
new uc_edi-997258.patch9.69 KB

Here is a patch for adding FTP delivery capabilities. This patch is mostly finished, but depends on #996970: Create a hook for providing delivery methods and move the filesystem and e-mail delivery methods into sub-modules., so we won't be able to add final touches until that patch is committed.

Cheers,

Antoine

jantoine’s picture

jantoine commented
Status: Active » Needs review

Forgot to update the status.

Cheers,

Antoine

bkosborne’s picture

bkosborne
Primary language English
Location New Jersey, USA
commented

Any update on this? I desire this functionality. I'll be working on this throughout my day today and will report back. Have to integrate your previous patch first...

jantoine’s picture

jantoine commented

Hi bkosborne,

I haven't had any feedback on the patch yet. It would be great if I could get a review of the patch before I commit it to the project. Let me know if you have any questions and how everything goes.

Cheers,

Antoine

bkosborne’s picture

bkosborne
Primary language English
Location New Jersey, USA
commented

Antoine,

I'm using both of your patches now... working well so far. One thing is that I added an "uc_edi_ftp_path" variable. This is the remote path of where the files should be moved to on the server. I felt this was needed because we can't assume that the home directory of the ftp account is where the files should go.

I'd like to see these patches integrated into the module and rolled, because I'd like to do a similar implementation for importing.

jantoine’s picture

jantoine commented

Hey bkosborne,

I am getting ready to apply this patch, but was wondering if you would like to provide a patch adding the uc_edi_ftp_path code?

Cheers,

Antoine

jantoine’s picture

jantoine commented
Status: Needs review » Fixed
StatusFileSize
new uc_edi-997258_1.patch10.46 KB

Committed the attached patch.

Cheers,

Antoine

Status: Fixed » Closed (fixed)

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

stanman’s picture

stanman commented

How do i apply this patch

Kind regards

Stany Berden

jantoine’s picture

jantoine commented

stanman,

This patch is already applied to the latest 6.x-1.x-dev snapshot release, so please use that if you need this functionality. If you need to apply patches in the future, please see http://drupal.org/patch/apply.

Cheers,

Antoine