Bug 537764 - Search and CA doesn't work if project name contain pipe
Summary: Search and CA doesn't work if project name contain pipe
Status: NEW
Alias: None
Product: JDT
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.9   Edit
Hardware: PC Mac OS X
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Jay Arthanareeswaran CLA
QA Contact:
URL:
Whiteboard: stalebug
Keywords: needinfo
Depends on:
Blocks:
 
Reported: 2018-08-07 11:07 EDT by Dawid Pakula CLA
Modified: 2023-03-10 10:37 EST (History)
1 user (show)

See Also:

Attachments
Test (3.62 KB, application/zip)
2018-08-08 13:04 EDT, Dawid Pakula CLA 		no flags Details
Test project for windows (10.23 KB, application/zip)
2018-08-09 06:26 EDT, Dawid Pakula 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 Dawid Pakula CLA 2018-08-07 11:07:57 EDT 
If project name contain pipe "|", for example "Test | Project", CA/search is not able to find classes.

Problematic is JavaSearchScope and how it play with pipe in resource path.

Same issue is in DLTK: https://bugs.eclipse.org/bugs/show_bug.cgi?id=537714
Comment 1 Dani Megert CLA 2018-08-07 11:20:45 EDT 
Please provide an example.

CA = content assist?
Comment 2 Dawid Pakula CLA 2018-08-08 13:04:32 EDT 
Created attachment 275316 [details]
Test
Comment 3 Dawid Pakula CLA 2018-08-08 13:09:22 EDT 
Import attached project and try search "MyClass" or "Another".

Same during Code Assist (CA), for example inside "Another" class write:
void test() { new My| } // Run CodeAssist on "|", MyClass expected, see nothing.

If you rename project to anything without "|", for example "Test Project" works fine.
Comment 4 Dani Megert CLA 2018-08-09 06:10:19 EDT 
I can't look at this since pipe is not valid on my Windows machine.

Jay, please have a look when you find some time.
Comment 5 Dawid Pakula CLA 2018-08-09 06:26:45 EDT 
Created attachment 275328 [details]
Test project for windows

Test Project for windows.
Comment 6 Dani Megert CLA 2018-08-09 08:53:19 EDT 
(In reply to Dawid Pakula from comment #5)
> Created attachment 275328 [details]
> Test project for windows
> 
> Test Project for windows.

Nope, this can't be imported. It gives:

Some projects cannot be imported because they already exist in the workspace or their project description file is corrupted
  Problems were encountered during import:
    | is an invalid character in resource name 'Test | Project'.
Comment 7 Eclipse Genie CLA 2020-10-07 01:14:41 EDT 
This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet.

If you have further information on the current state of the bug, please add it. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.

--
The automated Eclipse Genie.
Comment 8 Eclipse Genie CLA 2023-03-10 10:37:47 EST 
This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet.

If you have further information on the current state of the bug, please add it. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.

--
The automated Eclipse Genie.