Letting the user change the value of the refresh for themselves is not a bad thing but neither is having a non-zero default for the entire installation.  I very much would have liked to be able to set a non-zero default for the page refresh for the installation, the options I have seen and tried cause breakage of the system and require each user to change the preference each time they login to the system.  I do agree that undocumented additions and variables are bad but it would be very nice if both options were available.<br>
<br>James<br><br><div class="gmail_quote">On Thu, Feb 12, 2009 at 12:27 PM, Kevin Falcone <span dir="ltr"><<a href="mailto:falcone@bestpractical.com">falcone@bestpractical.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="Ih2E3d"><br>
On Feb 12, 2009, at 3:19 PM, Jo Rhett wrote:<br>
<br>
>> On Feb 12, 2009, at 2:43 PM, Jo Rhett wrote:<br>
>>> This patch is adapted from something Ian posted for 3.6.4 adapted<br>
>>> for 3.8.2.  Jesse, please consider adopting this for the mainline as<br>
>>> it is a popular question and this patch allows the person to<br>
>>> override the default.<br>
><br>
> On Feb 12, 2009, at 12:12 PM, Kevin Falcone wrote:<br>
>> Since 3.8 added user level preferences, this really looks like<br>
>> something which is<br>
>> better handled as a user preference.   RT::Config has docs about<br>
>> adding items<br>
>> to %META and marking them as user overridable.  It would also want<br>
>> some<br>
>> changes and docs for RT_Config.pm because you could set a global<br>
>> default<br>
>> (default would be 0)<br>
><br>
> Are you saying that you guys will implement it that way for a later<br>
> version? Or is this a request for a patch to implement these things?<br>
><br>
<br>
</div>That was a suggestion of how to make the patch better and more<br>
compatible with 3.8<br>
<br>
I wouldn't apply any patch that adds two new Config variables without<br>
documenting<br>
them and adding defaults to the shipped config file.<br>
<font color="#888888"><br>
-kevin<br>
</font><div><div></div><div class="Wj3C7c">_______________________________________________<br>
<a href="http://lists.bestpractical.com/cgi-bin/mailman/listinfo/rt-users" target="_blank">http://lists.bestpractical.com/cgi-bin/mailman/listinfo/rt-users</a><br>
<br>
Community help: <a href="http://wiki.bestpractical.com" target="_blank">http://wiki.bestpractical.com</a><br>
Commercial support: <a href="mailto:sales@bestpractical.com">sales@bestpractical.com</a><br>
<br>
<br>
Discover RT's hidden secrets with RT Essentials from O'Reilly Media.<br>
Buy a copy at <a href="http://rtbook.bestpractical.com" target="_blank">http://rtbook.bestpractical.com</a><br>
</div></div></blockquote></div><br>