I've enabled Administration Views to get an improved way of finding content, but am seeing many duplicate rows for each node, with identical content in each.

A quick check tells me that there's an extra duplicate for each taxonomy term that a node refers to (I have product nodes that have "taxonomy reference" fields.)
So for a product with one taxonomy checked, it shows up twice.
For a product with 11 taxonomy terms checked, it shows up 12 times.

Something wrong with the SQL that's generating the view...

CommentFileSizeAuthor
#9 admin_views-1332238-9.patch469 bytesfizk
#5 1332238.patch811 bytesdawehner
#1 admin_views_duplicates-1332238-1.patch859 byteszambrey

Comments

zambrey’s picture

zambrey commented
Status: Active » Needs review
StatusFileSize
new admin_views_duplicates-1332238-1.patch859 bytes

Here's the patch. For quick fix you can go to view edit page and check Distinct under Query settings.

EvanDonovan’s picture

EvanDonovan commented

I would agree that this is the resolution, though I did not test the patch, only configured in the interface.

kristen pol’s picture

kristen pol
she/her
Primary language English
Location Santa Cruz, CA, USA
commented

I ran into this problem independently and set the distinct checkbox and it worked for me. I didn't try the patch though.

Kristen

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Title: Duplicate nodes listed in admin/content with Administration Views enabled (taxonomy-related) » Duplicate nodes listed on admin/content (taxonomy-related)
Project: Administration menu » Administration Views
Version: 7.x-3.x-dev » 7.x-1.x-dev
Status: Needs review » Needs work

Needs to be re-rolled against admin_views.

dawehner’s picture

dawehner
Primary language German
commented
Status: Needs work » Needs review
StatusFileSize
new 1332238.patch811 bytes

Here is a rerole for 7.x-1.x

This indeed fixes the problem of the "one to many"-join

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Needs review » Reviewed & tested by the community
Issue tags: +Needs backport to D6

Thanks!

IIRC, a similar option existed in D6

dawehner’s picture

dawehner
Primary language German
commented
Status: Reviewed & tested by the community » Patch (to be ported)

Thanks for the review.

Committed to 7.x-1.x

To be able to backport it to d6 a 6.x branch is required. I will do this this evening.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Version: 7.x-1.x-dev » 6.x-1.x-dev

What's wrong this existing branch though? ;)

fizk’s picture

fizk commented
Status: Patch (to be ported) » Needs review
StatusFileSize
new admin_views-1332238-9.patch469 bytes

Re-rolled for D6. Will commit to D6 branch if it looks good.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented

Is that patch based on a re-export or is that manually edited in? We should always base patches on re-exports, so an earlier change from a different issue does not pop up in a later patch that re-exports a view.

If it's based on a re-export, then it's good to go.

fizk’s picture

fizk commented
Status: Needs review » Fixed

It's based on a re-export.

Committed to 6.x-dev:

http://drupalcode.org/project/admin_views.git/commit/c843eaa55b757c15dcf...

Automatically closed -- issue fixed for 2 weeks with no activity.