• Resolved midwestbroadcasting

    (@midwestbroadcasting)


    After installing the latest update from 2.4.0.3 to 2.4.0.4 I encountered a critical error issue. I had to remove the plugin in order to be able to access admin panel. I was able to restore a latest backup in order to have the previous 2.4.0.3 version back up. Fortunately this is a site it development. Please let me know if you would like access in order to resolve via email.

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

Viewing 15 replies - 1 through 15 (of 26 total)
  • Thread Starter midwestbroadcasting

    (@midwestbroadcasting)

    WordPress version 5.8.1
    Current theme: Uncode Child (version 1.0.0)
    Current plugin: Radio Station (version 2.4.0.4)
    PHP version 7.3.31-1+0~20210923.88+debian9~1.gbpac4058

    Error Details
    =============
    An error of type E_ERROR was caused in line 3514 of the file /home/588111.cloudwaysapps.com/djtzxwmngt/public_html/wp-content/plugins/radio-station/radio-station.php. Error message: Uncaught Error: Call to undefined function radio_station_get_setting() in /home/588111.cloudwaysapps.com/djtzxwmngt/public_html/wp-content/plugins/radio-station/radio-station.php:3514
    Stack trace:
    #0 /home/588111.cloudwaysapps.com/djtzxwmngt/public_html/wp-includes/class-wp-hook.php(303): radio_station_revoke_show_edit_cap(Array, Array, Array, Object(WP_User))
    #1 /home/588111.cloudwaysapps.com/djtzxwmngt/public_html/wp-includes/plugin.php(189): WP_Hook->apply_filters(Array, Array)
    #2 /home/588111.cloudwaysapps.com/djtzxwmngt/public_html/wp-includes/class-wp-user.php(786): apply_filters(‘user_has_cap’, Array, Array, Array, Object(WP_User))
    #3 /home/588111.cloudwaysapps.com/djtzxwmngt/public_html/wp-includes/capabilities.php(801): WP_User->has_cap(‘unfiltered_html’)
    #4 /home/588111.cloudwaysapps.com/djtzxwmngt/public_html/wp-includes/capabilities.php(693): user_can(Object(WP_User), ‘unfiltered_html’)
    #5 /home/588111.cloudwaysapps.com/djtzxwmngt/public_html/wp-includes/kses.php(2142): current_user_can(‘unfiltere

    Plugin Author Tony Zeoli

    (@tonyzeoli)

    Thanks for letting us know about his and posting the error log. We’re checking into it and will report back ASAP.

    Plugin Contributor Tony Hayes

    (@majick)

    @midwestbroadcasting Sorry this happened for you, we do our best to avoid such errors ever happening. However in this case, I can see it is a conflict with another plugin, which is checking some user capability after Radio Station has loaded, but *before* all plugins have finished loading. While this isn’t the best practice, it understandably happens in some plugins.

    Rather than trying to work out which plugin is conflicting, I’ve just added a patch to check for the existence of the missing function (that is loaded later on.) You can use the version on Github (master branch) which includes this patch. https://github.com/netmix/radio-station/ (As this is an edge case, we’ll probably just hotfix 2.4.0.4 in the repo rather than bumping to the next version.)

    Thread Starter midwestbroadcasting

    (@midwestbroadcasting)

    Moving forward when there is a release, the patch will be included correct? Wasn’t clear whether this was a fix for this version only or if the fix will be included moving forward so this issue does not occur in future releases?

    Thanks!

    Plugin Author Tony Zeoli

    (@tonyzeoli)

    @midwestbroadcasting We put it I the develop version so you could get it and update as fast as possible but o just pushed the radio-station.php file but did not update the version. So it’s in any new download but we didn’t want to push a new version because only people using a certain plugin combination might have this error and I don’t want two back tk back version releases for such a minor fix that only happens to some users.

    Plugin Author Tony Zeoli

    (@tonyzeoli)

    @jasva this morning I added the fix to the download repo on www.ads-software.com, so you may have already taken the “develop” version to fix, but if not just redownload from our page here on wordpress, org and the fix is in 2.4.0.4 now. Wasn’t worth adding a new version number and having a whole bunch of folks download a new update when it’s only affecting a small sub set of folks.

    Thread Starter midwestbroadcasting

    (@midwestbroadcasting)

    Thank you Tony!

    Plugin Author Tony Zeoli

    (@tonyzeoli)

    @midwestbroadcasting @jasva glad it’s fixed. Thanks for reporting the issue.

    Thread Starter midwestbroadcasting

    (@midwestbroadcasting)

    Tony,
    So I updated the plugin via the WordPress repository as mention “download repo on www.ads-software.com” and still got the same error. Can you please check to see if the necessary files got updated?

    Thread Starter midwestbroadcasting

    (@midwestbroadcasting)

    I actually tried it several ways by downloading the plugin and overwriting the plugin via the WordPress plugin console, and also by updating the plugin the usual way through plugins. Both gave me a critical error.

    Thread Starter midwestbroadcasting

    (@midwestbroadcasting)

    Tony, Following up on…
    So I updated the plugin via the WordPress repository as mention “download repo on www.ads-software.com” and still got the same error. Can you please check to see if the necessary files got updated?

    Plugin Author Tony Zeoli

    (@tonyzeoli)

    @midwestbroadcasting @retiredgamer I’m pretty certain I’ve added the fix and pushed it. I’ll ask @majick to check the GitHub repo file that is publicly available to compare and make sure. But he won’t be online until around 6 pm EST as he’s in Australia.

    In the meantime, can you both download the radio-station.php file from the “development” version on Git and upload via FTP, but renaming the original file to back it up so you don’t overwrite it. See if that works as @jasva may have taken the development version and not redownloaded the plugin. It’s not clear from his post what actually fixed the issue – whether he took the development version or whether you re-downloaded from the repo.

    I would also deactivate all plugins except radio station and then reactive each one to see which is the conflicting plugin. If radio station ends up still having the error without any other plugin active, then that’s on us. But if it’s fine and there’s an offending plugin that is conflicting, we’d be interested to know about it.

    Thread Starter midwestbroadcasting

    (@midwestbroadcasting)

    So comparing the two .zip files, radio-station-master.zip, the radio-station.php has 3720 lines of code. While the .zip file on the wordpress repository only has 3715 lines of code… some discrepancy at least in lines of code .

    Did a compare and what is missing in the wordpress repository is the following:

    // 2.4.0.4.1: fix for early capability check plugin conflict
    if ( !function_exists( ‘radio_station_get_setting’ ) ) {
    return $allcaps;
    }

    Plugin Author Tony Zeoli

    (@tonyzeoli)

    I just deleted the radio-station.php file from the repo and recommitted it. Even though I checked it and it had 3720 lines, I just thought it best to remove and recommit. Please check the download from www.ads-software.com again. It should have:

    // 2.4.0.4.1: fix for early capability check plugin conflict
    if ( !function_exists( ‘radio_station_get_setting’ ) ) {
    return $allcaps;
    }

    starting on lines 3495 to 3500

    And, it should total 3720 lines.

    Plugin Author Tony Zeoli

    (@tonyzeoli)

    @sounds @midwestbroadcasting @jctulip @wrwhradio I didn’t tag you all on my last reply, so doing so to make sure you get the message that I deleted the file from the repo and reuploaded it. Please read my last post above this one for instructions.

    • This reply was modified 3 years, 4 months ago by Tony Zeoli.
    • This reply was modified 3 years, 4 months ago by Tony Zeoli.
Viewing 15 replies - 1 through 15 (of 26 total)
  • The topic ‘There has been a critical error on this website.’ is closed to new replies.