Dirty Butter Posted June 13, 2015 Author Share Posted June 13, 2015 Brian, I'm too tired to keep working on this tonight. Will be back on in the morning. I'll double check my edits in the AM, since I have a bad habit of messing up your directions. Quote Link to comment Share on other sites More sharing options...
bsmither Posted June 13, 2015 Share Posted June 13, 2015 I think you missed my very last edit. https://forums.cubecart.com/topic/49895-usps-canada-parcel-rate-zone-change-causing-issue/?do=findComment&comment=209828That should be all the edits.NOW you can test.((Holy Moley! This editor has some serious problems!!)) Quote Link to comment Share on other sites More sharing options...
Dirty Butter Posted June 13, 2015 Author Share Posted June 13, 2015 Yes, I did miss that last edit. And YES! That fixed it. It doesn't help at all that the new code box does not provide line numbers. Quote Link to comment Share on other sites More sharing options...
bsmither Posted June 13, 2015 Share Posted June 13, 2015 I was trying to get a link to apply to some text, but the underlying link had a part that ended up looking like:?do=findComment&comment=209828which does not work as the href attribute of an anchor tag. Quote Link to comment Share on other sites More sharing options...
Dirty Butter Posted June 13, 2015 Author Share Posted June 13, 2015 I appreciate your time and effort to make this work. I had reported to CC that this USPS update was coming, but at the time they didn't think any changes would be needed. Well, obviously they did. Could you please re-open this one, with your fix?https://github.com/cubecart/v6/issues/588 Quote Link to comment Share on other sites More sharing options...
Al Brookbanks Posted June 17, 2015 Share Posted June 17, 2015 Rosemary would you mind sending me a zip of your new working USPS module (or just the one changed file)?[email protected]Cheers! Quote Link to comment Share on other sites More sharing options...
Dirty Butter Posted June 17, 2015 Author Share Posted June 17, 2015 Sent via email. Messaging here threw error on attachment. Quote Link to comment Share on other sites More sharing options...
Al Brookbanks Posted June 17, 2015 Share Posted June 17, 2015 Quote Link to comment Share on other sites More sharing options...
Al Brookbanks Posted June 17, 2015 Share Posted June 17, 2015 Thanks yes more forum issues I've raised a ticket with them. Quote Link to comment Share on other sites More sharing options...
Dirty Butter Posted June 17, 2015 Author Share Posted June 17, 2015 I forgot to mention, my copy does have an edit from Bsmither to leave the estimated Priority Mail "days" off the shipping choice names. Quote Link to comment Share on other sites More sharing options...
Al Brookbanks Posted June 18, 2015 Share Posted June 18, 2015 I did a comparison of the standard code and Rosemarys. I can see some bits are tweaks to get the services to display without timeframes etc but the rest of the changes don't seem that relevant. I can't see how these code changes fix this. I fear introducing issues from code changes I don't understand. Quote Link to comment Share on other sites More sharing options...
Dirty Butter Posted June 18, 2015 Author Share Posted June 18, 2015 shipping.class.phpMaybe I sent you the wrong one - sending again, just in case.(Might as well get rid of "bathroom language" in CC comment while you're at it :)) Quote Link to comment Share on other sites More sharing options...
Al Brookbanks Posted June 19, 2015 Share Posted June 19, 2015 Thanks to you and Brian! Quote Link to comment Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.