Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 328211 - [CA]Wrong file path inserted in the include statement
Summary: [CA]Wrong file path inserted in the include statement
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: PDT (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: PHP Core CLA
QA Contact: Ilina Stefanova CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-10-20 02:57 EDT by xu jiaxi CLA
Modified: 2020-05-14 11:17 EDT (History)
2 users (show)

See Also:


Attachments
sample file (2.58 KB, application/octet-stream)
2010-10-20 02:58 EDT, xu jiaxi CLA
no flags Details
patch (4.90 KB, patch)
2010-10-20 03:08 EDT, xu jiaxi CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description xu jiaxi CLA 2010-10-20 02:57:00 EDT
Wrong file path will be inserted in include statement when the included file is in the referred project which use a folder as source folder

Steps to reproduce:
1. create two projects: test1(use a folder as source folder), test2
2. change the include path of test2, add test1 to it.
3. create a PHP file in test2, and type include '|' // CA here
4. file path inserted: include 'src/a/Ac.php';  // the file path contains the source folder "src"

Expected:
include 'a/Ac.php';
Comment 1 xu jiaxi CLA 2010-10-20 02:58:00 EDT
Created attachment 181251 [details]
sample file

import to workspace
Comment 2 xu jiaxi CLA 2010-10-20 03:08:56 EDT
Created attachment 181253 [details]
patch
Comment 3 Zhongwei Zhao CLA 2010-11-08 05:47:38 EST
great job!
now I see we could insert the file which is in project in include statement correctly,now there is no proposals for external source folder and referenced project(the third project referenced by the referenced project,not sure if it is needed) and so on of the referenced project.We could consider support them later.applied to head,thanks!
Comment 4 Zhongwei Zhao CLA 2011-05-11 05:02:12 EDT
change status
Comment 5 Kalin CLA 2011-07-28 08:41:08 EDT
Verified.
Closing...