Ignore:
Timestamp:
06/04/10 12:15:30 (10 years ago)
Author:
mnagni
Message:

Not always the returned status message pointed to the bundle message.
Improved the general logging structure
Artifact promoted to version 1.1.8

Location:
TI01-discovery-API/trunk/MEDINDiscoveryService
Files:
7 edited

Legend:

Unmodified
Added
Removed
  • TI01-discovery-API/trunk/MEDINDiscoveryService/pom.xml

    r6773 r6799  
    44        <groupId>ndg.services.discovery</groupId> 
    55        <artifactId>MedinDiscoveryService</artifactId> 
    6         <version>1.1.7</version> 
     6        <version>1.1.8</version> 
    77        <packaging>aar</packaging> 
    88 
  • TI01-discovery-API/trunk/MEDINDiscoveryService/src/main/java/ndg/common/NdgLogger.java

    r6678 r6799  
    11package ndg.common; 
     2 
     3import ndg.common.exception.NdgException; 
    24 
    35import org.apache.log4j.Level; 
     
    2224                this.formatter = formatter; 
    2325        } 
     26                 
     27        public String getBundleMessageAndDoLog(NdgException e, Level priority){ 
     28                logMessage(e, priority); 
     29                return getBundleMessage(e); 
     30        }        
    2431         
    25         public void logMessage(NdgMessage msg, Level priority) { 
    26                 if (log.isEnabledFor(priority)) 
    27                         log.log(priority, formatter.getFormattedText(msg)); 
     32        public String getBundleMessageAndDoLog(NdgMessage msg, Level priority){ 
     33                logMessage(msg, priority); 
     34                return getBundleMessage(msg); 
     35        }        
     36         
     37        public String getBundleMessage(NdgException e){ 
     38                String ret = e.getMessage(); 
     39                if (e.getBundlMessage() != null) 
     40                        ret = formatter.getFormattedText(e.getBundlMessage()); 
     41                return ret; 
     42        }        
     43         
     44        public String getBundleMessage(NdgMessage msg){ 
     45                return formatter.getFormattedText(msg); 
     46        }        
     47         
     48        public void logMessage(NdgException e, Level priority) { 
     49                doLog(getBundleMessage(e), priority, e); 
     50        } 
     51         
     52        public void logMessage(NdgMessage e, Level priority) { 
     53                logMsg(new NdgException(e), priority); 
     54        } 
     55         
     56        public void logMessage(Throwable e, Level priority) { 
     57                doLog(null, priority, e); 
     58        } 
     59         
     60        public void logMessage(String msg, Level priority, Throwable e) { 
     61                doLog(null, priority, new NdgException(msg, e)); 
    2862        } 
    2963         
    3064        public void logMessage(String msg, Level priority) { 
    31                 log.log(priority, formatter.getFormattedText(msg)); 
     65                doLog(msg, priority); 
    3266        } 
    3367         
    34         public void logMessage(NdgMessage msg, Level priority, Throwable e) { 
    35                 if (log.isEnabledFor(priority)) 
    36                         log.log(priority, formatter.getFormattedText(msg), e); 
     68        private void logMsg(Throwable e, Level priority) { 
     69                doLog(null, priority, e); 
    3770        } 
    3871         
    39         public void logMessage(String msg, Level priority, Throwable e) { 
     72        private void doLog(String msg, Level priority) { 
    4073                if (log.isEnabledFor(priority)) 
    41                         log.log(priority, formatter.getFormattedText(msg), e); 
     74                        log.log(priority, msg); 
     75        } 
     76         
     77        private void doLog(String msg, Level priority, Throwable e) { 
     78                if (log.isEnabledFor(priority)) 
     79                        log.log(priority, msg, e); 
    4280        } 
    4381} 
  • TI01-discovery-API/trunk/MEDINDiscoveryService/src/main/java/ndg/common/exception/NdgException.java

    r6504 r6799  
    4444                this.bundleMessage = bundleMessage; 
    4545        } 
     46         
     47        public NdgMessage getBundlMessage() { 
     48                return bundleMessage; 
     49        } 
    4650} 
  • TI01-discovery-API/trunk/MEDINDiscoveryService/src/main/java/ndg/common/exception/NdgSQLException.java

    r6511 r6799  
    4040 
    4141        public NdgSQLException(NdgMessage bundleMessage, Throwable th) { 
    42                 super(th); 
     42                super(bundleMessage, th); 
    4343        } 
    4444} 
  • TI01-discovery-API/trunk/MEDINDiscoveryService/src/main/java/ndg/services/discovery/DiscoveryServiceMEDINSkeleton.java

    r6767 r6799  
    1313import ndg.common.NdgLogger; 
    1414import ndg.common.NdgMessage; 
     15import ndg.common.exception.NdgException; 
    1516import ndg.common.exception.NdgSQLException; 
    1617import ndg.common.exception.ResourceNotAvailable; 
     
    4445import ndg.services.discovery.medin.schema.SearchType; 
    4546import ndg.services.discovery.medin.schema.SimpleDocument1; 
    46 import ndg.services.discovery.medin.schema.SpatialType; 
    47 import ndg.services.discovery.medin.schema.TemporalType; 
    4847import ndg.services.discovery.medin.schema.TermSearchType; 
    4948import ndg.services.discovery.medin.schema.SearchType.SearchCriteria; 
     
    133132                                status = true; 
    134133                        } catch (NdgSQLException e) { 
    135                                 logHelper.logMessage(e.getMessage(), Level.ERROR, e); 
    136                                 statusMsg = e.getMessage(); 
     134                                statusMsg = logHelper.getBundleMessageAndDoLog(e, Level.ERROR); 
    137135                        } catch (ResourceNotAvailable e) { 
    138                                 logHelper.logMessage(e.getMessage(), Level.ERROR, e); 
    139                                 statusMsg = e.getMessage(); 
     136                                statusMsg = logHelper.getBundleMessageAndDoLog(e, Level.ERROR); 
    140137                        } finally { 
    141138                                responseContent.setHits(agent.getHits()); 
     
    146143                                                msg = new NdgMessage("search.no.results", new Object[] {}); 
    147144                                        } 
    148                                         statusMsg = DiscoveryBasic.getInstance().getFormatter().getFormattedText(msg); 
     145                                        statusMsg = logHelper.getBundleMessageAndDoLog(msg, Level.DEBUG); 
    149146                                } 
    150147                        } 
     
    173170                 
    174171                boolean status = false; 
    175                 String statusMessage = null; 
     172                String statusMsg = null; 
    176173                PresentAgent agent = new PresentAgent(); 
    177174                List<String> documentsId = new ArrayList<String>(); 
     
    188185                        status = true; 
    189186                } catch (NdgSQLException e) { 
    190                         statusMessage = e.getMessage(); 
     187                        statusMsg = logHelper.getBundleMessageAndDoLog(e, Level.ERROR); 
    191188                } catch (ResourceNotAvailable e) { 
    192                         statusMessage = e.getMessage(); 
     189                        statusMsg = logHelper.getBundleMessageAndDoLog(e, Level.ERROR); 
    193190                } finally { 
    194                         if (statusMessage == null) { 
     191                        if (statusMsg == null) { 
    195192                                NdgMessage msg = new NdgMessage("present.successful", new Object[] {}); 
    196                                 statusMessage = DiscoveryBasic.getInstance().getFormatter().getFormattedText(msg); 
     193                                statusMsg = logHelper.getBundleMessageAndDoLog(msg, Level.DEBUG); 
    197194                        } 
    198195                        responseContent.setStatus(status); 
    199                         responseContent.setStatusMessage(statusMessage); 
     196                        responseContent.setStatusMessage(statusMsg); 
    200197                        if (conn != null) 
    201198                                try { 
     
    409406                                newOMT.setMetricValue(value); 
    410407                        } catch (IllegalArgumentException e) { 
    411                                 logHelper.logMessage(msg, Level.ERROR, e); 
     408                                logHelper.logMessage(new NdgException(msg, e), Level.ERROR); 
    412409                        } catch (IllegalAccessException e) { 
    413                                 logHelper.logMessage(msg, Level.ERROR, e); 
     410                                logHelper.logMessage(new NdgException(msg, e), Level.ERROR); 
    414411                        } catch (InvocationTargetException e) { 
    415                                 logHelper.logMessage(msg, Level.ERROR, e); 
     412                                logHelper.logMessage(new NdgException(msg, e), Level.ERROR); 
    416413                        } 
    417414                } 
  • TI01-discovery-API/trunk/MEDINDiscoveryService/src/main/java/ndg/services/discovery/model/CommonMethods.java

    r6773 r6799  
    203203                        closeStatement(stat, rs); 
    204204                        NdgMessage msg = new NdgMessage("sql.execution.error", 
    205  
    206205                        new Object[] { sqlQuery }); 
    207                         throw new NdgSQLException(msg); 
     206                        throw new NdgSQLException(msg,e); 
    208207                } 
    209208 
  • TI01-discovery-API/trunk/MEDINDiscoveryService/src/main/java/ndg/services/discovery/model/PresentAgent.java

    r6767 r6799  
    1515import ndg.common.NdgLogger; 
    1616import ndg.common.NdgMessage; 
     17import ndg.common.exception.NdgException; 
    1718import ndg.common.exception.NdgSQLException; 
    1819import ndg.common.exception.ResourceNotAvailable; 
     
    183184                                        } 
    184185                                } catch (IllegalArgumentException e) { 
    185                                         logHelper.logMessage(msg, Level.ERROR, e); 
     186                                        logHelper.logMessage(new NdgException(msg, e), Level.ERROR); 
    186187                                } catch (IllegalAccessException e) { 
    187                                         logHelper.logMessage(msg, Level.ERROR, e); 
     188                                        logHelper.logMessage(new NdgException(msg, e), Level.ERROR); 
    188189                                } catch (InvocationTargetException e) { 
    189                                         logHelper.logMessage(msg, Level.ERROR, e); 
     190                                        logHelper.logMessage(new NdgException(msg, e), Level.ERROR); 
    190191                                } 
    191192                } 
Note: See TracChangeset for help on using the changeset viewer.