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

Issue when using post title as description #986

Closed
Toast707 opened this issue Jul 22, 2017 · 8 comments

Comments

Projects
None yet
4 participants
@Toast707
Copy link

commented Jul 22, 2017

Hi,

I'm currently trying to get AIO seo to use the post title as the description, but I'm having a bit of difficulty.

I've tried using both %post_title% and %wp_title% but neither are displaying. Is this a common issue?

Currently using php 7.1.7. WP version 4.8

@michaeltorbert

This comment has been minimized.

Copy link
Member

commented Jul 22, 2017

From preliminary testing, it looks like there may be a bug that causes the meta description to only show up if something is entered in the SEO description field for the post, regardless of the description template settings.
We should test (and look at the code) to determine any other conditions, and then see if it's a new problem or has been there for a while.

@Toast707

This comment has been minimized.

Copy link
Author

commented Jul 22, 2017

@michaeltorbert Hey, good to know it's not only me. Do you know what custom field the the descriptions are stored as?

@michaeltorbert

This comment has been minimized.

Copy link
Member

commented Jul 22, 2017

@Toast707 The field is _aioseop_description but that's only for manually entered ones. Otherwise they're generated dynamically (on the fly) and not stored in the database.

@Toast707

This comment has been minimized.

Copy link
Author

commented Jul 22, 2017

Yeah, I was just looking for a temporary work-around until it gets patched. Thanks for the heads up; looking forward to the update. I'll close the issue.

@Toast707 Toast707 closed this Jul 22, 2017

@michaeltorbert

This comment has been minimized.

Copy link
Member

commented Jul 22, 2017

Reopening, as this is a legitimate bug.

@Toast707

This comment has been minimized.

Copy link
Author

commented Jul 22, 2017

Sorry about that; I wasn't under the impression I was the first to post this issue.

@wpsmort

This comment has been minimized.

Copy link
Member

commented Jul 24, 2017

@michaeltorbert I tested on a new install with the current live version of AIOSEOP. There was no meta description until I activated the Autogenerate Descriptions option. Once I activated that option, the meta description was generated from my post excerpt or the post content as expected.

I then changed the Description Format to %post_title% and got the following meta description:

<meta name="description" content="Test post with excerpt | testsite" />

The above is the post title and site title which is a potential issue because I didn't use %blog_title% in the Description Format field. If I use %post_title% | %blog_title% in the Description Format field then I get the site title output twice.

I also tested with %wp_title% and that worked perfectly fine.

In fact I tested all the supported strings listed in the help for the Description Format field and they all worked fine with that one exception of the %post_title% including the site title.

The only other issue was that the placeholder text in the Description field in AIOSEOP did not show the correct description as per the format I selected, instead it always showed my post excerpt text.

@michaeltorbert michaeltorbert removed this from the 2.3.15 milestone Aug 2, 2017

@contactashish13

This comment has been minimized.

Copy link
Contributor

commented Dec 18, 2017

@wpsmort PR: #1421 solves part of the issue i.e. when you specify %post_title% in the description, now it will not add the blog title as the suffix.

Also seems related to #1408

contactashish13 added a commit to contactashish13/all-in-one-seo-pack that referenced this issue Mar 8, 2018

contactashish13 added a commit to contactashish13/all-in-one-seo-pack that referenced this issue Mar 8, 2018

@michaeltorbert michaeltorbert added this to the 2.7 milestone Mar 22, 2018

@wpsmort wpsmort modified the milestones: 2.7, 2.8 May 15, 2018

contactashish13 added a commit to contactashish13/all-in-one-seo-pack that referenced this issue Jul 27, 2018

@wpsmort wpsmort removed the Needs Testing label Aug 3, 2018

contactashish13 added a commit to contactashish13/all-in-one-seo-pack that referenced this issue Aug 21, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.