Jump to content
Latest News: (loading..)


  • Content count

  • Joined

  • Last visited

  • Days Won


BrockleyJohn last won the day on January 29

BrockleyJohn had the most liked content!


Profile Information

  1. NEW! Complete Order Editing Tool!

    That doesn't look like your mistake, it looks more like the code didn't work properly. Did you get that on the single order view (ie. while looking at admin/orders.php?oID=2&action=edit) or the list of orders view (admin/orders.php?oID=7) ? If you refresh that page and try again, do you still get the error?
  2. Sorted now. I had to install a separate up-to-date version of git and use that instead of the embedded one: https://stackoverflow.com/questions/48944875/sourcetree-error1407742essl-routinesssl23-get-server-hellotlsv1-alert-protoc Thanks for the help.
  3. Thanks @burt glad you're back up. After a lot of digging I tracked down my sourcetree putty installation in C:\Users\John\AppData\Local\SourceTree\app-2.4.8\tools\putty (at which point it decided it should only be available to one user, I know not) but changing the files there hasn't helped. It's not the only set of Putty files so I'll have to go through the lot, I guess.
  4. Yes - a bit off-topic but I guess most of the users are in this forum. https://githubengineering.com/crypto-removal-notice/ Since Github only accepts TLS1.2 on 22 Feb I can't push, pull or clone from Sourctree on Win7. error:1407742E:SSL routines:SSL23_GET_SERVER_HELLO:tlsv1 alert protocol version Have tried updating to very latest Sourcetree version, setting only TLS1.2 under Options / Network to no avail. Still investigating solutions... Quick poll - is yours working?
  5. NEW! Complete Order Editing Tool!

    @Tsimi I've no argument with that. A modularised approach with common code between the edit and ajax files is a very good idea too, and if well structured could readily support additional addons which would typically only hit one of the functional areas (eg. add/remove products for QTPro, SPPC, probably SSPP).
  6. NEW! Complete Order Editing Tool!

    ...but it turns out the javascript expects the images to be where they were in the package. Time to start making a list! Perhaps your version of the editor might make a better starting point @piernas. Is it on github?
  7. NEW! Complete Order Editing Tool!

    It explains why my test site doesn't have the pretty icons in Pierre's screenshot! I was commissioned to make a light-touch install of it so didn't go deeply into the actual editor itself, just plugged a lot of obvious holes and made sure that it stood up and appeared to work. That said I'm quite happy to apply bug fixes, test and put them to the apps area, so keep them coming. I can confirm that moving the images folder from where it was in the previous package does make them appear.
  8. NEW! Complete Order Editing Tool!

    @dr_lucas if you send me your edit_orders file I can look at merging in your bug fixes to the latest version and @Peper can test it. For the tax enhancement, we'll have to see how much work is involved...
  9. NEW! Complete Order Editing Tool!

    Have you tried it on an order with a shipping subtotal?
  10. NEW! Complete Order Editing Tool!

    yep - some are (int) casts for the ids and everything else is tep_db_input
  11. NEW! Complete Order Editing Tool!

    Sorry, you're right the rate is zone-specific; I should have asked what tax class applies. From a quick look at it, that fix should work as long as you add the extra subtotal after the shipping subtotal because it uses the tax rate from the shipping module to calculate the tax on the custom subtotal. If the custom subtotal is added before shipping, it'll come out zero. Your sample data doesn't seem to have a shipping subtotal at all. A proper solution would add a field on the additional custom order total for tax class. No class = no tax, otherwise apply tax based on class on subtotal. You have to be able to display the applicable tax rate on the invoice too.
  12. NEW! Complete Order Editing Tool!

    @Peper Yes I understand what it doesn't do. In order to be applied as you ask it needs to know: whether your custom subtotal is inclusive or exclusive of tax what tax rate applies
  13. NEW! Complete Order Editing Tool!

    @Peper what exactly are you wanting to happen?
  14. [CONTRIBUTION] Ultimate SEO URLs v2.1 - by Chemo

    ...but if you haven't installed Header Tags SEO and for some reason don't want to, the easiest change is to make the second model query the same as this too.
  15. [CONTRIBUTION] Ultimate SEO URLs v2.1 - by Chemo

    @brian_cee if in your store you want to output the product name unless you have overridden the breadcrumb in Header tags, you should change the first query from $model_query = tep_db_query("select coalesce(NULLIF(pd.products_seo_title, ''), p.products_model) as products_model from products p, products_description pd where p.products_id = '" . (int)$_GET['products_id'] . "' and p.products_id = pd.products_id and pd.language_id = '" . (int)$languages_id . "'"); to $model_query = tep_db_query("select coalesce(NULLIF(pd.products_seo_title, ''), p.products_name) as products_model from products p, products_description pd where p.products_id = '" . (int)$_GET['products_id'] . "' and p.products_id = pd.products_id and pd.language_id = '" . (int)$languages_id . "'"); and set Breadcrumb override to True