&fieldname (in combination with ¶mname) is an alternative way to create a where-condition, so you wonāt need this property.
You also donāt need the &tablename property.
You only need the preHook 'db2formit' if you want to fill the form with data from the database table when it is shown. Not sure if this is necessary to your case.
Thank you for taking the time to look at my issue.
I made the changes but the form did not send so no emails were triggered and the database was not updated.
Hereās the new code:
[[!FormIt?
&hooks=`FormItSaveForm,formit2db,email,redirect`
&emailTpl=`MyEmailChunk`
&prefix=`modx8_`
&packagename=`tfsurvey`
&classname=`TfSurvey`
&where=`{"id":[[!+tssrApproved]]}`
&emailSubject=`TSSR Approved`
&emailTo=`###@website.com,[[+email]]`
&emailCC=`###@website2.com`
&redirectTo=`https://website.com/tssrapprovednotice.html`
&validate=`name:required,email:required,tssrApproved:required`
&validationErrorMessage=`Please correct the following errors:[[+errors]]`
]]
It is probably important to know that this is a generic form that is called to a dynamic page that shows on the front end data relating to one specific row of the database. There are 100ās of these rows and one dynamically created web page for each row.
It is this one field called ātssrApprovedā that needs to be updated on the row of each page.
I always use migxDb for updating databases and never have an issue, but in this case I have to make one update from the front end, hence Iām trying to get formit2db working.
So Iām a bit confused about your setup.
To update a row you need two different values. One to select the right row and one with the new value to set.
$row_id = 10; //to select the row
$tssrApproved = 'xy'; //to set the new value
$obj = $modx->getObject('TfSurvey', array('id'=>$row_id)); //select the row
$obj->set('tssrApproved', $tssrApproved); //update the row
$obj->save();
//or maybe this is what you want to do
$tssrApproved_oldValue = 'xx';
$tssrApproved_newValue = 'yy';
$obj = $modx->getObject('TfSurvey', array('tssrApproved'=>$tssrApproved_oldValue));
$obj->set('tssrApproved', $tssrApproved_newValue);
$obj->save();
I think at the moment you are just providing one value and that would create code like this, that doesnāt make any sense.
$tssrApproved = 'xy'; //one value to select and set
$obj = $modx->getObject('TfSurvey', array('id'=>$tssrApproved));
$obj->set('tssrApproved', $tssrApproved);
$obj->save();
Each page is generated dynamically, then using migxLoopCollection I can fill the page with the data from the one row that is related to that page.
The form is a generic form that is called via a chunk to each page, so since every page relates to a different row on the database I cannot specifically name the row in the formit snippet.
What I am doing is pre-populating the (other) fields of the formit form, again using migxLoopCollection. This is specific information relating to the row associated with each dynamically created web page.
The only field that requires input is tssrApproved (value = yes/no) and then click submit. (Without Formit2db this is all fully functionally).
I can easily create and pre-populate a new field on the form with the row id number for that particular page, if I was to do this would there be any way I could use this form field for updating the database field for the field tssrApproved?
Canāt you just use the same where-clause you use in the migxLoopCollection-call and put it in the &where parameter of the FormIt-call?
[[!FormIt?&where=`{"id":"[[+row_id]]"}` ...
Btw, if you only want to update the tssrApproved field, you should define the removeFields property with the names of all the other form fields to exclude them.
I have tried a number of combinations of the āwhere-clauseā I used in the migxLoopCollection but none (I have tried so far) have worked in my formit2db added code.
This is the migxLoopCollection Iām using to create the dynamic data in each page:
āprepare_survey_whereā points to a plug-in that creates each page depending on its āsiteLrdā. (btw ā¦ I use this same code format to populate the Formit form fields, I just create a different TPL and the formit form fields fill with the correct data for that particular page)
I think I am straying from the initial direction of this post.
Going back to the first response from @halftrainedharry, it was mentioned that:
&fieldname in combination with ¶mname is an alternative way to create a where-condition
Maybe this is solution for what I want to do?
The only thing is, I do not know what information (text) to put into:
¶mname and &fieldname,
or in which order they go.
I thought I knew what the snippet was doing, but now I did some tests and the &where property behaves strangely. (Seems to work with fixed strings and in the preHook but loses its value in the hook). Sorry for that!
So as you said, it is probably better to focus on the ¶mname-property.
(I looked at the code and the &fieldname has no purpose at all. Not sure what it is used for!)
You could try this. Delete the &where-property, then add the ¶mname and a new input field with a corresponding name. The name has to be the name of the primary-key-column of your table.
(If the name is id you probably need to use friendly urls to avoid having two parameters with the same name.)
For testing purposes it is probably better to change the type from hidden to text so you can see if the id is correct and doesnāt lose its value if the form validation fails.
If this doesnāt work either I will write you a custom hook, which is probably better suited for your purpose. (For example: The FormIt2db-snippet automatically adds a new line to the table if it doesnāt find a row with the given id. You most likely donāt want that).
Thanks again for taking the time to assist me solve this issue. Alas it is to no avail.
I posted the following code but all it did was reset the page and posted to the 4 fields in the form all the corresponding data read straight from their fieldnames in the database.
Does your database table tfsurvey have a column with the name id?
Does the chunk [[$prepare_survey_where]] return the right value of this column id for the current row?
The snippet formit2db should create an entry in the error log if something fails. Can you check if something gets added to the error log when you try to send the form?
Can you check if a new row gets added to your database table when you try to send the form?
Maybe you could share the schema for the table tfsurvey or the code for [[$prepare_survey_where]] or [[!prepare_survey_where]] if all the things above are of no avail.
It did, it pointed to the &prefix, so I removed that line and tried again.
Then the form sent but it left me with some unintended consequences
Formit2db created another row in the datebase called 0 (zero). Oddly enough every field in the row ā0ā had a 0 in it apart from the field ātssrApprovedā which contained the text I inserted in the form.
Meaning your code worked for creating a new row, but did not input the text into the row of the actual webpage.
The code [[$prepare_survey_where]] points to a snippet that is part of creating new webpages on the fly. However I am also using it with migxLoopCollection to pre-populate 4 form fields of the Formit form, this works well.
However it has also created a new webpage called ā0ā. Itās no issue because I can just delete that row from the database.
So you can definitely claim partial success if not an answer to my problem, which is how to get formit2db to populate 1 field in an existing database row from a dynamic webpage.
I donāt know how to go about this for a generic form that is called to many different webpages whose URL is read from the database.
I am not sure how the request-parameter object_id is set when you first call the page with the form. But if it is somehow set on every subsequent call too (meaning when you send the form), you may not need the input-field.
