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

[Nethub Online]

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; ?>