Fixed errors related to creating a contact #20
Conversation
Previously, when sending data, a new contact was created with a new mtc_id. This problem was solved by correctly collecting cookies adding $request['header'][] = "Cookie: ". $data['COOKIES']; аfter public function prepareRequest(array $data)
add parsing fields from form
| $email = $_REQUEST['email'] ?? $_REQUEST['Email'] ?? $_REQUEST['E-mail'] ?? $_REQUEST['E-MAIL']; // mauticform_input_mkregistraciaslp_email | ||
| $name = $_REQUEST['name'] ?? $_REQUEST['Name']; // mauticform_input_mkregistraciaslp_name | ||
| $phone = $_REQUEST['phone'] ?? $_REQUEST['Phone']; // mauticform_input_mkregistraciaslp_phone |
There was a problem hiding this comment.
This is too specific. It shouldn't be in the docs. Why would anyone need to check for 4 different email keys?
There was a problem hiding this comment.
I tried using this file for various external services, such as website builders. It turned out that some people are passing parameters in upper case, and others in lower case. Somewhere it is necessary for third parties (web master, customer marketer) to register a variable manually. As a result, either we do the most universal and register-independent check of variables, or we take a long time to find out who made a mistake in writing a variable on the transmitting side.
| $form_id = $_REQUEST['form_id'] ?? $_REQUEST['Form_id']; // mauticform_input_mkregistraciaslp_form_id | ||
| if (is_null($form_id)) {$form_id = 1;} // if form_id is empty then use the 1'st Mautic form | ||
| $form = $mautic->getForm($form_id); |
There was a problem hiding this comment.
It must be a real Mautic form. The specific form that has the fields defined. It cannot default to 1.
There was a problem hiding this comment.
Again, to speed up the start of work if the transmitter is not on our side. We can send a link to our script, create 1 form and immediately start accepting data. And then when the form ids are configured on an external resource, the data will begin to be distributed to the required forms in mautic
| // Require Composer autoloader | ||
| require __DIR__.'/vendor/autoload.php'; | ||
| // or require __DIR__.'/../../vendor/autoload.php'; if you place it not in the root folder, but for example in the /docroot/webhook/ folder |
There was a problem hiding this comment.
Isn't this obvious? The autoloader path must be correct and depends on where the vendor dir is relative to this PHP script. It may be different for everyone. Copy-pasting code and hoping it will work without understanding it will be a big issue now with ChatGPT, I know. But even the young developers must thing about what the example code does.
There was a problem hiding this comment.
This is obvious to you, a senior programmer, but it is not obvious to a marketer who needs to send data to mautic from a website.
| // $request['header'][] = "Cookie: mautic_session_id=$sessionId"; | ||
| // $request['header'][] = "Cookie: mautic_device_id=$sessionId"; |
There was a problem hiding this comment.
What's wrong with this library to set the cookies for you? I think I'll need more context. And the changes need a bit of cleanup. Leaving commented out code in the production code isn't very professional.
There was a problem hiding this comment.
Look into https://github.com/escopecz/mautic-form-submit/issues
The library did not work correctly. A new lead was created with a new mtc_id (leaving the anonymous user with the old mtc_id) I and other people wrote about this. In the end I had to fix it myself. Now everything works.
There was a problem hiding this comment.
Hello @escopecz, thanks for the library, it helps us a lot. Hello @zaharovrd, we are facing the same problem you described. Submitting the form creates a new contact. Can you please share your solution to solve this problem?
There was a problem hiding this comment.
This is correct repo:
https://github.com/zaharovrd/mautic-form-submit
When creating a contact through the form, a new mtc_id was created
Contacts are now created correctly