Matoo125 / Vegapo

0 stars 2 forks source link

edit functionality #31

Closed mrkovec closed 7 years ago

mrkovec commented 7 years ago

added logging functionality for changes (issue #25) of:

Matoo125 commented 7 years ago

Je tam plno chýb, a neukazuje mi to, aká zmena bola vykonaná.

Matoo125 commented 7 years ago

Chýbajú tam namespace importy, začal som to opravovať ale je toho viac, čo nefunguje. Úpravu a abstrakciu zatial neriešim, keďže to nefunguje. Neviem, či si to testoval ale asi každý klik mi tam prihodí error.

Možno to je spôsobené iba nejakým detailom, ktorý prestal fungovať, keď si spravil pull z mastera (predpokladám). Pretože metóda getProductById tam už neexistuje.

je tam aj moja chyba spojená s loginom, ktorá už je opravená v mojom najnovšom pull requeste.

môžes to mergnuť znovu s masterom a skúsiť opraviť alebo iba opraviť a ja to mergnem. Ak to neupdatneš, login sa dá fixnuť v metóde user -> getById, tam je $this->save namiesto $this->fetch

Matoo125 commented 7 years ago

Nemôžem spraviť push do tejto vetvy, takže som spravil nový branch s mojimi úpravami

https://github.com/Matoo125/Vegapo/tree/mrkovec-adminedits https://github.com/Matoo125/Vegapo/commit/fa95a72849e08296d8d2b641023341f9a2b6dc4e

mrkovec commented 7 years ago

Cau, tak som sa k tomuto konecne dostal a chcem sa dohodnut ako dalej - stiahol som si najnovsiu verziu master branch upstreamu, rebasol ju do svojho adminedits branchu, opravil, otestoval a commitol. Pojde spravit merge do masteru z tohto pull requestu, alebo mam radsej dat do poriadku mrkovec-adminedits branch?

Matoo125 commented 7 years ago

Merge by spraviť išiel. Je tam ešte zopár múch

mrkovec commented 7 years ago

Taby otvorené a uzavreté mi hádžu sql error

Taby som odstranil - edity sa automaticky zatvaraju tajkze nemaju momentalne velke opodstatnenie.

Ak vymažem produkt typ je prázdny (mal by byť "vymazané")

Doplneny popis pre akciu trash/move. Tiez som doplnil logovanie move_to/.../sk/cz a naopak

Pri zmene produktu sa neloguje nič

Logovanie je zatial zavesene len na admin/produkty/upravit/ je aj ina cesta pre zmenu produktu?

Matoo125 commented 7 years ago

admin/produkty/upravit je jediná možnosť ale stále mi to neloguje pri upravovaní produktov.

Matoo125 commented 7 years ago

Teda loguje to, iba nezobrazuje tie logy v zmenách

mrkovec commented 7 years ago

Skus prosim ta pozriet view edit_details malo by byt:

CREATE or replace VIEW edit_details AS 
    #locale
    SELECT 
        q.id edit_id, q.type edit_type, null edit_sub_type, q.state edit_state, q.user_id edit_user_id, w.username edit_username, q.comment edit_comment, q.diff edit_diff, q.country edit_country, q.created_at edit_created_at, q.updated_at edit_updated_at,
        q.object_type, q.object_id, null object_name, null object_user_id, null object_username
    FROM edits q, users w 
    where q.object_type = 'locale' and q.user_id = w.user_id
    union
    #suggestion
    SELECT 
        q.id edit_id, q.type edit_type, w.type edit_sub_type, q.state edit_state, q.user_id edit_user_id, e.username edit_username, q.comment edit_comment, q.diff edit_diff, q.country edit_country, q.created_at edit_created_at, q.updated_at edit_updated_at,
        'product' object_type, w.product_id object_id, t.name object_name,  w.user_id object_user_id, r.username object_username
    FROM edits q, suggestions w, users e, users r, products t
    where q.object_type = 'suggestion' and q.object_id = w.id and q.user_id = e.user_id and w.user_id = r.user_id and t.id= w.product_id
    union
    #product    
    SELECT 
        q.id edit_id, q.type edit_type, null edit_sub_type, q.state edit_state, q.user_id edit_user_id, e.username edit_username, q.comment edit_comment, q.diff edit_diff, q.country edit_country, q.created_at edit_created_at, q.updated_at edit_updated_at,
        q.object_type, w.id object_id, w.name object_name,  w.author_id object_user_id, r.username object_username
    FROM edits q, products w, users e, users r
    where q.object_type = 'product' and q.object_id = w.id and q.user_id = e.user_id and w.author_id = r.user_id;
Matoo125 commented 7 years ago

Problém bol v tom, že keď product autor už neexistuje, edit_details preskočí riadok.

mrkovec commented 7 years ago

Super. Mozno by bolo teda dobre osetrit to outer joinom do buducnosti. Ale necham to zatial tak.