hasadna / open-shot

A simple, open question and answer site. Shot being an hebrew acronym for Q&A.
http://localshot.org.il
BSD 3-Clause "New" or "Revised" License
9 stars 30 forks source link

Fixed Issue #365 (Editing a question removed followers). Added a test. #371

Closed ofir-ov closed 11 years ago

ofir-ov commented 11 years ago

The problem was that upon edit, the previous instance of the question wasn't fed to the form. Thus, the form's rating was set to 1. The "follows" themselves were still in the database.

I also created a test that fails without the bug fix and passes with it. I'll be happy if you can review the test as well, because some of the code there wasn't intuitive.

Lastly, this is my first contribution to an open source project, so I hope I got everything alright, process-wise.