Would appreciate if someone would update this to Drupal 6.

Thanks

CommentFileSizeAuthor
#5 uc_po-automatic-numbering.module.patch7.58 KBfinex
#5 uc_po.tar_.gz3.38 KBfinex
#4 uc_po_upgrade_to_d6.patch5.35 KBjrglasgow
#4 uc_po-6.x.tar_.gz3.06 KBjrglasgow

Comments

Critical Tinkerer’s picture

Critical Tinkerer commented
Category: support » task
joshuautley’s picture

joshuautley commented

I second the motion.

jaydunford’s picture

jaydunford commented

Update to 6 would be really useful for me too.

Must learn how to do this myself one of these days!

jrglasgow’s picture

jrglasgow commented
StatusFileSize
new uc_po-6.x.tar_.gz3.06 KB
new uc_po_upgrade_to_d6.patch5.35 KB

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

finex’s picture

finex commented
StatusFileSize
new uc_po.tar_.gz3.38 KB
new uc_po-automatic-numbering.module.patch7.58 KB

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

mlsamuelson’s picture

mlsamuelson commented

I've tested the patch from #4 and it's works as expected.

Thanks jrglasgow.

Critical Tinkerer’s picture

Critical Tinkerer commented

Thanks for the hard work!

(Now I must make time to test it, since I was the one who requested it) :-P

finex’s picture

finex commented

Please, test it :-)

rszrama’s picture

rszrama commented
Assigned: Unassigned » rszrama
Status: Active » Fixed

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!

Status: Fixed » Closed (fixed)

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

andrews501’s picture

andrews501 commented

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