[imp] "Check Spelling" button is shown even no driver

[Jan Schneider]

Zitat von Nethub Online <nethubonline at gmail.com>:

> Nethub Online <nethubonline <at> gmail.com> writes:
>
>>
>> Hi all,
>>
>> In old version of IMP, if there is no spell driver, the spell check button
>> will be hidden.
>>
>> However now it is still shown even no driver is configured. Can it be hidden
>> again in future version?
>>
>> Horde Version
>>
>>     5.1.1
>>
>> Horde Applications
>>
>>     Agora [Forums]: unknown
>>     Ansel [Photos]: unknown
>>     Chora [Version Control]: unknown
>>     Content: 2.0.2
>>     Gollem [File Manager]: H5 (3.0.1) (run tests)
>>     Hermes [Time Tracking]: unknown
>>     Imp [Mail]: H5 (6.1.2) (run tests)
>>     Ingo [Filters]: H5 (3.1.1) (run tests)
>>     Jonah [News]: unknown
>>     Klutz [Comics]: unknown
>>     Kolab [Kolab]: unknown
>>     Kronolith [Calendar]: H5 (4.1.1) (run tests)
>>     Luxor [X-Ref]: unknown
>>     Mnemo [Notes]: H5 (4.1.0)
>>     Nag [Tasks]: H5 (4.1.0)
>>     Passwd [Password]: H5 (5.0.0) (run tests)
>>     Sam [Spam]: unknown
>>     Sesha [Inventory]: unknown
>>     Timeobjects: 2.0.4
>>     Trean [Bookmarks]: H5 (1.0.2)
>>     Turba [Address Book]: H5 (4.1.0) (run tests)
>>     Ulaform [Forms]: unknown
>>     Vilma [Mail Admin]: unknown
>>     Whups [Tickets]: unknown
>>     Wicked [Wiki]: unknown
>>
>> Nethub Online Limited
>> http://www.nethub.com.hk
>>
>
> I modify the source templates/dynamic/compose.html.php as below, can any
> developer verify whether it is correct and put it in future release? Many
> thanks :)
>
>
> "imp/templates/dynamic/compose.html.php" line 32
>    <li class="horde-icon">
>     <?php echo $this->actionButton(array('icon' => 'Spellcheck', 'id' =>
> 'spellcheck', 'title' => _("Check Spelling"))) ?>
>    </li>
>
> change to
>
>
> <?php if (!empty($GLOBALS['conf']['spell']['driver'])): ?>
>    <li class="horde-icon">
>     <?php echo $this->actionButton(array('icon' => 'Spellcheck', 'id' =>
> 'spellcheck', 'title' => _("Check Spelling"))) ?>
>    </li>
> <?php endif; ?>

Looks correct.
-- 
Jan Schneider
The Horde Project
http://www.horde.org/