[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-commits
Subject: [kdeplasma-addons] applets/comic: Enable QT_NO_CAST_FROM_ASCII also for applets/comic, string code a
From: Friedrich W. H. Kossebau <null () kde ! org>
Date: 2018-01-31 22:25:28
Message-ID: E1eh0oq-0007ys-O1 () code ! kde ! org
[Download RAW message or body]
Git commit e31b70aec96651a6202d732339c0d2ba2e710044 by Friedrich W. H. \
Kossebau. Committed on 31/01/2018 at 21:23.
Pushed by kossebau into branch 'master'.
Enable QT_NO_CAST_FROM_ASCII also for applets/comic, string code adapted
M +0 -3 applets/comic/CMakeLists.txt
M +1 -1 applets/comic/activecomicmodel.cpp
M +5 -5 applets/comic/checknewstrips.cpp
M +54 -46 applets/comic/comic.cpp
M +4 -4 applets/comic/comicarchivedialog.cpp
M +18 -18 applets/comic/comicarchivejob.cpp
M +34 -33 applets/comic/comicdata.cpp
M +3 -3 applets/comic/comicsaver.cpp
M +3 -3 applets/comic/stripselector.cpp
https://commits.kde.org/kdeplasma-addons/e31b70aec96651a6202d732339c0d2ba2e710044
diff --git a/applets/comic/CMakeLists.txt b/applets/comic/CMakeLists.txt
index bfead9607..9d7b20ca1 100644
--- a/applets/comic/CMakeLists.txt
+++ b/applets/comic/CMakeLists.txt
@@ -1,8 +1,5 @@
add_definitions(-DTRANSLATION_DOMAIN=\"plasma_applet_org.kde.plasma.comic\")
-remove_definitions(
- -DQT_NO_CAST_FROM_ASCII
-)
set(comic_SRCS
comic.cpp
comicmodel.cpp
diff --git a/applets/comic/activecomicmodel.cpp \
b/applets/comic/activecomicmodel.cpp index 5beeb412d..fdd65a055 100644
--- a/applets/comic/activecomicmodel.cpp
+++ b/applets/comic/activecomicmodel.cpp
@@ -61,7 +61,7 @@ QVariantHash ActiveComicModel::get(int row) const
const auto roleNames = this->roleNames();
hash.reserve(roleNames.size());
for (auto end = roleNames.constEnd(), it = roleNames.constBegin(); it \
!= end; ++it) {
- hash.insert(it.value(), data(idx, it.key()));
+ hash.insert(QString::fromUtf8(it.value()), data(idx, it.key()));
}
return hash;
diff --git a/applets/comic/checknewstrips.cpp \
b/applets/comic/checknewstrips.cpp index 88bacf60a..4a45a628a 100644
--- a/applets/comic/checknewstrips.cpp
+++ b/applets/comic/checknewstrips.cpp
@@ -41,8 +41,8 @@ void CheckNewStrips::dataUpdated( const QString &source, \
const Plasma::DataEngin {
QString lastIdentifierSuffix;
- if ( !data[ "Error" ].toBool() ) {
- lastIdentifierSuffix = data[ "Identifier" ].toString();
+ if (!data[QStringLiteral("Error")].toBool()) {
+ lastIdentifierSuffix = \
data[QStringLiteral("Identifier")].toString(); \
lastIdentifierSuffix.remove( source ); }
@@ -50,13 +50,13 @@ void CheckNewStrips::dataUpdated( const QString \
&source, const Plasma::DataEngin
if ( !lastIdentifierSuffix.isEmpty() ) {
QString temp = source;
- temp.remove( ':' );
+ temp.remove(QLatin1Char(':'));
emit lastStrip( mIndex, temp, lastIdentifierSuffix );
}
++mIndex;
if ( mIndex < mIdentifiers.count() ) {
- const QString newSource = mIdentifiers[mIndex] + ':';
+ const QString newSource = mIdentifiers[mIndex] + QLatin1Char(':');
mEngine->connectSource( newSource, this );
} else {
mIndex = 0;
@@ -71,7 +71,7 @@ void CheckNewStrips::start()
}
if ( mIndex < mIdentifiers.count() ) {
- const QString newSource = mIdentifiers[mIndex] + ':';
+ const QString newSource = mIdentifiers[mIndex] + QLatin1Char(':');
mEngine->connectSource( newSource, this );
}
}
diff --git a/applets/comic/comic.cpp b/applets/comic/comic.cpp
index 928139ca6..823fbafb5 100644
--- a/applets/comic/comic.cpp
+++ b/applets/comic/comic.cpp
@@ -84,8 +84,8 @@ void ComicApplet::init()
configChanged();
- mEngine = Plasma::PluginLoader::self()->loadDataEngine( "comic" );
- mModel = new ComicModel( mEngine, "providers", mTabIdentifier, this );
+ mEngine = Plasma::PluginLoader::self()->loadDataEngine(QStringLiteral("comic"));
+ mModel = new ComicModel(mEngine, QStringLiteral("providers"), \
mTabIdentifier, this); mProxy = new QSortFilterProxyModel( this );
mProxy->setSourceModel( mModel );
mProxy->setSortCaseSensitivity( Qt::CaseInsensitive );
@@ -104,21 +104,25 @@ void ComicApplet::init()
mDateChangedTimer->setInterval( 5 * 60 * 1000 ); // every 5 minutes
mDateChangedTimer->start();
- mActionNextNewStripTab = new QAction( QIcon::fromTheme( "go-next-view" \
), i18nc( "here strip means comic strip", "&Next Tab with a new Strip" ), \
this ); + mActionNextNewStripTab = new \
QAction(QIcon::fromTheme(QStringLiteral("go-next-view")), + \
i18nc("here strip means comic strip", "&Next Tab with a new Strip"), this); \
mActionNextNewStripTab->setShortcuts( \
KStandardShortcut::openNew() );
- actions()->addAction( "next new strip" , mActionNextNewStripTab );
+ actions()->addAction(QStringLiteral("next new strip"), \
mActionNextNewStripTab); mActions.append( mActionNextNewStripTab );
connect( mActionNextNewStripTab, &QAction::triggered, this, \
&ComicApplet::showNextNewStrip );
- mActionGoFirst = new QAction( QIcon::fromTheme( "go-first" ), i18n( \
"Jump to &first Strip" ), this ); + mActionGoFirst = new \
QAction(QIcon::fromTheme(QStringLiteral("go-first")), + \
i18n("Jump to &first Strip"), this); mActions.append( mActionGoFirst );
connect( mActionGoFirst, &QAction::triggered, this, \
&ComicApplet::slotFirstDay );
- mActionGoLast = new QAction( QIcon::fromTheme( "go-last" ), i18n( \
"Jump to ¤t Strip" ), this ); + mActionGoLast = new \
QAction(QIcon::fromTheme(QStringLiteral("go-last")), + \
i18n("Jump to ¤t Strip"), this); mActions.append( mActionGoLast );
connect( mActionGoLast, &QAction::triggered, this, \
&ComicApplet::slotCurrentDay );
- mActionGoJump = new QAction( QIcon::fromTheme( "go-jump" ), i18n( \
"Jump to Strip ..." ), this ); + mActionGoJump = new \
QAction(QIcon::fromTheme(QStringLiteral("go-jump")), + \
i18n("Jump to Strip ..."), this); mActions.append( mActionGoJump );
connect( mActionGoJump, &QAction::triggered, this, \
&ComicApplet::slotGoJump );
@@ -127,21 +131,25 @@ void ComicApplet::init()
mActions.append( mActionShop );
connect( mActionShop, &QAction::triggered, this, \
&ComicApplet::slotShop );
- QAction *action = new QAction( QIcon::fromTheme( "document-save-as" ), \
i18n( "&Save Comic As..." ), this ); + QAction *action = new \
QAction(QIcon::fromTheme(QStringLiteral("document-save-as")), + \
i18n("&Save Comic As..."), this); mActions.append( action );
connect( action, &QAction::triggered, this, \
&ComicApplet::slotSaveComicAs );
- action = new QAction( QIcon::fromTheme( "application-epub+zip" ), \
i18n( "&Create Comic Book Archive..." ), this ); + action = new \
QAction(QIcon::fromTheme(QStringLiteral("application-epub+zip")), + \
i18n("&Create Comic Book Archive..."), this); mActions.append( action );
connect( action, &QAction::triggered, this, \
&ComicApplet::createComicBook );
- mActionScaleContent = new QAction( QIcon::fromTheme( "zoom-original" \
), i18nc( "@option:check Context menu of comic image", "&Actual Size" ), \
this ); + mActionScaleContent = new \
QAction(QIcon::fromTheme(QStringLiteral("zoom-original")), + \
i18nc("@option:check Context menu of comic image", "&Actual Size"), this); \
mActionScaleContent->setCheckable( true ); \
mActionScaleContent->setChecked( mCurrent.scaleComic() ); mActions.append( \
mActionScaleContent );
connect( mActionScaleContent, &QAction::triggered, this, \
&ComicApplet::slotScaleToContent );
- mActionStorePosition = new QAction( QIcon::fromTheme( "go-home" ), \
i18nc( "@option:check Context menu of comic image", "Store current \
&Position" ), this); + mActionStorePosition = new \
QAction(QIcon::fromTheme(QStringLiteral("go-home")), + \
i18nc("@option:check Context menu of comic image", "Store current \
&Position"), this); mActionStorePosition->setCheckable( true );
mActionStorePosition->setChecked(mCurrent.hasStored());
mActions.append( mActionStorePosition );
@@ -179,10 +187,10 @@ void ComicApplet::dataUpdated( const QString &source, \
const Plasma::DataEngine:: setConfigurationRequired( false );
//there was an error, display information as image
- const bool hasError = data[ "Error" ].toBool();
- const bool errorAutoFixable = data[ "Error automatically fixable" \
].toBool(); + const bool hasError = \
data[QStringLiteral("Error")].toBool(); + const bool errorAutoFixable = \
data[QStringLiteral("Error automatically fixable")].toBool(); if ( \
hasError ) {
- const QString previousIdentifierSuffix = data[ "Previous \
identifier suffix" ].toString(); + const QString \
previousIdentifierSuffix = data[QStringLiteral("Previous identifier \
suffix")].toString();
if (mEngine && !mShowErrorPicture && \
!previousIdentifierSuffix.isEmpty() ) { mEngine->disconnectSource( source, \
this ); updateComic( previousIdentifierSuffix );
@@ -212,11 +220,11 @@ void ComicApplet::dataUpdated( const QString &source, \
const Plasma::DataEngine::
//prefetch the previous and following comic for faster navigation
if (mCurrent.hasNext()) {
- const QString prefetch = mCurrent.id() + ':' + \
mCurrent.next(); + const QString prefetch = mCurrent.id() + \
QLatin1Char(':') + mCurrent.next(); mEngine->connectSource( prefetch, this \
); }
if ( mCurrent.hasPrev()) {
- const QString prefetch = mCurrent.id() + ':' + \
mCurrent.prev(); + const QString prefetch = mCurrent.id() + \
QLatin1Char(':') + mCurrent.prev(); mEngine->connectSource( prefetch, this \
); }
}
@@ -237,7 +245,7 @@ void ComicApplet::getNewComics()
return;
}
if (!m_newStuffDialog) {
- m_newStuffDialog = new KNS3::DownloadDialog( \
QString::fromLatin1("comic.knsrc") ); + m_newStuffDialog = new \
KNS3::DownloadDialog( QStringLiteral("comic.knsrc") ); \
KNS3::DownloadDialog *strong = m_newStuffDialog.data(); \
strong->setTitle(i18n("Download Comics"));
connect(m_newStuffDialog.data(), SIGNAL(finished(int)), mEngine, \
SLOT(loadProviders())); @@ -299,7 +307,7 @@ void \
ComicApplet::updateUsedComics()
const QString iconPath = data.data( Qt::DecorationRole \
).value<QIcon>().name(); //found a newer strip last time, which was not \
visited
- if ( mCheckNewComicStripsInterval && !cg.readEntry( \
"lastStripVisited_" + identifier, true ) ) { + if \
(mCheckNewComicStripsInterval && \
!cg.readEntry(QLatin1String("lastStripVisited_") + \
identifier, true)) {
mActiveComicModel->addComic(identifier, name, iconPath, \
true); } else {
mActiveComicModel->addComic(identifier, name, iconPath);
@@ -408,9 +416,9 @@ void ComicApplet::slotFoundLastStrip( int index, const \
QString &identifier, cons Q_UNUSED(index)
KConfigGroup cg = config();
- if (suffix != cg.readEntry( "lastStrip_" + identifier, QString() ) ) {
+ if (suffix != cg.readEntry(QLatin1String("lastStrip_") + identifier, \
QString())) { qDebug() << identifier << "has a newer strip.";
- cg.writeEntry( "lastStripVisited_" + identifier, false );
+ cg.writeEntry(QLatin1String("lastStripVisited_") + identifier, \
false); updateComic(suffix);
}
}
@@ -453,8 +461,8 @@ void ComicApplet::slotArchive( int archiveType, const \
QUrl &dest, const QString const QString id = mCurrent.id();
qDebug() << "Archiving:" << id << archiveType << dest << \
fromIdentifier << toIdentifier;
ComicArchiveJob *job = new ComicArchiveJob(dest, mEngine, static_cast< \
ComicArchiveJob::ArchiveType >( archiveType ), \
mCurrent.type(), id, this);
- job->setFromIdentifier(id + ':' + fromIdentifier);
- job->setToIdentifier(id + ':' + toIdentifier);
+ job->setFromIdentifier(id + QLatin1Char(':') + fromIdentifier);
+ job->setToIdentifier(id + QLatin1Char(':') + toIdentifier);
if (job->isValid()) {
connect(job, &ComicArchiveJob::finished, this, \
&ComicApplet::slotArchiveFinished); \
KIO::getJobTracker()->registerJob(job); @@ -468,7 +476,7 @@ void \
ComicApplet::slotArchive( int archiveType, const QUrl &dest, const QString \
void ComicApplet::slotArchiveFinished (KJob *job ) {
if ( job->error() ) {
- KNotification::event( KNotification::Warning, i18n( "Archiving \
comic failed" ), job->errorText(), QIcon::fromTheme( "dialog-warning" \
).pixmap( KIconLoader::SizeMedium ) ); + KNotification::event( \
KNotification::Warning, i18n( "Archiving comic failed" ), job->errorText(), \
QIcon::fromTheme(QStringLiteral("dialog-warning")).pixmap(KIconLoader::SizeMedium));
}
}
@@ -486,7 +494,7 @@ void ComicApplet::updateComic( const QString \
&identifierSuffix )
setBusy(true);
- const QString identifier = id + ':' + identifierSuffix;
+ const QString identifier = id + QLatin1Char(':') + \
identifierSuffix;
//disconnecting of the oldSource is needed, otherwise you could \
get data for comics you are not looking at if you use tabs //if there was \
an error only disconnect the oldSource if it had nothing to do with the \
error or if the comic changed, that way updates of the error can come in @@ \
-668,28 +676,28 @@ void ComicApplet::setTabIdentifiers(const QStringList \
&tabs)
void ComicApplet::refreshComicData()
{
- mComicData["image"] = mCurrent.image();
- mComicData["prev"] = mCurrent.prev();
- mComicData["next"] = mCurrent.next();
- mComicData["additionalText"] = mCurrent.additionalText();
-
- mComicData["websiteUrl"] = mCurrent.websiteUrl().toString();
- mComicData["websiteHost"] = mCurrent.websiteUrl().host();
- mComicData["imageUrl"] = mCurrent.websiteUrl().toString();
- mComicData["shopUrl"] = mCurrent.websiteUrl().toString();
- mComicData["first"] = mCurrent.first();
- mComicData["stripTitle"] = mCurrent.stripTitle();
- mComicData["author"] = mCurrent.author();
- mComicData["title"] = mCurrent.title();
-
- mComicData["suffixType"] = "Date";
- mComicData["current"] = mCurrent.current();
- //mComicData["last"] = mCurrent.last();
- mComicData["currentReadable"] = mCurrent.currentReadable();
- mComicData["firstStripNum"] = mCurrent.firstStripNum();
- mComicData["maxStripNum"] = mCurrent.maxStripNum();
- mComicData["isLeftToRight"] = mCurrent.isLeftToRight();
- mComicData["isTopToBottom"] = mCurrent.isTopToBottom();
+ mComicData[QStringLiteral("image")] = mCurrent.image();
+ mComicData[QStringLiteral("prev")] = mCurrent.prev();
+ mComicData[QStringLiteral("next")] = mCurrent.next();
+ mComicData[QStringLiteral("additionalText")] = \
mCurrent.additionalText(); +
+ mComicData[QStringLiteral("websiteUrl")] = \
mCurrent.websiteUrl().toString(); + \
mComicData[QStringLiteral("websiteHost")] = mCurrent.websiteUrl().host(); + \
mComicData[QStringLiteral("imageUrl")] = mCurrent.websiteUrl().toString(); \
+ mComicData[QStringLiteral("shopUrl")] = \
mCurrent.websiteUrl().toString(); + mComicData[QStringLiteral("first")] \
= mCurrent.first(); + mComicData[QStringLiteral("stripTitle")] = \
mCurrent.stripTitle(); + mComicData[QStringLiteral("author")] = \
mCurrent.author(); + mComicData[QStringLiteral("title")] = \
mCurrent.title(); +
+ mComicData[QStringLiteral("suffixType")] = QStringLiteral("Date");
+ mComicData[QStringLiteral("current")] = mCurrent.current();
+ //mComicData[QStringLiteral("last")] = mCurrent.last();
+ mComicData[QStringLiteral("currentReadable")] = \
mCurrent.currentReadable(); + \
mComicData[QStringLiteral("firstStripNum")] = mCurrent.firstStripNum(); + \
mComicData[QStringLiteral("maxStripNum")] = mCurrent.maxStripNum(); + \
mComicData[QStringLiteral("isLeftToRight")] = mCurrent.isLeftToRight(); + \
mComicData[QStringLiteral("isTopToBottom")] = mCurrent.isTopToBottom();
emit comicDataChanged();
}
diff --git a/applets/comic/comicarchivedialog.cpp \
b/applets/comic/comicarchivedialog.cpp index 0b2ee943d..3306d8adf 100644
--- a/applets/comic/comicarchivedialog.cpp
+++ b/applets/comic/comicarchivedialog.cpp
@@ -40,8 +40,8 @@ ComicArchiveDialog::ComicArchiveDialog( const QString \
&pluginName, const QString
switch ( mIdentifierType ) {
case Date: {
- const QDate current = QDate::fromString( \
currentIdentifierSuffix, "yyyy-MM-dd" );
- const QDate first = QDate::fromString( firstIdentifierSuffix, \
"yyyy-MM-dd" ); + const QDate current = \
QDate::fromString(currentIdentifierSuffix, QStringLiteral("yyyy-MM-dd")); + \
const QDate first = QDate::fromString(firstIdentifierSuffix, \
QStringLiteral("yyyy-MM-dd")); const QDate today = QDate::currentDate();
QDate maxDate = today;
if ( current.isValid() ) {
@@ -162,8 +162,8 @@ void ComicArchiveDialog::slotOkClicked()
switch ( mIdentifierType ) {
case Date:
- fromIdentifier = ui.fromDate->date().toString( "yyyy-MM-dd" );
- toIdentifier = ui.toDate->date().toString( "yyyy-MM-dd" );
+ fromIdentifier = \
ui.fromDate->date().toString(QStringLiteral("yyyy-MM-dd")); + \
toIdentifier = ui.toDate->date().toString(QStringLiteral("yyyy-MM-dd")); \
break; case Number: {
fromIdentifier = QString::number( ui.fromNumber->value() );
diff --git a/applets/comic/comicarchivejob.cpp \
b/applets/comic/comicarchivejob.cpp index 7a67b373c..2698e7914 100644
--- a/applets/comic/comicarchivejob.cpp
+++ b/applets/comic/comicarchivejob.cpp
@@ -95,14 +95,14 @@ void ComicArchiveJob::setToIdentifier( const QString \
&toIdentifier ) {
mToIdentifier = toIdentifier;
mToIdentifierSuffix = mToIdentifier;
- mToIdentifierSuffix.remove( mPluginName + ':' );
+ mToIdentifierSuffix.remove(mPluginName + QLatin1Char(':'));
}
void ComicArchiveJob::setFromIdentifier( const QString &fromIdentifier )
{
mFromIdentifier = fromIdentifier;
mFromIdentifierSuffix = mFromIdentifier;
- mFromIdentifierSuffix.remove( mPluginName + ':' );
+ mFromIdentifierSuffix.remove(mPluginName + QLatin1Char(':'));
}
void ComicArchiveJob::start()
@@ -139,21 +139,21 @@ void ComicArchiveJob::dataUpdated( const QString \
&source, const Plasma::DataEngi return;
}
- const QString currentIdentifier = data[ "Identifier" ].toString();
+ const QString currentIdentifier = \
data[QStringLiteral("Identifier")].toString(); QString \
currentIdentifierSuffix = currentIdentifier;
- currentIdentifierSuffix.remove( mPluginName + ':' );
+ currentIdentifierSuffix.remove(mPluginName + QLatin1Char(':'));
- const QImage image = data[ "Image" ].value<QImage>();
- const bool hasError = data[ "Error" ].toBool() || image.isNull();
- const QString previousIdentifierSuffix = data[ "Previous identifier \
suffix" ].toString();
- const QString nextIdentifierSuffix = data[ "Next identifier suffix" \
].toString();
- const QString firstIdentifierSuffix = data[ "First strip identifier \
suffix" ].toString(); + const QImage image = \
data[QStringLiteral("Image")].value<QImage>(); + const bool hasError = \
data[QStringLiteral("Error")].toBool() || image.isNull(); + const \
QString previousIdentifierSuffix = data[QStringLiteral("Previous identifier \
suffix")].toString(); + const QString nextIdentifierSuffix = \
data[QStringLiteral("Next identifier suffix")].toString(); + const \
QString firstIdentifierSuffix = data[QStringLiteral("First strip identifier \
suffix")].toString();
- mAuthors << data[ "Comic Author" ].toString().split( ',', \
QString::SkipEmptyParts ); + mAuthors << data[QStringLiteral("Comic \
Author")].toString().split(QLatin1Char(','), QString::SkipEmptyParts); \
mAuthors.removeDuplicates();
if ( mComicTitle.isEmpty() ) {
- mComicTitle = data[ "Title" ].toString();
+ mComicTitle = data[QStringLiteral("Title")].toString();
}
if ( hasError ) {
@@ -272,8 +272,8 @@ void ComicArchiveJob::defineTotalNumber( const QString \
¤tSuffix ) //if there are no strips for certain days/numbers
if ( !currentSuffix.isEmpty() ) {
if ( mIdentifierType == Date ) {
- const QDate current = QDate::fromString( currentSuffix, \
"yyyy-MM-dd" );
- const QDate to = QDate::fromString( mToIdentifierSuffix, \
"yyyy-MM-dd" ); + const QDate current = \
QDate::fromString(currentSuffix, QStringLiteral("yyyy-MM-dd")); + \
const QDate to = QDate::fromString(mToIdentifierSuffix, \
QStringLiteral("yyyy-MM-dd")); if ( current.isValid() && to.isValid() ) {
//processed files + files still to download
mTotalFiles = mProcessedFiles + qAbs( current.daysTo( to ) \
); @@ -304,8 +304,8 @@ void ComicArchiveJob::findTotalNumberFromTo()
}
if ( mIdentifierType == Date ) {
- const QDate from = QDate::fromString( mFromIdentifierSuffix, \
"yyyy-MM-dd" );
- const QDate to = QDate::fromString( mToIdentifierSuffix, \
"yyyy-MM-dd" ); + const QDate from = QDate::fromString( \
mFromIdentifierSuffix, QStringLiteral("yyyy-MM-dd")); + const QDate \
to = QDate::fromString(mToIdentifierSuffix, QStringLiteral("yyyy-MM-dd")); \
if ( from.isValid() && to.isValid() ) { mTotalFiles = qAbs( from.daysTo( \
to ) ) + 1; }
@@ -324,7 +324,7 @@ void ComicArchiveJob::findTotalNumberFromTo()
QString ComicArchiveJob::suffixToIdentifier( const QString &suffix ) const
{
- return mPluginName + ':' + suffix;
+ return mPluginName + QLatin1Char(':') + suffix;
}
void ComicArchiveJob::requestComic( QString identifier ) \
//krazy:exclude=passbyvalue @@ -336,8 +336,8 @@ void \
ComicArchiveJob::requestComic( QString identifier ) //krazy:exclude=passbyv \
}
emit description( this, i18n( "Creating Comic Book Archive" ),
- qMakePair( QString( "source" ), identifier ),
- qMakePair( QString( "destination" ), \
mDest.toString() ) ); + \
qMakePair(QStringLiteral("source"), identifier), + \
qMakePair(QStringLiteral("destination"), mDest.toString()));
mEngine->connectSource( identifier, this );
// mEngine->query( identifier );
diff --git a/applets/comic/comicdata.cpp b/applets/comic/comicdata.cpp
index 2a03f6965..37f6248aa 100644
--- a/applets/comic/comicdata.cpp
+++ b/applets/comic/comicdata.cpp
@@ -37,21 +37,21 @@ void ComicData::init(const QString &id, const \
KConfigGroup &config)
void ComicData::load()
{
- mScaleComic = mCfg.readEntry("scaleToContent_" + mId, false);
- mMaxStripNum = mCfg.readEntry("maxStripNum_" + mId, 0);
- mStored = mCfg.readEntry("storedPosition_" + mId, QString());
+ mScaleComic = mCfg.readEntry(QLatin1String("scaleToContent_") + mId, \
false); + mMaxStripNum = mCfg.readEntry(QLatin1String("maxStripNum_") + \
mId, 0); + mStored = mCfg.readEntry(QLatin1String("storedPosition_") + \
mId, QString()); }
void ComicData::save()
{
- mCfg.writeEntry("scaleToContent_" + mId, mScaleComic);
- mCfg.writeEntry("maxStripNum_" + mId, mMaxStripNum);
- mCfg.writeEntry("storedPosition_" + id(), mStored);
+ mCfg.writeEntry(QLatin1String("scaleToContent_") + mId, mScaleComic);
+ mCfg.writeEntry(QLatin1String("maxStripNum_") + mId, mMaxStripNum);
+ mCfg.writeEntry(QLatin1String("storedPosition_") + id(), mStored);
// no next, thus the most recent strip
if (!hasNext()) {
- mCfg.writeEntry("lastStripVisited_" + mId, true);
- mCfg.writeEntry("lastStrip_" + mId, mLast);
+ mCfg.writeEntry(QLatin1String("lastStripVisited_") + mId, true);
+ mCfg.writeEntry(QLatin1String("lastStrip_") + mId, mLast);
}
}
@@ -69,40 +69,40 @@ void ComicData::storePosition(bool store)
void ComicData::setData(const Plasma::DataEngine::Data &data)
{
- const bool hasError = data[ "Error" ].toBool();
+ const bool hasError = data[QStringLiteral("Error")].toBool();
if (!hasError) {
- mImage = data["Image"].value<QImage>();
- mPrev = data["Previous identifier suffix"].toString();
- mNext = data["Next identifier suffix"].toString();
- mAdditionalText = data["Additional text"].toString();
+ mImage = data[QStringLiteral("Image")].value<QImage>();
+ mPrev = data[QStringLiteral("Previous identifier \
suffix")].toString(); + mNext = data[QStringLiteral("Next identifier \
suffix")].toString(); + mAdditionalText = \
data[QStringLiteral("Additional text")].toString(); }
- mWebsiteUrl = data[ "Website Url" ].value<QUrl>();
- mImageUrl = data["Image Url"].value<QUrl>();
- mShopUrl = data[ "Shop Url" ].value<QUrl>();
- mFirst = data[ "First strip identifier suffix" ].toString();
- mStripTitle = data[ "Strip title" ].toString();
- mAuthor = data[ "Comic Author" ].toString();
- mTitle = data[ "Title" ].toString();
+ mWebsiteUrl = data[QStringLiteral("Website Url")].value<QUrl>();
+ mImageUrl = data[QStringLiteral("Image Url")].value<QUrl>();
+ mShopUrl = data[QStringLiteral("Shop Url")].value<QUrl>();
+ mFirst = data[QStringLiteral("First strip identifier \
suffix")].toString(); + mStripTitle = data[QStringLiteral("Strip \
title")].toString(); + mAuthor = data[QStringLiteral("Comic \
Author")].toString(); + mTitle = \
data[QStringLiteral("Title")].toString();
- const QString suffixType = data[ "SuffixType" ].toString();
- if ( suffixType == "Date" ) {
+ const QString suffixType = \
data[QStringLiteral("SuffixType")].toString(); + if ( suffixType == \
QLatin1String("Date")) { mType = Date;
- } else if ( suffixType == "Number" ) {
+ } else if ( suffixType == QLatin1String("Number")) {
mType = Number;
} else {
mType = String;
}
- QString temp = data["Identifier"].toString();
- mCurrent = temp.remove(mId + ':');
+ QString temp = data[QStringLiteral("Identifier")].toString();
+ mCurrent = temp.remove(mId + QLatin1Char(':'));
//found a new last identifier
if (!hasNext()) {
mLast = mCurrent;
}
- mCurrentReadable = "";
+ mCurrentReadable.clear();
if ( mType == Number ) {
mCurrentReadable = i18nc("an abbreviation for Number", "# %1", \
mCurrent); int tempNum = mCurrent.toInt();
@@ -110,16 +110,16 @@ void ComicData::setData(const \
Plasma::DataEngine::Data &data) mMaxStripNum = tempNum;
}
- temp = mFirst.remove(mId + ':');
+ temp = mFirst.remove(mId + QLatin1Char(':'));
mFirstStripNum = temp.toInt();
- } else if ( mType == Date && QDate::fromString( temp, "yyyy-MM-dd" \
).isValid() ) { + } else if (mType == Date && QDate::fromString(temp, \
QStringLiteral("yyyy-MM-dd")).isValid()) { mCurrentReadable = mCurrent;
} else if ( mType == String ) {
mCurrentReadable = mCurrent;
}
- mIsLeftToRight = data["isLeftToRight"].toBool();
- mIsTopToBottom = data["isTopToBottom"].toBool();
+ mIsLeftToRight = data[QStringLiteral("isLeftToRight")].toBool();
+ mIsTopToBottom = data[QStringLiteral("isTopToBottom")].toBool();
save();
}
@@ -137,10 +137,11 @@ void ComicData::createErrorPicture(const \
Plasma::DataEngine::Data &data)
p.drawText( QRect( 10, 10 , 480, 100 ), Qt::TextWordWrap | \
Qt::AlignHCenter | Qt::AlignVCenter, title ); QString text = i18n( "Maybe \
there is no Internet connection.\nMaybe the comic plugin is \
broken.\nAnother reason might be that there is no comic for this \
day/number/string, so choosing a different one might work." );
- mPrev = data["Previous identifier suffix"].toString();
+ mPrev = data[QStringLiteral("Previous identifier suffix")].toString();
if (hasPrev()) {
- if (!data["Identifier"].toString().isEmpty() ) {
- mErrorStrip = data["Identifier"].toString();
+ const auto identifier = \
data[QStringLiteral("Identifier")].toString(); + if \
(!identifier.isEmpty()) { + mErrorStrip = identifier;
}
text.append( i18n( "\n\nChoose the previous strip to go to the \
last cached strip." ) ); }
diff --git a/applets/comic/comicsaver.cpp b/applets/comic/comicsaver.cpp
index 27cdd7170..36fa11a8f 100644
--- a/applets/comic/comicsaver.cpp
+++ b/applets/comic/comicsaver.cpp
@@ -36,10 +36,10 @@ bool ComicSaver::save(const ComicData &comic)
{
const QString title = comic.title();
- const QString name = title + " - " + comic.current() + ".png";
- QUrl destUrl = QUrl::fromLocalFile( mSavingDir->getDir() + '/'+ name \
); + const QString name = title + QLatin1String(" - ") + comic.current() \
+ QLatin1String(".png"); + QUrl destUrl = \
QUrl::fromLocalFile(mSavingDir->getDir() + QLatin1Char('/') + name);
- destUrl = QFileDialog::getSaveFileUrl(nullptr, QString(), destUrl, \
"*.png" ); + destUrl = QFileDialog::getSaveFileUrl(nullptr, QString(), \
destUrl, QStringLiteral("*.png"));
if ( !destUrl.isValid() ) {
return false;
diff --git a/applets/comic/stripselector.cpp \
b/applets/comic/stripselector.cpp index 94af85c09..8a71182ab 100644
--- a/applets/comic/stripselector.cpp
+++ b/applets/comic/stripselector.cpp
@@ -154,7 +154,7 @@ void DateStripSelector::select(const ComicData \
¤tStrip) KDatePicker *calendar = new KDatePicker;
calendar->setAttribute(Qt::WA_DeleteOnClose);//to have destroyed \
emitted upon closing calendar->setMinimumSize(calendar->sizeHint());
- calendar->setDate(QDate::fromString(currentStrip.current(), \
"yyyy-MM-dd")); + \
calendar->setDate(QDate::fromString(currentStrip.current(), \
QStringLiteral("yyyy-MM-dd")));
connect(calendar, &KDatePicker::dateSelected, this, \
&DateStripSelector::slotChosenDay);
connect(calendar, &KDatePicker::dateEntered, this, \
&DateStripSelector::slotChosenDay); @@ -167,11 +167,11 @@ void \
DateStripSelector::select(const ComicData ¤tStrip) void \
DateStripSelector::slotChosenDay(const QDate &date) {
if (date <= QDate::currentDate()) {
- QDate temp = QDate::fromString(mFirstIdentifierSuffix, \
"yyyy-MM-dd"); + QDate temp = \
QDate::fromString(mFirstIdentifierSuffix, \
QStringLiteral("yyyy-MM-dd"));
// only update if date >= first strip date, or if there is no \
first // strip date
if (temp.isValid() || date >= temp) {
- emit stripChosen(date.toString("yyyy-MM-dd"));
+ emit stripChosen(date.toString(QStringLiteral("yyyy-MM-dd")));
}
}
}
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic