Revert "Add a hook to determine if it's an updating clause"

This reverts commit eedb546.

cmd/api/src/queries/graph.go

@@ -367,7 +367,6 @@ type PreparedQuery struct {
 }
 
 func (s *GraphQuery) PrepareCypherQuery(rawCypher string) (PreparedQuery, error) {
-
 	var (
 		cypherFilters = []frontend.Visitor{
 			&frontend.ExplicitProcedureInvocationFilter{},
@@ -381,8 +380,6 @@ func (s *GraphQuery) PrepareCypherQuery(rawCypher string) (PreparedQuery, error)
 
 	// If cypher mutations are disabled, we want to add the updating clause filter to properly error as unsupported query
 	if !s.EnableCypherMutations {
-		cypherFilters = append(cypherFilters, &frontend.UpdatingNotAllowedClauseFilter{})
-	} else {
 		cypherFilters = append(cypherFilters, &frontend.UpdatingClauseFilter{})
 	}
 

packages/go/cypher/frontend/filter.go

@@ -25,7 +25,6 @@ import (
 // TODO: Review if relying on a deny model is less secure than explicit allow
 func DefaultCypherContext() *Context {
 	return NewContext(
-		&UpdatingNotAllowedClauseFilter{},
 		&UpdatingClauseFilter{},
 		&ExplicitProcedureInvocationFilter{},
 		&ImplicitProcedureInvocationFilter{},
@@ -57,18 +56,10 @@ func (s *SpecifiedParametersFilter) EnterOC_Parameter(ctx *parser.OC_ParameterCo
 	s.ctx.AddErrors(ErrUserSpecifiedParametersNotSupported)
 }
 
-type UpdatingNotAllowedClauseFilter struct {
-	BaseVisitor
-}
-
-func (s *UpdatingNotAllowedClauseFilter) EnterOC_UpdatingClause(ctx *parser.OC_UpdatingClauseContext) {
-	s.ctx.AddErrors(ErrUpdateClauseNotSupported)
-}
-
 type UpdatingClauseFilter struct {
 	BaseVisitor
 }
 
 func (s *UpdatingClauseFilter) EnterOC_UpdatingClause(ctx *parser.OC_UpdatingClauseContext) {
-	// Do something that marks this as an updating clause to check later (in pattern.go)
+	s.ctx.AddErrors(ErrUpdateClauseNotSupported)
 }