質問

このクエリは失敗し続けます

  

整合性制約違反:1048列 'login_name'はnullにできません

挿入ステートメントは...

    $insertUserQuery = 'INSERT INTO `users` (
                    `login_name`,
                    `password`, 
                    `first_name`,
                    `last_name`,
                    `company_name`,
                    `company_address`,
                    `country`,
                    `email`,
                    `phone_number`,
                    `agency_type`,
                    `sold_before`,
                    `authorised`,
                    `current_module`
                    )
                    VALUES (
                   :login_name, :login_password, :first_name, :last_name, :company_name, :company_address, :country, :email, :phone_number, :agency_type, :sold_before, 0, 0);';

    $bindings = array(':login_name'      => $loginName,
                      ':login_password'  => sha1($password . Config::PASSWORD_SALT),
                      ':first_name'      => $firstName,
                      ':last_name'       => $lastName,
                      ':company_name'    => $companyName,
                      ':company_address' => $companyAddress,
                      ':country'         => $country,
                      ':email'           => $emailAddress,
                      ':phone_number'     => $phone,
                      ':agency_type'     => null,
                      ':sold_before' => null  
                      );

print_r($bindings);                 

    Db::query($insertUserQuery, $bindings);

私のデータベースクラスは別の質問で。 print_r()は、配列に間違いなく値があることを示しています。

MySQL関数でもある「パスワード」という言葉を使用して、何か関係があるのでしょうか?

PDOは、SELECTを使用した場合と同じ方法で、INSERTを使用した準備済みステートメントをサポートしますか?

値を引用符で囲む必要がありますか、例: ':login_name'?

役に立ちましたか?

解決

PDO :: query はプリペアドステートメント構文をサポートしていませんか? PDO :: prepare および PDOStatement :: execute を読んでください。おそらく次のようなものが必要です:

$insertUserQuery = 'INSERT INTO `users` (`login_name`, ...) ' .
    'VALUES (:login_name, ...);';
$bindings = array(':login_name' => $loginName, ...);
$stmt = Db::prepare($insertUserQuery);
$stmt->execute($bindings);

バインディングの配列を作成する代わりに、 $ stmt-> bindValue()を呼び出すこともできます。すぐに型を検証できるため、各値を明示的にバインドする方が少し優れていると思います。

編集:ごめんなさいjcinacio、投稿した後まであなたのコメントがほとんど同じだとは思わなかった。

ライセンス: CC-BY-SA帰属
所属していません StackOverflow
scroll top