Jump to content

blucollarguy

Members
  • Content count

    739
  • Joined

  • Last visited

Everything posted by blucollarguy

  1. blucollarguy

    Multi_Vendor_Shipping new thread

    There are some instructions about modifying shipping modules included in the package. Take a look and give it a go, if you run into any problems. post the section of code you are having trouble with and what the error is and we will try to help you with it. Good luck, Craig :) I appreciate any input for the instructions, they are very difficult to keep up with. The ability to offer a compiled quote is an option we considered adding to MVS but have not done it yet. It would require a fair amount of planning and code work to get thru it and display all the correct prices with the options available prooerly. A good bit of work. If you want to try to tackle yourself, we would certainly be willing to help. Good luck with whatever you decide to do, Craig :)
  2. blucollarguy

    Multi_Vendor_Shipping new thread

    The thing you need to do here is see how this code does what it does. "order->...." is the data being used to pass the needed information to the pages in order to obtain the quote. This ordinarily comes from includes/classes/order.php, MVS changes some things there, vicually compare what is here with what is changed by MVS. Then, the code that collects and displays the quote, from checkout_shippng.php, is also re-written here. Again, visually compare the changes. This will not be very easy, but it can be done. Good luck, Craig :)
  3. blucollarguy

    Multi_Vendor_Shipping new thread

    Do you mean the "Oder Editor for MVS" or just upating the order status from the "orders" page? If it is the "Oder Editor for MVS" I can't say for sure since I havn't had the chance yet to really look at Stew's handywork, but, check this: at the top of admin\includes\classes\vendor_order_data.php, this was chnaged: <!-- MVS $Id: vendor_order_data.php By Craig Garrison Sr. (craig@blucollarsales.com) for Multi-Vendor Shipping osCommerce, Open Source E-Commerce Solutions http://www.oscommerce.com Copyright (c) 2005 osCommerce Released under the GNU General Public License --> <?php $index2 = 0; to: <?php /* MVS $Id: vendor_order_data.php By Craig Garrison Sr. (craig@blucollarsales.com) for Multi-Vendor Shipping osCommerce, Open Source E-Commerce Solutions http://www.oscommerce.com Copyright (c) 2005 osCommerce Released under the GNU General Public License */ $index2 = 0; The version may be different, or he may have used an old copy while developing the contrib. If that doesn't fix it, post back and we will see what we can figure out. Craig :)
  4. blucollarguy

    Multi_Vendor_Shipping new thread

    You can do this Wade, just use a file compare program, the lines will show you what is different about the files. I would suggest starting with ccgv and adding MVS. Try it on a test copy of your site and see what happens, only thing to lose is time. Good luck, Craig :)
  5. blucollarguy

    Multi_Vendor_Shipping new thread

    My fault, change this line: $vendor_delivery_address_list['delivery_name'] . ' phone: ' . $vendor_delivery_address_list['customers_telephone'] . 'Email: ' . $vendor_delivery_address_list['customers_email'] . '<br>' . to: $vendor_delivery_address_list['delivery_name'] . ' phone: ' . $vendor_delivery_address_list['customers_telephone'] . 'Email: ' . $vendor_delivery_address_list['customers_email_address'] . '<br>' . Good luck, Craig :)
  6. blucollarguy

    Multi_Vendor_Shipping new thread

    Try looking at admin/includes/classes/order.php That one will show you the block of code that is changed for MVS. All product info is collected in defferent query and put together from a larger "for" loop than the standard osC info. Craig :)
  7. blucollarguy

    Multi_Vendor_Shipping new thread

    This will add the telephone and email, the comments will take a bit more though as all the comments are added to the same table, both admin and customer, and we will need to determine which one is needed. Find the query that starts with this: // let's build the email // Get the delivery address and then replace with this code: // let's build the email // Get the delivery address $delivery_address_query = tep_db_query("select distinct delivery_company, delivery_name, delivery_street_address, delivery_city, delivery_state, delivery_postcode, cusstomers_telephone, customers_email_address from " . TABLE_ORDERS . " where orders_id='" . $oID ."'"); Then, find this code: '<br>Dropship deliver to:<br>' . $vendor_delivery_address_list['delivery_company'] .'<br>' . $vendor_delivery_address_list['delivery_name'] .'<br>' . $vendor_delivery_address_list['delivery_street_address'] .'<br>' . $vendor_delivery_address_list['delivery_city'] .', ' . $vendor_delivery_address_list['delivery_state'] . ' ' . $vendor_delivery_address_list['delivery_postcode'] . '<br><br>'; And add where you want the new fields to show, the following 2 lines: $vendor_delivery_address_list['customers_telephone'] and $vendor_delivery_address_list['customers_email'] So that the whole block would now look like this: '<br>Dropship deliver to:<br>' . $vendor_delivery_address_list['delivery_company'] .'<br>' . $vendor_delivery_address_list['delivery_name'] . ' phone: ' . $vendor_delivery_address_list['customers_telephone'] . 'Email: ' . $vendor_delivery_address_list['customers_email'] . '<br>' . $vendor_delivery_address_list['delivery_street_address'] .'<br>' . $vendor_delivery_address_list['delivery_city'] .', ' . $vendor_delivery_address_list['delivery_state'] . ' ' . $vendor_delivery_address_list['delivery_postcode'] . '<br><br>'; That would put the phone and email on the same line as the customer's name, with labels. Good luck, Craig :) Both USPS and UPS are changing all their web APIs to an SSL connection, for the most part it will mean nothing more that changing the URL that the code sends the information for the quote. For instance, USPS will be changing "http://production.shippingapis.com/" to "https://secure.shippingapis.com/", it will be that easy. I plan to have this tested and ready in the next release of MVS. I don't have the time to get into this much, but if you take a good look at the MVS modifications to catalog/includes/classes/order.php you will see what is done to change the products list so that they are called properly per vendor, or as if it were a standard osC order, there are some major differences between the 2. I hope to get into the PDF system to build emails as an option but have not gotten into it yet and may not for quite some time. good luck, Craig :)
  8. blucollarguy

    Multi_Vendor_Shipping new thread

    You cannot overwrite and run these 2 mods, they must be installed separately and by editing each individual file that is common between the 2. I suggest adding all the files from ccgv that do not overwrite any MVS edited files, then use a compare program to install the ccgv code on the files already modified for MVS. Good luck, Craig :)
  9. blucollarguy

    Multi_Vendor_Shipping new thread

    I'm not Jim, but you should have no trouble adding CCGV, just pay attention to the placement of the code changes, some are critical, others are not. Good luck, Craig :)
  10. blucollarguy

    Multi_Vendor_Shipping new thread

    Are you using MVS V1.1? That version has an updated UPSXML module, you should not upgrade yet the rest of the code yet though, I have the next update coming to MVS soon. You can try to use the file in 1.1 and see if that one can do what you need. Good luck, Craig :)
  11. blucollarguy

    Multi_Vendor_Shipping new thread

    You need to add the "CUSTOMERS_TELEPHONE" to the query first: $delivery_address_query = tep_db_query("select distinct delivery_company, delivery_name, delivery_street_address, delivery_city, delivery_state, delivery_postcode, customers_email_address, customers_telephone from " . TABLE_ORDERS . " where orders_id='" . $oID ."'"); The add the call you are looking for: '<br>Please deliver to:<br>' . $vendor_delivery_address_list['delivery_company'] .'<br>' . $vendor_delivery_address_list['delivery_name'] .'<br>' . $vendor_delivery_address_list['delivery_street_address'] .'<br>' . $vendor_delivery_address_list['delivery_suburb'] .'<br>' . $vendor_delivery_address_list['delivery_city'] .'<br>' . $vendor_delivery_address_list['delivery_state'] .'<br>' . $vendor_delivery_address_list['delivery_postcode'] . '<br><br>' . '>>>' . $vendor_delivery_address_list['customers_telephone'] . '<<<' That should do it just fine, good luck, Craig :)
  12. blucollarguy

    Multi_Vendor_Shipping new thread

    You can reference this page in the forum for the Vendors Auto Email thread, VAE Add Address field to email There are a couple of posts on that page that address exactly what you are trying to do, just check that you use the correct table name(customer_suburb... or what ever it is) in the database. Good luck, Craig :)
  13. blucollarguy

    Multi_Vendor_Shipping new thread

    See quoted post below: Also, for anyone using the IPN module from the osC team, I have found the problem with the emails. Starting at about line 408: function before_process() { global $customer_id, $order, $sendto, $billto, $payment, $currencies, $cart, $cart_PayPal_IPN_ID; global $$payment; include(DIR_WS_CLASSES . 'order_total.php'); $order_total_modules = new order_total; $order_totals = $order_total_modules->process(); $order_id = substr($cart_PayPal_IPN_ID, strpos($cart_PayPal_IPN_ID, '-')+1); $sql_data_array = array('orders_id' => $order_id, 'orders_status_id' => $order->info['order_status'], 'date_added' => 'now()', 'customer_notified' => (SEND_EMAILS == 'true') ? '1' : '0', 'comments' => $order->info['comments']); tep_db_perform(TABLE_ORDERS_STATUS_HISTORY, $sql_data_array); Note that they did the same thing as the other IPN modules, skipped the checkout_process.php and did all the work within the module, what a pain in the !@%&, why would they do that!? I thought the osC team would try to be a little compliant with their own code. This means that the code for the emails(from checkout_process.php) needs to be added here as well in order for the emails to go out correctly. I will write this up for the next release as soon as I get the time, I have a number of other things to chase down first. Craig :)
  14. blucollarguy

    Vendor_Auto_E-mail (In_Development)

    Take a look further up this thread and you will see where the customer's telephone was added to the email by another user. You should be able to do the exact same thing with the customer's email address. Instead of "customers_telephone" you would use "customers_email_address". You would need to this in both checkout_process.php and admin/orders.php to have full functionality. Good luck, Craig :)
  15. blucollarguy

    Multi_Vendor_Shipping new thread

    The IPN module from the osC team seems stable, but I have been personally been having problems with PayPal myself, so I am not sure what to tell you. I guess I am going to have to take some steps to better deal with this in general, if I come up with something I will post here. Craig :)
  16. blucollarguy

    Multi_Vendor_Shipping new thread

    Your problem is simple, this contrib along with all the other PayPal contribs, sets the status via it's own script rather than using the system within osC. I am assuming you are working with the latest version: around line 1186 of catalog/includes/modules/payment/paypal_wpp.php function after_process() { global $insert_id; tep_db_query("update ".TABLE_ORDERS_STATUS_HISTORY. " set comments = concat(if(trim(comments) != '', concat(trim(comments), '\n'), ''), 'Transaction ID: ".$this->trans_id."\nPayment Type: ".$this->payment_type."\nPayment Status: ".$this->payment_status.($this->avs != 'N/A' ? "\nAVS Code: ".$this->avs."\nCVV2 Code: ".$this->cvv2 : '')."') where orders_id = ".$insert_id); } Note that the status get's set here. That is the problem. You will need to integrate the MVS code(the function and the call to it, and perhaps all the table updates as well) from checkout_process.php. This will be a bit tricky, so be careful and make backups, but you can do it. :thumbsup: Good luck, Craig :)
  17. blucollarguy

    Multi_Vendor_Shipping new thread

    The missing argument is the vendors_id, for some reason it is not being passed to the function properly. You could suppress part of this: adding the @ symbol before the call to the "constant" will get rid of the "warning" about not finding the constant, but that is ultimately not the real issue. It would look like this: @constant(MODULE_SHIPPING_AIRBORNE_DIMENSIONAL_TABLE ....) This would need to be done in the file, where it tries to reference the constant. I have not worked with this module yet, so I can't say much else for sure. Good luck, Craig :)
  18. blucollarguy

    Multi_Vendor_Shipping new thread

    Which version of MVS are you working with? It sounds like a PHP5 issue, but V1.1 should be fully compatible, if you are not using V1.1, you need to upgrade. If you are running 1.1 then you need to double check ALL your edits, it must be in there somewhere. Good luck So the IPN module from the osC team is not working with the email portion of MVS? Are your customers coming back to your site after they pay at PayPal properly? Is the order status updating correctly when a customer pays(updating from pending(default osC) to processing(default osC))? Craig :)
  19. blucollarguy

    Multi_Vendor_Shipping new thread

    Do you mean the one by the osC team? That one should work just fine. If you are using one of the other versions, the problem is simple: you must install the MVS code from "checkout_process.php" into "includes\modules\payment\paypal\catalog\checkout_process.inc.php". A pain in the @%#! If it is the one from the osC team, then there is something else going on, as it is fully compatible with the structure that MVS was designed around. Craig :)
  20. blucollarguy

    Multi_Vendor_Shipping new thread

    This is the only contribution that will do what you need. And it is usable by most anybody, but can be difficult to install because of so much code work. The checkout process is basically being re-written by MVS, so there is a lot to it. Just take your time, and be sure to back EVERY file before you start. I suggest using file differencing tool while installing as well, makes things easier in my opinion. I use WinMerge myself, but there are many available. Good luck, Craig :)
  21. blucollarguy

    Multi_Vendor_Shipping new thread

    Here is the code that displays that message, some like it nice and bold, others will want it a little "nicer", easy enough. // MVS start if a shipping method has not been selected for all vendors, redirect the customer to the shipping method selection page if ( (SELECT_VENDOR_SHIPPING == 'true') && (($cart->content_type != 'virtual') || ($total_weight > 0 )) ){ // This test only works under MVS, but is skipped for virtual and/or zero weight if (!is_array ($shipping['vendor']) || count($shipping['vendor']) != count($cart->vendor_shipping)) { // No shipping selected or not all selected tep_redirect(tep_href_link(FILENAME_CHECKOUT_SHIPPING, 'error_message=' . ERROR_NO_SHIPPING_SELECTED_SELECTED, 'SSL')); } } // MVS end Rather than sending it back as an "error_message", one could send back in the "message_stack" which would be a "nicer" message. We could even write our own "error" class for this purpose as well. Not all that difficult, it just depends on the individual, some people have posted that they really like it as is, some don't, we need common ground, probably need to build in some options related to this. The define should be in the language file. Craig :)
  22. blucollarguy

    Multi_Vendor_Shipping new thread

    That's great Ken, thanks for contributing. Craig :)
  23. blucollarguy

    Multi_Vendor_Shipping new thread

    Check that you have "vendor_order_info.php" on the server and that it is intact, and then check the mods in admin/includes/classes/order.php and admin/orders.php. These files are the ones involved in displaying all the MVS order data. One or more of them must be either corrupt or perhaps an accidental reversion to the original osC version. This will be where your issue is. Craig :)
  24. blucollarguy

    Multi_Vendor_Shipping new thread

    I agree as well, we shouldn't jump into coding this until get a better picture of our users want with this and the code for the gift vouchers will need to be dealt with one way or the other. This is what I was suggesting, another list that would be created when there are "virtual" or "downloadable" products in the order. I would show the products but no shipping methods at all. Maybe a note about what they are: "These products do not ship" or something. But we need input on this for sure, as much as possible. And as soon as possible so we can get this included in the next update. Craig :)
  25. blucollarguy

    Multi_Vendor_Shipping new thread

    Perhaps Jim, we could add some code to list the products in "checkout_shipping" without them being in the quote list? We could remove them from the shipping array, and add them to another array to create a list of all products or something of this sort? This type of feature would have to have options as this a circumstance that not everyone will aggree on how to handle. No doubt about that, which is what I was going to do with the 0 weight removal I have on another site, but I haven't added it at all yet. I dunno, Jim, just brainstorming. Craig :)
×