Bug 186416 - Servlet doesn't spellcheck
Summary: Servlet doesn't spellcheck
Status: NEW
Alias: None
Product: WTP Java EE Tools
Classification: WebTools
Component: jst.servlet (show other bugs)
Version: 2.0   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 4.0   Edit
Assignee: Chuck Bridgham CLA
QA Contact: Chuck Bridgham CLA
URL:
Whiteboard:
Keywords: bugday
Depends on: 198152
Blocks:
  Show dependency tree
 
Reported: 2007-05-10 13:29 EDT by Tim deBoer CLA
Modified: 2010-05-27 11:24 EDT (History)
4 users (show)

See Also:

Attachments
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 Tim deBoer CLA 2007-05-10 13:29:06 EDT 
With the new platform spell-checking feature on by default, the new servlet wizard creates a class with a spelling error in it - the word 'Servlet' in the class level javadoc. We should make sure that Servlet gets added into the dictionary somehow so that it doesn't show up as a typo in this case or user created code & javadoc.

I assume this happens elsewhere for words like 'EJB' and we should fix those as well. Opening here just because this is the first one I saw.
Comment 1 Nitin Dahyabhai CLA 2007-06-14 02:20:41 EDT 
Perhaps we need to ask the platform to allow contributions of "safe" words by plugins?
Comment 2 David Carver CLA 2007-07-27 09:43:45 EDT 
Wouldn't this require creating a whole new dictionary, that had the servlet spelling merged.  I don't think there is the ability to define more than one dictionary to use to check at a time.  In fact the Dictionary only allows for one dictionary to be used, so a user would need to use a custom dictionary that contained the original and the new words that needs to be added.

What is really needed is the ability with in the Spell checking program to be able to add supplemental dictionaries that are scanned in addition to the main one specified.
Comment 3 Nitin Dahyabhai CLA 2007-07-27 12:47:47 EDT 
That sounds like a very useful RFE for the main Platform :)
Comment 4 David Carver CLA 2007-07-27 14:57:45 EDT 
I'll do some checking and see if there is an open bug for this already or not.  If not, I'll open an enhancement request, and put a link to the bug here or update the dependency.
Comment 5 David Carver CLA 2007-07-27 15:05:27 EDT 
I've created bug 198152, in the platform forum to help address this issue.
Comment 6 Tim deBoer CLA 2008-06-19 09:02:36 EDT 
David - Why did you assign this to me (the originator)? Any chance you just confused 'servlet' with 'server'?
Comment 7 David Williams CLA 2008-06-19 09:27:52 EDT 
(In reply to comment #6)
> David - Why did you assign this to me (the originator)? Any chance you just
> confused 'servlet' with 'server'?
> 

it was only because I didn't know who to assign it to, and figured you would. 

(I was just trying to do some mass updates of some bugs improperly triaged, so didn't have much time to spend on each). 

As always, fee free to assign to more appropriate person. But we shouldn't leave a "targetted" bug sitting in 'inbox'.
Comment 8 Tim deBoer CLA 2008-06-19 09:57:00 EDT 
Ok, assigning to Chuck as lead of this component.