Closed mikoim closed 8 years ago
前のコミットに入れ忘れた分を追加しました.
quoteIdentifier()
$s->execute()
query($sql, Adapter::QUERY_MODE_EXECUTE)
$_obj->type にアクセスすることが出来ず,isset($_obj->type) が常に FALSE になってしまう. e6e2aabf60a5b9dbdb4ff2b7e76598d234833ad5 までは動作していた模様.
$_obj->type
isset($_obj->type)
FALSE
https://github.com/logue/pukiwiki_adv/blob/master/wiki-common/lib/PukiWiki/Relational.php#L139 https://github.com/logue/pukiwiki_adv/blob/master/wiki-common/lib/PukiWiki/Relational.php#L213
これは、割と初期の頃に組んだコードで、単にファイル単位で処理していたものをSqliteで置き換えたものです。 ベタ移植したのであまりプリペアドステートメントなど考慮していませんでした。 今後、もっと抜本的な改善が必要かもしれませんね。
複合演算子に置き換え
前のコミットに入れ忘れた分を追加しました.
プリペアドステートメントを使用するように変更
quoteIdentifier()
を結合していたのをプリペアドステートメントへ変更$s->execute()
をquery($sql, Adapter::QUERY_MODE_EXECUTE)
へ変更リンクのデータベースが更新されない問題を修正
$_obj->type
にアクセスすることが出来ず,isset($_obj->type)
が常にFALSE
になってしまう. e6e2aabf60a5b9dbdb4ff2b7e76598d234833ad5 までは動作していた模様.問題の箇所
https://github.com/logue/pukiwiki_adv/blob/master/wiki-common/lib/PukiWiki/Relational.php#L139 https://github.com/logue/pukiwiki_adv/blob/master/wiki-common/lib/PukiWiki/Relational.php#L213