Simplified code

git-svn-id: https://svn.apache.org/repos/asf/maven/plugins/trunk@1674106 13f79535-47bb-0310-9956-ffa450edef68

Author: krosenvold

src/main/java/org/apache/maven/plugin/assembly/archive/task/AddDependencySetsTask.java

@@ -134,20 +134,15 @@ void addDependencySet( final DependencySet dependencySet, final Archiver archive
 
         final Set<Artifact> dependencyArtifacts = resolveDependencyArtifacts( dependencySet );
 
-        final UnpackOptions opts = dependencySet.getUnpackOptions();
-        if ( dependencySet.isUnpack() && opts != null && ( opts.isFiltered() || opts.getLineEnding() != null ) )
-        {
-            // find out if we can just ditch this empty block
-        }
-        else if ( dependencyArtifacts.size() > 1 )
+        if ( !unpackTransformsContent( dependencySet ) && dependencyArtifacts.size() > 1 )
         {
             checkMultiArtifactOutputConfig( dependencySet );
         }
 
         logger.debug( "Adding " + dependencyArtifacts.size() + " dependency artifacts." );
 
         InputStreamTransformer fileSetTransformers =
-            dependencySet.isUnpack() && dependencySet.getUnpackOptions() != null
+            isUnpackWithOptions( dependencySet )
                 ? ReaderFormatter.getFileSetTransformers( configSource, dependencySet.getUnpackOptions().isFiltered(),
                                                           dependencySet.getUnpackOptions().getLineEnding() )
                 : null;
@@ -181,6 +176,21 @@ else if ( dependencyArtifacts.size() > 1 )
         }
     }
 
+    private boolean isUnpackWithOptions( DependencySet dependencySet )
+    {
+        return dependencySet.isUnpack() && dependencySet.getUnpackOptions() != null;
+    }
+
+    private boolean unpackTransformsContent( DependencySet dependencySet )
+    {
+        return isUnpackWithOptions( dependencySet ) && isConentModifyingOption( dependencySet.getUnpackOptions() );
+    }
+
+    private boolean isConentModifyingOption( UnpackOptions opts )
+    {
+        return ( opts.isFiltered() || opts.getLineEnding() != null );
+    }
+
     private void checkMultiArtifactOutputConfig( final DependencySet dependencySet )
     {
         String dir = dependencySet.getOutputDirectory();
@@ -198,8 +208,9 @@ private void checkMultiArtifactOutputConfig( final DependencySet dependencySet )
         if ( ( dir == null || !dir.contains( "${" ) ) && ( mapping == null || !mapping.contains( "${" ) ) )
         {
             logger.warn( "NOTE: Your assembly specifies a dependencySet that matches multiple artifacts, but "
-                + "specifies a concrete output format. THIS MAY RESULT IN ONE OR MORE ARTIFACTS BEING OBSCURED!\n\n"
-                + "Output directory: '" + dir + "'\nOutput filename mapping: '" + mapping + "'" );
+                             + "specifies a concrete output format. THIS MAY RESULT IN ONE OR MORE ARTIFACTS BEING "
+                             + "OBSCURED!\n\n" + "Output directory: '" + dir + "'\nOutput filename mapping: '" + mapping
+                             + "'" );
         }
     }
 
@@ -234,7 +245,7 @@ private void addNormalArtifact( final DependencySet dependencySet, final Artifac
         task.setUnpack( dependencySet.isUnpack() );
 
         final UnpackOptions opts = dependencySet.getUnpackOptions();
-        if ( dependencySet.isUnpack() && ( opts != null ) )
+        if ( isUnpackWithOptions( dependencySet ) )
         {
             task.setIncludes( opts.getIncludes() );
             task.setExcludes( opts.getExcludes() );