[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    [kdeplasma-addons/plasma/sreich/youtube-runner] runners/youtube: loop a bit less
From:       Shaun Reich <shaun.reich () kdemail ! net>
Date:       2012-02-26 19:50:18
Message-ID: 20120226195018.C7B01A60B9 () git ! kde ! org
[Download RAW message or body]

Git commit 555b4cdd49e1dd53f73846d00fa5bf3e980d5068 by Shaun Reich.
Committed on 26/02/2012 at 20:49.
Pushed by sreich into branch 'plasma/sreich/youtube-runner'.

loop a bit less

M  +3    -10   runners/youtube/youtube.cpp

http://commits.kde.org/kdeplasma-addons/555b4cdd49e1dd53f73846d00fa5bf3e980d5068

diff --git a/runners/youtube/youtube.cpp b/runners/youtube/youtube.cpp
index 7ce0ca5..57e6a2b 100644
--- a/runners/youtube/youtube.cpp
+++ b/runners/youtube/youtube.cpp
@@ -44,11 +44,11 @@ YouTube::YouTube(QObject *parent, const QVariantList& args)
     setObjectName(QLatin1String("YouTube"));
     setIgnoredTypes(Plasma::RunnerContext::FileSystem | \
Plasma::RunnerContext::Directory | Plasma::RunnerContext::NetworkLocation);  
-    Plasma::RunnerSyntax s(QLatin1String(":q:"), i18n("Finds YouTube video matching \
:q:.")); +    Plasma::RunnerSyntax s(QLatin1String("youtube :q:"), i18n("Finds \
YouTube video matching :q:."));  s.addExampleQuery(QLatin1String("youtube :q:"));
     addSyntax(s);
 
-    addSyntax(Plasma::RunnerSyntax(QLatin1String("youtube"), i18n("Lists the videos \
matching the query, using YouTube search"))); +//    \
addSyntax(Plasma::RunnerSyntax(QLatin1String("youtube"), i18n("Lists the videos \
matching the query, using YouTube search")));  setSpeed(SlowSpeed);
     setPriority(LowPriority);
 }
@@ -71,8 +71,6 @@ void YouTube::match(Plasma::RunnerContext &context)
         return;
     }
 
-    kDebug() << "YouTube Runner, Sleeping for 1 seconds";
-
     QEventLoop loop;
 
     TubeJob tubeJob(term);
@@ -115,12 +113,7 @@ void YouTube::parseJson(const QByteArray& data, \
                Plasma::RunnerContext &context)
         QVariantList thumbnailList = subSubMap.value("media$thumbnail").toList();
 
         QString thumbnail;
-        foreach (const QVariant& variant, thumbnailList) {
-            //FIXME: is this even reliable?
-            kDebug() << variant.toMap().keys();
-                thumbnail = thumbnailList.at(2).toMap().value("url").toString();
-            kDebug() << "DOWNLOADING URL:" << thumbnail;
-        }
+        thumbnail = thumbnailList.at(2).toMap().value("url").toString();
 
         QEventLoop loop;
         m_thumbnailDownloader = new QNetworkAccessManager();


[prev in list] [next in list] [prev in thread] [next in thread]