binbin4649 / bbm

0 stars 3 forks source link

user_all_count is wrong. #99

Closed binbin4649 closed 10 years ago

binbin4649 commented 10 years ago

Please fix. (Bet.php function saveNewBet)

Present condition : bet count Ideal condition : user count

for exsample. http://bookbookmaker.com/books/15

https://dl.dropboxusercontent.com/u/9913763/bug/fc85e2af739239451e178bc268ef8e0e.png

shivram2609 commented 10 years ago

@binbin4649 , I have reviewed the issue and checked the code, actually it is showing the total count of bets for the book, i user can bet more than one for a same content and everytime a user will bet for content count on user-all_count in table books will increase so it is showing count accordingly, please let me know if have any query or concern.

binbin4649 commented 10 years ago

For example, $conditions = array('Bet.book_id' => $book_id, 'Bet.user_id' => $user_id); $user_all_count = $this->Bet->find('count', $conditions);

How about that? Incidentally, have not tried.

binbin4649 commented 10 years ago

fa81ebd5587a5057c79ec397ca5b97220070cc89