Using the commons_search_solr_user feature, /people shows the users who have "Recently contributed to Array."

People___The_Academy.png

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ezra-g’s picture

Status: Active » Postponed (maintainer needs more info)

Can you post steps to reproduce this on a fresh instal of Commons packaged with build-commons-dev.make? I'm unable to do so.

dstol’s picture

Status: Postponed (maintainer needs more info) » Active

Gladly, http://youtu.be/IdkGbzD65b4. I forgot to show that I ran a drush search-index and cron to confirm that the index was 100%

ezra-g’s picture

The resolution isn't high enough to read the error you get during installation at http://www.youtube.com/watch?v=IdkGbzD65b4&feature=youtu.be, but since I haven't seen that error message recently, I wonder if can explicitly confirm that you did a git pull on the Commons repo and packaged with build-commons-dev.make.

japerry’s picture

The problem is reproducible. Remove all content and groups and you'll get this error.

dstol’s picture

That's not what I'm doing at all. This is a completely fresh install. Built from 25c628742787cb6e15925af2281a811578498d3a

New video, now with more pixels, http://youtu.be/wWx1XqYC5Jo

ezra-g’s picture

Priority: Normal » Major
Issue tags: +Commons 7.x-3.3 radar

I was able to reproduce on Commons packaged this morning on an Acquia Cloud server.

This is a pretty unfortunate first experience for folks installing a service that we hope they'll buy. Bringing into the 3.3 radar.

japerry’s picture

Status: Active » Needs review
FileSize
2.46 KB

After trying to reproduce how I reproduced this issue earlier, I can no longer reproduce it. However, I could get it to do some funky things like "recently contributed to ."

This patch should remove the line if the og_title is not a string.

japerry’s picture

Status: Needs review » Needs work

This patch is based off of work done with Nick at from the solr search group. Will need to re-roll.

japerry’s picture

Assigned: Unassigned » Zarabadoo
Status: Needs work » Needs review
FileSize
917 bytes

This one is odd, because I think there is something more fundamentally wrong with the theme layer in regards to search and the template used to display results.

However, in the interim this patch will fix the problem. The existing template was printing the group array, which is an array (figure that! ;-) ) .. instead we should be passing it through render.

Zarabadoo’s picture

Status: Needs review » Reviewed & tested by the community

This looks good and will solve the specific issue mentioned in the title of this.

ezra-g’s picture

Status: Reviewed & tested by the community » Fixed

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

Anonymous’s picture

Issue summary: View changes

Update name of feature