From 7d0dab121ac9a03fc0ae3a12350422d9e1613093 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Stephan=20A=C3=9Fmus?= <superstippi@gmx.de>
Date: Sun, 24 Aug 2014 19:17:19 +0200
Subject: [PATCH] PackageInfo: Added setting rating summary before any ratings
 are known.

Fleshed out RatingSummary.
---
 src/apps/haikudepot/PackageInfo.cpp | 76 +++++++++++++++++++++++++++++
 src/apps/haikudepot/PackageInfo.h   | 10 ++++
 2 files changed, 86 insertions(+)

diff --git a/src/apps/haikudepot/PackageInfo.cpp b/src/apps/haikudepot/PackageInfo.cpp
index 37176d8406..2aaaf2d878 100644
--- a/src/apps/haikudepot/PackageInfo.cpp
+++ b/src/apps/haikudepot/PackageInfo.cpp
@@ -374,6 +374,62 @@ UserRating::operator!=(const UserRating& other) const
 }
 
 
+// #pragma mark - RatingSummary
+
+
+RatingSummary::RatingSummary()
+	:
+	averageRating(0.0f),
+	ratingCount(0)
+{
+	for (int i = 0; i < 5; i++)
+		ratingCountByStar[i] = 0;
+}
+
+
+RatingSummary::RatingSummary(const RatingSummary& other)
+{
+	*this = other;
+}
+
+
+RatingSummary&
+RatingSummary::operator=(const RatingSummary& other)
+{
+	averageRating = other.averageRating;
+	ratingCount = other.ratingCount;
+
+	for (int i = 0; i < 5; i++)
+		ratingCountByStar[i] = other.ratingCountByStar[i];
+
+	return *this;
+}
+
+
+bool
+RatingSummary::operator==(const RatingSummary& other) const
+{
+	if (averageRating != other.averageRating
+		|| ratingCount != other.ratingCount) {
+		return false;
+	}
+
+	for (int i = 0; i < 5; i++) {
+		if (ratingCountByStar[i] != other.ratingCountByStar[i])
+			return false;
+	}
+	
+	return true;
+}
+
+
+bool
+RatingSummary::operator!=(const RatingSummary& other) const
+{
+	return !(*this == other);
+}
+
+
 // #pragma mark - PublisherInfo
 
 
@@ -508,6 +564,7 @@ PackageInfo::PackageInfo()
 	fFullDescription(),
 	fChangelog(),
 	fUserRatings(),
+	fCachedRatingSummary(),
 	fScreenshots(),
 	fState(NONE),
 	fDownloadProgress(0.0),
@@ -531,6 +588,7 @@ PackageInfo::PackageInfo(const BString& title,
 	fChangelog(),
 	fCategories(),
 	fUserRatings(),
+	fCachedRatingSummary(),
 	fScreenshots(),
 	fState(NONE),
 	fDownloadProgress(0.0),
@@ -551,6 +609,7 @@ PackageInfo::PackageInfo(const PackageInfo& other)
 	fChangelog(other.fChangelog),
 	fCategories(other.fCategories),
 	fUserRatings(other.fUserRatings),
+	fCachedRatingSummary(other.fCachedRatingSummary),
 	fScreenshots(other.fScreenshots),
 	fState(other.fState),
 	fInstallationLocations(other.fInstallationLocations),
@@ -573,6 +632,7 @@ PackageInfo::operator=(const PackageInfo& other)
 	fChangelog = other.fChangelog;
 	fCategories = other.fCategories;
 	fUserRatings = other.fUserRatings;
+	fCachedRatingSummary = other.fCachedRatingSummary;
 	fScreenshots = other.fScreenshots;
 	fState = other.fState;
 	fInstallationLocations = other.fInstallationLocations;
@@ -595,6 +655,7 @@ PackageInfo::operator==(const PackageInfo& other) const
 		&& fChangelog == other.fChangelog
 		&& fCategories == other.fCategories
 		&& fUserRatings == other.fUserRatings
+		&& fCachedRatingSummary == other.fCachedRatingSummary
 		&& fScreenshots == other.fScreenshots
 		&& fState == other.fState
 		&& fFlags == other.fFlags
@@ -697,9 +758,24 @@ PackageInfo::AddUserRating(const UserRating& rating)
 }
 
 
+void
+PackageInfo::SetRatingSummary(const RatingSummary& summary)
+{
+	if (fCachedRatingSummary == summary)
+		return;
+
+	fCachedRatingSummary = summary;
+
+	_NotifyListeners(PKG_CHANGED_RATINGS);
+}
+
+
 RatingSummary
 PackageInfo::CalculateRatingSummary() const
 {
+	if (fUserRatings.CountItems() == 0)
+		return fCachedRatingSummary;
+	
 	RatingSummary summary;
 	summary.ratingCount = fUserRatings.CountItems();
 	summary.averageRating = 0.0f;
diff --git a/src/apps/haikudepot/PackageInfo.h b/src/apps/haikudepot/PackageInfo.h
index 30d42182c8..dc62aaf2a5 100644
--- a/src/apps/haikudepot/PackageInfo.h
+++ b/src/apps/haikudepot/PackageInfo.h
@@ -127,6 +127,14 @@ typedef List<UserRating, false> UserRatingList;
 
 
 class RatingSummary {
+public:
+								RatingSummary();
+								RatingSummary(const RatingSummary& other);
+
+			RatingSummary&		operator=(const RatingSummary& other);
+			bool				operator==(const RatingSummary& other) const;
+			bool				operator!=(const RatingSummary& other) const;
+
 public:
 			float				averageRating;
 			int					ratingCount;
@@ -269,6 +277,7 @@ public:
 			bool				AddUserRating(const UserRating& rating);
 			const UserRatingList& UserRatings() const
 									{ return fUserRatings; }
+			void				SetRatingSummary(const RatingSummary& summary);
 			RatingSummary		CalculateRatingSummary() const;
 
 			bool				AddScreenshot(const BitmapRef& screenshot);
@@ -293,6 +302,7 @@ private:
 			BString				fChangelog;
 			CategoryList		fCategories;
 			UserRatingList		fUserRatings;
+			RatingSummary		fCachedRatingSummary;
 			BitmapList			fScreenshots;
 			PackageState		fState;
 			PackageInstallationLocationSet