Git commit 5698b821796d977a309717a2ddc56dac39571589 by Christoph Feck. Committed on 15/09/2016 at 01:43. Pushed by cfeck into branch 'kf5'.
Merge remote-tracking branch 'origin/master' into kf5 Please review the resolution of the conflicts. CCMAIL: amarok-devel@kde.org M +9 -9 cmake/modules/FindMySQLAmarok.cmake M +0 -2 src/configdialog/dialogs/DatabaseConfig.cpp M +5 -7 src/context/applets/lyrics/LyricsBrowser.cpp M +1 -7 src/context/applets/wikipedia/WikipediaApplet.cpp M +0 -1 src/context/applets/wikipedia/WikipediaApplet_p.h M +1 -1 src/context/engines/tabs/TabsEngine.cpp M +3 -15 src/context/engines/wikipedia/WikipediaEngine.cpp M +34 -11 src/core-impl/collections/audiocd/AudioCdCollection.cpp M +7 -0 src/core-impl/collections/audiocd/AudioCdCollection.h M +2 -2 src/core-impl/collections/db/sql/SqlScanResultProcessor.cpp M +4 -1 src/core-impl/collections/db/sql/mysqlcollection/CMakeLists.txt M +1 -1 src/core-impl/collections/ipodcollection/IpodCollectionFactory.cpp M +1 -1 src/core-impl/collections/umscollection/podcasts/UmsPodcastMeta.cpp M +2 -1 src/core-impl/playlists/types/file/xspf/XSPFPlaylist.cpp M +1 -0 src/core-impl/storage/sql/mysqlserverstorage/MySqlServerStorageFactory.cpp M +1 -1 src/core/podcasts/PodcastReader.cpp M +1 -5 src/dbus/mpris2/MediaPlayer2.cpp M +0 -2 src/dialogs/OrganizeCollectionDialog.h M +1 -1 src/dialogs/TagDialog.cpp M +1 -1 src/dynamic/biases/EchoNestBias.h M +1 -1 src/importers/ImporterSqlConnection.h M +1 -1 src/scripting/scriptconsole/ScriptConsole.cpp M +4 -6 src/widgets/FilenameLayoutWidget.h M +5 -1 src/widgets/MetaQueryWidget.cpp http://commits.kde.org/amarok/5698b821796d977a309717a2ddc56dac39571589 diff --cc src/context/applets/wikipedia/WikipediaApplet.cpp index d635f61,705b9db..59cc81a --- a/src/context/applets/wikipedia/WikipediaApplet.cpp +++ b/src/context/applets/wikipedia/WikipediaApplet.cpp @@@ -417,8 -410,8 +414,8 @@@ WikipediaAppletPrivate::_getLangMap( languageSettingsUi.progressBar->setMaximum( 100 ); languageSettingsUi.progressBar->setValue( 0 ); - KUrl url; + QUrl url; - url.setScheme( "http" ); + url.setScheme( "https" ); url.setHost( "en.wikipedia.org" ); url.setPath( "/w/api.php" ); url.addQueryItem( "action", "query" ); @@@ -804,10 -787,9 +799,9 @@@ WikipediaApplet::createConfigurationInt QWidget *genSettings = new QWidget; d->generalSettingsUi.setupUi( genSettings ); d->generalSettingsUi.mobileCheckBox->setCheckState( d->useMobileWikipedia ? Qt::Checked : Qt::Unchecked ); - d->generalSettingsUi.sslCheckBox->setCheckState( d->useSSL ? Qt::Checked : Qt::Unchecked ); connect( d->languageSettingsUi.downloadButton, SIGNAL(clicked()), this, SLOT(_getLangMap()) ); - connect( parent, SIGNAL(okClicked()), this, SLOT(_loadSettings()) ); + connect( parent, SIGNAL(clicked()), this, SLOT(_loadSettings()) ); parent->addPage( genSettings, i18n( "Wikipedia General Settings" ), "configure" ); parent->addPage( langSettings, i18n( "Wikipedia Language Settings" ), "applications-education-language" ); diff --cc src/context/engines/tabs/TabsEngine.cpp index 87bcd34,6c0504c..97b39cb --- a/src/context/engines/tabs/TabsEngine.cpp +++ b/src/context/engines/tabs/TabsEngine.cpp @@@ -243,8 -243,8 +243,8 @@@ voi TabsEngine::queryUltimateGuitar( const QString &artist, const QString &title ) { // Query UltimateGuitar.com (filtering guitar (tabs + chords) and bass tabs) - KUrl ultimateGuitarUrl; + QUrl ultimateGuitarUrl; - ultimateGuitarUrl.setScheme( "http" ); + ultimateGuitarUrl.setScheme( "https" ); ultimateGuitarUrl.setHost( "www.ultimate-guitar.com" ); ultimateGuitarUrl.setPath( "/search.php" ); ultimateGuitarUrl.addQueryItem( "view_state", "advanced" ); diff --cc src/context/engines/wikipedia/WikipediaEngine.cpp index ca4ede6,d23a327..b3bb6b3 --- a/src/context/engines/wikipedia/WikipediaEngine.cpp +++ b/src/context/engines/wikipedia/WikipediaEngine.cpp @@@ -542,9 -530,9 +530,9 @@@ voi WikipediaEnginePrivate::fetchWikiUrl( const QString &title, const QString &urlPrefix ) { Q_Q( WikipediaEngine ); - KUrl pageUrl; + QUrl pageUrl; QString host( ".wikipedia.org" ); - pageUrl.setScheme( useSSL ? QLatin1String( "https" ) : QLatin1String( "http" ) ); + pageUrl.setScheme( QLatin1String( "https" ) ); if( useMobileVersion ) { @@@ -581,8 -569,8 +569,8 @@@ WikipediaEnginePrivate::fetchLangLinks const QString &llcontinue ) { Q_Q( WikipediaEngine ); - KUrl url; + QUrl url; - url.setScheme( useSSL ? QLatin1String( "https" ) : QLatin1String( "http" ) ); + url.setScheme( QLatin1String( "https" ) ); url.setHost( hostLang + QLatin1String(".wikipedia.org") ); url.setPath( QLatin1String("/w/api.php") ); url.addQueryItem( QLatin1String("action"), QLatin1String("query") ); @@@ -603,8 -591,8 +591,8 @@@ voi WikipediaEnginePrivate::fetchListing( const QString &title, const QString &hostLang ) { Q_Q( WikipediaEngine ); - KUrl url; + QUrl url; - url.setScheme( useSSL ? QLatin1String( "https" ) : QLatin1String( "http" ) ); + url.setScheme( QLatin1String( "https" ) ); url.setHost( hostLang + QLatin1String(".wikipedia.org") ); url.setPath( QLatin1String("/w/api.php") ); url.addQueryItem( QLatin1String("action"), QLatin1String("query") ); diff --cc src/core-impl/collections/audiocd/AudioCdCollection.cpp index 82e52d0,3af6e36..a8c8ac2 --- a/src/core-impl/collections/audiocd/AudioCdCollection.cpp +++ b/src/core-impl/collections/audiocd/AudioCdCollection.cpp @@@ -348,10 -348,32 +348,32 @@@ AudioCdCollection::checkForStartPlayReq } } + + QString + AudioCdCollection::trackBaseFileName( int i ) const + { + return QString( "Track%1" ).arg( i, 2, 10, QChar('0') ); + } + + + QString + AudioCdCollection::trackWavFileName( int i ) const + { + return trackBaseFileName( i ) + ".wav"; + } + + + QString + AudioCdCollection::trackDisplayName( int i ) const + { + return i18n( "Track" ) + ' ' + QString::number( i ); + } + + qint64 - AudioCdCollection::trackLength(int i) const + AudioCdCollection::trackLength( int i ) const { - QUrl kioUrl = audiocdUrl( QString("Track%1.wav").arg(i, 2, 10, QChar('0') ) ); - KUrl kioUrl = audiocdUrl( trackWavFileName( i ) ); ++ QUrl kioUrl = audiocdUrl( trackWavFileName( i ) ); KIO::UDSEntry uds; if ( KIO::NetAccess::stat(kioUrl, uds, NULL) ) { @@@ -482,19 -504,21 +504,21 @@@ AudioCdCollection::noInfoAvailable( int i = 1; - QString prefix( "0" ); - QString trackName = "Track " + prefix + QString::number( i ); + QString trackWav = trackWavFileName( i ); - while( KIO::NetAccess::exists( KUrl( "audiocd:/" + trackName + ".wav" ), KIO::NetAccess::SourceSide, 0 ) ) + // This will find also data tracks on mixed CDs: + // a better way to discover the available audio tracks should be found + while( KIO::NetAccess::exists( audiocdUrl( trackWav ), KIO::NetAccess::SourceSide, 0 ) ) { - debug() << "got track: " << "audiocd:/" + trackName + ".wav"; + debug() << "got track url: " << audiocdUrl( trackWav ); - QString baseUrl = "audiocd:/" + m_discCddbId + '/' + QString::number( i ); + //we hack the url so the engine controller knows what track on the CD to play.. - KUrl baseUrl = audiocdUrl( m_discCddbId + '/' + QString::number( i ) ); ++ QUrl baseUrl = audiocdUrl( m_discCddbId + '/' + QString::number( i ) ); - Meta::AudioCdTrackPtr trackPtr = Meta::AudioCdTrackPtr( new Meta::AudioCdTrack( this, trackName, QUrl(baseUrl) ) ); + Meta::AudioCdTrackPtr trackPtr = Meta::AudioCdTrackPtr( new Meta::AudioCdTrack( this, trackDisplayName( i ), baseUrl ) ); trackPtr->setTrackNumber( i ); - trackPtr->setFileNameBase( trackName ); + trackPtr->setFileNameBase( trackBaseFileName( i ) ); trackPtr->setLength( trackLength( i ) ); memoryCollection()->addTrack( Meta::TrackPtr::staticCast( trackPtr ) ); diff --cc src/core-impl/collections/audiocd/AudioCdCollection.h index 08a43fe,e829368..4c942e0 --- a/src/core-impl/collections/audiocd/AudioCdCollection.h +++ b/src/core-impl/collections/audiocd/AudioCdCollection.h @@@ -102,7 -102,14 +102,14 @@@ private void readAudioCdSettings(); // Helper function to build the audiocd url. - KUrl audiocdUrl( const QString &path = "" ) const; + QUrl audiocdUrl( const QString &path = "" ) const; + // The file name of the track without extension + QString trackBaseFileName( int i ) const; + // The file name of the track in .wav format + QString trackWavFileName( int i ) const; + // The name of the track that should be displayed + QString trackDisplayName( int i ) const; + // The length of the track in milliseconds qint64 trackLength( int i ) const; /** diff --cc src/core-impl/collections/db/sql/mysqlcollection/CMakeLists.txt index 52b4707,4c618fc..faf0a17 --- a/src/core-impl/collections/db/sql/mysqlcollection/CMakeLists.txt +++ b/src/core-impl/collections/db/sql/mysqlcollection/CMakeLists.txt @@@ -18,17 -18,20 +18,20 @@@ target_link_libraries(amarok_collection amarok-sqlcollection amarokcore amaroklib - ${KDE4_KDECORE_LIBS} - ${QT_QTCORE_LIBRARY} - ${QT_QTGUI_LIBRARY} + KF5::KDELibs4Support + Qt5::Core + Qt5::Gui - ${MYSQL_EMBEDDED_LIBRARIES} ${CMAKE_DL_LIBS} ${ZLIB_LIBRARIES} ) + if(WITH_MYSQL_EMBEDDED) + target_link_libraries( amarok_collection-mysqlcollection ${MYSQL_EMBEDDED_LIBRARIES} ) + endif(WITH_MYSQL_EMBEDDED) + if(NOT WIN32 AND NOT APPLE) target_link_libraries( amarok_collection-mysqlcollection crypt pthread ) -endif(NOT WIN32 AND NOT APPLE) +endif() if(APPLE) SET_TARGET_PROPERTIES(amarok_collection-mysqlcollection PROPERTIES LINK_FLAGS "-undefined dynamic_lookup") diff --cc src/core-impl/playlists/types/file/xspf/XSPFPlaylist.cpp index 8330c3d,e327354..6997b45 --- a/src/core-impl/playlists/types/file/xspf/XSPFPlaylist.cpp +++ b/src/core-impl/playlists/types/file/xspf/XSPFPlaylist.cpp @@@ -28,8 -28,9 +28,9 @@@ #include "playlist/PlaylistModelStack.h" using namespace Playlists; + using namespace Playlist; -XSPFPlaylist::XSPFPlaylist( const KUrl &url, Playlists::PlaylistProvider *provider, OnLoadAction onLoad ) +XSPFPlaylist::XSPFPlaylist( const QUrl &url, Playlists::PlaylistProvider *provider, OnLoadAction onLoad ) : PlaylistFile( url, provider ) , QDomDocument() , m_autoAppendAfterLoad( onLoad == AppendToPlaylist ) diff --cc src/scripting/scriptconsole/ScriptConsole.cpp index 779bebc,e4e0fab..a34a299 --- a/src/scripting/scriptconsole/ScriptConsole.cpp +++ b/src/scripting/scriptconsole/ScriptConsole.cpp @@@ -156,7 -161,7 +156,7 @@@ ScriptConsole::ScriptConsole( QWidget * settings.endGroup(); if( m_savePath.isEmpty() ) - m_savePath = QUrl( KStandardDirs::locate( "data", "amarok/scriptconsole/" ) ).path(); - m_savePath = KUrl( KStandardDirs::locateLocal( "data", "amarok/scriptconsole/" ) ).path(); ++ m_savePath = QUrl( KStandardDirs::locateLocal( "data", "amarok/scriptconsole/" ) ).path(); slotNewScript(); connect( m_debugger, SIGNAL(evaluationSuspended()), SLOT(slotEvaluationSuspended()) ); diff --cc src/widgets/FilenameLayoutWidget.h index c29aba9,8f6e35a..62c846a --- a/src/widgets/FilenameLayoutWidget.h +++ b/src/widgets/FilenameLayoutWidget.h @@@ -78,11 -77,10 +77,10 @@@ class AMAROK_EXPORT FilenameLayoutWidge void setScheme( const QString &scheme ); - - public slots: + public Q_SLOTS: virtual void onAccept(); - signals: + Q_SIGNALS: /** emitted when either the scheme, option checkboxes or the replace edits change */ void schemeChanged();