hughie.molloy

Link to github bug report: https://github.com/Yoast/wordpress-seo/issues/19062

I see you noted that there was a bug on the private forum – the bug has not actually been resolved.

And it would be useful for the public to know that

I think this has fixed it

Here is my stack trace in my error log:

Trying to get property of non-object
/wp-content/plugins/publish-to-apple-news/includes/apple-exporter/class-exporter-content.php – 231 – HandleError
/wp-content/plugins/publish-to-apple-news/includes/apple-exporter/builders/class-builder.php – 123 – nodes
/wp-content/plugins/publish-to-apple-news/includes/apple-exporter/builders/class-components.php – 731 – content_nodes
/wp-content/plugins/publish-to-apple-news/includes/apple-exporter/builders/class-components.php – 41 – _split_into_components
/wp-content/plugins/publish-to-apple-news/includes/apple-exporter/builders/class-builder.php – 66 – build
/wp-content/plugins/publish-to-apple-news/includes/apple-exporter/class-exporter.php – 243 – to_array
/wp-content/plugins/publish-to-apple-news/includes/apple-exporter/class-exporter.php – 171 – generate_json
/wp-content/plugins/publish-to-apple-news/admin/apple-actions/index/class-push.php – 421 – generate
/wp-content/plugins/publish-to-apple-news/admin/apple-actions/index/class-push.php – 183 – generate_article
/wp-content/plugins/publish-to-apple-news/admin/apple-actions/index/class-push.php – 84 – push
/wp-content/plugins/publish-to-apple-news/admin/class-admin-apple-post-sync.php – 92 – perform
/wp-includes/class-wp-hook.php – 288 – do_publish
/wp-includes/class-wp-hook.php – 310 – apply_filters
/wp-includes/plugin.php – 453 – do_action
/wp-includes/post.php – 3640 – do_action
REDACTED.php – 110 – wp_insert_post

The apple news article gets made but not the wordpress post.

My fix above does not actually work for php5 so ignore it.

From my testing this is an issue and happens if you don’t have your api channel, id and secret set.

I still think it is a major bug as it makes the entire rest of the site unusable. Apple news should not stop the site being usable if it is not configured properly.

And actually a default: break; in the switch loop would be good too.

Thanks. I identified it on my local. Prod is hiding the warnings.

Sorry it’s line 523. I had added some debug a few lines above.

I am on Stable tag: 1.3.0, Tested up to: 4.7.4 according to readme.txt. WP 4.8.2 but I don’t think the error is WP related