WordPress.org

Ready to get started?Download WordPress

Forums

Yahoo! Weather Forecasts
Reposting bug report - credit not defined (5 posts)

  1. needle
    Member
    Posted 11 months ago #

    Well, that's odd - the forums seem to have marked my post as closed before it's even been opened.

    So, reposting my simple bug report:

    On line 96 of yahoo!-wheather.php, if credits are not shown, $credit is undefined, throwing a "Notice: Undefined variable" when WP_DEBUG is set to true.

    The simple fix is to define it as empty on the previous line, ie:

    $credit = '';
    if ( $show_credit ) $credit = ' & <a href="http://zourbuth.com/plugins/yahoo-weather/">zourbuth.com</a>';

    Cheers,

    Christian

    http://wordpress.org/extend/plugins/yahoo-weather-forecasts/

  2. Well, that's odd - the forums seem to have marked my post as closed before it's even been opened.

    Spam filter. It happens and I've un-spammed this post.

    If the credits are not opt-in (they had better be) then this plugin violates the terms of hosting a plugin on WordPress.ORG's repository.

    http://wordpress.org/extend/plugins/about/guidelines/

    If that's not the case (I've not yet installed this plugin) please report it to plugins@wordpress.org or perhaps the author will reply to this topic.

  3. needle
    Member
    Posted 11 months ago #

    Sad to report that credits are opt-out. Perhaps it's a simple oversight. I'll wait a while for the author to reply before reporting.

    Cheers,
    Christian

  4. zourbuth
    Member
    Plugin Author

    Posted 11 months ago #

    Hi needle, since this widget uses the Yahoo! Weather Forecast feeds, Yahoo need an attribution to it http://developer.yahoo.com/weather/#terms.

  5. needle
    Member
    Posted 11 months ago #

    Hi zourbuth, I don't have any say over the WordPress hosting terms - your plugin has been flagged because the bug report I submitted went through forum moderation.

    My guess would be that if the plugin has to violate the Yahoo terms to be included in the WordPress plugin directory, then it should be removed. Either that, or you need to sort out the attribution issue with Yahoo themselves, since it is presumably in their interests to have a plugin in the directory that uses their weather API.

    Cheers,
    Christian

Reply

You must log in to post.

About this Plugin

About this Topic

Tags

No tags yet.