x/stackalytics
Code Issues Proposed changes

Don't mix days and timestamps

Browse Source 
get_first_record_day() returns a number of days, not a timestamp.

Change-Id: I5e3b6f1946e7ee1c8c8b04edb9bc8bbdc542a83d
Signed-off-by: Pierre-Louis Bonicoli <pierre-louis.bonicoli@libregerbil.fr>
This commit is contained in:
Pierre-Louis Bonicoli
2017-03-10 15:14:16 +01:00
parent 280559ab97
commit de3c107ac9
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
stackalytics/dashboard/decorators.py
View File

@@ -122,14 +122,15 @@ def record_filter(ignore=None):
def _filter_records_by_days(start_date, end_date, memory_storage_inst):
if start_date:
start_date = utils.date_to_timestamp_ext(start_date[0])
start_day = utils.timestamp_to_day(start_date)
else:
start_date = memory_storage_inst.get_first_record_day()
start_day = memory_storage_inst.get_first_record_day()
if end_date:
end_date = utils.date_to_timestamp_ext(end_date[0])
else:
end_date = utils.date_to_timestamp_ext('now')
start_day = utils.timestamp_to_day(start_date)
end_day = utils.timestamp_to_day(end_date)
return memory_storage_inst.get_record_ids_by_days(