Jump to content

TomB01

Members
  • Content count

    406
  • Joined

  • Last visited

  • Days Won

    2

TomB01 last won the day on November 28 2019

TomB01 had the most liked content!

About TomB01

  • Birthday 12/24/1954

Profile Information

Recent Profile Visitors

8,746 profile views
  1. TomB01

    NEW! Complete Order Editing Tool!

    Thanks for replying, John. I'm running Phoenix 1.0.5.0. You actually gave or told me how to modify the orders_edit_order.php file back on 12/24. I think you uploaded it all in your 1.2.8 version. Note that your later versions after Denkster still can't be found on the Apps Marketplace. I went searching earlier this morning and couldn't find anything. This worked at the time, when I installed this from you: Somewhere along the way to 1.0.5.0 and Supporters Code updates, I lost the Edit button again, but don't know when - or obviously, how.
  2. TomB01

    NEW! Complete Order Editing Tool!

    I discovered recently that my "Edit" button has disappeared from my Order Details view in Admin, where the order's Summary, Products, and Status History are displayed. I checked by toggling the Edit button feature in the Order Editor configuration and the Order Editor is functioning correctly. I normally do not like to have that Edit button in the orders list view go directly to an order editing screen, but to the orders detail screen, instead. So, I typically have that feature disabled in the configuration. However, I get no "Edit" button at all in the order details view. I first noticed this because the Details box has the following characters displayed erroneously at the bottom left of each Summary, Products, and Status History box: " '); " I traced these three characters to this code in the HTML for orders.php when displaying the order details in Admin: Sure enough, what little I can interpret is that string of script code controls the appearance of the "Edit" button in the order details view. I suspect those ending characters are invalidating the script code. I believe that script should end in only " </script> " instead of " </script>'); " Can anyone direct me to which file that script is developed, so that I can try to find the error? I've checked orders.php and edit_orders.php and it doesn't appear that any of that code is included in those files. My orders.php file was somehow modded, because I have it archived under a "FixEditButton" folder. Unfortunately, it doesn't appear that I commented the code and I can't remember what was done. 😕
  3. @dculley, Your first one up there had a red period before "key" in "configuration_key". Maybe it was a copy-paste thing?
  4. Here is the post with the SQL (on the previous page):
  5. Yes, of course! Please note my previous posts. There are two SQL statements included that you need to poke the appropriate values into the configuration table. They are in my most recent posts (typing this on the phone right now, or I'd copy them again for you).
  6. @dculley I'm sorry - this is beyond me. As for the "(;" I see something similar at the bottom of my Order-Summary-Products-Status History-Paypal page, but it seems innocuous, so I left it alone. It's some sort of trailing code that the PHP is not recognizing. Hopefully, someone else will have some suggestions. Keep me in mind if you end up willing to pay someone to do a USPS module for Phoenix. I might be able to chip in something. EDIT: The other possible issue is going well beyond Phoenix 1.0.5.0. The changes they made in going to 1.0.5.1 were severe. It's why I stopped at 1.0.5.0. I'm not sure we can even make the shipping modules work under the newer changes.
  7. @dculley modulesphx.php shows this from lines 40 to 47: foreach ($_POST['configuration'] as $key => $value) { if (is_array($value)) { $value = implode(';', $value); } $key = tep_db_prepare_input($key); $value = tep_db_prepare_input($value); tep_db_query("UPDATE configuration SET configuration_value = '" . tep_db_input($value) . "' WHERE configuration_key = '" . tep_db_input($key) . "'"); modulesyours.php shows this from lines 35 to 47: foreach ($_POST['configuration'] as $key => $value) { /* kymation mod for Phoenix (11/26/2019) - https://forums.oscommerce.com/topic/383307-usps-rate-v4-intl-rate-v2-official-support-thread/?do=findComment&comment=1793962 */ // if( is_array( $value ) ) $value = implode( ', ', $value ); //End kymation mod for Phoenix // /* Updated mod in greasemonkey post (11/30/2019) - https://forums.oscommerce.com/topic/494460-ups-xml-17-for-phoenix/?do=findComment&comment=1799248 */ if (is_array($value) ) { $value = implode( ", ", $value); $value = preg_replace ("/, --none--/", "", $value); } // End greasemonkey mod // Continued from line 36 USPS V4 tep_db_query("update " . TABLE_CONFIGURATION . " set configuration_value = '" . $value . "' where configuration_key = '" . $key . "'"); Except for the comments, they are essentially the same, except for these two lines: $key = tep_db_prepare_input($key); $value = tep_db_prepare_input($value); There was an additional line included by ecartz in the version that I use, but it was mainly applied by user greasemonkey when he was trying to get UPS XML to work. I've tried it both ways on my installs and it doesn't seem to make a difference, but I kept it just in case. So, the operative code is this in the NEW Phoenix modules.php: if (is_array($value)) { $value = implode(';', $value); } Whereas, we were previously including this: if (is_array($value) ) { $value = implode( ", ", $value); $value = preg_replace ("/, --none--/", "", $value); } Without knowing exactly what these changes mean, it appears to me that the newest Phoenix version included the corrective code in the modules.php file by default, except for that seemingly superfluous code ecartz suggested for greasemonkey: $value = preg_replace ("/, --none--/", "", $value); The newest Phoenix modules.php includes those new "$key" and "$value" statements above, but those are assigning variables, so I'm not sure there going to interfere with anything (probably something with the Hooks call). My guess is that you should simply try the newest Phoenix modules.php file - without editing anything - and see if it works. Do you have a test store installed? If not, I'd work on that so that you can simply test these things without worry of crashing your live store. Try it and see if it works. As stated, there's definitely extra code and features in the newest modules.php file. However, it appears that Burt and crew have included the important "if (is_array($value)) { $value = implode(", etc., correcting code as standard in modules.php now. Remember that the only thing that this modules.php correction changed was how the module settings in Admin were saved. Without that code (it appears it's included by default, now), if you went to change/edit the module settings in Admin, it would lose/change all the configuration values for the usps.php module. Worst case, you could manually punch those values back into the configuration table in phpAdmin. (Granted, that would be a lot of work for the USPS module, because there's a ton of settings.) If you try it and it doesn't work, maybe we can elevate this to ecartz or someone else and see if they can figure it out.
  8. The only part you insert is that part bounded by the comments for "//Correction …" and "//End correction …" // //Correction for legacy shipping modules // if (is_array($value) ) { $value = implode( ", ", $value); $value = preg_replace ("/, --none--/", "", $value); } // //End correction for legacy shipping modules // Use the rest of the code that I pasted in with that to find the proper place in the modules.php. IOW, look for this section: if (tep_not_null($action)) { switch ($action) { case 'save': foreach ($_POST['configuration'] as $key => $value) { Insert the "//Correction - //End correction" code on the next line after the last "{" shown immediately above. Go back up to my previous post if you don't understand, and look for where that "//Correction - //End correction" code is placed.
  9. Well, you're way ahead of me. I stopped at 1.0.5.0 and am waiting until 1.0.6.0 to upgrade. That said, this is the section of code that is in the modules.php file (/catalog/admin/) that has the corrections. I insert the lines bordered by the comments into the modules.php file, every time I load a new update to Phoenix. I don't think it really matters if there are other changes to the file, as long as the section below stays the same. If you don't insert the commented code below, when you attempt to set the parameters for the shipping module in Admin, you'll lose all of the settings and future attempts won't save it back to the database. (IOW, it becomes corrupted.) The code that I insert is bordered below by the "Correction/End Correction for legacy shipping modules" section (it also applies to UPSxml add-on and I think the FedEx Web services add-on): if (tep_not_null($action)) { switch ($action) { case 'save': foreach ($_POST['configuration'] as $key => $value) { // //Correction for legacy shipping modules // if (is_array($value) ) { $value = implode( ", ", $value); $value = preg_replace ("/, --none--/", "", $value); } // //End correction for legacy shipping modules // tep_db_query("update configuration set configuration_value = '" . $value . "' where configuration_key = '" . $key . "'"); } tep_redirect(tep_href_link('modules.php', 'set=' . $set . '&module=' . $_GET['module'])); break; case 'install': case 'remove': $file_extension = substr($PHP_SELF, strrpos($PHP_SELF, '.')); $class = basename($_GET['module']); if (file_exists($module_directory . $class . $file_extension)) { // include lang file include($module_language_directory . $language . '/modules/' . $module_type . '/' . $class . $file_extension); include($module_directory . $class . $file_extension); $module = new $class; if ($action == 'install') { if ($module->check() > 0) { // remove module if already installed $module->remove(); } $module->install(); $modules_installed = explode(';', constant($module_key)); if (!in_array($class . $file_extension, $modules_installed)) { $modules_installed[] = $class . $file_extension; } tep_db_query("update configuration set configuration_value = '" . implode(';', $modules_installed) . "' where configuration_key = '" . $module_key . "'"); tep_redirect(tep_href_link('modules.php', 'set=' . $set . '&module=' . $class)); } elseif ($action == 'remove') { $module->remove(); $modules_installed = explode(';', constant($module_key)); if (in_array($class . $file_extension, $modules_installed)) { unset($modules_installed[array_search($class . $file_extension, $modules_installed)]); } tep_db_query("update configuration set configuration_value = '" . implode(';', $modules_installed) . "' where configuration_key = '" . $module_key . "'"); tep_redirect(tep_href_link('modules.php', 'set=' . $set)); } } tep_redirect(tep_href_link('modules.php', 'set=' . $set . '&module=' . $class)); break; } } Note that as mentioned in an earlier post up there, you must add these items to the configuration table in the database: Also note the change recommended by ecartz that I had to add to the usps.php file (line 126): Anyway, see if that works -
  10. I was not going to post in this thread again, since I sort of chimed in with the complaints. However, my complaint was centered around the frequency and speed of updates and my inability to keep up with a live store. I've since come to the conclusion that as a Phoenix Club member, I'm more directly exposed to the development environment than ever before and that was what was frustrating. Perhaps back in the day, 99% of us would never have been exposed to that and we were happy to implement major upgrades when they were announced (or not, as the case used to be). So, I'm trying to find my place, learn along the way, and hope that I can perform the major upgrades when they occur, as in 1.0.6.0, 1.0.7.0, etc. However, I do not know why anyone would think "Access to the code is severely limited." Perhaps becoming a Supporter allows one access to certain code, but the fund-raising is absolutely necessary for any of it. The basic Phoenix is still left untouched, consistently updated, and completely transparent. Meanwhile, there are many add-ons that are universally available and some of the key Phoenix Supporters/Coders take it upon themselves to keep them updated - for free.
  11. Oh no - so sorry for that. I hope that your business returns soon.
  12. I don't think we are trying to be discouraging; just realistic. It's not actually a fair assessment. I can only speak for myself, but I have at least 3 add-ons whose original developers are either long gone, or refuse to support the newer changes required. If I lose one particular add-on (USPS shipping), I'm out of business. So, if I applied the 5-10 minute upgrade, it could result in days or longer before getting help with an error. Meanwhile ... if I beg for help, I get responses suggesting that I pay for an outside developer to fix things. If I could pay for an outside developer, I'd never have gone with OsCommerce in the first place. (I've actually paid for something in the past, but never got anything for it.) Yeah, if we back things up we can put it back in short order, but that's still scary as hell with a live store. Sometimes, even after checking as extensively as you can, things break a week or two later on something you never predicted. Meanwhile, you've had some new orders and new customers added to the database. How do you go back to the backup then? Perform an incremental database update/conversion again? Your store is broken and it might even be in the database. Heck, I'm taking a chance just going with Phoenix as it is. What do I do when USPS changes something again? Please understand. I didn't set out to complain. Phoenix is a wonderful project and I'm 100% behind it and you. But you seem to be asking where these views come from. 😉 EDIT: Just to be clear, all those Russian fake user accounts with 2.3.4 was driving me crazy. Phoenix and the Supporters Code stopped that completely (well, I've had one in about a month - used to get 6-12 per day). So, taking the chance with Phoenix has been very, very, well worth it. There's just some nagging worries that continue ...
  13. Another one here with the same strategy - went live with 1.0.5.0, but will wait for 1.0.6.0 before attempting another upgrade. Kind've getting antsy about it now, though.
  14. @ecartz and @BrockleyJohn, I greatly appreciate you both adding to the thread and offering suggestions! I agree that USPS regularly breaks the API from time-to-time. That's definitely been the history since I've been using OsCommerce, over a decade ago. However, this is not new. @ArtcoInc had the very same issue a couple of pages back in this thread, in early September of 2019: About the only difference is that his error message says line 132; mine is line 126 - but it's the same foreach statement. I implemented the very first suggestion that @ecartz made like so: foreach (($Package['ExtraServices']['ExtraService'] ?? []) as $key => $val) { That appears to have worked! As BrockleyJohn said, maybe it was just a warning (but horribly presented to customers) and it looks like it's now suppressed. MANY THANKS!
  15. OK, it's not the USPS server. The error is 100% consistent and repeatable this morning on my live store: Warning: Invalid argument supplied for foreach() in/***/***/***/***/catalog/includes/modules/shipping/usps.php on line 126 The foreach loop in the usps.php file is as posted above: 126. foreach ($Package['ExtraServices']['ExtraService'] as $key => $val) { if ( is_array ( $val ) && array_key_exists ( 'ServiceName', $val ) && isset ( $iExtras[$val['ServiceName']] ) && tep_not_null($iExtras[$val['ServiceName']]) && ((MODULE_SHIPPING_USPS_RATE_TYPE == 'Online' && $val['AvailableOnline'] == 'True') || (MODULE_SHIPPING_USPS_RATE_TYPE == 'Retail' && $val['Available'] == 'True'))) { $val['ServiceAdmin'] = $iExtras[$val['ServiceName']]; $Services[] = $val; } This only happens on International addresses, not domestic (US) ones. The ExtraServices are as follows: Registered Mail, N, Insurance, N, Return Receipt, N, Restricted Delivery, N, Pick-Up, N, Certificate of Mailing, N As can be seen, I've selected NO (N) ExtraServices. The error is printed 12 times ahead of drawing in the checkout_shipping.php page on the browser. Once the checkout_shipping.php page is drawn in, there are 3 USPS quotes, and a couple each of UPS and FedEx quotes, now all showing correctly. I guess it's going through the foreach loop 12 times, which is why the error message prints out that many times. As best I can tell using file comparisons, etc., the files and database settings on both my live store and test store are identical, but only the live store shows the errors. In addition, I am manually re-writing the admin/modules.php file with each Phoenix update (I'm at 1.0.5.0) with the following code inserted: // //Correction for legacy shipping modules // if (is_array($value) ) { $value = implode( ", ", $value); $value = preg_replace ("/, --none--/", "", $value); } // //End correction for legacy shipping modules // Ideas? Help?
×