diff options
-rw-r--r-- | App.php | 3 | ||||
-rw-r--r-- | Controller.php | 28 | ||||
-rw-r--r-- | Db.php | 6 |
3 files changed, 14 insertions, 23 deletions
@@ -138,8 +138,9 @@ class App return false; } + $msg = trim($msg); $fh = fopen($file, "a+"); - fwrite($fh, $msg); + fwrite($fh, "$msg\n"); fclose($fh); } diff --git a/Controller.php b/Controller.php index d5813a1..e25c575 100644 --- a/Controller.php +++ b/Controller.php @@ -107,7 +107,7 @@ class Controller $sql .= " LIMIT $limit"; } - $result = DB::query($sql, $values)->fetchAll(PDO::FETCH_ASSOC); + $result = DB::fetch($sql, $values); for ($i=0; $i < count($result); $i++) { @@ -216,14 +216,9 @@ class Controller * Insert this thing in the DB and return inserted * thing */ - public function insert($data = null) + public function insert() { - if (!$data) { - $data = $_POST; - } - if (empty($data)) { - App::log("Attempted to insert empty data"); return App::error("Nothing to insert"); } @@ -247,17 +242,16 @@ class Controller /** * Delete this thing */ - public function delete() + protected function delete($where) { - if (empty($_POST)) { - exit(App::error("Delete without arguments")); + if (empty($where)) { + return App::error("Delete without arguments"); } - list($key, $value) = each($_POST); + list($key, $value) = each($where); $sql = "DELETE FROM `{$this->table}` WHERE `{$key}` = ?"; - - return DB::query($sql, $value); + return Db::query($sql, array($value)); } @@ -266,13 +260,9 @@ class Controller * We keep this in the Controller since it might know * more about the topmost class */ - public function update($data = null, $where = null) + public function update($data, $where = null) { - if (!$data) { - $data = $_POST; - } - - if (empty($data)) { + if (!is_array($data)) { return App::error("Update with empty parameters"); } @@ -54,7 +54,7 @@ class Db * Execute an SQL query * @return mixed */ - public function query($sql, $data = null) + public static function query($sql, $data = null) { if (!$data) { $data = array(); @@ -104,9 +104,9 @@ class Db } - public function fetch($sql, $data = null) + public static function fetch($sql, $data = null) { - $stmt = DB::query($sql, $data); + $stmt = self::query($sql, $data); return $stmt->fetchAll(PDO::FETCH_ASSOC); } |