plasma: const-ify the service job parameters

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
This commit is contained in:
Ivailo Monev 2023-06-02 02:45:51 +03:00
parent 7c8aed76ae
commit 2858f3be71
9 changed files with 13 additions and 14 deletions

View file

@ -36,7 +36,7 @@ DataEngineService::DataEngineService(DataEngine *engine)
}
Plasma::ServiceJob* DataEngineService::createJob(const QString& operation,
QMap<QString,QVariant>& parameters)
const QMap<QString,QVariant>& parameters)
{
return new GetSource(m_engine, operation, parameters, this);
}

View file

@ -42,7 +42,7 @@ class DataEngineService : public Plasma::Service
protected:
Plasma::ServiceJob* createJob(const QString& operation,
QMap<QString,QVariant>& parameters);
const QMap<QString,QVariant>& parameters);
private Q_SLOTS:
void sourceAdded(QString source);

View file

@ -30,9 +30,9 @@ namespace Plasma
{
GetSource::GetSource(DataEngine *engine, const QString& operation,
QMap<QString,QVariant>& parameters,
DataEngineService *service)
: ServiceJob(QString("publickey"), operation, parameters, service),
const QMap<QString,QVariant>& parameters,
DataEngineService *service)
: ServiceJob(QString::fromLatin1("publickey"), operation, parameters, service),
m_engine(engine),
m_service(service)
{

View file

@ -33,7 +33,7 @@ class GetSource : public Plasma::ServiceJob
public:
GetSource(DataEngine *engine, const QString& operation,
QMap<QString,QVariant>& parameters,
const QMap<QString,QVariant>& parameters,
DataEngineService *service = 0);
void start();

View file

@ -37,7 +37,7 @@ namespace Plasma
PlasmoidServiceJob::PlasmoidServiceJob(const QString &destination,
const QString &operation,
QMap<QString,QVariant>& parameters,
const QMap<QString,QVariant>& parameters,
PlasmoidService *service)
: Plasma::ServiceJob(destination, operation, parameters,
static_cast<Plasma::Service*>(service)),
@ -89,7 +89,7 @@ PackageMetadata PlasmoidService::metadata() const
}
Plasma::ServiceJob* PlasmoidService::createJob(const QString& operation,
QMap<QString,QVariant>& parameters)
const QMap<QString,QVariant>& parameters)
{
return new PlasmoidServiceJob(destination(), operation, parameters, this);
}

View file

@ -39,7 +39,7 @@ class PlasmoidServiceJob : public ServiceJob
public:
PlasmoidServiceJob(const QString &destination,
const QString &operation,
QMap<QString,QVariant>& parameters,
const QMap<QString,QVariant>& parameters,
PlasmoidService *parent = 0);
void start();
@ -60,7 +60,7 @@ class PlasmoidService : public Service, DataEngineConsumer
protected:
Plasma::ServiceJob* createJob(const QString& operation,
QMap<QString,QVariant>& parameters);
const QMap<QString,QVariant>& parameters);
private:
QString m_packagePath;

View file

@ -61,7 +61,7 @@ public:
setName("NullService");
}
ServiceJob *createJob(const QString &operation, QMap<QString, QVariant> &)
ServiceJob *createJob(const QString &operation, const QMap<QString, QVariant> &)
{
return new NullServiceJob(destination(), operation, this);
}

View file

@ -159,8 +159,7 @@ ServiceJob *Service::startOperationCall(const KConfigGroup &description, QObject
if (d->disabledOperations.contains(op)) {
kDebug() << "Operation" << op << "is disabled";
} else {
QMap<QString, QVariant> params = parametersFromDescription(description);
job = createJob(op, params);
job = createJob(op, parametersFromDescription(description));
}
} else {
kDebug() << op << "is not a valid group; valid groups are:" << d->config->groupList();

View file

@ -276,7 +276,7 @@ protected:
* @return a ServiceJob that can be started and monitored by the consumer
*/
virtual ServiceJob *createJob(const QString &operation,
QMap<QString, QVariant> &parameters) = 0;
const QMap<QString, QVariant> &parameters) = 0;
/**
* By default this is based on the file in plasma/services/name.operations, but can be