Simple Contact Form

[Pixrael]

Since I haven't received answers, I add the code directly in the module (I know is bad but I need to finish)
Line 264:

$inputfield->attr('class','form-control');

It would be great if this option was in the configuration as well as the current option for the Send button

[justb3a]

On 28/06/2017 at 9:53 PM, Pixrael said:

Since I haven't received answers, I add the code directly in the module (I know is bad but I need to finish)
Line 264:

$inputfield->attr('class','form-control');

It would be great if this option was in the configuration as well as the current option for the Send button

Hi, I spent the last days without a computer (holidays!!) so I didn't read my emails
But the forum offers a lot of good advice and you're not the first one who asking this. So please have a look at the following posts / answers:

[Pixrael]

I like the second solution, I will implement it later. Thank you!

[Pixrael]

1 hour ago, Pixrael said:

I like the second solution, I will implement it later. Thank you!

It works, but applies the classes to all inputfields, including the form and buttons. Is there any way to ask the type of input (Ex. InputfieldText) before applying the classes? I check the API Reference for inputfield but nothing..

[Claus]

I just tried to install it a few times on the current release version, but I get this same error every time:

Error: Uncaught Error: [] operator not supported for strings in /Users/me/git/site.net/site/modules/SimpleContactForm/SimpleContactForm.module:131
Stack trace:
#0 /Users/me/git/site.net/wire/core/Modules.php(602): ProcessWire\SimpleContactForm->init()
#1 /Users/me/git/site.net/wire/core/Modules.php(477): ProcessWire\Modules->initModule(Object(ProcessWire\SimpleContactForm))
#2 /Users/me/git/site.net/wire/core/ProcessWire.php(495): ProcessWire\Modules->triggerInit()
#3 /Users/me/git/site.net/wire/core/Wire.php(380): ProcessWire\ProcessWire->___init()
#4 /Users/me/git/site.net/wire/core/WireHooks.php(698): ProcessWire\Wire->_callMethod('___init', Array)
#5 /Users/me/git/site.net/wire/core/Wire.php(442): ProcessWire\WireHooks->runHooks(Object(ProcessWire\ProcessWire), 'init', Array)
#6 /Users/me/git/site.net/wire/core/ProcessWire.php(587) (line 131 of /Users/me/git/site.net/site/modules/SimpleContactForm/SimpleContactForm.module) 

This error message was shown because: you are logged in as a Superuser. Error has been logged.

 

[szabesz]

18 minutes ago, Claus said:

[] operator not supported

PHP version? 7.1? Or something way older?

https://www.google.hu/search?q="[]+operator+not+supported"+php&oq="[]+operator+not+supported"+php

[Claus]

10 hours ago, szabesz said:

PHP version? 7.1? Or something way older?

https://www.google.hu/search?q="[]+operator+not+supported"+php&oq="[]+operator+not+supported"+php

I’m on 7.1.5

[szabesz]

3 hours ago, Claus said:

I’m on 7.1.5

So you either have to downgrade to PHP 7.0.x or wait for @justb3a to update the module. You can "hack it" too, either by actually changing the module's code or you can try the "multiple copies of the same module" technique:

https://processwire.com/blog/posts/processwire-core-updates-2.5.14/

I never did the latter, BTW.

[Claus]

I downgraded to 7.0.5 and that works without issues. Thanks for the help!

[mel47]

Hi,

I feel very stupid. I read all the thread and all the documentation. But I'm unable to hide scf-website by css. How I can add class='hidden' to this specific field?

			$scf = $modules->get('SimpleContactForm');

						$options = array(
						 'btnClass' => 'button is-primary',
						 'successMessage'  => 'Message Sent!',
						 'errorMessage'  => 'Sorry - error, your message was not sent!',
						 
						  'markup' => array(
							'list' => "<div class='field' {attrs}>{out}</div>",
							'item_label' => "<label class='label' for='{for}'>{out}</label>",
							'item' => "<div class='control'>{out}</div>",
							'item_error' => "<article class='message is-warning'><div class='message-body'>{out}</div></article>",
						  ),
			
						);

						echo $scf->render($options);

Thanks

Melanie

[alexm]

@mel47 How do you mean?

If you add the following to your CSS file this should work

#wrap_Inputfield_scf-website, 
#wrap_scf-date {
	display: none;
}

That should work

[justb3a]

It's not that easy to access only this field, you could add a class for every "InputfieldText" for example.

But why not extending your CSS as @alexmercenary suggested? If you do not want to use an ID as CSS selector you could use classes as well.

This is the default markup:

<div class="form__item form__item--scf-website" id="wrap_Inputfield_scf-website">...</div>
<div class="form__item form__item--scf-date field--required" id="wrap_scf-date">...</div>

As you can see, every field gets wrapped using the class form__item--{fieldname}. Just extend your CSS:

.hidden, 
.form__item--scf-website {
    display: none;
    visibility: hidden;
}

[mel47]

Ok thanks! Have played too much with markup and classes options (incorrectly). Everything is fine now.

Mel

[Mike Rockett]

On 7/19/2017 at 7:16 AM, Claus said:

I just tried to install it a few times on the current release version, but I get this same error every time:

[...]

As a follow-up to this, where exacty is allFields first declared before init() is called? Not sure what's going on there, because that's supposed to be an array, but it looks like PHP is calling it a string...?

[justb3a]

@Mike Rockett It's declared in `SimpleContactFormConfig.php:19`. It was declared as string (oops, my fault ) but should be of type array. I've fixed it.

[Mike Rockett]

3 hours ago, justb3a said:

@Mike Rockett It's declared in `SimpleContactFormConfig.php:19`. It was declared as string (oops, my fault ) but should be of type array. I've fixed it.

Ah right, didn't actually think it was a config property (wasn't looking properly). 

[Claus]

I just realised that a user can omit filling out any of the form fields, then press 'Send', and an email will be sent. Am I doing something wrong, or how can I stop this from happening?

[szabesz]

5 hours ago, Claus said:

how can I stop this from happening

The module uses ProcessWire's own form API; you just need to set the required fields to be required in the admin.

[Claus]

2 hours ago, szabesz said:

The module uses ProcessWire's own form API; you just need to set the required fields to be required in the admin.

Bingo! Thanks.

[maxf5]

Uff, after a few weeks i have a problem with the checkboxes in the form.
They 're not required, but the user can't send the form when not checked.

Anyone with same problems?

Iam using two different forms on my website (different pages) which haven't the same fields,
i set the fields for the particular form with:

$options = array(
  'allFields' => 'sfc_tel,scf_fax,scf_email,scf_subject,scf_message,scf_haendlerwerden',
  ..
);

scf_haendlerwerden is a checkbox, not-required. 
Is that where the error comes from, selecting the fields via $options["allFields"] ?

ProcessWire 3.0.75

[maxf5]

Any ideas? How do you set up checkboxes?

[justb3a]

On 21/09/2017 at 9:14 PM, maxf5 said:

Any ideas? How do you set up checkboxes?

Hei, I found some time to reproduce your issue. The logfile (BE > Logs > Simplecontactform-log) contains the reason what's going wrong:

> [FAILURE] Number of fields does not match.

This means that the number of submitted fields does not match the number of fields which are present in the form. Looking further it turned out that $this->input->post does not contain the checkbox field (if it wasn't checked). 

It's a standard browser behaviour that the value of a checkbox is only sent if the checkbox was checked. This means I have to exclude this fieldtype when comparing number of fields / counting fields. coming soon...

 

Edit: Just published a new release / version 1.0.5. Please update.

[maxf5]

@justb3a, it's working now with the update to 1.0.5, huge thanks for your time and patience!

[titanium]

Hi justb3a, the module is amazing! I think I found a small glitch: if one tries to enter more than one field name in "Create and add fields" (comma separated), ProcessWire throws an error:

 ProcessWire: ProcessModule: Field name 'scf__contact_lastname' may not have double underscores because this usage is reserved by the core

Field names I tried with are like this: contact_title, contact_firstname, contact_lastname...
If this field names are entered in one by one, all is well.

PW version 3.0.76

[justb3a]

@titanium The problem wasn't that you entered multiple fieldnames at once. The comma-separated may not include spaces. So entering `contact_title,contact_firstname,contact_lastname...` does work. I added a hotfix which removes the spaces before parsing the string.