Olly

Forum Replies Created

Viewing 20 posts - 41 through 60 (of 4,430 total)
  • Author
    Posts
  • 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 ?

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

                                          I can see where this might come in useful, so as of v5.3 of the ingredients plugin, you can now add/assign your defined allergens and/or additives to each ingredient.

                                          Hope that helps

                                          in reply to: interface #68772
                                          Olly
                                          Admin & Mod

                                            As I do not know anything about your cash register – or indeed what it is you want it to do – I can only refer you here in general

                                            https://docs.wp-pizza.com/developers/

                                            and here in particular (the most likely “interface” you are looking for , but I cannot know)

                                            https://docs.wp-pizza.com/developers/?section=action-wppizza_on_order_execute

                                            Hope that helps

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