[Rt-commit] rt branch 5.0/allow-unset-defaultcatalog created. rt-5.0.2-61-g4461a6e551

[BPS Git Server]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "rt".

The branch, 5.0/allow-unset-defaultcatalog has been created
        at  4461a6e551250248d8336df84a6eb205584b9fb4 (commit)

- Log -----------------------------------------------------------------
commit 4461a6e551250248d8336df84a6eb205584b9fb4
Author: Brian Conry <bconry at bestpractical.com>
Date:   Tue Feb 8 15:39:22 2022 -0600

    Allow DefualtCatalog to be unset in Web Interface
    
    Previously it was listed as needing to be set, even though the default
    is unset, so the selectbox for it in the Assets subsection of the System
    Configuration page was created without a "(no value)" option.  The
    result of this is that any change made in that subsection would set the
    DefaultCatalog to whichever catalog happened to be first in the list.
    
    It also prevented it from processing a "Reset" as the handling of the
    form data wouldn't accept having nothing selected.

diff --git a/lib/RT/Config.pm b/lib/RT/Config.pm
index 6286872212..ab97f2cad7 100644
--- a/lib/RT/Config.pm
+++ b/lib/RT/Config.pm
@@ -1962,6 +1962,7 @@ our %META;
         Widget          => '/Widgets/Form/Select',
         WidgetArguments => {
             Description => 'Default catalog',    #loc
+            Default     => 1, # allow user to unset it on EditConfig.html
             Callback    => sub {
                 my $ret = { Values => [], ValuesLabel => {} };
                 my $c = RT::Catalogs->new( $HTML::Mason::Commands::session{'CurrentUser'} );

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


hooks/post-receive
-- 
rt