Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 363676

Summary: [RBD] HyperLink widget: The href could not found items in WebContent
Product: z_Archived Reporter: Thomas Wu <wxwu>
Component: EDTAssignee: Xiao Bin Chen <xiaobinc>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: jqian, svihovec, xiaobinc
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
error page none

Description Thomas Wu CLA 2011-11-14 03:01:26 EST
Build Identifier: 20111113

There is the same issue in RBD.
The href could not found the children of WebContent if not giving "WebContent/" at the beginning. But referring to HTML widget, users need not give "WebContent/" because that is default location.

Reproducible: Always

Steps to Reproduce:
1.DD a HyperLink widget into a handler.
2.New a html file in a child dictionary of WebContent, such as WebContent/1/myTest.html
3.Select and open the properties view of HyperLink.
4.Set href property as "/1/myTest.html"
5.Preview the page, and click the HyperLink.

Actaul results: The page report the following error.
document.write("Could not open /1/myTest.html");
Comment 1 Thomas Wu CLA 2011-11-14 03:03:07 EST
Created attachment 206919 [details]
error page
Comment 2 Thomas Wu CLA 2011-11-14 03:15:33 EST
The item also could not be found in deployed mode.
Comment 3 Xiao Bin Chen CLA 2011-11-14 04:54:45 EST
fixed
Comment 4 Brian Svihovec CLA 2011-11-14 10:47:03 EST
This fix breaks the Tic Tac Toe demo in the RUI Technical Samples - /org.eclipse.edt.rui.samples_0.7.0 (KAN-SAV CVS)

Since this issue happens in RBD, I think we should revert this change and defer this defect until 1.0.  

Also, the Tic Tac Toe Demo uses "draganddrop/x.png" as the URL for the image, which is similar to the "1/myTest.html" in this example, so we should be able to make this work without any changes.  My guess is that the leading "/" needs to be removed.
Comment 5 Xiao Bin Chen CLA 2011-11-14 21:14:52 EST
(In reply to comment #4)
> This fix breaks the Tic Tac Toe demo in the RUI Technical Samples -
> /org.eclipse.edt.rui.samples_0.7.0 (KAN-SAV CVS)
> 
> Since this issue happens in RBD, I think we should revert this change and defer
> this defect until 1.0.  
> 
> Also, the Tic Tac Toe Demo uses "draganddrop/x.png" as the URL for the image,
> which is similar to the "1/myTest.html" in this example, so we should be able
> to make this work without any changes.  My guess is that the leading "/" needs
> to be removed.

Current I will comment this fix,and leave it to the future release.
Comment 6 Brian Svihovec CLA 2011-11-14 22:12:49 EST
I would rather not ship commented code, so please remove this code for now.  We can use CVS history as the starting point in a future release, or we can attach a patch here.
Comment 7 Xiao Bin Chen CLA 2011-11-15 00:14:21 EST
Removed these comment code.
Comment 8 Thomas Wu CLA 2011-11-17 20:43:10 EST
It works if user inputs "1/myTest.html" here, so it is low impact. Close it now.