psmith      2003/09/14 23:38:35

  Modified:    src/java/org/apache/log4j/chainsaw LogUI.java
  Log:
  Fixed bug where a settings Listener was never given a chance to load settings
  and so the previous configs used were not presented
  in the No Receivers Warning dialog.
  
  Revision  Changes    Path
  1.28      +25 -23    jakarta-log4j/src/java/org/apache/log4j/chainsaw/LogUI.java
  
  Index: LogUI.java
  ===================================================================
  RCS file: /home/cvs/jakarta-log4j/src/java/org/apache/log4j/chainsaw/LogUI.java,v
  retrieving revision 1.27
  retrieving revision 1.28
  diff -u -r1.27 -r1.28
  --- LogUI.java        14 Sep 2003 23:47:26 -0000      1.27
  +++ LogUI.java        15 Sep 2003 06:38:35 -0000      1.28
  @@ -589,33 +589,34 @@
        final NoReceiversWarningPanel noReceiversWarningPanel =
         new NoReceiversWarningPanel();
   
  +     final SettingsListener sl = new SettingsListener() {
  +       public void loadSettings(LoadSettingsEvent event) {
  +             int size = event.asInt("SavedConfigs.Size");
  +             Object[] configs = new Object[size];
  +
  +             for (int i = 0; i < size; i++) {
  +               configs[i] = event.getSetting("SavedConfigs." + i);
  +             }
  +
  +             noReceiversWarningPanel.getModel().setRememberedConfigs(configs);
  +       }
  +
  +       public void saveSettings(SaveSettingsEvent event) {
  +             Object[] configs =
  +               noReceiversWarningPanel.getModel().getRememberedConfigs();
  +             event.saveSetting("SavedConfigs.Size", configs.length);
  +
  +             for (int i = 0; i < configs.length; i++) {
  +               event.saveSetting("SavedConfigs." + i, configs[i].toString());
  +             }
  +       }
  +     };
      /**
        * This listener sets up the NoReciversWarningPanel and
        * loads saves the configs/logfiles
        */
  -   getSettingsManager().addSettingsListener(
  -      new SettingsListener() {
  -        public void loadSettings(LoadSettingsEvent event) {
  -              int size = event.asInt("SavedConfigs.Size");
  -              Object[] configs = new Object[size];
  -
  -              for (int i = 0; i < size; i++) {
  -                configs[i] = event.getSetting("SavedConfigs." + i);
  -              }
  -
  -              noReceiversWarningPanel.getModel().setRememberedConfigs(configs);
  -        }
  -
  -        public void saveSettings(SaveSettingsEvent event) {
  -              Object[] configs =
  -                noReceiversWarningPanel.getModel().getRememberedConfigs();
  -              event.saveSetting("SavedConfigs.Size", configs.length);
  -
  -              for (int i = 0; i < configs.length; i++) {
  -                event.saveSetting("SavedConfigs." + i, configs[i].toString());
  -              }
  -        }
  -      });
  +   getSettingsManager().addSettingsListener(sl);
  +   getSettingsManager().configure(sl);
   
       SwingUtilities.invokeLater(
         new Runnable() {
  @@ -690,6 +691,7 @@
                       } catch (Exception e) {
                         LogLog.error("Error initializing Log4j", e);
                       }
  +                    
LogManager.getLoggerRepository().getRootLogger().addAppender(handler);
   
                       receiversPanel.updateReceiverTreeInDispatchThread();
                     }
  
  
  

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to