Error in content administration using taxonomy filter

Still present in 6.12

no patch, but still exists.

bump up, still present in 6.14

Patch attached. Killes found / fixed it

Let's get that into 6.15

Bugs are first fixed on Drupal 7 and then backported to Drupal 6 to ensure we do not introduce regressions.

i would schedule this complicated fix and review for around the year 2015, or even slighly later. This is also a little to bloody-edge for 8.x

In D8, one can't filter nodes tagged with a certain tag. The menu callback function for the page admin/content does not propose a way to filter nodes by term. The only filters proposed by node_filters() are the status, the node type and optionally the language. The function is the same in D7. But in D6 we can filter by terms (seen in code and in the UI). What do we do for D8 ? Do we allow the possibility to filter by term ? Or do we remove the legacy code ?
Here's a patch for the second option (the most easy one), even if I would prefer the first option. I don't know why it was removed.

Forgot to change status.

Very odd. The patch addresses cruft that was left in after an existing feature was removed.

My opinion is that we should commit the patch and then work on fixing the issue by providing better content admin tools in core.

I want to agree with you here. What about D7 and D6 ? I would say backport the patch to D7 (should be easy).

And for d6 the patch from eugenmayer should be used, as it's a bug there.

D7, yes. But D6 is supposed to allow taxonomy filtering.

We can't write a patch which tests a non existing feature, or?

The code which removes it looks fine!

#15: Yeah, I agree there's no test to write here.

Committed to 8.x. Moving to 7.x.

Let's see what the testbot thinks, if it works fine, it's RTBC.

#9: 262498-remove-taxonomy-content-filter.patch queued for re-testing.

So #9 is RTBC for D7.

Committed and pushed to 7.x too.

Marking this issue fixed, though it sounds like we need a follow-up to replace this functionality in D8.

well, isnt that fixed when its also applied to D6?

Needs a current D6 port. :)

durr

tried with #0 but as patch ;-) for D 6.x (dev)

(and meanwhile some people agree it's better to speak of versions if you mean versions).

Yep, that looks good and fixes the issue for me.

I applied patch #26 successfully as well.

Fixed in 6.x via #648516: content/admin/node pager_query with taxonomy filter returns incorrect results when multiple versions of nodes are kept.