(causeway) branch CAUSEWAY-3676 updated: CAUSEWAY-3676: wip on addActions, to move into GqlvObjectSpec (2)

[danhaywood]

This is an automated email from the ASF dual-hosted git repository.

danhaywood pushed a commit to branch CAUSEWAY-3676
in repository https://gitbox.apache.org/repos/asf/causeway.git


The following commit(s) were added to refs/heads/CAUSEWAY-3676 by this push:
     new e06b3b71e4 CAUSEWAY-3676: wip on addActions, to move into 
GqlvObjectSpec (2)
e06b3b71e4 is described below

commit e06b3b71e44dbd9a5e3b85f00bd3204028e8b261
Author: danhaywood <d...@haywood-associates.co.uk>
AuthorDate: Fri Jan 19 12:56:43 2024 +0000

    CAUSEWAY-3676: wip on addActions, to move into GqlvObjectSpec (2)
---
 .../viewer/graphql/viewer/source/ObjectTypeFactory.java       | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git 
a/incubator/viewers/graphql/viewer/src/main/java/org/apache/causeway/viewer/graphql/viewer/source/ObjectTypeFactory.java
 
b/incubator/viewers/graphql/viewer/src/main/java/org/apache/causeway/viewer/graphql/viewer/source/ObjectTypeFactory.java
index feb1115443..ad702c1a6a 100644
--- 
a/incubator/viewers/graphql/viewer/src/main/java/org/apache/causeway/viewer/graphql/viewer/source/ObjectTypeFactory.java
+++ 
b/incubator/viewers/graphql/viewer/src/main/java/org/apache/causeway/viewer/graphql/viewer/source/ObjectTypeFactory.java
@@ -23,7 +23,6 @@ import static 
graphql.schema.GraphQLInputObjectType.newInputObject;
 import static graphql.schema.GraphQLNonNull.nonNull;
 import static graphql.schema.GraphQLObjectType.newObject;
 
-import java.util.ArrayList;
 import java.util.List;
 import java.util.Set;
 import java.util.stream.Collectors;
@@ -166,7 +165,7 @@ public class ObjectTypeFactory {
 
         gqlvObjectSpec.getObjectSpec().streamActions(ActionScope.PRODUCTION, 
MixedIn.INCLUDED)
                 .forEach(objectAction ->
-                        addAction(objectAction, 
gqlvObjectSpec.getGqlObjectTypeBuilder(), gqlvObjectSpec.mutatorsTypeBuilder, 
gqlvObjectSpec.mutatorsTypeFields)
+                        addAction(gqlvObjectSpec, objectAction)
                 );
 
         if (!gqlvObjectSpec.mutatorsTypeFields.isEmpty()){
@@ -209,7 +208,13 @@ public class ObjectTypeFactory {
     }
 
 
-    private static void addAction(ObjectAction objectAction, 
GraphQLObjectType.Builder objectTypeBuilder, GraphQLObjectType.Builder 
mutatorsTypeBuilder, List<GraphQLFieldDefinition> mutatorsTypeFields) {
+    private static void addAction(
+            final GqlvObjectSpec gqlvObjectSpec, final ObjectAction 
objectAction) {
+
+        final GraphQLObjectType.Builder objectTypeBuilder = 
gqlvObjectSpec.getGqlObjectTypeBuilder();
+        final GraphQLObjectType.Builder mutatorsTypeBuilder = 
gqlvObjectSpec.mutatorsTypeBuilder;
+        final List<GraphQLFieldDefinition> mutatorsTypeFields = 
gqlvObjectSpec.mutatorsTypeFields;
+
         if (objectAction.getSemantics().isSafeInNature()) {
 
             String fieldName = objectAction.getId();