From 5b6b21722f6f34f7a4ec3935360828074d94cc22 Mon Sep 17 00:00:00 2001
From: Milan Crha <mcrha@redhat.com>
Date: Fri, 6 Sep 2024 10:13:14 +0200
Subject: [PATCH] dnfdaemon: Correct argument direction in interface definition
 of Goal::cancel method

Both the arguments are output arguments, to be read by client, not
to be set by the client, thus mark them as such.

Related to https://github.com/rpm-software-management/dnf5/pull/1665#discussion_r1745152462
---
 dnf5daemon-server/dbus/interfaces/org.rpm.dnf.v0.Goal.xml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/dnf5daemon-server/dbus/interfaces/org.rpm.dnf.v0.Goal.xml b/dnf5daemon-server/dbus/interfaces/org.rpm.dnf.v0.Goal.xml
index c94804c54..ab142e21f 100644
--- a/dnf5daemon-server/dbus/interfaces/org.rpm.dnf.v0.Goal.xml
+++ b/dnf5daemon-server/dbus/interfaces/org.rpm.dnf.v0.Goal.xml
@@ -102,8 +102,8 @@ along with libdnf.  If not, see <https://www.gnu.org/licenses/>.
         Cancel the transaction that was initiated by `do_transaction()`. The transaction can only be canceled during the package download phase. Once the RPM transaction has begun, cancellation is no longer permitted.
     -->
     <method name="cancel">
-        <arg name="success" type="b" />
-        <arg name="error_msg" type="s" />
+        <arg name="success" type="b" direction="out" />
+        <arg name="error_msg" type="s" direction="out" />
     </method>
 
 </interface>