Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 312880 - same alias for different namespace
Summary: same alias for different namespace
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: PDT (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: xu jiaxi CLA
QA Contact: Ilina Stefanova CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-05-14 05:05 EDT by Zhongwei Zhao CLA
Modified: 2020-05-14 11:18 EDT (History)
1 user (show)

See Also:


Attachments
patch (6.08 KB, patch)
2010-11-11 03:14 EST, 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 Zhongwei Zhao CLA 2010-05-14 05:05:35 EDT
php code:
<?php
namespace ns1\ns11\ns111 {
      class c1 {  }
}
namespace ns1\ns11\ns112 {
      class c1 {  }
}
namespace ns2\n21 {
      $a = new c1|      //| is cursor position
}

Then choose \ns1\ns11\ns111\c1,we will get:
<?php
namespace ns1\ns11\ns111 {
      class c1 {  }
}
namespace ns1\ns11\ns112 {
      class c1 {  }
}
namespace ns2\n21 {
      use ns1\ns11\ns111\c1;

      $a = new c1()
}
then type $a = new c1,we get:
namespace ns1\ns11\ns111 {
      class c1 {  }
}
namespace ns1\ns11\ns112 {
      class c1 {  }
}
namespace ns2\n21 {
      use ns1\ns11\ns111\c1;

      $a = new ns111\c1();
      $a = new c1|      //| is cursor position
}

Then choose \ns1\ns11\ns112\c1,we will get:
<?php
namespace ns1\ns11\ns111 {
      class c1 {  }
}
namespace ns1\ns11\ns112 {
      class c1 {  }
}
namespace ns2\n21 {
      use ns1\ns11\ns111\c1;

      use ns1\ns11\ns112\c1;

      $a = new c1();
      $a = new c1()
}

now 
      use ns1\ns11\ns111\c1;

      use ns1\ns11\ns112\c1;
have the same alias.


So we need use namespace alias or full qualified class name for \ns1\ns11\ns112\c1.
Comment 1 xu jiaxi CLA 2010-11-11 03:14:16 EST
Created attachment 182884 [details]
patch

I will try to attach unit tests later if possible
Comment 2 Zhongwei Zhao CLA 2011-05-23 01:58:25 EDT
applied the patch,thanks!
Comment 3 Ilina Stefanova CLA 2011-07-27 03:57:32 EDT
Verified