This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 189814 - Allow the Spellchecker to also spell check comments
Summary: Allow the Spellchecker to also spell check comments
Status: RESOLVED FIXED
Alias: None
Product: php
Classification: Unclassified
Component: Editor (show other bugs)
Version: 6.x
Hardware: All All
: P3 normal with 16 votes (vote)
Assignee: Tomas Mysik
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-08-24 17:31 UTC by Chowarmaan
Modified: 2016-02-14 04:12 UTC (History)
3 users (show)

See Also:
Issue Type: ENHANCEMENT
Exception Reporter:

Attachments
patch (51.22 KB, patch)
2016-02-10 11:17 UTC, junichi11 		Details | Diff
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 Chowarmaan 2010-08-24 17:31:03 UTC 
The ability to spell check HTML and the PHP code is good for syntax matching, but it would be very helpful to also be able to spell check comments.

The reason is that a lot of developers include good comments, but spelling mistakes may change the meaning of the comment.
Comment 1 Filip Zamboj 2010-09-15 12:33:05 UTC 
batch reassigning
Comment 2 zeleznypa 2015-02-10 08:18:47 UTC 
Please, add support for checking phpdoc commnents like // or /** 
It will be very usefull.
Comment 3 junichi11 2016-02-10 11:17:23 UTC 
Created attachment 158447 [details]
patch

I've created a patch.
Comment 4 junichi11 2016-02-10 11:20:21 UTC 
It's available in the only PHPDoc blocks (/** */).

Tomas, could you please review it when you can make time?

Thanks!
Comment 5 Tomas Mysik 2016-02-10 13:37:55 UTC 
Thanks a lot for your work, Junichi! Will review all your patches this week.

Thanks!
Comment 6 Tomas Mysik 2016-02-11 13:09:44 UTC 
Patch applied with slight modifications. Junichi, please review it if you are interested.

Thanks a lot for your contribution!

http://hg.netbeans.org/web-main/rev/9e43af72a621
Comment 7 junichi11 2016-02-11 15:05:11 UTC 
No problem. Thanks a lot for reviewing and adjusting it!
Comment 8 Quality Engineering 2016-02-12 02:36:36 UTC 
Integrated into 'main-silver', will be available in build *201602120002* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)

Changeset: http://hg.netbeans.org/main-silver/rev/9e43af72a621
User: Tomas Mysik <tmysik@netbeans.org>
Log: #189814 - Allow the Spellchecker to also spell check comments

Patch contributed by junichi11@netbeans.org. Patch has been slighly improved.
Comment 9 dicexq 2016-02-13 10:16:44 UTC 
There seems to be an issue with this code

/**
 * @since       2016-02-12
 * 
 * Test
 */

Every keypress or click will cause `A org.openide.util.RequestProcessor$FastItem exception`.

This is caused by the date.

Trace: http://pastebin.com/4upA0TBD
Comment 10 junichi11 2016-02-14 04:12:16 UTC 
(In reply to dicexq from comment #9)
> There seems to be an issue with this code
> 
> /**
>  * @since       2016-02-12
>  * 
>  * Test
>  */
> 
> Every keypress or click will cause `A
> org.openide.util.RequestProcessor$FastItem exception`.
> 
> This is caused by the date.
> 
> Trace: http://pastebin.com/4upA0TBD

@dicexq :
Thanks for reporting it! I submit it as a new issue[1]. I'll fix it.
If you find issues for spellchecker support, please submit them to the NetBeans bugzilla. (please add me to CC because I can notice/look at it.)

[1] https://netbeans.org/bugzilla/show_bug.cgi?id=257977

Thanks!