Olly

Forum Replies Created

Viewing 20 posts - 41 through 60 (of 4,432 total)
  • Author
    Posts
  • in reply to: Error message for Delivery By Post/ZipCode #69233
    Olly
    Admin & Mod

      that would indicate you either have a serious issue somewhere (see your debug.log – https://docs.wp-pizza.com/troubleshooting/ ) or you have been/are caching your pages (or both)

      in reply to: Error message for Delivery By Post/ZipCode #69214
      Olly
      Admin & Mod

        Hi

        looking at your site, you are running a version of the postcode plugin that is over 7 years old (v 4.0.7 – Oct 2017) , so I would first of all suggest you update it to the most recent version (v5.2.3 as of writing).

        in reply to: WP-PIZZA Language Setting Changed After WordPress 6.7 Update #69204
        Olly
        Admin & Mod

          Just for info:

          It appears that WP 6.7+ does not want to read two text domains in the same plugin (anymore).
          The wisdom of that escapes me, but alas this is what it seemingly is.
          The good thing is – talking about wppizza related stuff – that this only affects the main wppizza plugin.
          All/any wppizza extensions should be unaffected I believe.

          I’ll see if I can still somewhat keep admin and frontend translation files separate in a different way and will release an update asap.
          Worst case scenario, I’ll have to merge the 2 language files – which would not really be that dramatic, just a bit of a nuisance

          We’ll see how it goes. I would think it will all be good again in the next few days – allowing for some testing .

          Thanks for the heads-up (for temporary workaround see above)

          in reply to: WP-PIZZA Language Setting Changed After WordPress 6.7 Update #69197
          Olly
          Admin & Mod

            PS: as a quick fix until either WP updates things their end or I find a suitable workaround – whatever happens first –
            you could copy the language files you need – i suspect in your case wppizza-admin-de_DE.mo and wppizza-admin-de_DE.po from wp-content/plugins/wppizza/lang into wp-content/languages/plugins
            so they will be read form there

            in reply to: WP-PIZZA Language Setting Changed After WordPress 6.7 Update #69194
            Olly
            Admin & Mod

              on second thoughts , it really looks like WP messed up here and broke things …
              see: https://core.trac.wordpress.org/ticket/62337

              I’ll follow that thread for a while and dig around a bit anyway, but will probably wait until WP 6.7.1 until I do something about this ( for a few days anyway).

              in reply to: WP-PIZZA Language Setting Changed After WordPress 6.7 Update #69193
              Olly
              Admin & Mod

                Not sure if this was intentional or a mistake by WP’s devs, but suffice to say that’s quite a nasty change that happened there and I suspect will affect lots of plugins and theme’s (WPPizza or not)

                Relatively easily fixed I think (though I have not yet 100% worked through all the the implications) but needs an update to the wppizza plugin (as well as – almost all – associated extensions) .
                So that’ll be me being busy for a few days then…
                I’ll make updates available asap – starting with the main WPPizza plugin of course..

                Thanks for flagging it, I probably would not have noticed for quite some time otherwise.

                in reply to: New ingredients functions #69173
                Olly
                Admin & Mod

                  While I was messing around with various things it occurred to me that this sort of thing already exists in the pickup prices plugin – https://www.wp-pizza.com/downloads/wppizza-pickup-prices/

                  you simply need to enter -0 into any pickup price for an item . If you apply the -0 to all pickup-prices (i.e variations) of an item, the whole item will become unavailable on pickup…

                  in reply to: New ingredients functions #69104
                  Olly
                  Admin & Mod

                    wppizza is a custom post type, category__not_in will not work there. you will need taxonomy queries

                    
                    $args['tax_query']['relation'] = 'AND' ;
                    $args['tax_query'][1]['taxonomy'] 			=  WPPIZZA_TAXONOMY ;
                    $args['tax_query'][1]['field'] 			=  'id' ;
                    $args['tax_query'][1]['terms'] 			=  array(16) ;/* 16 being your beer/wine cat */ 
                    $args['tax_query'][1]['include_children'] 			=  false ;
                    $args['tax_query'][1]['operator'] 			=  'NOT IN' ;
                    

                    that said, having thought about this a bit more, I think this is actually going down the wrong path as you would also have to find a way to reload the page when switching between pickup/delivery . Do-able but somewhat unnecessarily complicated altogether if you ask me.

                    perhaps a (somewhat) better approach is actually some js here .

                    every item has a title with an id , that is made up of blogid, catid, postid, sizes(tier)id, in tier id like

                    
                    <h2 id="wppizza-article-1-2-8-3-0" class="wppizza-article-h2">
                    

                    so you can select all ‘wppizza-article-h2’ on a page , get the id’s , split by ‘-‘ to then get all ‘cat ids’ so to speak
                    and show/hide depending on pickup/delivery value and when switching between the two (you’d still need to set the session in the cart accordingly with a php filter)

                    advantage being you do not have to reload any pages so much faster (still many pitfalls on the way to be encountered , but personally i think a better approach)

                    I might make this sort of thing into a plugin myself as i can see the possible need for this in certain circumstances, but at the moment we are where we are

                    in reply to: New ingredients functions #69100
                    Olly
                    Admin & Mod

                      PS: taking items off the page would not stop an item from being added to the cart first and then pickup being changed to delivery (or vice versa).
                      If you want to prevent that too , it gets a lot more complicated although you could perhaps run/add the filter below as well and removing the non-permitted items in the cart as required

                      
                      $session = apply_filters('wppizza_fltr_session', $session);
                      

                      (though not tested in any shape or form – just an idea off the top of my head)

                      Alternatively you of course have two sites (multisite setup perhaps) one for pickup, one for delivery

                      in reply to: New ingredients functions #69099
                      Olly
                      Admin & Mod

                        you would need to put your dough selection it into it’s own (custom) group . (with a “…choose one only…” setting I would think)
                        Otherwise it’s just another ingredient.

                        Regarding pickup / delivery :
                        there’s a ‘wppizza_filter_loop_args’ filter that lets you set the arguments (i.e the posts returned). so in conjunction with

                        
                        if(wppizza_is_pickup()){
                        /* do something if pickup selected*/
                        }
                        

                        for example (https://docs.wp-pizza.com/developers/?section=global-wppizza-functions)
                        I would think you could exclude posts (i.e post id’s ) as required . Probably simply getting all wppizza posts(ids) that belong to your beverages category and exclude/include as required

                        alternatively, some conditional – again , based on wppizza_is_pickup() that simply removes the beverages shortcode of the page where you have it (or exclude that page entirely from your site when it’s pickup)

                        Up to you which way you want to do this of course

                        in reply to: New ingredients functions #69068
                        Olly
                        Admin & Mod

                          sounds like you need to setup/add an “exclude” ingredients custom group

                          in reply to: Add Additives to Ingredients #68929
                          Olly
                          Admin & Mod

                            Ok, now you are describing this, I can reproduce this (it’s got to do with your first one having an ident of zero most likely)

                            I’ll fix that

                            in reply to: Add Additives to Ingredients #68927
                            Olly
                            Admin & Mod

                              works just fine here. did you check your debug log and browsr console for errors ?

                              in reply to: Add Additives to Ingredients #68880
                              Olly
                              Admin & Mod

                                FYI: v5.3.1 should have this fixed now

                                in reply to: Add Ingredients Plugin Version 5.3 – Error Message Appears #68879
                                Olly
                                Admin & Mod

                                  as of 5.3.1 this should be fixed now, but of course , let me know if you still have issues

                                  in reply to: Add Ingredients Plugin Version 5.3 – Error Message Appears #68863
                                  Olly
                                  Admin & Mod

                                    I’ve no idea what happened there.

                                    I’ll fix this in the next couple of hours or so with 5.3.1 update .
                                    In the meantime, you could simply change

                                    'name'=>$this->render_tags($v['item']) . $this->render_additives($k, $v),
                                    back to
                                    'name'=>$this->render_tags($v['item']) ,

                                    so it stops throwing those errors (or of course downgrade back to 5.2.2 – whatever is easiest)

                                    Apologies for messing this up

                                    in reply to: Add Additives to Ingredients #68857
                                    Olly
                                    Admin & Mod

                                      hold off on updating. I introduced an errror in 5.3
                                      will be fixed asap

                                      in reply to: Add Ingredients Plugin Version 5.3 – Error Message Appears #68854
                                      Olly
                                      Admin & Mod

                                        forget all of that . the render_additives function got lost in the upload somehow.
                                        I’ll fix that asap.

                                        thansk for reporting

                                        in reply to: Add Ingredients Plugin Version 5.3 – Error Message Appears #68853
                                        Olly
                                        Admin & Mod

                                          also, have you checked your debug.log ?
                                          is there anything i there ?

                                          in reply to: Add Ingredients Plugin Version 5.3 – Error Message Appears #68852
                                          Olly
                                          Admin & Mod

                                            can you go to the (admin) ingredients tab and simply save your ingredients as they are once ?
                                            if you do that , does your error go away ?

                                          Viewing 20 posts - 41 through 60 (of 4,432 total)