Skip to content

Commit d50ccf7

Browse files
committed
[fix] dump and toString methods of UserDefinedFunction
Output argument names and add "as" before function return type.
1 parent 9ab11b0 commit d50ccf7

File tree

1 file changed

+32
-25
lines changed

1 file changed

+32
-25
lines changed

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

Lines changed: 32 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -166,40 +166,47 @@ public Sequence eval(Sequence contextSequence, Item contextItem) throws XPathExc
166166
context.stackLeave(this);
167167
// context.expressionEnd(this);
168168
}
169-
}
170-
171-
/* (non-Javadoc)
172-
* @see org.exist.xquery.Function#dump(org.exist.xquery.util.ExpressionDumper)
173-
*/
169+
}
170+
171+
@Override
174172
public void dump(ExpressionDumper dumper) {
175173
final FunctionSignature signature = getSignature();
176-
if (signature.getName() != null)
177-
{dumper.display(signature.getName());}
174+
if (signature.getName() != null) {
175+
dumper.display(signature.getName());
176+
}
178177
dumper.display('(');
179-
for(int i = 0; i < signature.getArgumentTypes().length; i++) {
180-
if(i > 0)
181-
{dumper.display(", ");}
182-
dumper.display(signature.getArgumentTypes()[i]);
183-
}
184-
dumper.display(") ");
178+
for (int i = 0; i < signature.getArgumentTypes().length; i++) {
179+
if (i > 0) {
180+
dumper.display(", ");
181+
}
182+
dumper.display('$');
183+
dumper.display(getParameters().get(i));
184+
dumper.display(" as ");
185+
dumper.display(signature.getArgumentTypes()[i]);
186+
}
187+
dumper.display(") as ");
185188
dumper.display(signature.getReturnType().toString());
186189
}
187-
188-
/* (non-Javadoc)
189-
* @see java.lang.Object#toString()
190-
*/
190+
191+
@Override
191192
public String toString() {
192193
final FunctionSignature signature = getSignature();
193194
final StringBuilder buf = new StringBuilder();
194-
if (signature.getName() != null)
195-
{buf.append(signature.getName());}
195+
if (signature.getName() != null) {
196+
buf.append(signature.getName());
197+
}
196198
buf.append('(');
197-
for(int i = 0; i < signature.getArgumentTypes().length; i++) {
198-
if(i > 0)
199-
{buf.append(", ");}
200-
buf.append(signature.getArgumentTypes()[i]);
201-
}
202-
buf.append(')');
199+
for (int i = 0; i < signature.getArgumentTypes().length; i++) {
200+
if (i > 0) {
201+
buf.append(", ");
202+
}
203+
buf.append('$');
204+
buf.append(getParameters().get(i));
205+
buf.append(" as ");
206+
buf.append(signature.getArgumentTypes()[i]);
207+
}
208+
buf.append(") as ");
209+
buf.append(signature.getReturnType());
203210
return buf.toString();
204211
}
205212

0 commit comments

Comments
 (0)