[bugfix] Show prefix of XPath standard library functions in the profiling output

Author: adamretter

exist-core/pom.xml

@@ -802,6 +802,7 @@
                                 <include>src/test/java/org/exist/IndexerTest.java</include>
                                 <include>src/test/java/org/exist/IndexerTest2.java</include>
                                 <include>src/test/java/org/exist/IndexerTest3.java</include>
+                                <include>src/main/java/org/exist/Namespaces.java</include>
                                 <include>src/main/resources-filtered/org/exist/system.properties</include>
                                 <include>src/main/java/org/exist/backup/ExportGUI.java</include>
                                 <include>src/main/java/org/exist/backup/ExportMain.java</include>
@@ -1236,6 +1237,7 @@
                                 <exclude>src/test/java/org/exist/IndexerTest.java</exclude>
                                 <exclude>src/test/java/org/exist/IndexerTest2.java</exclude>
                                 <exclude>src/test/java/org/exist/IndexerTest3.java</exclude>
+                                <exclude>src/main/java/org/exist/Namespaces.java</exclude>
                                 <exclude>src/main/resources-filtered/org/exist/system.properties</exclude>
                                 <exclude>src/main/java/org/exist/backup/ExportGUI.java</exclude>
                                 <exclude>src/main/java/org/exist/backup/ExportMain.java</exclude>

exist-core/src/main/java/org/exist/Namespaces.java

@@ -1,4 +1,28 @@
 /*
+ * Elemental
+ * Copyright (C) 2024, Evolved Binary Ltd
+ *
+ * [email protected]
+ * https://www.evolvedbinary.com | https://www.elemental.xyz
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; version 2.1.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this library; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301  USA
+ *
+ * NOTE: Parts of this file contain code from 'The eXist-db Authors'.
+ *       The original license header is included below.
+ *
+ * =====================================================================
+ *
  * eXist-db Open Source Native XML Database
  * Copyright (C) 2001 The eXist-db Authors
  *
@@ -41,7 +65,8 @@ public interface Namespaces {
 	String SCHEMA_INSTANCE_NS = XMLConstants.W3C_XML_SCHEMA_INSTANCE_NS_URI;
 
     // Move this here from Function.BUILTIN_FUNCTION_NS? /ljo
-	String XPATH_FUNCTIONS_NS = "http://www.w3.org/2005/xpath-functions";	
+	String XPATH_FUNCTIONS_NS = "http://www.w3.org/2005/xpath-functions";
+	String XPATH_FUNCTIONS_PREFIX = "fn";
     String XQUERY_LOCAL_NS = "http://www.w3.org/2005/xquery-local-functions";
 	String XPATH_DATATYPES_NS = "http://www.w3.org/2003/05/xpath-datatypes";
 

exist-core/src/main/java/org/exist/xquery/PerformanceStatsImpl.java

@@ -62,6 +62,9 @@
 import java.util.Map;
 import java.util.Set;
 
+import static org.exist.Namespaces.XPATH_FUNCTIONS_NS;
+import static org.exist.Namespaces.XPATH_FUNCTIONS_PREFIX;
+
 /**
  * Implementation of a PerformanceStats that is designed
  * to be used from a single XQuery via its {@link XQueryContext}
@@ -271,11 +274,16 @@ public void recordQuery(final String source, final long elapsed) {
     }
 
     @Override
-    public void recordFunctionCall(final QName qname, final String source, final long elapsed) {
+    public void recordFunctionCall(QName qname, final String source, final long elapsed) {
         if (!isEnabled()) {
             return;
         }
 
+        if (XPATH_FUNCTIONS_NS.equals(qname.getNamespaceURI()) && qname.getPrefix() == null) {
+            // make sure that functions from the XPath/XQuery standard library are shown correctly in the output
+            qname = new QName(qname.getLocalPart(), qname.getNamespaceURI(), XPATH_FUNCTIONS_PREFIX, qname.getNameType());
+        }
+
         final FunctionStats newStats = new FunctionStats(source, qname);
         final FunctionStats stats = functions.get(newStats);
         if (stats == null) {
@@ -437,4 +445,4 @@ public void reset() {
         indexStats.clear();
         optimizations.clear();
     }
-}
+}