From 3dc79838b5484aba0d805a3c5dac4c0e26503ac6 Mon Sep 17 00:00:00 2001
From: "Javier S. Pedro" <maemo@javispedro.com>
Date: Wed, 28 Sep 2011 03:15:12 +0200
Subject: trying to fix more weird albumart behaviour

---
 qmafwwatchlet/metawatch-digital.qml   |  2 +-
 qmafwwatchlet/qmafwwatchletplayer.cpp | 37 +++++++++++++++++++++--------------
 qmafwwatchlet/qmafwwatchletplayer.h   |  1 +
 3 files changed, 24 insertions(+), 16 deletions(-)

diff --git a/qmafwwatchlet/metawatch-digital.qml b/qmafwwatchlet/metawatch-digital.qml
index 22c7833..345aab6 100644
--- a/qmafwwatchlet/metawatch-digital.qml
+++ b/qmafwwatchlet/metawatch-digital.qml
@@ -22,7 +22,7 @@ Rectangle {
 
 		Text {
 			anchors.fill: parent
-			text: player.title
+			text: player.title.length > 0 ? player.title : qsTr("No media");
 			color: "black"
 		}
 	}
diff --git a/qmafwwatchlet/qmafwwatchletplayer.cpp b/qmafwwatchlet/qmafwwatchletplayer.cpp
index d4f9686..cc63508 100644
--- a/qmafwwatchlet/qmafwwatchletplayer.cpp
+++ b/qmafwwatchlet/qmafwwatchletplayer.cpp
@@ -16,7 +16,7 @@ QMafwWatchletPlayer::QMafwWatchletPlayer(QMafwWatchlet* watchlet) :
 	_active(false),
 	_renderer(0),
 	_state(MafwRenderer::Stopped),
-	_title(tr("No media")),
+	_title(),
 	_album(),
 	_artist(),
 	_rendererArt(),
@@ -90,7 +90,6 @@ void QMafwWatchletPlayer::stop()
 void QMafwWatchletPlayer::next()
 {
 	if (!_renderer) return;
-	qDebug() << "next";
 	_renderer->next();
 }
 
@@ -118,25 +117,22 @@ void QMafwWatchletPlayer::setRenderer(MafwRenderer * renderer)
 {
 	if (_renderer && _active) {
 		disconnect(_renderer, 0, this, 0);
-	}
-	_renderer = renderer;
-	reconnect();
-	if (!_renderer && _active) {
-		_title = tr("No media");
-		emit titleChanged();
+		_title.clear();
 		_album.clear();
-		emit albumChanged();
 		_artist.clear();
-		emit artistChanged();
-		_rendererArt.clear();
-		_mediaArt.clear();
-		emit mediaArtChanged();
+		handleMediaChange();
+		emit titleChanged();
+		updateMediaArt();
 	}
+	_renderer = renderer;
+	reconnect();
 }
 
 void QMafwWatchletPlayer::reconnect()
 {
 	if (_renderer && _active) {
+		connect(_renderer, SIGNAL(mediaChanged(int, const MafwContent&)),
+			this, SLOT(handleMediaChange()));
 		connect(_renderer, SIGNAL(metadataChanged(const QString&, const QList<QVariant>&)),
 			this, SLOT(handleChangedMetadata(const QString&, const QList<QVariant>&)));
 		connect(_renderer, SIGNAL(stateChanged(MafwRenderer::State)),
@@ -155,6 +151,11 @@ QString QMafwWatchletPlayer::stripAlbumArtComponent(const QString& component)
 	static QRegExp stripB("^[()_{}[]!@#$^&*+=|\\\\/\"'?<>~`\\s\\t]*");
 	static QRegExp stripE("[()_{}[]!@#$^&*+=|\\\\/\"'?<>~`\\s\\t]*$");
 	QString s(component);
+
+	if (s.isEmpty()) {
+		return QString(" ");
+	}
+
 	s = s.replace(rsb, "");
 	s = s.replace(rfb, "");
 	s = s.replace(rrb, "");
@@ -162,8 +163,8 @@ QString QMafwWatchletPlayer::stripAlbumArtComponent(const QString& component)
 	s = s.replace(stripE, "");
 	s = s.replace("  ", " ");
 	s = s.replace("\t", " ");
-	s = s.toLower();
-	return s;
+
+	return s.toLower();
 }
 
 QString QMafwWatchletPlayer::mediaArtPath() const
@@ -209,6 +210,12 @@ void QMafwWatchletPlayer::updateMediaArt()
 	}
 }
 
+void QMafwWatchletPlayer::handleMediaChange()
+{
+	// Sometimes qmafw does not send the renderer-art metadata, so we assume it's been cleared here.
+	_rendererArt.clear();
+}
+
 void QMafwWatchletPlayer::handleChangedMetadata(const QString &s, const QList<QVariant> &l)
 {
 	if (s == MAFW_METADATA_KEY_TITLE) {
diff --git a/qmafwwatchlet/qmafwwatchletplayer.h b/qmafwwatchlet/qmafwwatchletplayer.h
index 4a587a6..0d035e2 100644
--- a/qmafwwatchlet/qmafwwatchletplayer.h
+++ b/qmafwwatchlet/qmafwwatchletplayer.h
@@ -68,6 +68,7 @@ private:
 	void updateMediaArt();
 
 private slots:
+	void handleMediaChange();
 	void handleChangedMetadata(const QString& s, const QList<QVariant>& l);
 	void handleChangedState(MafwRenderer::State state);
 	void handleMediaInfo(const MafwMediaInfo& info);
-- 
cgit v1.2.3