• Hey, thanks for your help regarding the other issue (although I am not sure why it is marked as resolved already). I went ahead and uploaded the WPBakery source for your testing

    This is an entirely separate issue regarding the upload of variation images inside of the stock WooCommerce App.

    You will notice that images uploaded as “Additional Images” inside of the variations: https://i.imgur.com/JvslzP4.png

    Are not translated into the offloaded URL once the options are selected for that product: https://i.imgur.com/9d29DTF.png

    The page I need help with: [log in to see the link]

Viewing 15 replies - 1 through 15 (of 17 total)
  • Plugin Author Anton Vanyukov

    (@vanyukov)

    Hi @ellasbubbles,

    I have checked WooCommerce and I do not see the option to add additional images to variations, as in your screenshot. Could you please let me know how to enable this functionality so I can test this out?

    Best regards,
    Anton

    Thread Starter ellasbubbles

    (@ellasbubbles)

    Thank you for looking into this further, as this is about 80% of relevant images on our website (and most woocommerce sites).

    Please add a variation to your product, and then add the main image, there will then be blue text “Add additional images”: https://i.imgur.com/6AXT3xP.png

    I have dropped a test admin login through the contact form at https://vcore.au/contact-us/ as well

    Plugin Author Anton Vanyukov

    (@vanyukov)

    @ellasbubbles , I have updated the beta with the fix: https://github.com/av3nger/cf-images/releases/tag/1.9.2-beta.1

    Best regards,
    Anton

    Thread Starter ellasbubbles

    (@ellasbubbles)

    NOT RESOLVED: I just updated the plugin on site, and can confirm that the first image of the chosen variations does deliver from imagedelivery; however please check the additional images (by changing the slider to a different image). These do NOT deliver from imagedelivery still: https://ellasbubbles.com/walk-in-tubs/stainless-steel-door/elite/

    Plugin Author Anton Vanyukov

    (@vanyukov)

    @ellasbubbles, you’re right. Fixed. Updated the same beta

    Best regards,
    Anton

    Thread Starter ellasbubbles

    (@ellasbubbles)

    RESOLVED: Tested + 5 Starred Plugin.

    Thank you

    Thread Starter ellasbubbles

    (@ellasbubbles)

    UPDATE [60s+ Load Time]: We are forced to downgrade the plugin due to a 60s+ load time introduced to product pages: https://i.imgur.com/qJm4yac.png

    Please note that Plugin: cf-images remains the slowest query on ALL pages

    Queries should be optimized not to fetch repetitively

    Plugin Author Anton Vanyukov

    (@vanyukov)

    Hi @ellasbubbles,

    I had a look at your screenshot and I was able to replicate that query. Although, it doesn’t take 60s+ on my end, I’ll try to explain what’s happening there.

    Regarding caching and repetitive fetches. The query runs 1 time per each new image. As of WordPress 6.1 the WP_Query calls are cached by default, unless explicitly requested not to cache. So those queries should be performing well. If there’s excessive use on the database, the query time can stack up, but my plugin is not the root cause of the slowness here.

    Now, why do we even need those in the first place, right? The queries originate from RankMath Images SEO module. If you have that enabled in Rank Math and are using the %filename% attribute in either the title or the alt value for the image, my plugin will need to find the original file name for offloaded images. Hence, the query to find the original image post and get the title.

    You can remove this functionality by removing the %filename% attribute from RankMath Images SEO module, and this will remove the extra query. You can also comment out the add_filter( 'rank_math/replacements', array( $this, 'fix_file_name_replacement' ), 10, 2 ); line in my plugin in file app/integrations/class-rank-math.php on line 41. However, it will break the %filename% functionality in RankMath due to the way Cloudflare Image URLs are formed. So it’s just easier to remove the attribute from RankMath, if you absolutely must get rid of the query.

    In all my tests, this did not add any noticeable time to the page load (within 0.0001 seconds per query). However, I’m always open to suggestions if you have any ideas on how to make this query more performant or improve this functionality in any way.

    Best regards,
    Anton

    Thread Starter ellasbubbles

    (@ellasbubbles)

    Hi Anton, thank you for the elaborate reply. Please note that the individual query does NOT take 60s, yet the query is duplicated over 100+ times at 0.2s each which is building the total load time. We have about 30 images on about 48 unique variations.

    Please note that you are more than welcome to test this out on our site, although after jumping beta’s we are now experiencing a new issue with 0 images being recognized as offloaded: https://i.imgur.com/29YmYc3.png

    Before testing any further, we ask that you please investigate this on your end. I have resubmitted login information through https://vcore.au/contact-us/ for your convenience

    • This reply was modified 4 months, 2 weeks ago by ellasbubbles.
    Plugin Author Anton Vanyukov

    (@vanyukov)

    @ellasbubbles,

    I might have found a way to avoid those queries. I’ve updated the beta here: https://github.com/av3nger/cf-images/releases/tag/1.9.2-beta.1

    I have checked the offload status on your site. The only time the image should be listed as not offloaded – if there’s no _cloudflare_image_id meta data. I’m guessing you might have bulk removed the images from Cloudflare? That’s the only way this meta field can be removed from within the plugin.

    Best regards,
    Anton

    Thread Starter ellasbubbles

    (@ellasbubbles)

    The beta does not see to have the “Settings” Panel: https://i.imgur.com/Hr0z8O4.png

    Although upon installing, I am not seeing our images being served by the CDN

    Please note that we have not removed or altered anything in any way regarding the images at cloudflare or meta information: https://i.imgur.com/PXPpSDy.png

    Plugin Author Anton Vanyukov

    (@vanyukov)

    @ellasbubbles,

    The plugin folder should not be renamed. The “Settings” link filter in WordPress is based on the plugin folder name. You can also just go to Media – Offload Settings in the right side panel.

    Best regards,
    Anton

    Thread Starter ellasbubbles

    (@ellasbubbles)

    Our plugin is reading 0 offloaded. Again we have made no DB modifications and only followed the beta install instructions as provided here: https://i.imgur.com/jUx0BRf.png

    All images are at cloudflare still

    SELECT?*?FROM?postmeta?WHERE?meta_key?LIKE?'%_cloudflare_image_id%'
    Showing rows 0 – 24 (20456 total, Query took 1.9794 seconds.)

    Plugin Author Anton Vanyukov

    (@vanyukov)

    @ellasbubbles, ok, no problem, just bulk offload all the images again. Not sure why they all got removed, but re-offloading is the easiest fix.

    Best regards,
    Anton

    Thread Starter ellasbubbles

    (@ellasbubbles)

    Can you please log in and witness the issue I am experiencing vs. me typing it here step by step. It is a little more prominent and vivid than what you understand.

    For example, in Media I clealy have more than 6+ images offloaded, as recognized by your plugin: https://i.imgur.com/kEIEnyh.png

    However, in the Settings Panel, I only “have 6 images offloaded”, as others are not recognized by your plugin: https://i.imgur.com/xpby7Tz.png

    What caused the inconsistency and how can you resolve this? We are talking about over 7,000 images to Re-Offload, so would appreciate a mediocre fix if possible

Viewing 15 replies - 1 through 15 (of 17 total)
  • You must be logged in to reply to this topic.