Download & Extend
Ubercart EDI » Issues

Create an FTP delivery method sub-module.

Posted by AntoineSolutions on December 13, 2010 at 9:53pm
Project:Ubercart EDI
Version:6.x-1.x-dev
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

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

Cheers,

Antoine

Login or register to post comments

Comments

#1

Posted by AntoineSolutions on December 13, 2010 at 9:54pm

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

AttachmentSize
uc_edi-997258.patch 9.69 KB

#2

Posted by AntoineSolutions on December 13, 2010 at 9:54pm
Status:active» needs review

Forgot to update the status.

Cheers,

Antoine

#3

Posted by bkosborne on January 3, 2011 at 3:43pm

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...

#4

Posted by AntoineSolutions on January 3, 2011 at 6:57pm

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

#5

Posted by bkosborne on January 11, 2011 at 9:45pm

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.

#6

Posted by AntoineSolutions on January 21, 2011 at 8:53pm

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

#7

Posted by AntoineSolutions on January 25, 2011 at 7:33am
Status:needs review» fixed

Committed the attached patch.

Cheers,

Antoine

AttachmentSize
uc_edi-997258_1.patch 10.46 KB

#8

Posted by System Message on February 8, 2011 at 7:40am
Status:fixed» closed (fixed)

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

#9

Posted by stanman on March 7, 2011 at 11:01am

How do i apply this patch

Kind regards

Stany Berden

#10

Posted by AntoineSolutions on March 7, 2011 at 3:16pm

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

nobody click here