Make WordPress Core

Opened 15 years ago

Closed 15 years ago

#19251 closed defect (bug) (fixed)

Respect more restrictive blog_public settings when disabling crawling

Reported by: ryan Owned by: ryan
Priority: normal Milestone: 3.3
Component: General Version: 3.2.1
Severity: normal Keywords:
Cc: Focuses:

Description

[17891] introduced a pre_option_blog_public that returns 0 when replytocom is set in GET. This works fine for core but could thwart some privacy plugins that use blog_public values other than 0 or 1. To accommodate plugins, blog_public should be overridden only when it is currently set to 1.

Attachments (2)

19251.diff (1.7 KB ) - added by ryan 15 years ago.
19251.2.diff (3.5 KB ) - added by nacin 15 years ago.
Patch I worked up independently.

Download all attachments as: .zip

Change History (5)

@ryan
15 years ago

#1 @ryan
15 years ago

That patch plays with adding a disallow_crawling filter. Not sure I want to go there, but have a look.

@nacin
15 years ago

Patch I worked up independently.

#2 @ryan
15 years ago

I like .2.diff. It avoids unintended consequences with manipulating blog_public and is more obvious.

#3 @ryan
15 years ago

  • Owner set to ryan
  • Resolutionfixed
  • Status newclosed

In [19304]:

Introduce wp_no_robots(). Call it for pages that should never be indexed, regardless of blog privacy settings. Props nacin. fixes #19251

Note: See TracTickets for help on using tickets.