--- a/flys-client/src/main/java/de/intevation/flys/client/server/CollectionHelper.java	Tue Aug 28 11:51:12 2012 +0000
+++ b/flys-client/src/main/java/de/intevation/flys/client/server/CollectionHelper.java	Tue Aug 28 13:10:14 2012 +0000
@@ -607,7 +607,8 @@
 
 
     /**
-     *
+     * From a document, parse the settings for an output and create an
+     * OutputSettings object.
      */
     protected static Settings parseSettings(String outName, Element node) {
         OutputSettings set = new OutputSettings(outName);
@@ -620,7 +621,7 @@
 
         Element settings = (Element)elements.item(0);
 
-        // get the categories
+        // Get the categories
         NodeList catNodes = settings.getChildNodes();
         for (int i = 0; i < catNodes.getLength(); i++) {
             Element catNode = (Element)catNodes.item(i);
@@ -634,7 +635,7 @@
             // iterate through all properties or groups.
             List<Property> props = new ArrayList<Property> ();
             for (int j = 0; j < list.getLength(); j++) {
-                Property p = new PropertySetting();
+                Property p;
                 Element e = (Element)list.item(j);
                 if (e.hasChildNodes() &&
                     e.getFirstChild().getNodeType() != Node.TEXT_NODE) {
@@ -669,7 +670,7 @@
 
 
     /**
-     *
+     * From a property element create a Property object.
      */
     protected static Property parseSetting(Element property){
         NamedNodeMap attrMap = property.getAttributes();
@@ -720,7 +721,7 @@
         Element node,
         boolean outs
     ) {
-        logger.debug("AddArtifactServiceImpl.parseCollectionItem");
+        logger.debug("CollectionHelper.parseCollectionItem");
 
         if (node == null) {
             logger.debug("The node for parsing CollectionItem is null!");