[Coffee] [PATCH coffee-flask] Fix coffee graph for start of the month

Michal Sojka michal.sojka at cvut.cz
Mon Sep 3 15:43:06 CEST 2018


Dík. Na první pohled se zdá, že to problém opravdu řeší, ale okomentuj
to prosím trochu v "commit message" - v čem je problém a jak ho to
vyřeší. Zdá se, že číslo dne nahrazuješ počtem sekund, ale není mi jasné
proč to funguje ve funkci coffee_graph_history(). Tam se výsledek
z databáze rozděluje do proměnných days, counts a flavors. Proměnná
"days" má teď tedy zavádějící název - měla by se to přejmenovat na
seconds nebo secs_since_epoch. Ale asi se někde v kódu ty sekundy
převedou na dny, protože graf je správně, ale já to tam nevidím. Můžeš
upravit i kód coffee_graph_history, aby to tam bylo vidět a poslat 2.
verzi patche?

-Michal

On Mon, Sep 03 2018, Klapalek, Jaroslav wrote:
> ---
>  coffee_db.py | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/coffee_db.py b/coffee_db.py
> index 0abb5a4..8c18e63 100644
> --- a/coffee_db.py
> +++ b/coffee_db.py
> @@ -95,7 +95,7 @@ def coffee_history(uid=None):
>  
>      if uid is None:
>          res = list(c.execute("""
> -            select strftime('%d', ds.d),count(c.flavor),c.flavor from
> +            select strftime('%s', ds.d),count(c.flavor),c.flavor from
>              (select num,date('now',-num || ' days') as d from days) ds
>              left join coffees c
>              on d = date(c.time) group by d, c.flavor
> @@ -103,7 +103,7 @@ def coffee_history(uid=None):
>      else:
>          res = list(c.execute(
>              """
> -            select strftime('%d', ds.d),count(c.flavor),c.flavor from
> +            select strftime('%s', ds.d),count(c.flavor),c.flavor from
>              (select num,date('now',-num || ' days') as d from days) ds
>              left join
>              (select time,flavor from coffees where id = ?) c
> -- 
> 2.7.4
>
>
> _______________________________________________
> Coffee mailing list
> Coffee at rtime.felk.cvut.cz
> https://rtime.felk.cvut.cz/mailman/listinfo/coffee



More information about the Coffee mailing list