You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@creadur.apache.org by rd...@apache.org on 2013/04/01 09:31:37 UTC
svn commit: r1463102 - in
/creadur/tentacles/trunk/src/main/java/org/apache/creadur/tentacles:
License.java LicenseType.java Licenses.java Main.java
Author: rdonkin
Date: Mon Apr 1 07:31:36 2013
New Revision: 1463102
URL: http://svn.apache.org/r1463102
Log:
Improve fluency of interface
Modified:
creadur/tentacles/trunk/src/main/java/org/apache/creadur/tentacles/License.java
creadur/tentacles/trunk/src/main/java/org/apache/creadur/tentacles/LicenseType.java
creadur/tentacles/trunk/src/main/java/org/apache/creadur/tentacles/Licenses.java
creadur/tentacles/trunk/src/main/java/org/apache/creadur/tentacles/Main.java
Modified: creadur/tentacles/trunk/src/main/java/org/apache/creadur/tentacles/License.java
URL: http://svn.apache.org/viewvc/creadur/tentacles/trunk/src/main/java/org/apache/creadur/tentacles/License.java?rev=1463102&r1=1463101&r2=1463102&view=diff
==============================================================================
--- creadur/tentacles/trunk/src/main/java/org/apache/creadur/tentacles/License.java (original)
+++ creadur/tentacles/trunk/src/main/java/org/apache/creadur/tentacles/License.java Mon Apr 1 07:31:36 2013
@@ -29,7 +29,7 @@ public class License {
private final String text;
private final String key;
private final Set<Archive> archives = new HashSet<Archive>();
- final List<File> locations = new ArrayList<File>();
+ private final List<File> locations = new ArrayList<File>();
public License(final String key, final String text) {
this.text = text;
@@ -48,6 +48,10 @@ public class License {
return this.archives;
}
+ public List<File> getLocations() {
+ return this.locations;
+ }
+
public Set<URI> locations(final Archive archive) {
final URI contents = archive.contentsURI();
final Set<URI> locations = new HashSet<URI>();
Modified: creadur/tentacles/trunk/src/main/java/org/apache/creadur/tentacles/LicenseType.java
URL: http://svn.apache.org/viewvc/creadur/tentacles/trunk/src/main/java/org/apache/creadur/tentacles/LicenseType.java?rev=1463102&r1=1463101&r2=1463102&view=diff
==============================================================================
--- creadur/tentacles/trunk/src/main/java/org/apache/creadur/tentacles/LicenseType.java (original)
+++ creadur/tentacles/trunk/src/main/java/org/apache/creadur/tentacles/LicenseType.java Mon Apr 1 07:31:36 2013
@@ -26,14 +26,14 @@ public enum LicenseType {
ASL_2_0("asl-2.0"), CPL_1_0("cpl-1.0"), CDDL_1_0("cddl-1.0");
- public static Licenses loadLicensesFrom(
- final TentaclesResources tentaclesResources) throws IOException {
+ public static Licenses loadLicensesFrom(final Platform platform)
+ throws IOException {
final Map<String, String> licenses =
new ConcurrentHashMap<String, String>();
for (final LicenseType type : LicenseType.values()) {
- type.putTextInto(licenses, tentaclesResources);
+ type.putTextInto(licenses, platform.getTentaclesResources());
}
- return new Licenses(licenses);
+ return new Licenses(licenses, platform);
}
private final String resourceName;
Modified: creadur/tentacles/trunk/src/main/java/org/apache/creadur/tentacles/Licenses.java
URL: http://svn.apache.org/viewvc/creadur/tentacles/trunk/src/main/java/org/apache/creadur/tentacles/Licenses.java?rev=1463102&r1=1463101&r2=1463102&view=diff
==============================================================================
--- creadur/tentacles/trunk/src/main/java/org/apache/creadur/tentacles/Licenses.java (original)
+++ creadur/tentacles/trunk/src/main/java/org/apache/creadur/tentacles/Licenses.java Mon Apr 1 07:31:36 2013
@@ -18,19 +18,27 @@
*/
package org.apache.creadur.tentacles;
+import java.io.File;
+import java.io.IOException;
import java.util.Collections;
import java.util.Map;
public class Licenses {
+ private final IOSystem ioSystem;
private final Map<String, String> licenses;
- public Licenses(final Map<String, String> licenses) {
+ public Licenses(final Map<String, String> licenses, final Platform platform) {
super();
+ this.ioSystem = platform.getIoSystem();
this.licenses = Collections.unmodifiableMap(licenses);
}
- public License license(final String text) {
+ public License from(final File document) throws IOException {
+ return license(this.ioSystem.slurp(document));
+ }
+
+ private License license(final String text) {
final String key = toKey(text);
return new License(key, normalize(text));
}
Modified: creadur/tentacles/trunk/src/main/java/org/apache/creadur/tentacles/Main.java
URL: http://svn.apache.org/viewvc/creadur/tentacles/trunk/src/main/java/org/apache/creadur/tentacles/Main.java?rev=1463102&r1=1463101&r2=1463102&view=diff
==============================================================================
--- creadur/tentacles/trunk/src/main/java/org/apache/creadur/tentacles/Main.java (original)
+++ creadur/tentacles/trunk/src/main/java/org/apache/creadur/tentacles/Main.java Mon Apr 1 07:31:36 2013
@@ -95,7 +95,7 @@ public class Main {
this.tentaclesResources.copyTo("legal/style.css",
new File(this.layout.getOutputDirectory(), "style.css"));
- this.licenses = loadLicensesFrom(this.tentaclesResources);
+ this.licenses = loadLicensesFrom(platform);
}
public static void main(final String[] args) throws Exception {
@@ -154,8 +154,7 @@ public class Main {
final List<File> files =
this.fileSystem.licensesFrom(archive.contentsDirectory());
for (final File file : files) {
- final License license =
- this.licenses.license(this.ioSystem.slurp(file));
+ final License license = this.licenses.from(file);
License existing = licenses.get(license);
if (existing == null) {
@@ -163,7 +162,7 @@ public class Main {
existing = license;
}
- existing.locations.add(file);
+ existing.getLocations().add(file);
existing.getArchives().add(archive);
archive.getLicenses().add(existing);
}
@@ -206,10 +205,7 @@ public class Main {
for (final File file : files) {
- final License license =
- this.licenses.license(this.ioSystem.slurp(file));
-
- undeclared.remove(license);
+ undeclared.remove(this.licenses.from(file));
}