From 106fe2777dd3314dc750c5a4aade8a5822a32ab1 Mon Sep 17 00:00:00 2001
From: Peter Philipp <peter.philipp@cando-image.com>
Date: Tue, 23 Jul 2024 11:36:45 +0200
Subject: [PATCH] feat: Add a way to pass arguments to operators. Why this
 wasn't added to the operator call in
 \Pimcore\Bundle\DataHubBundle\GraphQL\Resolver\Base::resolve() in the first
 place is a mystery - dropping information just like that is a shame. And now
 we can't simply adjusting the interface without breaking stuff but have to
 create more interfaces.

---
 .../ArgumentsAwareOperatorInterface.php       | 25 +++++++++++++++++++
 .../ContextAwareOperatorInterface.php         | 25 +++++++++++++++++++
 src/GraphQL/Resolver/Base.php                 | 11 ++++++++
 3 files changed, 61 insertions(+)
 create mode 100644 src/GraphQL/Query/Operator/ArgumentsAwareOperatorInterface.php
 create mode 100644 src/GraphQL/Query/Operator/ContextAwareOperatorInterface.php

diff --git a/src/GraphQL/Query/Operator/ArgumentsAwareOperatorInterface.php b/src/GraphQL/Query/Operator/ArgumentsAwareOperatorInterface.php
new file mode 100644
index 00000000..1dc3d8ec
--- /dev/null
+++ b/src/GraphQL/Query/Operator/ArgumentsAwareOperatorInterface.php
@@ -0,0 +1,25 @@
+<?php
+
+declare(strict_types=1);
+
+/**
+ * Pimcore
+ *
+ * This source file is available under two different licenses:
+ * - GNU General Public License version 3 (GPLv3)
+ * - Pimcore Commercial License (PCL)
+ * Full copyright and license information is available in
+ * LICENSE.md which is distributed with this source code.
+ *
+ *  @copyright  Copyright (c) Pimcore GmbH (http://www.pimcore.org)
+ *  @license    http://www.pimcore.org/license     GPLv3 and PCL
+ */
+
+namespace Pimcore\Bundle\DataHubBundle\GraphQL\Query\Operator;
+
+interface ArgumentsAwareOperatorInterface extends OperatorInterface
+{
+    public function setArguments(array $arguments);
+
+    public function getArguments();
+}
diff --git a/src/GraphQL/Query/Operator/ContextAwareOperatorInterface.php b/src/GraphQL/Query/Operator/ContextAwareOperatorInterface.php
new file mode 100644
index 00000000..22fbc758
--- /dev/null
+++ b/src/GraphQL/Query/Operator/ContextAwareOperatorInterface.php
@@ -0,0 +1,25 @@
+<?php
+
+declare(strict_types=1);
+
+/**
+ * Pimcore
+ *
+ * This source file is available under two different licenses:
+ * - GNU General Public License version 3 (GPLv3)
+ * - Pimcore Commercial License (PCL)
+ * Full copyright and license information is available in
+ * LICENSE.md which is distributed with this source code.
+ *
+ *  @copyright  Copyright (c) Pimcore GmbH (http://www.pimcore.org)
+ *  @license    http://www.pimcore.org/license     GPLv3 and PCL
+ */
+
+namespace Pimcore\Bundle\DataHubBundle\GraphQL\Query\Operator;
+
+interface ContextAwareOperatorInterface extends OperatorInterface
+{
+    public function setContext(array $context);
+
+    public function getContext();
+}
diff --git a/src/GraphQL/Resolver/Base.php b/src/GraphQL/Resolver/Base.php
index af3215d1..cc304164 100644
--- a/src/GraphQL/Resolver/Base.php
+++ b/src/GraphQL/Resolver/Base.php
@@ -19,6 +19,8 @@
 use GraphQL\Executor\Promise\Adapter\SyncPromise;
 use GraphQL\Language\AST\FieldNode;
 use GraphQL\Type\Definition\ResolveInfo;
+use Pimcore\Bundle\DataHubBundle\GraphQL\Query\Operator\ArgumentsAwareOperatorInterface;
+use Pimcore\Bundle\DataHubBundle\GraphQL\Query\Operator\ContextAwareOperatorInterface;
 use Pimcore\Bundle\DataHubBundle\GraphQL\Traits\ElementLoaderTrait;
 use Pimcore\Bundle\DataHubBundle\GraphQL\Traits\ServiceTrait;
 use Pimcore\Model\DataObject\ClassDefinition;
@@ -63,6 +65,9 @@ public function __construct($typeName, $attributes, $class, $container)
      */
     public function resolve($value = null, $args = [], $context = [], ResolveInfo $resolveInfo = null)
     {
+        // If value is an array it is assumed that the data were resolved
+        // upstream and don't need further evaluation but simple delivery later
+        // on. Which is achieved by using a deferred wrapper.
         if (is_array($value)) {
             $result = $value[$resolveInfo->fieldName] ?? null;
             // check for alias as we cache the properties with aliases
@@ -86,6 +91,12 @@ public function resolve($value = null, $args = [], $context = [], ResolveInfo $r
         }
         /** @var \Pimcore\Bundle\DataHubBundle\GraphQL\Query\Operator\AbstractOperator $operatorImpl */
         $operatorImpl = $this->getGraphQlService()->buildQueryOperator($this->typeName, $this->attributes);
+        if ($operatorImpl instanceof ArgumentsAwareOperatorInterface) {
+            $operatorImpl->setArguments($args ?? []);
+        }
+        if ($operatorImpl instanceof ContextAwareOperatorInterface) {
+            $operatorImpl->setContext($context ?? []);
+        }
 
         $element = $this->loadDataElement($value, 'object');
         $valueFromOperator = $operatorImpl->getLabeledValue($element, $resolveInfo);