Web 3.0 Semantic approach

Thanks for the link and making an issue, this is on the TODO list already.

The concept is rather simple, but there's often a lot of data that is outside of Fivestar's control regarding the markup. For example, trying to make a complete "review" requires not only the Fivestar rating, but also the title and contents of the reviews, which is outside the scope of the output Fivestar provides.

<div class="hreview">
   <span class="item">
     <span class="fn">L'Amourita Pizza</span>
   </span>
   <span class="rating">3.5</span> <-- This is the only line Fivestar is responsible for.
   <span class="reviewer">Ulysses Grant</span>
   <span class="dtreviewed">2009-01-06</span>
   <span class="summary">"Delicious, tasty pizza in Eastlake."</span>
</div>

Regarding "aggregated" reviews, Fivestar at least has a little more responsibility here.

<div class="hreview-aggregate">
   <div class="item">
      <span class="fn">L'Amourita Pizza</span>
   </div>
   <div class="rating">
      <span class="average">4.4</span>
   </div> 
   <span class="count">1,313</span>
</div>

Is actually not far off from what we're using already:

<div class="fivestar-summary fivestar-summary-combo fivestar-feedback-enabled">
  <span class="user-rating">Your rating: <span>4</span></span>
  <span class="average-rating">Average: <span>3</span></span>
  <span class="total-votes">(<span>2</span> votes)</span>
</div>

So at least that could be fixed just by changing the name of some classes for the most part, but again the "title" of the item being reviewed isn't within the scope of Fivestar's output. Changing these classes will most definitely break existing CSS on sites that have themed their Fivestar widgets, so this is definitely in the "2.0" category for Fivestar since we don't want to break existing sites in the 1.x version.

Moving to active, "needs review" is for issues with patches or documentation changes.

It shouldn't be fivestar's job to output a complete hReview. To sufficiently offer hReview functionality, it only needs to output the 'rating' class in the appropriate areas:

For static display, all you have to do is add the class 'rating' around the initial div which contains the rating (line 1384 of fivestar.module,v 1.13.2.56)

$first = $n == 1 ? ' star-first rating' : '';

For vote-enabled display, wrap rating around the !stars output for the Average rating (line 1418 of fivestar.module,v 1.13.2.56)

$output .= '<span class="average-rating">'. t('Average: <span class="rating">!stars</span>', array('!stars' => $average_stars)) .'</span>';

I downloaded the latest copy of fivestar (1.18) and attempted to create a patch like I described above (http://drupal.org/node/495118#comment-1883448). This is my first patch so please let me know if I'm messing anything up!

Even thought this issue is older, I'm going to close it as a duplicate of #901260: Google microdata markup? mainly because there's been more discussion on that issue.