public async Task <IEnumerable <InvalidCaseReportExpanded> > Get()
{
var invalidCaseReports = await _invalidCaseReports.GetAllAsync();
// Comment from woksin - 15/02-2018
// By fetching all dataCollectors to memory we should get reduced latency,
// which was the case when I tested this method using the old method versus fetching the data prior to querying it.
// In my opinion, the best way to do this is to have a cache-system for these databases (preferably in the classes that deals directly
// with IMongoDatabase and IMongoCollection
var dataCollectors = await _dataCollectors.GetAllAsync();
// Following over from CaseReportController...
// (half -as bad, but still half-assed)
//
// Comment from review; einari - 23rd of October 2017
// Todo: This is a N+1 query - potentially incredibly slow
// an optimization would be to get all data collectors and all healthrisks and then
// do inmemory lookups. Also moved this away from being done inside the CaseReportExtended
// object - as it should not be having a relationship to repositories
return(invalidCaseReports.Select(caseReport =>
{
var dataCollector = dataCollectors.FirstOrDefault(collector => collector.Id == caseReport.DataCollectorId);
//var dataCollector2 = _dataCollectors.GetById(caseReport.DataCollectorId);
return new InvalidCaseReportExpanded(caseReport, dataCollector);
}));
}
public async Task <IActionResult> Export(string format = "excel")
{
if (!exporters.ContainsKey(format))
{
return(NotFound());
}
var exporter = exporters[format];
var dataCollectors = await _dataCollectors.GetAllAsync();
var stream = new MemoryStream();
var result = exporter.WriteDataCollectors(dataCollectors, stream);
if (result)
{
stream.Seek(0, SeekOrigin.Begin);
return(File(stream, exporter.GetMIMEType(), "datacollectors-" + DateTimeOffset.UtcNow.ToString("yyyy-MMMM-dd") + exporter.GetFileExtension()));
}
stream.Close();
return(StatusCode(500));
}
public async Task <IActionResult> Get()
{
var items = await _dataCollectors.GetAllAsync();
return(Ok(items));
}