EDI file options - split files, filter products
| Project: | Ubercart EDI |
| Version: | 5.x-1.x-dev |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | kussmaul |
| Status: | active |
Jump to:
More enhancements, for which I'd appreciate feedback and suggestions:
1. My EDI partner requires the header record in one file and each detail record in a separate file.
I could a) split the record in the non-Drupal script that uploads files to the FTP site,
or b) add a "file break" syntax to uc_edi.
2. Some of my products are not shippable, and should not be included in the EDI files.
I could a) filter these out in the non-Drupal upload script, or b) filter them out in uc_edi.
In this particular case I could use the "Product and its derivatives are shippable." setting,
but this might not be appropriate for other EDI applications.
If other people would find these useful, I'm happy to add them to uc_edi
(I'm using a 6.x version, but changes could also be applied to 5.x).
Otherwise, I can use approach b) on my own.
Thanks,
Clif
#1
1) Since FTP is generally considered to be insecure, the second method sounds better
2) Me too, and this seems like an extremely common use case that should definitely be fulfilled by this module. I made this change for a client, but their version of uc_edi is unhacked and needs to be generalized so I can merge development efforts here. I found that some of the code flow could be optimized a bit, since it seemed inconsistent and was hard to follow.
#2
1) I don't have much choice in uploading via FTP. Now the EDI partner wants 2 files - one with all of the header records, and one with all of the detail records. If we can't find a general way to handle multiple possibilities, it might be better
to create a single file and use separate code to sort lines into appropriate files.
2) This would be great - I guess the module needs some way to indicate whether or not to export non-shippable products, or products need an option indicating whether they should be exported.