• Resolved Plokamis

    (@plokamis)


    HEllo, at first i want to say that the plugin is awesome and really funcional but i get an conflict message in 2 site that i have tried it

    Warning: Invalid argument supplied for foreach() in /home2/dustin88/public_html/realestateroofer.com/wp-content/plugins/popup-maker/includes/google-fonts.php on line 28

    It appears everytime the pop up auto triggers. IS this a bug? or what?

    https://www.ads-software.com/plugins/popup-maker/

Viewing 11 replies - 1 through 11 (of 11 total)
  • Plugin Author Daniel Iser

    (@danieliser)

    @plokamis – Adding the issue to github now. You can follow up on it there to see how its coming.

    Off the top of my head without looking sounds like it is unable to connect to the Google Font API or the theme settings. Can you try editing the theme and simply hitting the update button?

    I will add a check that will at least not generate an error like that if the api is unavailable.

    I am recording tutorial videos today and will be working on bug fixes the rest of the week. There should be an update later this week but I’ll see if I can find a temporary solution for now either by modifying plugin files with the patch or using filters to remove GFonts until resolved.

    Thread Starter Plokamis

    (@plokamis)

    @danieliser,
    Thanks for the quick reply!
    Unfortunately my knowledge dont get to modifying the theme but i check that you have a checkbox that deactivates loading google fonts so the problem solved. But it would be nice to intergrate them without error.

    So thank you anyway! ??

    Plugin Rocks!

    Thread Starter Plokamis

    (@plokamis)

    @danieliser,

    It appears to all pages Now! What should i do?????

    Thank you! The site is alive and online.

    Can you drop me a hand?

    Thank you!

    Link page : https://realestateroofer.com/buyers/

    Thread Starter Plokamis

    (@plokamis)

    Hello again i found a solution temporary.
    I commented this in the google-fonts.php

    `if( !is_wp_error( $response ) ) {
    $data = json_decode($response, true);
    $items = $data[‘items’];
    $font_list = array();
    //foreach($items as $item) {
    // $font_list[$item[‘family’]] = $item;
    //}
    set_transient( ‘popmake-google-fonts-list’, $font_list, WEEK_IN_SECONDS );
    return $font_list;
    `

    I will be waiting news
    Thank you:D

    Plugin Author Daniel Iser

    (@danieliser)

    @plokamis – I am working on this an several other issues today. I will be trying to get an update out by tomorow.

    Thread Starter Plokamis

    (@plokamis)

    @danieliser

    Hey Bro dont worry, temporary i solved it! Do your think!

    The plugin Rocks Either ways ??

    Plugin Author Daniel Iser

    (@danieliser)

    @plokamis – Great, glad to hear it, if you don’t mind please take a moment to rate and review the plugin and or support

    Thread Starter Plokamis

    (@plokamis)

    @danieliser

    Did it already! ??

    Hey guys.

    Just checking in to let you know that I’m seeing this error in the current version of the plugin: Version 1.1.10

    I’m going to see what I can do about it now. If I find a fix I’ll post here.

    Hmmm… I can’t find the bug at the moment.

    Maybe I was seeing it in a cached page from before a site-wide plugin update?

    I’ll let you know if I see it again, and how I’m squishing it.

    Plugin Author Daniel Iser

    (@danieliser)

    @sunilwilliams this exact error cannot occur anymore, we now check to be sure that the font array is not empty before processing the foreach. So likely your correct, it was a cache issue.

    Please don’t hesitate to make a new thread if you find a new bug.

    PS Please take a moment to rate and review the plugin and or support.

Viewing 11 replies - 1 through 11 (of 11 total)
  • The topic ‘Conflict of the plugin – Bug’ is closed to new replies.