1
0
mirror of https://github.com/mitb-archive/filebot synced 2025-03-10 06:20:27 -04:00

Refactor ExpressionFormat into generic ExpressionFormat (used by -mediainfo and -list calls) and ExpressionFileFormat (used when generating output paths) that performs all the file path validation logic (strip spaces, don't allow /\ in binding values, etc)

This commit is contained in:
Reinhard Pointner 2017-02-28 00:59:20 +08:00
parent 06493e2940
commit 6996e4bf72
2 changed files with 5 additions and 5 deletions

View File

@ -147,7 +147,7 @@ public class CmdlineOperations implements CmdlineInterface {
return renameAll(renameMap, renameAction, conflict, null); return renameAll(renameMap, renameAction, conflict, null);
} }
public List<File> renameSeries(Collection<File> files, RenameAction renameAction, ConflictAction conflictAction, File outputDir, ExpressionFormat format, EpisodeListProvider db, String query, SortOrder sortOrder, ExpressionFilter filter, Locale locale, boolean strict) throws Exception { public List<File> renameSeries(Collection<File> files, RenameAction renameAction, ConflictAction conflictAction, File outputDir, ExpressionFileFormat format, EpisodeListProvider db, String query, SortOrder sortOrder, ExpressionFilter filter, Locale locale, boolean strict) throws Exception {
log.config(format("Rename episodes using [%s]", db.getName())); log.config(format("Rename episodes using [%s]", db.getName()));
// ignore sample files // ignore sample files
@ -302,7 +302,7 @@ public class CmdlineOperations implements CmdlineInterface {
return new ArrayList<Episode>(episodes); return new ArrayList<Episode>(episodes);
} }
public List<File> renameMovie(Collection<File> files, RenameAction renameAction, ConflictAction conflictAction, File outputDir, ExpressionFormat format, MovieIdentificationService service, String query, ExpressionFilter filter, Locale locale, boolean strict) throws Exception { public List<File> renameMovie(Collection<File> files, RenameAction renameAction, ConflictAction conflictAction, File outputDir, ExpressionFileFormat format, MovieIdentificationService service, String query, ExpressionFilter filter, Locale locale, boolean strict) throws Exception {
log.config(format("Rename movies using [%s]", service.getName())); log.config(format("Rename movies using [%s]", service.getName()));
// ignore sample files // ignore sample files
@ -493,7 +493,7 @@ public class CmdlineOperations implements CmdlineInterface {
return renameAll(renameMap, renameAction, conflictAction, matches); return renameAll(renameMap, renameAction, conflictAction, matches);
} }
public List<File> renameMusic(Collection<File> files, RenameAction renameAction, ConflictAction conflictAction, File outputDir, ExpressionFormat format, MusicIdentificationService... services) throws Exception { public List<File> renameMusic(Collection<File> files, RenameAction renameAction, ConflictAction conflictAction, File outputDir, ExpressionFileFormat format, MusicIdentificationService... services) throws Exception {
List<File> audioFiles = sortByUniquePath(filter(files, AUDIO_FILES, VIDEO_FILES)); List<File> audioFiles = sortByUniquePath(filter(files, AUDIO_FILES, VIDEO_FILES));
// check audio files against all services if necessary // check audio files against all services if necessary
@ -529,7 +529,7 @@ public class CmdlineOperations implements CmdlineInterface {
return renameAll(renameMap, renameAction, conflictAction, null); return renameAll(renameMap, renameAction, conflictAction, null);
} }
public List<File> renameFiles(Collection<File> files, RenameAction renameAction, ConflictAction conflictAction, File outputDir, ExpressionFormat format, XattrMetaInfoProvider service, ExpressionFilter filter, boolean strict) throws Exception { public List<File> renameFiles(Collection<File> files, RenameAction renameAction, ConflictAction conflictAction, File outputDir, ExpressionFileFormat format, XattrMetaInfoProvider service, ExpressionFilter filter, boolean strict) throws Exception {
log.config(format("Rename files using [%s]", service.getName())); log.config(format("Rename files using [%s]", service.getName()));
// match to xattr metadata object or the file itself // match to xattr metadata object or the file itself

View File

@ -222,7 +222,7 @@ public abstract class ScriptShellBaseClass extends Script {
try { try {
return formatter.format(new MediaBindingBean(xattr.getMetaInfo(file), file)); return formatter.format(new MediaBindingBean(xattr.getMetaInfo(file), file));
} catch (SuppressedThrowables e) { } catch (SuppressedThrowables e) {
debug.finest(format("%s => %s", format, e.getMessage())); debug.finest(format("%s => %s", format, e));
} }
return null; return null;