There is currently not an overall score for each review. It would be good if one could be calculated from all of the axes.

Comments

colan’s picture

Assigned: colan » Unassigned
Status: Active » Needs review
StatusFileSize
new4.58 KB

Here's a simple patch for this. I got it working, but I spent very little time theming. We should come up with a nicer way of displaying this. I like the way overall scores/weights are displayed in userreview. It seems complicated, but it might be worth it. Here is a screenshot. Or maybe the number could just be on the right side of the table in a large font. Thoughts?

If anybody out there can embellish this patch, please feel free. ;)

colan’s picture

StatusFileSize
new4.97 KB

Here's an up-to-date patch that works with the new preview code.

jo1ene’s picture

Status: Needs review » Needs work

+1 for functionality

Works correctly in my tests

-1 for UI

I believe the overall score should be at the top. That's where most users would expect to find such a thing. I further think that if we are calculating an overall score, it should aslo be displayed on the reviewed node itself. I understand having the views-generated list of reviews on another tab, but some kind of score should be displayed on the node itself. There should be some sort of handle besides a tab to let users know that there are reviews to be seen.

We should also be looking at a more robust, clearer display of scores, but maybe that's a different issue. I too wonder if this could go the way of userreview. Mabybe there could be collaboration? The two projects are similar with different presentations.

Crell’s picture

Version: master » 5.x-1.x-dev

4.7 is now in maintenance mode, so new features should go against Drupal 5 only. Thanks.

drupal_noob’s picture

Is this feature going to be added in a upcoming version of nodereview for drupal 5? That would be so awesome:)

Sophia’s picture

Any news on this yet? :)

danwassink’s picture

I can't get the patch to work for the overall score. Anyone have any ideas?

parajeff’s picture

This would be very useful to have in the 5.x version. Anyone working on this?

Fayna’s picture

I would love this feature as well. Subscribing to this thread.

SamRose’s picture

Hmmm.. this is a wee bit misleading (I thought this patch would work with 5.x, but apparently it was posted as a 4.7 issue originally)

SamRose’s picture

I am trying to look through this code now, and see if I can work it out from 4.7 to 5.x...

summit’s picture

Version: 5.x-1.x-dev » 6.x-1.x-dev

Needing this in D6 version..greetings, Martijn

kompressaur’s picture

subscribing

liliplanet’s picture

subscribe thx!

danielhonrade’s picture

Status: Needs work » Needs review
StatusFileSize
new19.79 KB

just committed a solution to this, please see attached file

danielhonrade’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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