wxcontact.php page errors!

[William Grimsley]

Hi Ken,

I've just installed the more detailed version of the wxcontact.php page, successfully.

The only problem I have is the "Message:" field.

I've made it the same width as all the other fields, but I don't know how to change the height of the field?

I wan't to this so that people can see more of there message, instead of scrolling back through a long single line.

How do I do this?

Thanks,

William

[beteljuice]

William - It is not an error just because you don't like it !

You could of course look up how to handle forms and not be so impatient

You could try changing:

Code: Select all

 <div style="text-align:left;">
                 <input style="text-align:left; margin:0;" type="text" id="fsc_message1" name="fsc_message" value=""  size="40" />
        </div>

To something like:

Code: Select all

 <div style="text-align:left;">
                 <textarea style="text-align:left; margin:0;" id="fsc_message1" name="fsc_message" value=""  cols="40"  rows="6" />
        </div>

[William Grimsley]

Hi beteljuice,

Yes, I have done that before. But, like I always am, I wan't to be precise. The message field is not the same width as all the other fields.

Thanks for your help,

William

[William Grimsley]

Hi Ken,

Ok, now I've got this error.

When I type in the subject field "Hello" and then I send the email, the email comes back as "Feedback from http://www.newton-poppleford-weather.co.uk site for Web Master"

Why is this happening?

Here is some images to help:

contact.jpg

sent.jpg

Thanks,

William

[beteljuice]

William, you have moments of brilliance, and sometimes you blow it !

Take a look at how the form information is handled in wxcontact-send.php

Note that Hello is still part of the (extended) subject content.

[William Grimsley]

Yes, I know. I blow it all the time, just like with my ajax-dashboard.php file this evening, that I had to re-do!

Ok, I've looked through the wxcontact-send.php file and found this:

Code: Select all

Subject: <?php echo "$subject"; ?>

Also, I've looked through the file wxcontact.php file and found this:

Code: Select all

value="<?php echo $subject; ?>"

To me, that should use the subject in the wxcontact.php file, as the same as in the wxcontact-send.php file...

[beteljuice]

You keep using the emotive word 'error' when it isn't - no better way to get a script writers back up

The script (in common with many mail forms) is designed to inform you, the receiver, where the mail truly originated from. ie. your site.

Somewhere, most likely in wxcontact-send.php, that phrase / formating you are taking an exception to, is being ADDED to the passed subject value.

I personally would leave it alone, not least because as we are always telling people who modify templates - YOU ARE ON YOUR OWN WITH UPDATES.

PS. Hey look, your message field is what you want now ...... Take a deep breath, hold it, do not use it to blurt out babblecrap faster than you can think, savour it, then take another and do something else entirely different.

[William Grimsley]

Hi beteljuice,

Hehe! I've managed to do it now, after putting this code into like you said "wxcontact-send.php"

I changed:

Code: Select all

// Get the request variables
$visitormail = $_POST['visitormail'];
$visitor = $_POST['visitor'];
$notes = $_POST['notes'];
$attn  = $_POST['attn'];
$ip    = $_POST['ip'];
$httpref = $_POST['httpref'];
$httpagent = $_POST['httpagent'];

To

Code: Select all

// Get the request variables
$visitorsubject = $_POST['visitorsubject'];
$visitormail = $_POST['visitormail'];
$visitor = $_POST['visitor'];
$notes = $_POST['notes'];
$attn  = $_POST['attn'];
$ip    = $_POST['ip'];
$httpref = $_POST['httpref'];
$httpagent = $_POST['httpagent'];

Then,

Code: Select all

$subject = "Feedback from " . $_POST['visitorsubject'] ." site for " . trim($attn);

To

Code: Select all

$subject = "" . $_POST['visitorsubject'] ."";

Thanks for you help,

William