Make WordPress Core

Opened 16 years ago

Closed 16 years ago

#15573 closed defect (bug) (fixed)

Post Count Links are Bogus When query_var = false

Reported by: miqrogroove Owned by: scribu
Priority: normal Milestone: 3.1
Component: Taxonomy Version: 3.0.1
Severity: normal Keywords:
Cc: Focuses:

Description

If I call register_taxonomy() with $args = ('query_var' => FALSE) then the links listed in the post count column of the taxonomy editor don't work as expected. The links should not be displayed if they are defective like this.

Attachments (2)

15573.diff (803 bytes ) - added by scribu 16 years ago.
Use 'taxonomy' => 'foo', 'term' => 'bar'
15573.2.diff (897 bytes ) - added by scribu 16 years ago.
Also check public flag

Download all attachments as: .zip

Change History (9)

#1 @scribu
16 years ago

  • Milestone Awaiting Review3.1
  • Owner set to scribu
  • Status newaccepted

#2 @scribu
16 years ago

  • Resolutionfixed
  • Status acceptedclosed

(In [16578]) Don't link post counts if taxonomy isn't queryable. Fixes #15573

#3 follow-up: @nacin
16 years ago

  • Resolution fixed
  • Status closedreopened

Just because it doesn't have a query var, doesn't make it not queryable. We have taxonomy and term query vars for that. You have to make a separate publicly queryable check.

@scribu
16 years ago

Use 'taxonomy' => 'foo', 'term' => 'bar'

#4 in reply to: ↑ 3 @scribu
16 years ago

Replying to nacin:

Just because it doesn't have a query var, doesn't make it not queryable. We have taxonomy and term query vars for that.

15573.diff does that.

You have to make a separate publicly queryable check.

The 'public' option reads: @TODO not implemented.

Hm...

@scribu
16 years ago

Also check public flag

#5 @scribu
16 years ago

The only way the columns would not be linked is with public=false and show_ui=true, which doesn't make much sense, but hey...

#6 @scribu
16 years ago

(In [16579]) Check public flag instead of query_var to decide if counts should be linked. See #15573

#7 @scribu
16 years ago

  • Resolutionfixed
  • Status reopenedclosed
Note: See TracTickets for help on using tickets.