Matoo125 / Vegapo

0 stars 2 forks source link

Database statistics #52

Closed mrkovec closed 7 years ago

mrkovec commented 7 years ago

Riesenie #18, do info pageu (footer Nieco ine) som pridal statistiky pre:

Potreboval som pridat novu lib pre generovanie "peknych" farebnych paliet a nasiel som tuto od googlu, tak dufam ze to bude okej. Tak isto dizajn je len taky nastrel.

Matoo125 commented 7 years ago

Vyzerá to super.

Zopár pripomienok

  1. Ten kód by nemal ísť cez index controller ale vlastný stats controller. Potom aj view by bol oddelený twig súbor.
  2. To, že tie queries sú komplikované by mi ani tak nevadilo ale tie čísla, ktoré produkujú niesu úplne správne napr. zoznam kategórií s počtom produktov táto query funguje správne
    select c.id, c.name, count(p.id) total from categories c
    inner join products p on p.category_id = c.id 
    where c.country = 'sk' 
    group by c.id
    order by total

    a tá ktorá tam je mi ukazuje nesprávne čisla.

tiež tagy toto funguje

select distinct t.id, t.name, count(mt.id) from tags t
inner join matching_tags mt on mt.tag_id = t.id  
where t.country = 'sk' and mt.country = 'sk'
group by t.id
mrkovec commented 7 years ago

To ze som statistikam nespravil vlastny kontroler bolo pol na pol z lenivosti a z toho ze som videl, ze aj dalsie footer polozky su tahane cez info stranku - okej oddelim to. K tym selectom, su komplikovane lebo som sa snazil vytiahnut pocty za oba jazyky, teda nie len sk pre slovakov atd. A kvoli odstraneniu duplicit je tam aj to parovanie cez value a to viacnasobne groupovanie. Na cc je potom v podstate zavysle len dotahovanie nazvu kategorii/tagov/shopov.

Matoo125 commented 7 years ago

Ok takže tie queries sú v pohode. Ja som aj zabudol na ten value column.

mrkovec commented 7 years ago

Jedine ma napadlo, ze mozno by bolo lepsie slovakom ukazat hodnoty len za sk a cechom za cz. Nech to nieje matuce. Aby slovaci zbytocne nehladali Penny markt alebo podobne.

Matoo125 commented 7 years ago

Ono sčítavanie nieje ideálne hlavne preto, že je tam mnoho rovnakých produktov CZ a SK

mrkovec commented 7 years ago

Jasne, to ma nenapadlo. Upravim to na sumarizaciu podla cc uzivatela. Este taka technicka otazka, ja ako oraclista ked napisem products q, categories w implicitne ocakavam ze je to inner join. Pre mysql je to lepsie uviest priamo, tak ako v tvojich prikladoch?

Matoo125 commented 7 years ago

Kým to funguje tak je to asi jedno. Explicitný join je podľa mňa čitatelnejší.

mrkovec commented 7 years ago

Obe zmeny zapracovane.

Matoo125 commented 7 years ago

Zle som sa vyjadril, myslel som novú metódu v info controller ... upravil som to 8d932f5dc727b044c01b8e7f8857b57dd585fa95

mrkovec commented 7 years ago

👍 len mam teraz problem, ze mi master konci s Undefined offset chybou na Vegapo\index.php on line 44.

Matoo125 commented 7 years ago

composer update

mrkovec commented 7 years ago

Este jeden problem, updateol som aj bower package a chyba sice je uz odstranena, ale mam uplne rozhaseny layout - a bol aj pred updateom na bootstrap4. Mozno nejake chybajuce css?

Matoo125 commented 7 years ago

https://github.com/Matoo125/Vegapo/issues/50#issuecomment-334159381

Odstránil som CSS súbor úplne pretože som tam importoval všetky css z knižnce. Teraz to treba exportovať zo SASS alebo stiahnuť z odkazu