[Rt-commit] rt branch, 4.0/delete-default-summary-rows-pref, created. rt-4.0.6-216-g82c5217

Alex Vandiver alexmv at bestpractical.com
Thu Jun 21 12:56:03 EDT 2012


The branch, 4.0/delete-default-summary-rows-pref has been created
        at  82c5217789ac0f5d347602b5bad1c40c55072ebc (commit)

- Log -----------------------------------------------------------------
commit 82c5217789ac0f5d347602b5bad1c40c55072ebc
Author: Alex Vandiver <alexmv at bestpractical.com>
Date:   Thu Jun 21 12:42:08 2012 -0400

    Remove unimplemented user override of DefaultSummaryRows
    
    479a311 added DefaultSummaryRows to the lsit of user-overridable
    options, but failed to change any of the calls to
    ->Get('DefaultSummaryRows') to respect the user's setting.  Remove the
    value from the list of user-configurable settings, as it would merely
    serve as a default for the per-user preference already found on the
    /Prefs/MyRT.html page.  Having a per-user configurable default setting
    for a per-user configurable preference makes no sense.

diff --git a/lib/RT/Config.pm b/lib/RT/Config.pm
index d26624f..cbf0cc8 100644
--- a/lib/RT/Config.pm
+++ b/lib/RT/Config.pm
@@ -302,15 +302,6 @@ our %META = (
     },
 
     # User overridable options for RT at a glance
-    DefaultSummaryRows => {
-        Section         => 'RT at a glance',    #loc
-        Overridable     => 1,
-        SortOrder       => 1,
-        Widget          => '/Widgets/Form/Integer',
-        WidgetArguments => {
-            Description => 'Number of search results',    #loc
-        },
-    },
     HomePageRefreshInterval => {
         Section         => 'RT at a glance',                       #loc
         Overridable     => 1,

-----------------------------------------------------------------------


More information about the Rt-commit mailing list