[Bf-blender-cvs] [955748ab1e3] master: Fix: Duplicate link search entries for attribute statistic node

[Hans Goudey]

Commit: 955748ab1e35d92d9c60ce81f43681e715768eb2
Author: Hans Goudey
Date:   Tue Dec 28 12:44:36 2021 -0600
Branches: master
https://developer.blender.org/rB955748ab1e35d92d9c60ce81f43681e715768eb2

Fix: Duplicate link search entries for attribute statistic node

Using the output declarations is incorrect because there is a
declaration for each type. Instead loop over the names directly,
since it will make it easier to add an integer mode that only
supports some of the outputs.

===================================================================

M       source/blender/nodes/geometry/nodes/node_geo_attribute_statistic.cc

===================================================================

diff --git 
a/source/blender/nodes/geometry/nodes/node_geo_attribute_statistic.cc 
b/source/blender/nodes/geometry/nodes/node_geo_attribute_statistic.cc
index 4d6fb328ad9..e97badeb83a 100644
--- a/source/blender/nodes/geometry/nodes/node_geo_attribute_statistic.cc
+++ b/source/blender/nodes/geometry/nodes/node_geo_attribute_statistic.cc
@@ -148,8 +148,8 @@ static void 
node_gather_link_searches(GatherLinkSearchOpParams &params)
     });
   }
   else {
-    for (const SocketDeclarationPtr &socket_decl : declaration.outputs()) {
-      StringRefNull name = socket_decl->name();
+    for (const StringRefNull name :
+         {"Mean", "Median", "Sum", "Min", "Max", "Range", "Standard 
Deviation", "Variance"}) {
       params.add_item(IFACE_(name.c_str()), [node_type, name, 
type](LinkSearchOpParams &params) {
         bNode &node = params.add_node(node_type);
         node.custom1 = *type;

_______________________________________________
Bf-blender-cvs mailing list
Bf-blender-cvs@blender.org
List details, subscription details or unsubscribe:
https://lists.blender.org/mailman/listinfo/bf-blender-cvs