Opened 7 years ago

Closed 7 years ago

#2825 closed defect (bug) (fixed)

Category Exclusion for real

Reported by: skeltoac Owned by: anonymous
Priority: normal Milestone: 2.1
Component: Administration Version: 2.1
Severity: normal Keywords: has-patch 2nd-opinion
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 7 years ago.

Download all attachments as: .zip

Change History (7)

  • 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.

Bump! Anybody have anything to say about this patch?

comment:3   ryan7 years ago

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

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

comment:4   sdiz7 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.

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?

comment:6   ryan7 years ago

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