Commit 1d30388f authored by Harald Sitter's avatar Harald Sitter 🏳🌈
Browse files

use httpAssembleURI not httpAssembleURIf

Summary:
the latter is for when the last argument is a format string. which I
believe is not the case here. or perhaps more importantly if it was a
format string it'd be an invalid call because we pass no values in

Test Plan:
- make two printers
- pause them
- print job to one printer
- open printer queue
- right click job to move to other printer
- job is now on other printer

Reviewers: dantti

Reviewed By: dantti

Subscribers: kde-utils-devel

Differential Revision: https://phabricator.kde.org/D28479
parent 65cc1210
......@@ -188,8 +188,8 @@ QString KIppRequest::assembleUrif(const QString &name, bool isClass)
destination = QLatin1String("/printers/") + name;
}
httpAssembleURIf(HTTP_URI_CODING_ALL, uri, sizeof(uri), "ipp", cupsUser(), "localhost",
ippPort(), destination.toUtf8().constData());
httpAssembleURI(HTTP_URI_CODING_ALL, uri, sizeof(uri), "ipp", cupsUser(), "localhost",
ippPort(), destination.toUtf8().constData());
return QString::fromLatin1(uri);
}
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment