cacert/cacert-testmgr
9
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

db->update() sucks for complicated where clauses

Browse source 
Signed-off-by: Michael Tänzer <neo@nhng.de>
This commit is contained in:
Michael Tänzer 2010-04-29 21:55:57 +02:00
parent e9d2e83670
commit 727e4ff282
1 changed files with 10 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

21
manager/application/controllers/AddPointsController.php
View file

@ -102,17 +102,16 @@ class AddPointsController extends Zend_Controller_Action
// Fix the assurer flag
$where = array();
$query = '`users`.`id` = ?';
$where[] = $this->db->quoteInto($query, $user['id']);
$query = 'exists(select * from `cats_passed` as `cp`, ' .
'`cats_variant` as `cv` where `cp`.`variant_id` = `cv`.`id` and ' .
'`cv`.`type_id` = 1 and `cp`.`user_id` = ?';
$where[] = $this->db->quoteInto($query, $user['id']);
$query = '(select sum(`points`) from `notary` where `to`= ? and ' .
'`expire` > now()) >= 100';
$where[] = $this->db->quoteInto($query, $user['id']);
$this->db->update('users', array('assurer' => 1), $where);
$query = 'UPDATE `users` SET `assurer` = 1 WHERE `users`.`id` = :user AND '.
'EXISTS(SELECT * FROM `cats_passed` AS `cp`, `cats_variant` AS `cv` '.
'WHERE `cp`.`variant_id` = `cv`.`id` AND `cv`.`type_id` = 1 AND '.
'`cp`.`user_id` = :user) AND '.
'(SELECT SUM(`points`) FROM `notary` WHERE `to` = :user AND '.
'`expire` < now()) >= 100';
$query_params['user'] = $user['id'];
$this->db->query($query, $query_params);
return;
}