Propagate delimiter changes to LocalBlobStore.

Propagates the changes to add delimiter support to the local blob
store. The prefix option no longer works with the non-recursive
option. The caller _must_ set the delimiter option to enable the same
functionality.
This commit is contained in:
Timur Alperovich 2015-07-13 17:19:13 -07:00 committed by Andrew Gaul
parent e4161a59bb
commit 19d54369db
1 changed files with 3 additions and 10 deletions

View File

@ -344,8 +344,8 @@ public final class LocalBlobStore implements BlobStore {
}
}));
if (!options.isRecursive()) {
return extractCommonPrefixes(contents, storageStrategy.getSeparator(), options.getPrefix());
if (options.getDelimiter() != null) {
return extractCommonPrefixes(contents, options.getDelimiter(), options.getPrefix());
}
return contents;
@ -472,14 +472,7 @@ public final class LocalBlobStore implements BlobStore {
if (prefix == null || prefix.isEmpty()) {
return name.indexOf(delimiter) == -1;
}
String prefixMatch;
if (prefix.endsWith(delimiter)) {
prefixMatch = "^" + Pattern.quote(prefix) + ".*";
} else {
// We should correctly match strings like "foobar/" where the prefix is only "foo"
prefixMatch = "^" + Pattern.quote(prefix) + ".*" + Pattern.quote(delimiter) + ".*";
}
if (name.matches(prefixMatch)) {
if (name.startsWith(prefix)) {
String unprefixedName = name.replaceFirst(prefix, "");
if (unprefixedName.equals("")) {
// we are the prefix in this case, return false