Removed option to call sum() with string literal argument

Lwiel · Jan 16, 2023 · f13618a · f13618a
1 parent d324407
commit f13618a
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 13 deletions.
diff --git a/src/Raven.Server/Documents/Patch/JavaScriptUtils.cs b/src/Raven.Server/Documents/Patch/JavaScriptUtils.cs
@@ -115,14 +115,6 @@ internal JsValue Sum(JsValue self, JsValue[] args)
                     }
                 }
             }
-
-            if (args[0] is JsString fieldName)
-            {
-                doc.TryGetValue(fieldName, out var sumValue);
-
-                return sumValue;
-            }
-
             return null;
         }        
 

diff --git a/src/Raven.Server/Documents/Queries/HasSpecialMethodVisitor.cs b/src/Raven.Server/Documents/Queries/HasSpecialMethodVisitor.cs
@@ -43,11 +43,6 @@ public override void VisitCallExpression(CallExpression callExpression)
                                     }
                                 }
                             }
-
-                            if (callExpression.Arguments.AsNodes()[0] is Literal fieldNameLiteral)
-                            {
-                                _queryMetadata.SumInJs = fieldNameLiteral.StringValue;
-                            }
                         }
 
                         break;
-Original file line number
+Diff line change
@@ Expand Up @@
                                         }
                                     }
                                 }
-                                if (callExpression.Arguments.AsNodes()[0] is Literal fieldNameLiteral)
-                                {
-                                    _queryMetadata.SumInJs = fieldNameLiteral.StringValue;
-                                }
                             }
                             break;
@@ Expand Down @@