SEO title and description fields of Real-time SEO (Yoast SEO) doesn't get saved when used with a seo_ui module.

As "SEO UI" module changes the form structure of metatag module fields, Real-time SEO (Yoast SEO) doesn't able to save it correctly.

So, we need to update the variables if SEO UI is installed

CommentFileSizeAuthor
#5 yoast_seo_doesnt_works-2813823-5.patch1.31 KBnileshlohar
#2 yoast_seo_with_seo_ui_2813823-2.patch1.32 KBnileshlohar
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

nileshlohar created an issue. See original summary.

nileshlohar’s picture

nileshlohar CreditAttribution: nileshlohar commented
Status: Active » Needs review
FileSize
yoast_seo_with_seo_ui_2813823-2.patch1.32 KB

Here is the patch

naveenvalecha’s picture

naveenvalecha
He/Him
Primary language Hindi
CreditAttribution: naveenvalecha as a volunteer commented
Version: 7.x-1.0 » 7.x-1.x-dev
Status: Needs review » Reviewed & tested by the community
Issue tags: +Novice

Thanks!

naveenvalecha’s picture

naveenvalecha
He/Him
Primary language Hindi
CreditAttribution: naveenvalecha as a volunteer commented
Status: Reviewed & tested by the community » Needs work 
+++ b/yoast_seo.module
@@ -334,11 +334,11 @@ function yoast_seo_configuration_form_after_build($form, &$form_state) {
+      'description' => (module_exists('seo_ui')) ? $form['seo_vtab']['metatags'][$langcode]['basic']['description']['value']['#id'] : $form['metatags'][$langcode]['basic']['description']['value']['#id'],    ¶

Coding standard missing this can be take care at the time of commit.

nileshlohar’s picture

nileshlohar CreditAttribution: nileshlohar commented
Title: Yoast SEO doesnt works with SEO UI module » Real-time SEO (Yoast SEO) is not compatible with SEO UI module.
Issue summary: View changes
Status: Needs work » Needs review
FileSize
yoast_seo_doesnt_works-2813823-5.patch1.31 KB

Corrected.

naveenvalecha’s picture

naveenvalecha
He/Him
Primary language Hindi
CreditAttribution: naveenvalecha as a volunteer commented
Status: Needs review » Reviewed & tested by the community

Thanks!

naveenvalecha’s picture

naveenvalecha
He/Him
Primary language Hindi
CreditAttribution: naveenvalecha as a volunteer commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 7.x-1.x Will be included in next release.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.