Yes, of course. Fixed in CVS.

Changes have been made in CVS for this ticket:

http://cvs.horde.org/diff.php/ingo/lib/Ingo.php?r1=1.106&r2=1.107&ty=u

I think there is an error in the applied patch.

I thought script icon would not be displayed if the auto_update is 
locked and set.

Right now is not displayed is only auto_update is locked and unset



Shouldn't the patch rather be:



@@ -395,7 +395,9 @@ class Ingo {

          if (in_array(INGO_STORAGE_ACTION_SPAM, 
$_SESSION['ingo']['script_categories'])) {

              $menu->add(Horde::applicationUrl('spam.php'), 
_("S_pam"), 'spam.png');

          }

-        if ($_SESSION['ingo']['script_generate']) {

+        if ($_SESSION['ingo']['script_generate'] &&

+            !($GLOBALS['prefs']->isLocked('auto_update') &&

+             $GLOBALS['prefs']->getValue('auto_update'))) {

              $menu->add(Horde::applicationUrl('script.php'), 
_("_Script"), 'script.png');

          }

          if (!empty($GLOBALS['ingo_shares']) && 
empty($GLOBALS['conf']['share']['no_sharing'])) {

Changes have been made in CVS for this ticket:

http://cvs.horde.org/diff.php/ingo/docs/CHANGES?r1=1.55.2.106&r2=1.55.2.107&ty=u
http://cvs.horde.org/diff.php/ingo/lib/Ingo.php?r1=1.69.6.19&r2=1.69.6.20&ty=u

Easy enough. Committed for Ingo 1.2.1.

Changes have been made in CVS for this ticket:

http://cvs.horde.org/diff.php/ingo/docs/CHANGES?r1=1.201&r2=1.202&ty=u
http://cvs.horde.org/diff.php/ingo/lib/Ingo.php?r1=1.105&r2=1.106&ty=u

We already locked and set this option, so that would indeed make sense 
for our setup.

If you want to remove that icon, you have to make sure that the user 
preference to enable scripts automatically is set and locked. We could 
hide the button under that condition. Does that make sense?

In order to not confuse our users, we would like to hide the script 
icon displayed in the top menu bar in ingo.



Would it be possible to add an option for this ?