Bug 495342

Summary: org.eclipse.jdt.internal.corext.refactoring.structure.PullUpRefactoringProcessor.checkAccesses(IProgressMonitor) doesn't check constructors for accessibility
Product: [Eclipse Project] JDT Reporter: Raffi Khatchadourian <raffi.khatchadourian>
Component: UIAssignee: JDT-UI-Inbox <jdt-ui-inbox>
Status: NEW --- QA Contact:
Severity: enhancement    
Priority: P3 CC: jarthana, raffi.khatchadourian
Version: 3.1   
Target Milestone: ---   
Hardware: Macintosh   
OS: Mac OS X   
Whiteboard:

Description Raffi Khatchadourian CLA 2016-06-02 15:46:37 EDT 
org.eclipse.jdt.internal.corext.refactoring.structure.PullUpRefactoringProcessor.checkAccesses(IProgressMonitor) checks accessed methods of the pulled up members but this method actually ignores constructors. Particularly, org.eclipse.jdt.internal.corext.refactoring.structure.PullUpRefactoringProcessor.checkAccessedMethods(IProgressMonitor, ITypeHierarchy) uses org.eclipse.jdt.internal.corext.refactoring.structure.ReferenceFinderUtil.getMethodsReferencedIn(IJavaElement[], WorkingCopyOwner, IProgressMonitor) to find methods accessed by the pulled up members. That method doesn't return referenced constructors.


-- Configuration Details --
Product: Eclipse 4.5.2.20160218-0600 (org.eclipse.epp.package.committers.product)
Installed Features:
 org.eclipse.jdt 3.11.2.v20160212-1500
Comment 1 Jay Arthanareeswaran CLA 2016-06-03 11:05:15 EDT 
Moving to UI