D28479: use httpAssembleURI not httpAssembleURIf

Harald Sitter noreply at phabricator.kde.org
Wed Apr 1 11:37:10 BST 2020


sitter created this revision.
sitter added a reviewer: dantti.
Herald added a subscriber: kde-utils-devel.
sitter requested review of this revision.

REVISION 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

REPOSITORY
  R363 Print Manager

BRANCH
  release/20.04

REVISION DETAIL
  https://phabricator.kde.org/D28479

AFFECTED FILES
  libkcups/KIppRequest.cpp

To: sitter, dantti
Cc: kde-utils-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-utils-devel/attachments/20200401/07389da2/attachment.html>


More information about the Kde-utils-devel mailing list