Robots not saved, keeps being noindex/nofollow [#835172]

Comment	File	Size	Author
#23	nodewords-noindex_nofollow_defaults-6.x.patch	1.13 KB	AlexisWilke
#23	nodewords-index_follow_is_empty-6.x.patch	1 KB	AlexisWilke

Comment #2

powery commented 23 June 2010 at 06:39

subscribing

Log in or register to post comments

Comment #3

summit commented 23 June 2010 at 08:01

Hi, I think this is duplicate of: http://drupal.org/node/774652.

Will try tonight (thanks to Druppy :):

To change the default robot tag for all pages to "index, follow", you can go to

nodewords/nodewords_basic/includes/nodewords_basic.nodewords.tags.inc

line 255

and change

'#default_value' => empty($content['index_follow']) ? 'noindex,nofollow' : $content['index_follow'],

to

'#default_value' => empty($content['index_follow']) ? 'index,follow' : $content['index_follow'],

But it would be great to have a structural solution!
greetings, Martijn

Log in or register to post comments

Comment #4

powery commented 23 June 2010 at 08:35

This solution #3 did not solve the same problem for me

Log in or register to post comments

Comment #5

summit commented 23 June 2010 at 09:43

Hi Kiam, as stated on: http://drupal.org/node/786270#comment-2929824 you "attacked" a different function to get this working.
Was this the change you referred to: http://drupalcode.org/viewvc/drupal/contributions/modules/nodewords/node... ?

I can't go to latest .dev of nodewords, because I did this yesterday, and then no metatags are shown on my node-pages anymore (off course did update.php). So I would very much like the 90% working 1.12-beta9 to improve with only this issue.

Thanks a lot for all your support so far!
greetings, Martijn

Log in or register to post comments

Comment #6

summit commented 23 June 2010 at 11:06

Powery, can you check whether this solution works? I am not behind a pc on which I can change code.
Thanks! greetings, Martijn

Log in or register to post comments

Comment #7

powery commented 23 June 2010 at 11:21

And shall I try patch from #3 and patch from #5? Or from #5 only?

Log in or register to post comments

Comment #8

powery commented 23 June 2010 at 12:01

I think, that only #5 works. But "Meta tag ROBOTS for lists" can not be saved to others then INDEX, FOLLOW.

Log in or register to post comments

Comment #9

summit commented 23 June 2010 at 16:30

For me sufficient! Thanks. Will try patch #5 also when I am home.
EDIT: Yes patch #5 worked for this issue!
greetings,
Martijn

Log in or register to post comments

Comment #10

powery commented 23 June 2010 at 16:32

And can you change "Meta tag ROBOTS for lists"?

Log in or register to post comments

Comment #11

summit commented 23 June 2010 at 22:19

Why should I use that please?

Log in or register to post comments

Comment #12

powery commented 24 June 2010 at 06:27

As is written as help under Meta tag ROBOTS for lists:

The value to use for the meta tag ROBOTS for the pages that use a pager; this setting is not used for the first page of the list, or when the option to repeat the meta tags for all the pages in the same list is selected.

If you don't want to index pages with url like example.com/something?page=4, you can choose NOINDEX, FOLLOW for these pages a index only page example.com/something.

Log in or register to post comments

Comment #13

summit commented 24 June 2010 at 09:26

Hi,
And that is not working with patch #5?
greetings, Martijn

Log in or register to post comments

Comment #14

powery commented 24 June 2010 at 11:15

Yes. I can not save other value.

Log in or register to post comments

Comment #15

summit commented 24 June 2010 at 11:29

I think would be best to search through the commits, or place a new issue for this, right?
greetings, Martijn

Log in or register to post comments

Comment #16

powery commented 24 June 2010 at 11:34

ok

Log in or register to post comments

Comment #17

Mike Kennedy commented 2 July 2010 at 14:52

Honestly this was driving me absolutely crazy, I couldnt get the metatags to say robots "index,follow" which I feel is an important part of metatags. I even thought about switching back to the old node words. After a week of searching I finally found this "hack". I dont know if it is the correct way to fix this problem, as I am not a php guy but it was simple to do and worked for me. Here is the link good luck http://www.answerflip.com/how-fix-missing-metatags-robots-index-follow-n...

Log in or register to post comments

Comment #18

czeky commented 2 July 2010 at 19:33

Hi, on my instalation, robots changed in node edit are not saved..

Log in or register to post comments

Comment #19

wheelercreek commented 6 July 2010 at 17:07

I was able to correct the issue partially by using #3 and #5 above, but I found that the string was output as "indexfollow" without the comma.. I tried searching up the code stream to see how this was happening, but in the end I just added an if statement to correct that line and rewrite it. It's at line 297, in the function "nodewords_basic_robots_prepare":

if (!empty($value)) {
     $tags['robots'] = implode(',', $value);
     if($tags['robots'] == 'indexfollow'){ 
         $tags['robots'] = 'index,follow'; 
     }		
}

I did find that the value, by the time it gets to this implode statement, is already a string instead of an array.

Log in or register to post comments

Comment #20

trentharlem commented 7 July 2010 at 07:44

"How to write a Robots Meta Tag
Where to put it

Like any tag it should be placed in the HEAD section of an HTML page, as in the example above. You should put it in every page on your site, because a robot can encounter a deep link to any page on your site.
What to put into it

The "NAME" attribute must be "ROBOTS".

Valid values for the "CONTENT" attribute are: "INDEX", "NOINDEX", "FOLLOW", "NOFOLLOW". Multiple comma-separated values are allowed, but obviously only some combinations make sense. If there is no robots tag, the default is "INDEX,FOLLOW", so there's no need to spell that out. That leaves:

<META NAME="ROBOTS" CONTENT="NOINDEX, FOLLOW">
<META NAME="ROBOTS" CONTENT="INDEX, NOFOLLOW">
<META NAME="ROBOTS" CONTENT="NOINDEX, NOFOLLOW">

"

-From: http://www.robotstxt.org/meta.html

If I'm reading this right, if i want my page to be "INDEX,FOLLOW", I should uncheck the "ROBOTS" meta tag in the "meta tags to output" section at /admin/content/nodewords...
YES?

Log in or register to post comments

Comment #21

wiredescape commented 20 August 2010 at 03:47

The 'no comma' problem is a different issue.

See #785654: Commas missing in ROBOTS tag for previously created nodes for more info.

Log in or register to post comments

Comment #22

laura s commented 24 August 2010 at 22:43

Just had a friendly reader report to me that "NOINDEX,NOFOLLOW" was on all of my blog posts, even though the module is set for INDEX,FOLLOW. I disabled the field in the module. Rather unfortunate SEO impact on an SEO module.

Log in or register to post comments

Comment #23

AlexisWilke commented 29 September 2010 at 08:13

Assigned:	Unassigned	» AlexisWilke
Priority:	Normal	» Critical
Status:	Active	» Needs review

Status	File	Size
new	nodewords-index_follow_is_empty-6.x.patch	1 KB
new	nodewords-noindex_nofollow_defaults-6.x.patch	1.13 KB

HI guys,

This is a really bad one! Making all those websites NOINDEX and NOFOLLOW. You should have fixed that the day it was reported!!!

There are two patches.

#3 was not sufficient since you had another place with the same wrong defaults.

There is a patch for #20 although that's much less important to not issue the tag if just and only "index,follow". Up to you whether you want to apply that one.

Thank you.
Alexis