From 54fee8ffed6f4765a1823e41fc12002cc644eca9 Mon Sep 17 00:00:00 2001
From: = <=>
Date: Sun, 30 Sep 2012 20:31:22 +0100
Subject: [PATCH] Select all facility - code review fixes

---
 .../accounts/spotify/SpotifyAccountConfig.cpp          | 10 +++-------
 .../accounts/spotify/SpotifyAccountConfig.ui           |  6 +++---
 2 files changed, 6 insertions(+), 10 deletions(-)

diff --git a/src/libtomahawk/accounts/spotify/SpotifyAccountConfig.cpp b/src/libtomahawk/accounts/spotify/SpotifyAccountConfig.cpp
index 1b7f537cf..09ce54add 100644
--- a/src/libtomahawk/accounts/spotify/SpotifyAccountConfig.cpp
+++ b/src/libtomahawk/accounts/spotify/SpotifyAccountConfig.cpp
@@ -52,7 +52,7 @@ SpotifyAccountConfig::SpotifyAccountConfig( SpotifyAccount *account )
 
     connect( m_ui->usernameEdit, SIGNAL( textEdited( QString ) ), this, SLOT( resetLoginButton() ) );
     connect( m_ui->passwordEdit, SIGNAL( textEdited( QString ) ), this, SLOT( resetLoginButton() ) );
-    connect( m_ui->selectAllCheckbox, SIGNAL( stateChanged( int ) ), this, SLOT( selectAllPlaylists()));
+    connect( m_ui->selectAllCheckbox, SIGNAL( stateChanged( int ) ), this, SLOT( selectAllPlaylists() ) );
     loadFromConfig();
 
     m_playlistsLoading = new AnimatedSpinner( m_ui->playlistList );
@@ -203,17 +203,13 @@ SpotifyAccountConfig::loginResponse( bool success, const QString& msg, const QSt
 
 }
 
-
-
 void
 SpotifyAccountConfig::selectAllPlaylists()
 {
-    qDebug() << Q_FUNC_INFO << "Setting playlists to sync";
-
-    for(int i = 0; i < m_ui->playlistList->count(); i++)
+    for( int i = 0; i < m_ui->playlistList->count(); i++ )
     {
         QListWidgetItem* item = m_ui->playlistList->item( i );
-        item->setCheckState(m_ui->selectAllCheckbox->checkState());
+        item->setCheckState( m_ui->selectAllCheckbox->checkState() );
     }
 }
 
diff --git a/src/libtomahawk/accounts/spotify/SpotifyAccountConfig.ui b/src/libtomahawk/accounts/spotify/SpotifyAccountConfig.ui
index 8f2cadd17..d465487a1 100644
--- a/src/libtomahawk/accounts/spotify/SpotifyAccountConfig.ui
+++ b/src/libtomahawk/accounts/spotify/SpotifyAccountConfig.ui
@@ -6,7 +6,7 @@
    <rect>
     <x>0</x>
     <y>0</y>
-    <width>435</width>
+    <width>375</width>
     <height>487</height>
    </rect>
   </property>
@@ -34,7 +34,7 @@
         <string/>
        </property>
        <property name="pixmap">
-        <pixmap resource="../../../../resources.qrc">:/data/images/spotify-logo.png</pixmap>
+        <pixmap resource="../../../resources.qrc">:/data/images/spotify-logo.png</pixmap>
        </property>
        <property name="scaledContents">
         <bool>true</bool>
@@ -168,7 +168,7 @@
   </layout>
  </widget>
  <resources>
-  <include location="../../../../resources.qrc"/>
+  <include location="../../../resources.qrc"/>
  </resources>
  <connections/>
 </ui>