I have, everything seems to be working good. although i am still testing with orders,shippers and customers to make sure everything is working good.
Printable View
Well, I think the newest version of this must have some problems in admin/confirm_track.php -- I went through the file and found some small issues, but it still didn't work. I must be missing something. Here's what I found:
A couple of %2 are in there instead of a space.
A couple <? are in there instead of <?php
Could you post or PM me the copy of admin/confirm_track.php that you are using successfully? I bet this will fix my problem.
Thanks!
I found a little bug. If you split an order in two between two suppliers/drop shippers only the first tracking information will be sent to them. To fix this, I changed the following in confirm_track_sub.php
Around line 170 change $customer_notified=1; to this:
if($order_shipping_complete==1) {
$customer_notified=1;
} else {
$customer_notified=0; }
I found a few other little bugs, but they only seem to affect the resending of old P.O.s:
1. Old POs will only resend to the original dropshipper no matter what you choose.
2. Old POs won't combine items onto one PO like New POs does for the same orders.
3. Old POs won't include product attributes.
I tried to fix the third one, but wasn't able to figure it out.
By the way, I've gone ahead and made some changes to make this contribution easier to use. I realized that I was going to need more information to decide who I would drop ship with. So, I added Customer Comments, State, and Shipping Choice to the New Orders page. This way I can evaluate what the cheapest method to ship something is. I can also see if there are any special requests in the comments that I need to accomodate before blindly sending a PO out. If people are interested I'd be happy to share the changes to do this.
If anyone figures out how to fix the Old PO resend problems please let me know.
Can someone post an old copy of confirm_track.php that is working for them. I still get nothing on Admin-Customers-PO Enter Tracking #. :( I'd really like to try an older version and see if something is up with the newest one.
I've made some serious changes to this mod, and I am considering re-releasing it. First, I'd just like to ask if that is ok with anyone who has worked on this. I know I have the right to do so as there is no restrictive license on this code, but I'd hate to step on anyone's toes.
Here's what I've done so far:
1) Implemented PDF packing lists that automatically attach to the email purchase orders. These have customer comments, and only include the products that ship with each purchase order. If the purchase order is a partial shipment, this is indicated on the PDF packing list.
2) Reworked confirm_track.php. As stated in my previous post, this was badly broken. I fixed a lot of bugs and cut a lot of code. I also added who the purchase order was sent to. It seems to work fine now.
3)As noted in my last post, I added a ton of extra information to the send PO page.
4)Emails to customers now include all products shipped regardless of whether or not the shipment was a partial shipment or not. The old one only listed the products if it was a partial shipment
5)I simplified the PO # and Order # system. Now, PO #'s are what used to be Order #-PO #. So for example, if the PO # is 5 and the order # is 10 the new PO# would be 10-5. This way, there is only one number that your subcontractor needs to know, but you have all the information you might need in this number.
6)Changed the tracking input page so that all products are checked by default. The most common scenario is that all products shipped at once, so this is the easiest way to handle this. If not, you can still easily uncheck products.
Here are two things that I still have not addressed that I would like to address before re-releasing this:
1) The Resend PO page has a ton of problems. In addition to the problems I have already posted, the PO is always resent to the same subcontractor! Yikes!! Big problems... I have not worked on the resend page at all.
2) I'd like to remove one of the two "own stock" selections. Why do we need two of them?
That sounds amazing. Most of those mods you have planned are very good. Keepsake I think originally did this mod and has been excellent for support on it as it grew. Perhaps they can weigh in on your question.
I figure if I don't hear from anyone before this is ready, I'll just release it. I may not be able to tackle the resend POs; I keep finding more and more problems with it. I'll certainly give it a try, but I'm not going to kill myself trying to fix it.
By the way, if you are using this module, please be aware that there is a bug with the shipping when sending new POs. When sending multiple orders, it will only use one type of shipping for all of your orders! So, be careful, and send orders out one at a time. Otherwise, you may end up with an expensive shipping bill from your subcontractors! :) I've fixed this problem, and when I can pull everything together, my new release will include this fix.
If I don't end up fixing the resend problems, all I really need to do is take out the code that is specific to my site. I've been using the modified system for a little over a week now, and the only problem I had was with the shipping. (Luckily, I stocked the product that led me to discover this error!!)
i noticed an issue recently with the email that gets sent in the OLD version (which I am still using). Where it says {delivery adress} in the email, it is not sending the ENTIRE shipping address. Instead, if an address has more than one line, for example:
John Doe
1234 Street
Apt. #4
City, State 12345
it only sends:
John Doe
1234 Street
City, State 12345
It is missing the other line. This is a serious issue. Does anyone else have this problem? How can I fix this?
Thank you!
Jaime
I already fixed that problem for sending new POs. When I release my changes, it should work with the second address line.
With old POs, I haven't bothered. The resend function has too many other serious problems to even begin tackling the little problems like this one.
If you want to fix it yourself, the variables you need to call from the database for address line 2 are customer_suburb, delivery_suburb, and shipping_suburb. You'll have to create a simple function to check if there is any data, otherwise you will end up with a blank line for all the addresses without a second line.
Hope that helps!