Olly

Forum Replies Created

Viewing 20 posts - 41 through 60 (of 4,441 total)
  • Author
    Posts
  • Olly
    Admin & Mod

      Ok, so:
      Elementor adds lots and lots of surrounding elements to the orderpage widget (about a dozen or more)
      Interestingly, after payment (i.e on the so-called “thankyou page”) it does not add those wrapper elements anymore.
      Why ? I have no idea.
      That said – given the popularity of elementor – I’ll have a look as to whether it’s something they should perhaps be doing or I should perhaps be doing and/or find a workaround/fix if applicable. I simply do not know yet where the issue is exactly
      so will probably not be something that can be done quickly as it may have many implications elsewhere.

      That said, if the only page where you are having issues is your thank you page, there is certainly an easy fix by simply adding the following (adjust as required) to your css somewhere (see https://docs.wp-pizza.com/advanced/?section=frontend-css-styles-layout). This is a very specific override to that widget there so should be quite safe to use

      
      .wppizza-order-wrap-thankyoupage{
      padding-top: 100px;
      margin: 0 20%;
      }
      

      PS: if you enable debug on your site you will see that elementor throws lots and lots of database errors. which may – or may not – be related. Just mentioning it though that really is something for them to fix.

      That’s it from me for now on this subject

      Olly
      Admin & Mod
        This reply has been marked as private.
        Olly
        Admin & Mod
          This reply has been marked as private.
          Olly
          Admin & Mod

            out of curiosity, if you want to make a duplicator package available for me somewhere, I can have a look (i assume you know what i am talking about)

            Olly
            Admin & Mod

              works just fine here with Elementor …

              Olly
              Admin & Mod

                as you are using a commercial theme I can only advise you to the usual troubleshooting as per
                https://docs.wp-pizza.com/troubleshooting/

                if you disable all plugins except the main wppizza one and only get the issue with the theme you are using but not any default theme , you will have to speak to the theme author.
                I will not be able to advise you as to what your theme’s issues may be (as it is commercial, I will not be buying it as i have no use for it)

                PS: You should make sure to also check your javascript console too for some clues

                Olly
                Admin & Mod

                  As I know nothing about the “Pizzon” theme, all I can only really say is :
                  Does it work as expected with a default 20something theme ? If so , it’s a question for the “Pizzon” theme developer

                  As it’s all a bit abstract though, do you have a screenshot of your thank you page you can share ?
                  Ideally with some comments as to what you expect there to be different if it’s not that obvious

                  PS: woocommerce usage (or not) is not relevant .

                  Olly
                  Admin & Mod

                    >but the widget loads in an empty Theme site
                    what widget please ?

                    That said, if you are using the “Avada” theme, then the fault is there and they need to fix this – they know about this but seemingly refuse to do anything about this .
                    (Regardless of what they say, they are simply using the ‘the_content’ filter multiple times on the same page – with the results you can see )

                    Olly
                    Admin & Mod

                      Hi
                      apologies for the delay – I had responded in my head, but never put it to paper so to speak.
                      Regardless, looking at your site , your (wppizza related ) plugins – specifically the post/zipcode plugin – are several years out of date, so I would suggest you update those to to most recent version to start off with.
                      Furthermore, enabling debugging is also always a good thing to do if you have any issues ( see https://docs.wp-pizza.com/troubleshooting/ )

                      PS: your site also links to localhost images/links that do not work obviously

                      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

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