Merge branch 'cc-5709-airtime-analyzer' into saas

This commit is contained in:
Albert Santoni 2014-11-18 17:14:01 -05:00
commit 162995bb49
3 changed files with 9 additions and 3 deletions

View file

@ -355,7 +355,13 @@ SQL;
{ {
$exists = false; $exists = false;
try { try {
//Explicitly check filepath because if it's blank, getFilePath() can
//still return a directory that exists.
if (!$this->_file->getDbFilepath()) {
$exists = false;
} else {
$exists = file_exists($this->getFilePath()); $exists = file_exists($this->getFilePath());
}
} catch (Exception $e) { } catch (Exception $e) {
return false; return false;
} }

View file

@ -52,7 +52,7 @@ class AnalyzerPipeline:
metadata = dict() metadata = dict()
metadata = MetadataAnalyzer.analyze(audio_file_path, metadata) metadata = MetadataAnalyzer.analyze(audio_file_path, metadata)
metadata = FileMoverAnalyzer.move(audio_file_path, import_directory, original_filename, metadata) metadata = FileMoverAnalyzer.move(audio_file_path, import_directory, original_filename, metadata)
metadata["import_status"] = 0 # imported metadata["import_status"] = 0 # Successfully imported
# Note that the queue we're putting the results into is our interprocess communication # Note that the queue we're putting the results into is our interprocess communication
# back to the main process. # back to the main process.

View file

@ -26,7 +26,7 @@ QUEUE = "airtime-uploads"
Airtime's music library directory. Lastly, the extracted metadata is Airtime's music library directory. Lastly, the extracted metadata is
reported back to the Airtime web application. reported back to the Airtime web application.
There's a couple of Very Important technical details and contraints that you There's a couple of Very Important technical details and constraints that you
need to know if you're going to work on this code: need to know if you're going to work on this code:
1) airtime_analyzer is designed so it doesn't have to run on the same 1) airtime_analyzer is designed so it doesn't have to run on the same