In reading the docs more carefully, there is no property called 'rating'. The property should be aggregateRating or reviewRating or starRating. The 'rating' property would just get ignored. Also, rating is not recognized and validated by Google, they require aggregateRating for many of the currently supported schemas, so this needs to be fixed.

CommentFileSizeAuthor
#4 aggregate-rating-cleanup-2945920.patch35.56 KBKarenS
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

KarenS created an issue. See original summary.

ChristophWeber’s picture

ChristophWeber CreditAttribution: ChristophWeber at BIZX LLC commented

Oh dang, I could have told you if I had more time to check your patch in depth. Good catch!

ChristophWeber’s picture

ChristophWeber CreditAttribution: ChristophWeber at BIZX LLC commented
Title: AggregateRating cleanup » AggregateRating cleanup (D8)
Issue summary: View changes
KarenS’s picture

KarenS CreditAttribution: KarenS at Lullabot commented
Status: Active » Needs review
FileSize
aggregate-rating-cleanup-2945920.patch35.56 KB

  • KarenS committed 4eac0f5 on 8.x-1.x 
    Issue #2945920 by KarenS: AggregateRating cleanup (D8)
KarenS’s picture

KarenS CreditAttribution: KarenS at Lullabot commented
Status: Needs review » Fixed

This is fixed, includes fixing tests and adding an update hook to alter any "rating" values that might have already been there.

Status: Fixed » Closed (fixed)

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