Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 361802 - Default variable namespace – no namespace.
Summary: Default variable namespace – no namespace.
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-24 09:35 EDT by Lukasz Wycisk CLA
Modified: 2011-11-01 19:45 EDT (History)
1 user (show)

See Also:

Attachments
Patch (761 bytes, text/plain)
2011-10-24 09:36 EDT, Lukasz Wycisk CLA 		no flags Details
Testcase (1.56 KB, text/plain)
2011-10-24 09:36 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-24 09:35:33 EDT 
According to the standard
http://www.w3.org/TR/xpath20/#id-variables
“An unprefixed variable reference is in no namespace.” - not in default as it is in the current implementation.
Changing test base:
AbstractPsychoPathTest:setVariable to null namespaces causes test
TestBugs:testDefNamespaceOnbuiltInTypes() to fail.
Small change in StaticNameResolver solves problem.
Comment 1 Lukasz Wycisk CLA 2011-10-24 09:36:35 EDT 
Created attachment 205813 [details]
Patch

Patch file
Comment 2 Lukasz Wycisk CLA 2011-10-24 09:36:59 EDT 
Created attachment 205815 [details]
Testcase
Comment 3 Jesper Moller CLA 2011-10-30 20:04:55 EDT 
Patch looks good ... Reviewing and testing against HEAD and maintenance branches!
Comment 4 Jesper Moller CLA 2011-11-01 18:37:55 EDT 
Applied to HEAD and 3_3_maintenance.

Mukul, if you want to port this to the 3_2 branch, please do so, if it is an issue with 3.2.
Comment 5 Jesper Moller CLA 2011-11-01 19:45:01 EDT 
Comment on attachment 205815 [details]
Testcase

Oops, forgot that this MAY be relevant for 3.2 branch, so not marking as obsolete.