search

ese-unibe-ch / ese2012-team4

5 stars 1 forks source link

Fix: Wrong activity log when buying items #70

Closed r-burn closed 11 years ago

r-burn commented 11 years ago

Die buy_new_item methode wird in unseren controllern von dem user.working_for verwendet, wenn sie loggt sollte sie ja eigentlich den user der etwas gekauft hat loggen. Dies klappt so natürlich nicht mehr. Habe es in den tests ebenfalls notiert.

LukasSkywalker commented 11 years ago

Ich hatte das in gemacht, aber @gianlucamateo hat das wieder zurückgeändert... Das Problem ist diese Zeile: https://github.com/ese-unibe-ch/ese2012-team4/commit/b48699a1f41d08011598edb5b889cdef5dec4fd1#L0L248 Ich habe sie in @session_user.buy_new_item geändert, damit die buy-funktion den Scope des Users hat. Erst in dewr Funktion beachte ich dann das working_for... Ich finde meine Lösung schöner, weil wir dann eine avarisble weniger brauchen (@session_user, die nur für den Log da ist). Mein changeset: https://github.com/ese-unibe-ch/ese2012-team4/commit/ec5157d8d075a06b6438feef2c2329aec33180cb

gianlucamateo commented 11 years ago

ja ich habe lange überlegt ob ich das ändern soll, aber so kam das item nach auktionsschluss ins nirvana, falls seller und buyer denselben account hatten (z. B. ese und Coding Inc.) ich bin einverstanden, deine Lösung ist eleganter, aber ich habe sie nicht zum laufen gekriegt. aktuell sollten diese Bugs gefixt sein.