inpsyde / MultilingualPress-Yoast-SEO-Sync

This is a simple add-on for the MultilingualPress plugin to synchronize the post metadata of the Yoast SEO plugin between translated posts.
GNU General Public License v2.0
7 stars 3 forks source link

Fatal error: Uncaught DomainException: Empty Term. #9

Closed Dinamiko closed 4 years ago

Dinamiko commented 4 years ago

Steps to reproduce

  1. Install latest versions of MLP Yoast (2.1.1) and MLP (3.3.0)
  2. Connect one site
  3. Create a new post category term

What should happen I should see the Yoast tab correctly in translation metabox.

What happened instead A fatal error is displayed: Fatal error: Uncaught DomainException: Empty Term. in .../wp-content/plugins/MultilingualPress-Yoast-SEO-Sync/src/TranslationUi/Term/Repository.php on line 41

mklasen commented 4 years ago

Aside from the Yoast tab not displaying, this also breaks the full edit term page.

Kdamen1 commented 4 years ago

MLP 3.3.0 does not fix the earlier mentioned issues. Like Marinus also mentioned:

Please have this fixed ASAP since this is killing for SEO ratings.

YourMark commented 4 years ago

I have also encountered this.

Dinamiko commented 4 years ago

@mklasen, @Kdamen1, @YourMark we have just released new version 2.1.2 (https://github.com/inpsyde/MultilingualPress-Yoast-SEO-Sync/releases/tag/v2.1.2) which should fix the issue, when you have time please give it a try to see if it works for you now, otherwise let us know, thanks.

mklasen commented 4 years ago

Hi @Dinamiko, Thanks for your prompt reply and action. The issue seems resolved. @Kdamen1 - Can you confirm this?

Kdamen1 commented 4 years ago

@mklasen looks good now!