Write Data in Custom Table

[dragonstyleGG]

Hello,

I created a database table with the name modx_mails.

Now I created a snippet with that code:

<?php
define('MODX_CONFIG_KEY','config');

$test_msid = "11111";
$test_name = "Duck";
$test_vorname = "Donald";
$test_email = "[email protected]";
$test_status = "not";



$sql = "INSERT INTO modx_mails (msid, name, vorname, email, status) VALUES (:msid, :name, :vorname, :email, :status)";
$query = $modx->query($sql);

$pdo->prepare($query)->execute([$test_msid, $test_name, $test_vorname, $test_email, $test_status]);

But I get an Error 500. I think there ist something wrong with the SQL-Stuff. Do have someone a hint?

[halftrainedharry]

@dragonstyleGG

$pdo->prepare($query)

Try using $modx instead of $pdo.

[dragonstyleGG]

Ah, yes the Error 500 does not appear then. But it don’t write the data to the table.

[halftrainedharry]

I don’t think you need the query. Use the SQL statement directly with prepare:

$modx->prepare($sql)->execute(...)

[halftrainedharry]

You also need to use an associative array

$modx->prepare($sql)->execute(['msid' => $test_msid, 'name' => $test_name, ... ])

[bruno17]

maybe better to create a proper xPDO package and use the modx/xPDO api methods

[dragonstyleGG]

Hm,

it’s still don’t write the data:

$sql = "INSERT INTO modx_mails (msid, name, vorname, email, status) VALUES (:msid, :name, :vorname, :email, :status)";
$query = $modx->query($sql);

$modx->prepare($query)->execute(['msid' => $test_msid, 'name' => $test_name, 'vorname' => $test_vorname, 'email' => $test_email, 'status' => $test_status]);

I don’t think you need the query . Use the SQL statement directly with prepare
What do you mean with that?

[dragonstyleGG]

maybe better to create a proper xPDO package and use the modx/xPDO api methods

Yes, that might be a better solution. But for a “not every day programmer” like me it is far to overload.
And the doc is not really “not-every-day-programmer” friendly

[halftrainedharry]

$sql = "INSERT INTO modx_mails (msid, name, vorname, email, status) VALUES (:msid, :name, :vorname, :email, :status)";
$modx->prepare($sql)->execute(['msid' => $test_msid, 'name' => $test_name, 'vorname' => $test_vorname, 'email' => $test_email, 'status' => $test_status]);

[dragonstyleGG]

@ halftrainedharry:

Thank you. That is working.

[bobray]

The ClassExtender extra can help you generate classes that xPDO can use to read and write to your custom table. A new version of the extra is due out soon.