addthis / metrics-reporter-config

Apache License 2.0
74 stars 52 forks source link

CsvReporterConfig returns null if directory exists #24

Open anupbanerjee opened 8 years ago

anupbanerjee commented 8 years ago

Class: CsvReporterConfig Method: enable Line 39: File foutDir = createFile(); The line above invokes the Parent AbstractCsvReporterConfig's createFile() method. In createFile() method's
Line Number 45: boolean success = foutDir.mkdirs(); This method will return false, if the directory already exists. Since the value returned is false in CsvReporterConfig the enable method will return false. I don't know if this is correct implementation, the reporter should continue to update the CSV files if the directory and file already exists.