Skip to content

修改 Pix_Table 的 MySQL adapter 以正確產生資料表 #21

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
yookoala wants to merge 3 commits into
base: master
Choose a base branch
from
Open

修改 Pix_Table 的 MySQL adapter 以正確產生資料表 #21

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
eec225c
修改 Pix_Table 的 MySQL adapter 以正確產生資料表
yookoala Dec 20, 2014
fff9367
改進 news-raw-tables.php 產生表格的方法
yookoala Jan 7, 2015
b73989a
改進 table-build.php 處理錯誤的方法
yookoala Jan 7, 2015
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 2 additions & 7 deletions webdata/scripts/news-raw-tables.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,14 +5,9 @@
* to create 12 tables for NewsRaw
*/
$theTime = mktime();
$str = "CREATE TABLE IF NOT EXISTS `news_raw_%s` (
`news_id` int(11) NOT NULL,
`time` int(11) NOT NULL,
`raw` text NOT NULL,
PRIMARY KEY (`news_id`,`time`)
) ENGINE=InnoDB;";
$str = 'CREATE TABLE IF NOT EXISTS `news_raw_%s` LIKE news_raw';
$db = NewsRaw::getDb();
for ($i = 0; $i < 12; $i++) {
$db->query(sprintf($str, date('Ym', $theTime)));
$theTime = strtotime('+1 month', $theTime);
}
}
15 changes: 13 additions & 2 deletions webdata/scripts/table-build.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,18 @@
$p = pathinfo($m);
$o = new $p['filename'];
if($o instanceof Pix_Table) {
$o->dropTable();
try {
$o->dropTable();
} catch (Pix_Table_Exception $e) {
echo 'Unable to drop table "'.$p['filename'].'". ';
if (preg_match('/^Table: \w+SQL Error: \(1051\)Unknown table/',
$e->getMessage())) {
echo 'Table missing. Ignored.'."\n";
} else {
echo "\n";
die($e->getMessage());
}
}
$o->createTable();
}
}
}
4 changes: 2 additions & 2 deletions webdata/stdlibs/pixframework/Pix/Table/Db/Adapter/MysqlCommon.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ abstract class Pix_Table_Db_Adapter_MysqlCommon extends Pix_Table_Db_Adapter_SQL
public function createTable($table)
{
$sql = "CREATE TABLE " . $this->column_quote($table->getTableName());
$types = array('bigint', 'tinyint', 'int', 'varchar', 'char', 'text', 'float', 'double', 'binary');
$types = array('bigint', 'tinyint', 'int', 'varchar', 'char', 'text', 'longtext', 'float', 'double', 'binary');

foreach ($table->_columns as $name => $column) {
$s = $this->column_quote($name) . ' ';
Expand Down Expand Up @@ -73,7 +73,7 @@ public function createTable($table)
$column_sql[] = $s;
}

$sql .= " (\n" . implode(", \n", $column_sql) . ") ENGINE = InnoDB\n";
$sql .= " (\n" . implode(", \n", $column_sql) . ") ENGINE = InnoDB DEFAULT CHARSET=utf8mb4\n";

return $this->query($sql, $table);
}
Expand Down