Make WordPress Core

Opened 20 years ago

Closed 19 years ago

#2825 closed defect (bug) (fixed)

Category Exclusion for real

Reported by: skeltoac's profile skeltoac Owned by:
Milestone: 2.1 Priority: normal
Severity: normal Version: 2.1
Component: Administration Keywords: has-patch 2nd-opinion
Focuses: Cc:

Description

Query vars like "cat=-1" will exclude all posts with ONLY category 1. Posts severally categorized cannot be excluded by category.

Attachments (1)

true-cat-exclusion.diff (1.2 KB) - added by skeltoac 20 years ago.

Download all attachments as: .zip

Change History (7)

#1 @skeltoac
20 years ago

  • Keywords has-patch 2nd-opinion added

Attached patch true-cat-exclusion.diff ensures all posts in the excluded categories are excluded. It costs one query. This will help folks with asides or other kinds of multiple-loop systems.

#2 @skeltoac
20 years ago

Bump! Anybody have anything to say about this patch?

#3 @ryan
20 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [3886]) Cat exclusion from skeltoac. fixes #2825

#4 @sdiz
20 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

This change break RSS feed.

When there are a post with the more then one category, it is emitted more then once, causeing duplicated GUID.

#5 @skeltoac
20 years ago

sdiz: I'm not seeing the effect you describe. I have a post with several categories and when I view one of its categories as a feed it appears only once. Are you running a trunk checkout or did you apply this patch manually? See #2709, might be related. What steps to reproduce? Can you get a printout of the offending query?

#6 @ryan
19 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed
Note: See TracTickets for help on using tickets.