Bug 171802 - [javadoc][select] F3 does not work on method which have deprecated type as argument
Summary: [javadoc][select] F3 does not work on method which have deprecated type as ar...
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 3.3   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.3 M6   Edit
Assignee: Eric Jodet CLA
QA Contact:
URL:
Whiteboard:
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2007-01-26 10:04 EST by Frederic Fusier CLA
Modified: 2007-06-06 13:39 EDT (History)
1 user (show)

See Also:

Attachments
Proposed fix (965 bytes, patch)
2007-02-07 04:01 EST, Eric Jodet CLA 		no flags Details | Diff
patch + test case (3.46 KB, patch)
2007-02-20 12:04 EST, Eric Jodet CLA 		no flags Details | Diff
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 Frederic Fusier CLA 2007-01-26 10:04:10 EST 
Having org.eclipse.jdt.core project in my workspace, I open SelectionEngine and go to the deprecated method:
 	public void searchAllTypeNames(
		IWorkspace workspace,
		final char[] packageName, 
		final char[] typeName,
		final int matchMode, 
		final boolean isCaseSensitive,
		int searchFor, 
		IJavaSearchScope scope, 
		final ITypeNameRequestor nameRequestor,
		int waitingPolicy,
		IProgressMonitor progressMonitor)  throws JavaModelException
In this method Javadoc, there's a @deprecated tag telling to use another searchTypeNames method instead. Trying to select this method fails with a bip and 'Current text selection does not resolve to a Java element'...
Comment 1 Eric Jodet CLA 2007-02-07 04:01:20 EST 
Created attachment 58430 [details]
Proposed fix

This patch fixes the issue and deprecated method is correctly resolved.
Comment 2 Eric Jodet CLA 2007-02-20 12:04:30 EST 
Created attachment 59391 [details]
patch + test case

Contains the same fix
+ corresponding test case: SelectionJavadocModelTests::testBug171802()
Comment 3 Frederic Fusier CLA 2007-02-20 12:42:39 EST 
Thanks Eric => released for 3.3 M6 in HEAD stream.

P.S. Comment 0 should read SearchEngine instead of SelectionEngine...
Comment 4 Olivier Thomann CLA 2007-03-20 09:04:13 EDT 
Verified for 3.3 M6 using build I20070320-0010