[java-metadata-aggregator COMMIT] in /trunk/aggregator-pipeline/src/main/java/net/shibboleth/metadata: FirstItemIdIte...

[noreply at shibboleth.net]

Author: lajoie
Date: Sun Oct 30 13:13:08 2011
New Revision: 157

URL: http://svn.shibboleth.net/view/java-metadata-aggregator?rev=157&view=rev
Log:
correct inconsistent quoting of item IDs
fix uninitialized unidentified item ID string
shorten up some log messages

Modified:
    trunk/aggregator-pipeline/src/main/java/net/shibboleth/metadata/FirstItemIdItemIdentificationStrategy.java
    trunk/aggregator-pipeline/src/main/java/net/shibboleth/metadata/pipeline/StatusMetadataLoggingStage.java

Modified: trunk/aggregator-pipeline/src/main/java/net/shibboleth/metadata/FirstItemIdItemIdentificationStrategy.java
URL: http://svn.shibboleth.net/view/java-metadata-aggregator/trunk/aggregator-pipeline/src/main/java/net/shibboleth/metadata/FirstItemIdItemIdentificationStrategy.java?rev=157&r1=156&r2=157&view=diff
==============================================================================
--- trunk/aggregator-pipeline/src/main/java/net/shibboleth/metadata/FirstItemIdItemIdentificationStrategy.java (original)
+++ trunk/aggregator-pipeline/src/main/java/net/shibboleth/metadata/FirstItemIdItemIdentificationStrategy.java Sun Oct 30 13:13:08 2011
@@ -29,7 +29,7 @@
 public class FirstItemIdItemIdentificationStrategy implements ItemIdentificationStrategy {
 
     /** Identifier to use if an {@link Item} does have an {@link ItemId}. Default value: unidentified */
-    private String noItemIdIdentifier;
+    private String noItemIdIdentifier = "unidentified";
 
     /**
      * Gets the identifier to use if an {@link Item} does have an {@link ItemId}.

Modified: trunk/aggregator-pipeline/src/main/java/net/shibboleth/metadata/pipeline/StatusMetadataLoggingStage.java
URL: http://svn.shibboleth.net/view/java-metadata-aggregator/trunk/aggregator-pipeline/src/main/java/net/shibboleth/metadata/pipeline/StatusMetadataLoggingStage.java?rev=157&r1=156&r2=157&view=diff
==============================================================================
--- trunk/aggregator-pipeline/src/main/java/net/shibboleth/metadata/pipeline/StatusMetadataLoggingStage.java (original)
+++ trunk/aggregator-pipeline/src/main/java/net/shibboleth/metadata/pipeline/StatusMetadataLoggingStage.java Sun Oct 30 13:13:08 2011
@@ -50,23 +50,23 @@
         if (statuses != null && !statuses.isEmpty() && log.isInfoEnabled()) {
             log.info("Item {} was marked with the following Info status messages", itemId);
             for (StatusMetadata status : statuses) {
-                log.info("    Component {} reported: {}", status.getComponentId(), status.getStatusMessage());
+                log.info("    {} reported: {}", status.getComponentId(), status.getStatusMessage());
             }
         }
 
         statuses = (List<StatusMetadata>) matchingMetadata.get(WarningStatus.class);
         if (statuses != null && !statuses.isEmpty() && log.isWarnEnabled()) {
-            log.warn("Item '{}' was marked with the following Warning status messages", itemId);
+            log.warn("Item {} was marked with the following Warning status messages", itemId);
             for (StatusMetadata status : statuses) {
-                log.warn("    Component {} reported: {}", status.getComponentId(), status.getStatusMessage());
+                log.warn("    {} reported: {}", status.getComponentId(), status.getStatusMessage());
             }
         }
 
         statuses = (List<StatusMetadata>) matchingMetadata.get(ErrorStatus.class);
         if (statuses != null && !statuses.isEmpty() && log.isErrorEnabled()) {
-            log.error("Item '{}' was marked with the following Error status messages", itemId);
+            log.error("Item {} was marked with the following Error status messages", itemId);
             for (StatusMetadata status : statuses) {
-                log.error("    Component {} reported: {}", status.getComponentId(), status.getStatusMessage());
+                log.error("    {} reported: {}", status.getComponentId(), status.getStatusMessage());
             }
         }
     }