Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 361056 - fn:abs - typeError on sequence
Summary: fn:abs - typeError on sequence
Status: RESOLVED FIXED
Alias: None
Product: WTP Source Editing
Classification: WebTools
Component: wst.xpath (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Jesper Moller CLA
QA Contact: Jesper Moller CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-10-15 10:22 EDT by Lukasz Wycisk CLA
Modified: 2011-12-13 18:55 EST (History)
0 users

See Also:

Attachments
testcase patch and processor patch (2.94 KB, text/plain)
2011-10-15 10:22 EDT, Lukasz Wycisk CLA 		no flags Details
processor patch (2.47 KB, text/plain)
2011-10-15 10:23 EDT, Lukasz Wycisk CLA 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Lukasz Wycisk CLA 2011-10-15 10:22:14 EDT 
Created attachment 205253 [details]
testcase patch and processor patch

1. FnAbs: http://www.w3.org/TR/xquery-operators/#func-abs

According to http://www.w3.org/TR/xpath20/#id-function-calls if sequence is expected (indicator *, +, or ?) the atomization of arguments should be applied. Now in case of node passed to method type error occur. However there is some support for atomization - FnDistinctValues was fixed lately. Test case:

TestBugs:testFunctionAtomization()

We think that in other numeric function similar errors can occur, suppose other tests should be added. Surprisingly there are no tests for that sort of situation in conformance suit (at least we haven’t found any).
Comment 1 Lukasz Wycisk CLA 2011-10-15 10:23:04 EDT 
Created attachment 205254 [details]
processor patch
Comment 2 Jesper Moller CLA 2011-10-30 20:04:54 EDT 
Patch looks good ... Reviewing and testing against HEAD and maintenance branches!
Comment 3 Jesper Moller CLA 2011-11-26 06:21:17 EST 
The patch contained both the QName fix and the fb:abs part, but I separated the relevant one out.
Applied to HEAD, next up: 3.3.x branch.
Comment 4 Jesper Moller CLA 2011-12-13 18:55:06 EST 
Committed to the 3.3 service branch