From: Andre Noll <maan@systemlinux.org>
Date: Sun, 5 Jun 2011 17:14:37 +0000 (+0200)
Subject: Clear score table on mood reload.
X-Git-Tag: v0.4.8~31
X-Git-Url: http://git.tue.mpg.de/?a=commitdiff_plain;h=75805d92f62468a4beb20b590a7273f25bb64882;p=paraslash.git

Clear score table on mood reload.

Whenever a blob is added, the mood event handler is called
which may reload the current mood in order to react to the
change being made. However, we missed to clear the score table
first. This is necessary to to re-insert all admissible files.
Currently this fails with

	afs_event: table moods, event 8: key already exists in rbtree
---

diff --git a/mood.c b/mood.c
index d82d5a52..93461ee8 100644
--- a/mood.c
+++ b/mood.c
@@ -904,6 +904,8 @@ static int reload_current_mood(void)
 int moods_event_handler(enum afs_events event, __a_unused struct para_buffer *pb,
 		void *data)
 {
+	int ret;
+
 	if (!current_mood)
 		return 0;
 	switch (event) {
@@ -916,6 +918,10 @@ int moods_event_handler(enum afs_events event, __a_unused struct para_buffer *pb
 	case BLOB_ADD:
 		if (data == moods_table || data == playlists_table)
 			return 1; /* no reload necessary for these */
+		ret = clear_score_table();
+		if (ret < 0)
+			PARA_CRIT_LOG("clear score table returned %s\n",
+				para_strerror(-ret));
 		return reload_current_mood();
 	/* these also require reload of the score table */
 	case ATTRIBUTE_ADD: