Posted by hardfocus on January 21, 2009 at 6:20am
| Project: | UC Purchase Order |
| Version: | 5.x-1.x-dev |
| Component: | Code |
| Category: | task |
| Priority: | normal |
| Assigned: | rszrama |
| Status: | closed (fixed) |
Issue Summary
Would appreciate if someone would update this to Drupal 6.
Thanks
Comments
#1
#2
I second the motion.
#3
Update to 6 would be really useful for me too.
Must learn how to do this myself one of these days!
#4
The attached patch against HEAD will make this module ready for D6. I have also included a tarball for those who prefer not to patch.
This patch was sponsored by Tribute Media
#5
The patch by jrglasgow works perfectly!
I've created a patch over it for adding the possibility to skip the manual insert of the product order number. I'll attach both the patch and the patched code.
I hope your patch will be branched to 6.x with my patch (or something better) included
#6
I've tested the patch from #4 and it's works as expected.
Thanks jrglasgow.
#7
Thanks for the hard work!
(Now I must make time to test it, since I was the one who requested it) :-P
#8
Please, test it :-)
#9
The patch in #4 lacked an update function and needed a few more changes, so I've completed those and committed them to 6.x-1.0 version. Regarding the patch in #5, I don't think adding features in a Drupal update patch is a good idea. Please submit a separate feature request with your changes rolled into a new patch against the 6--1 branch for consideration. Thanks for the help, everyone!
#10
Automatically closed -- issue fixed for 2 weeks with no activity.
#11
Hello,
I installed the patched code from #5 and then went to PO settings to try to disable "Enable manual Purchase Order input" in order to make PO number automatically generated. When I deselected the box and save, the system gave me the following message "Enable manual Purchase Order input field is required" and the option couldn't be disabled. I gave permission to "Pay with PO" to a user role.
Using Ubercart 6.2.4.
Also tried with UC PO 6.x.1.0 and UC PO 6.x.2.x dev but the option cannot be disabled neither.
Any suggestions, please?
Many thanks