Mercurial > dive4elements > river
comparison artifacts/src/main/java/org/dive4elements/river/collections/AttributeWriter.java @ 8325:02782ed6c8de
Reduce misspelled, superfluous logging.
| author | "Tom Gottfried <tom@intevation.de>" |
|---|---|
| date | Thu, 25 Sep 2014 12:21:47 +0200 |
| parents | e4606eae8ea5 |
| children | 5e38e2924c07 |
comparison
equal
deleted
inserted
replaced
| 8324:d6c03537970e | 8325:02782ed6c8de |
|---|---|
| 140 List<Facet> newOutFacets, | 140 List<Facet> newOutFacets, |
| 141 List<Facet> oldOutFacets | 141 List<Facet> oldOutFacets |
| 142 ) { | 142 ) { |
| 143 List<String> compatFacets = this.compatibilities.get(outputName); | 143 List<String> compatFacets = this.compatibilities.get(outputName); |
| 144 | 144 |
| 145 if (log.isDebugEnabled() && compatFacets != null) { | |
| 146 log.debug("Compabitle Facets:"); | |
| 147 for (String compatible: compatFacets) { | |
| 148 log.debug( "- " + compatible); | |
| 149 } | |
| 150 } | |
| 151 | |
| 152 try { | 145 try { |
| 153 writeFacets(outputName, newOutFacets, oldOutFacets, compatFacets); | 146 writeFacets(outputName, newOutFacets, oldOutFacets, compatFacets); |
| 154 } | 147 } |
| 155 catch (ArtifactDatabaseException ade) { | 148 catch (ArtifactDatabaseException ade) { |
| 156 log.error(ade, ade); | 149 log.error(ade, ade); |
| 189 } | 182 } |
| 190 | 183 |
| 191 for (Facet fac: newFacets) { | 184 for (Facet fac: newFacets) { |
| 192 ManagedFacet facet = (ManagedFacet) fac; | 185 ManagedFacet facet = (ManagedFacet) fac; |
| 193 | 186 |
| 194 if (debug) { | |
| 195 log.debug("Try to add Facet: " + facet.getName()); | |
| 196 } | |
| 197 | |
| 198 String bondage = facet.getBoundToOut(); | 187 String bondage = facet.getBoundToOut(); |
| 199 if (bondage != null && bondage.equals(outputName)) { | 188 if (bondage != null && bondage.equals(outputName)) { |
| 200 log.debug("Adding bound facet regardless of compatibility: " + | 189 log.debug("Adding bound facet regardless of compatibility: " + |
| 201 facet.getName()); | 190 facet.getName()); |
| 202 } else if (!compatibleFacets.contains(facet.getName())) { | 191 } else if (!compatibleFacets.contains(facet.getName())) { |
| 203 log.debug("Have incompatible facet, skip: " + facet.getName()); | 192 log.debug("Skip incompatible facet " + facet.getName()); |
| 204 continue; | 193 continue; |
| 205 } else if (facet.getBoundToOut() != null && | 194 } else if (facet.getBoundToOut() != null && |
| 206 !facet.getBoundToOut().equals(outputName)) { | 195 !facet.getBoundToOut().equals(outputName)) { |
| 207 log.debug("Skip facet " + facet.getName() + | 196 log.debug("Skip facet " + facet.getName() + |
| 208 " because it is bound to: " + facet.getBoundToOut()); | 197 " because it is bound to: " + facet.getBoundToOut()); |
| 209 continue; | 198 continue; |
| 210 } else { | 199 } else { |
| 211 log.debug("Facet is bound to: " + facet.getBoundToOut()); | 200 log.debug("Compatible facet " + facet.getName() + |
| 212 log.debug("Have compatible facet: " + facet.getName()); | 201 " is bound to: " + facet.getBoundToOut()); |
| 213 } | 202 } |
| 214 | 203 |
| 215 ManagedFacet picked = pickFacet(facet, oldFacets); | 204 ManagedFacet picked = pickFacet(facet, oldFacets); |
| 216 | 205 |
| 217 if (facet.equals(picked)) { | 206 if (facet.equals(picked)) { |