In working on a view for drupal.org as part of #1772152: Please evaluate profile spam view I ended up creating a "User: Comments authored" relationship. Very useful. Combined with aggregation and using "count distinct" it allows a user view to include # comments authored and # of nodes authored.

Patch attached.

CommentFileSizeAuthor
#15 1773966-15.patch778 bytesjoshmiller
#12 views-uid_comment_relationship-1773966-3.patch771 bytesScott Robertson
#3 views-uid_comment_relationship-1773966-3.patch771 bytesparisek
views-uid_comment_relationship.patch794 bytesWorldFallz

Comments

Status: Needs review » Needs work

The last submitted patch, views-uid_comment_relationship.patch, failed testing.

Jason Dean’s picture

Jason Dean commented

ooh really useful and working for me - thanks!

parisek’s picture

parisek
Primary language Czech
commented
Priority: Normal » Major
Status: Needs work » Needs review
StatusFileSize
new views-uid_comment_relationship-1773966-3.patch771 bytes

I hope correct patch is attached and soon will be merged

pixelsweatshop’s picture

pixelsweatshop commented
Priority: Major » Normal

I can confirm that the patch in #3 works. Moving from major to normal as this is a feature request, not a bug.

mstrelan’s picture

mstrelan commented
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community
alan d.’s picture

alan d. commented

+rtbtc - used with activity streams

albie001’s picture

albie001 commented

Works great. Thanks

Scott Robertson’s picture

Scott Robertson commented

Working great for me as well.

sergeypavlenko’s picture

sergeypavlenko commented

Hi all

Thanks @parisek. Patch worked for me. Issue created a long time ago, but not commited, need to fix this!

Kimberley_p’s picture

Kimberley_p commented

Working well for me too. Hopefully be committed soon...?

colan’s picture

colan
he/him
Primary language English
Location Toronto 🇨🇦
commented

We've recently switched our testing from the old qa.drupal.org to DrupalCI. Because of a bug in the new system, #2623840: Views (D7) patches not being tested, older patches must be re-uploaded. On re-uploading the patch, please set the status to "Needs Review" so that the test bot will add it to its queue.

If all tests pass, change the Status back to "Reviewed & tested by the community". We'll most likely commit the patch immediately without having to go through another round of peer review.

We apologize for the trouble, and appreciate your patience.

Scott Robertson’s picture

Scott Robertson commented
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new views-uid_comment_relationship-1773966-3.patch771 bytes

Re-uploading patch, per #11.

Scott Robertson’s picture

Scott Robertson commented
Status: Needs review » Reviewed & tested by the community

Marking as RTBC again now that the new tests have passed.

jurriaanroelofs’s picture

jurriaanroelofs commented

Good patch, tested and works great. +1

joshmiller’s picture

joshmiller
he/him
Primary language English
Location Indianapolis, Indiana, USA
commented
StatusFileSize
new 1773966-15.patch778 bytes

Re-rolled.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Reviewed & tested by the community » Fixed
Parent issue: » #2960871: Plan for Views 7.x-3.23 release

Committed. Thanks!

Status: Fixed » Closed (fixed)

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