Jump to content

KirkM

Member
  • Posts

    182
  • Joined

  • Last visited

  • Days Won

    5

Everything posted by KirkM

  1. No no. I have notes of what files have been changed in the skin. I will run DiffMerge on each of them and carefully update the code on the new CC version skin. I have very clear remarks on every change. I just need to carefully examine the code between the old and new to make sure I understand any changes in the new that may affect my modifications. It is slow and tedious. And, as I said, I have to think. Hate that.
  2. Thanks Brian. I have a filediff program. I was just wondering if I needed to go through it file by file. Just lazy but since things are pretty significantly re-arranged on my Foundation skin variant, I have to sit and transpose the differences instead of viewing the difference pages and approving the automatic code block updates left to right in the file diff edit window. I hate it when I have to actually do some work. Even worse when that work requires me to think. Thinking is hard.
  3. Just wanted to pop back in and thank Brian and Al (and whomever else worked on the update) for the php 8.1 compatibility. Finally got a rare minute to jump over and do some store upgrades and then switch them to php 8.1. As always, your work is very much appreciated. One question - Do you know if anything changed in the Foundation skin between 6.4.4 and 6.4.7? I have a skin that is based on Foundation but is significantly altered in its element arrangement so I would like to not rebuild it if there is no reason to. Thanks
  4. Thanks so much for finding that Brian. Definitely a silly oversight in the code. Your fix seems to have solved the issue. This is certainly a bug that needs to be fixed. Will you be reporting it or would you prefer I do it? As always, your thorough work and prompt solutions are greatly appreciated.
  5. Yes. The owner and I both have accounts on his store so we can simulate purchase and checkout. It did the same thing for both of us when we were logged in. Out of curiosity, would that make a difference? I thought that CC tracked customers by email address, whether they have an account and are signed in or not. I assume that would be relational to the coupon uses counting.
  6. No. In phpMyAdmin sorting by customer_id shows the lowest number to be 130. No zero values in that column. Not sure I understand this one. There are hundreds of rows with the same coupon code as he has had hundreds of customers use coupons over the years. Do you mean if the first test found a zero? If so, this test is moot, I suppose.
  7. Sorry, I normally always post that info but was rushing out the door when I typed it and completely forgot. "There is never enough time to do it right, but there is always enough time to do it over.” ~John W. Bergman PHP 7.4, CC 6.4.4
  8. A customer put in a coupon code with global unlimited uses and one use per customer. When used the first time, it displays an error message that the limit has been reached even though this is the first use. If you set it to 2 uses per customer, it works on the first use and not the second. Leave it blank for unlimited per customer and it works fine too. This would make it appear that the code logic is counting the use BEFORE checking if the limit has been reached. It sounds kind of silly for a mistake like that to be in the code flow, but testing sure makes it seem that way. Anyone else see this?
  9. Thanks Brian for confirming my suspicion that there was individual item cascade rounding going on instead of sum subtotal rounding. I am pretty sure in the U.S. that the tax is always applied to a taxable subtotal. Even if there are exempt items in a purchase, they are just supposed to be excluded from the subtotal taxability (sort of a ghost subtotal for tax calculation). A penny isn't a huge deal, but his problem is it doesn't square up with his accounting books and for his state tax reporting. Technically, he is illegally charging too much tax when this anomaly occurs. While no one thinks he is looking at jail time for a few cents here and there, it is wrong, at least in this country and / or state. It would be interesting to know if this is a response to UK or EU tax codes. If not, then it is a flawed process that has yielded this issue. Alas, it appears I will have to go back to him, shrug my shoulders and tell him not much I can do at this point. Appreciate your help sir, as always.
  10. OK, so I guess nobody cares about this bug...
  11. A client in California has noticed that occasionally the tax is rounded to the wrong penny. As illustrated below, his tax rate is a flat 9% and the total of $140.04 at 9% would be .09 x 140.04 = 12.6036. This should round DOWN to $12.60. However, CC 6.4.4 is rounding it UP to $12.61. My wild guess is that there is other rounding taking place upstream that is delivering something above 12.605 to the final tax rounding or something like that but I really have no idea. Has this been seen before?
  12. My guess: Boooooom! Really, you don't have to go through all that. I don't want to hack CC just because I am a type A PITA. I can wait for everything to catch up in a future release... I swear I can... Really... ahhhhhhhhhhhh!
  13. That would be very generous of you to do. PHP 8 has only been out since the very end of November 2020, so that's just under 18 months. Lots of changes and depreciated / removed functions and conventions so it will understandably take a while for everyone to clean their code. All my stuff works fine on 7.4 too, I just am a bit impatient to work with some of the 8.1 improvements. I have it installed in my MAMP Pro testing server and develop only with that to future-proof as far out as possible so it is important that I have it on my production server, which I do for the domains I use. For CC, I can just leave it on 7.4 until all is ready and then I will clean 7.4 off the server. As always, thanks for your help and response Brian. It is greatly appreciated.
  14. Thanks Brian. Do you know if Smarty 4 in CC645? Obviously, if a single third-party component isn't compatible, then the whole thing isn't. I run PHP as a PFM Application per domain so I will keep 7.4 for the CC clients and wait for the plugins to catch up if they haven't yet. It would be great if Al specifically announces when the CC version is released that is tested compatible with PHP 8.1. (wink, wink, nudge, nudge)..
  15. Searched around and didn't see much about CC compatibility with the latest php version 8.1. Just a single post back in 2021 or so about problems with php 8.0 breaking CC. What's happening with CC and php 8.1? I am using CC 6.4.4 with php 7.4 right now (CC still recommends using php 7.3, which is now outdated), but all my data systems and some client wordpress sites are moved to php 8.1. CC is the only thing hanging me up from having 8.1 as the only php version on my server. Any info on a timeline for this would be greatly appreciated.
  16. Well, I feel like an idiot (not the first time). I was totally focused on where the coupon was applied and not on the shipping module. Uncommenting that line of code did the trick and makes perfect sense. It was right there in the shipping.class.php file, with a big flag of a comment to tell dummies like me "HEY! right here!". Sheesh. Thanks Brian for always jumping in to help. Your remark made me take a look over at the AIOS class after being so focused on the coupon class code.
  17. This is commented in the AIOS class: $this->_value = (float)$this->_basket['subtotal']; // XXX May want to remove coupon discount, i.e. // $this->_value -= $this->_basket['discount']; Could it really be as easy as uncommenting that line of code?
  18. Sorry, I realize this is not really clear. The example of what he needs is over-simplified with just the one product. There could be multiple products. The coupon in this case is for $15 off the whole order. Plus, his promotion also has free shipping for all orders $49 or more. He needs the shipping to be considered AFTER all coupons and any other discounts are deducted. The way the store works now, the coupon looks at the subtotal and the shipping looks at the subtotal ignoring the subtraction for the coupon, because the coupon also happens AFTER the subtotal. There is no consideration by the shipping for a modified subtotal because of the coupon. He will always need the shipping promotion to be on the final amount AFTER any discounts. To me, he is right, that is the way this should work or at least have the option for it to work that way. I hope that is a bit clearer on what the issue is with the current way CC handles the checkout flow.
  19. Been trying to figure out a way to get the coupon to be part of the calculation of subtotal. The more I examine the function of the cart / checkout process from the store owner's point of view and possible marketing approaches, the more I see that the coupon logic is deficient in it's ability to be used in conjunction with other marketing tools. I realize my point of view is rather simplistic since I am not intimately familiar with the code, but I have been reading through the relevant scripts and it seems that even with the complication of coupons for particular products or other variables, those discounts should be taken BEFORE the subtotal that all other things act upon, like shipping. Do you have any suggestions or ideas of how to hack this? Combining coupons with variable shipping zones / rates is pretty common for my client and seems to me to be a pretty basic marketing approach. Thanks.
  20. All in one shipping. What I was thinking was that the coupon should have the user choice of either calculating on the subtotal or as part of the calculation of the subtotal. That way the shipping or any other module can stay looking at the subtotal for their calculations. It would be a really nice flexibility feature in the coupon module.
  21. Interesting situation with a client on CC 6.4.4: He has shipping set up for a fixed rate of $7.95 for orders $0 - $48.99. He has another rate of free shipping for $49.00 - $100,000. He also has a coupon for $15 off for new customers. He needs the coupon to be subtracted BEFORE the store calculates the shipping rate. Here what is happening: Product Z 49.96 Subtotal 49.96 Certificate -15.00 Free Shipping Total 34.96 What he needs: Product Z 49.96 Certificate -15.00 Subtotal 34.96 Ship Charge 7.95 Total 42.91 I don't see a way to get this to happen. Anyone know if this is possible?
  22. OK, I got it. Once again, I got bitten by the "security by obscurity" silliness of CC. Updated the admin folder but not the old obscured admin folder before I ran the upgrade. Loaded the files manually to the new obscured folder and all is good now. I REALLY hate that "feature".
  23. No. Only Authorize.net Accept Hosted, Paypal Commerce Platform and All In One Shipping modules are enabled.
  24. OK, I see the bug you are referring to appears to be in the Print Order Form extension. I don't use that extension. I am getting this in the admin interface when viewing an order. Did I miss something in that bug thread on Github?
×
×
  • Create New...