Tickets :: [#3650] compose_spellcheck broken?

Summary	compose_spellcheck broken?
Queue	IMP
Queue Version	4.1
Type	Bug
State	Duplicate
Priority	1. Low
Owners
Requester	matthew.dunham (at) ic (dot) ucsb (dot) edu
Created	03/18/2006 (7130 days ago)
Due
Updated	03/18/2006 (7130 days ago)
Assigned
Resolved	03/18/2006 (7130 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	No

03/18/2006 01:20:00 AM	Michael Slusarz	Comment #3 State ⇒ Duplicate	Reply to this comment
Duplicate of ~~Bug 3589~~.

03/18/2006 12:41:57 AM	matthew (dot) dunham (at) ic (dot) ucsb (dot) edu	Comment #2	Reply to this comment
the 4.1 compose.php _getActionID() function appears to skip the compose spellcheck feature. the first codeblock in _getActionID() rewrites $aid and returns control immediately thereafter. the next codeblock is where the auto spellcheck is initiated, but execution never reaches this block when aid = $send_message oops. meant to write "when $aid = send_message" [Show Quoted Text - 12 lines][Hide Quoted Text] function _getActionID() { if ($aid = Util::getFormData('actionID')) { return str_replace('btn_', '', $aid); } if (Util::getFormData('btn_send_message')) { /* Alter the 'Send Message' action if the "Spell check before sending * message?" preference has been enabled. */ ....

03/18/2006 12:40:29 AM	matthew (dot) dunham (at) ic (dot) ucsb (dot) edu	Comment #1 State ⇒ Unconfirmed Priority ⇒ 1. Low Type ⇒ Bug Summary ⇒ compose_spellcheck broken? Queue ⇒ IMP	Reply to this comment
the 4.1 compose.php _getActionID() function appears to skip the compose spellcheck feature. the first codeblock in _getActionID() rewrites $aid and returns control immediately thereafter. the next codeblock is where the auto spellcheck is initiated, but execution never reaches this block when aid = $send_message function _getActionID() { if ($aid = Util::getFormData('actionID')) { return str_replace('btn_', '', $aid); } if (Util::getFormData('btn_send_message')) { /* Alter the 'Send Message' action if the "Spell check before sending * message?" preference has been enabled. */ ....