Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Critical error when using jetpack/13.0-a.1 #34627

Closed
helenaartmann opened this issue Dec 13, 2023 · 13 comments · Fixed by #34636
Closed

Critical error when using jetpack/13.0-a.1 #34627

helenaartmann opened this issue Dec 13, 2023 · 13 comments · Fixed by #34636
Assignees
Labels
Customer Report Issues or PRs that were reported via Happiness. aka "Happiness Request", or "User Report" [Feature] Publicize Now Jetpack Social, auto-sharing [Package] Publicize [Plugin] Jetpack Issues about the Jetpack plugin. https://wordpress.org/plugins/jetpack/ [Pri] High [Status] Priority Review Triggered The guild in charge of triage has been notified of this issue in Slack Triaged [Type] Bug When a feature is broken and / or not performing as intended

Comments

@helenaartmann
Copy link

helenaartmann commented Dec 13, 2023

Impacted plugin

Jetpack

Quick summary

Jetpack was automatically updated and when the user deactivated Classic Editor plugin, the critical error appeared on the site.

We found a temporary workaround by changing the PHP version to 7.4.

Steps to reproduce

  1. Jetpack was automatically updated to 13.0-a.1
  2. When the user deactivated the Classic Editor plugin, a critical error appeared on the site.
  3. To see the error, we had to go to the pages and click on editing a page
  4. Screenshot 2023-12-13 at 9 46 46 AM

A clear and concise description of what you expected to happen.

Be able to edit the page without the error appearing.

What actually happened

We don't have access to edit the page.

Impact

One

Available workarounds?

Yes, easy to implement

Platform (Simple and/or Atomic)

No response

Logs or notes

CLI error:

[13-Dec-2023 17:48:19 UTC] PHP Fatal error:  Uncaught TypeError: Cannot access offset of type string on string in /wordpress/plugins/jetpack/13.0-a.1/jetpack_vendor/automattic/jetpack-publicize/src/jetpack-social-settings/class-settings.php:155
Stack trace:
#0 /wordpress/plugins/jetpack/13.0-a.1/class.jetpack-gutenberg.php(743): Automattic\Jetpack\Publicize\Jetpack_Social_Settings\Settings->get_settings(true)
#1 /wordpress/core/6.4.2/wp-includes/class-wp-hook.php(324): Jetpack_Gutenberg::enqueue_block_editor_assets('')
#2 /wordpress/core/6.4.2/wp-includes/class-wp-hook.php(348): WP_Hook->apply_filters(NULL, Array)
#3 /wordpress/core/6.4.2/wp-includes/plugin.php(517): WP_Hook->do_action(Array)
#4 /wordpress/core/6.4.2/wp-includes/script-loader.php(2446): do_action('enqueue_block_a...')
#5 /wordpress/core/6.4.2/wp-includes/class-wp-hook.php(324): wp_common_block_scripts_and_styles('post.php')
#6 /wordpress/core/6.4.2/wp-includes/class-wp-hook.php(348): WP_Hook->apply_filters(NULL, Array)
#7 /wordpress/core/6.4.2/wp-includes/plugin.php(517): WP_Hook->do_action(Array)
#8 /wordpress/core/6.4.2/wp-admin/admin-header.php(118): do_action('admin_enqueue_s...', 'post.php')
#9 /wordpress/core/6.4.2/wp-admin/edit-form-blocks.php(309): require_once('/wordpress/core...')
#10 /wordpress/core/6.4.2/wp-admin/post.php(187): require('/wordpress/core...')
#11 {main}
  thrown in /wordpress/plugins/jetpack/13.0-a.1/jetpack_vendor/automattic/jetpack-publicize/src/jetpack-social-settings/class-settings.php on line 155
@helenaartmann helenaartmann added [Type] Bug When a feature is broken and / or not performing as intended Needs triage Ticket needs to be triaged labels Dec 13, 2023
@github-actions github-actions bot added [Plugin] Jetpack Issues about the Jetpack plugin. https://wordpress.org/plugins/jetpack/ [Pri] Low labels Dec 13, 2023
@helenaartmann
Copy link
Author

helenaartmann commented Dec 13, 2023

7447661-zd-a8c - follow up

Copy link
Contributor

github-actions bot commented Dec 13, 2023

Support References

This comment is automatically generated. Please do not edit it.

  • 7447661-zen
  • 7448953-zen
  • 7449292-zen
  • 7445308-zen
  • p9F6qB-dTZ-p2#comment-56925
  • 7453477-zen
  • p9F6qB-dTZ-p2#comment-56956
  • 7455700-zen
  • 7458133-zen
  • 7459451-zen

@github-actions github-actions bot added the Customer Report Issues or PRs that were reported via Happiness. aka "Happiness Request", or "User Report" label Dec 13, 2023
@jeherve jeherve added [Feature] Publicize Now Jetpack Social, auto-sharing [Package] Publicize [Pri] High Triaged and removed [Pri] Low Needs triage Ticket needs to be triaged labels Dec 13, 2023
@jeherve jeherve moved this from Needs Triage to Triaged in Automattic Prioritization: The One Board ™ Dec 13, 2023
@github-actions github-actions bot added the [Status] Priority Review Triggered The guild in charge of triage has been notified of this issue in Slack label Dec 13, 2023
@jeherve
Copy link
Member

jeherve commented Dec 13, 2023

cc @gmjuhasz who worked on #34113. Could you take a look? This was only a warning in PHP 7.4 but was bumped to an error on PHP 8.0.

Thanks!

@Greatdane
Copy link

Greatdane commented Dec 14, 2023

7448953-zd-a8c
7449292-zd-a8c
7445308-zd-a8c

Thom also shared more info on the issue here;
p9F6qB-dTZ-p2#comment-56925

@cuemarie
Copy link

cuemarie commented Dec 14, 2023

I just had a customer with this issue here: 7453477-zen ; it was isolated to their third-party theme ad-mania, and occurring when they tried accessing the Customizer. Downgrading to PHP 7.4 resolved.

@gmjuhasz is there anything we should inform customers about that they would need to bring to their theme authors? Or should #34636 resolve this for any site? (also asked this here: p9F6qB-dTZ-p2#comment-56956)

@pablinos
Copy link
Contributor

This got closed by merging the PR, but it won't be fixed until it's released on Atomic.

The fix is to delete the offending option:

wp option delete jetpack_social_autoconvert_images

And then not save settings on the General page /wp-admin/options-general.php until the fix is released. Downgrading PHP is working because it's more lax with things like treating strings as arrays!

Or should #34636 resolve this for any site?

Yes, once the fix is deployed it will be fixed for all sites.

@pablinos pablinos reopened this Dec 14, 2023
@paxblueribbon
Copy link

Issue came up in zdc-7454795, resolved by rolling back php version

@Greatdane
Copy link

7455700-zd-a8c

@zdenys
Copy link
Contributor

zdenys commented Dec 15, 2023

7458133-zd-a8c

@sdixon194
Copy link
Contributor

Just noting this fix was about to be deployed before other issues surfaced, so we needed another fix on top of it. I'll cut another version today but it might not be deployed until next week at this rate.

@dhackneya8c
Copy link

another report in 7459451-zd-a8c

@mdbitz
Copy link
Contributor

mdbitz commented Dec 15, 2023

latest Jetpack has been released to Atomic

@anomiex
Copy link
Contributor

anomiex commented Dec 18, 2023

Closing this per the above, the fix should be on Atomic with 13.0-a.5.

@anomiex anomiex closed this as completed Dec 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Customer Report Issues or PRs that were reported via Happiness. aka "Happiness Request", or "User Report" [Feature] Publicize Now Jetpack Social, auto-sharing [Package] Publicize [Plugin] Jetpack Issues about the Jetpack plugin. https://wordpress.org/plugins/jetpack/ [Pri] High [Status] Priority Review Triggered The guild in charge of triage has been notified of this issue in Slack Triaged [Type] Bug When a feature is broken and / or not performing as intended
Development

Successfully merging a pull request may close this issue.