Looks like you need to copy and paste those links into your browser for them to work - cheers.

Only thing I can think of... Do you have the placeholder for the telephone number? Is is the correct casing?

Cheers,

/Dirk

Having a quick skim through the code this seems fine. Did you build this into a DLL and drop it into the BIN directory?
Did you update the XSLT to show the new field in /data/form_log.xslt ?

Warren

In your code file I cannot see any reference at the top to the phonenumber string ?

I changed the contact form on my siteto this, but it appears to be missing from yours...

private string _EmailTo, _EmailSubject, _EmailBody, _EmailReplyFrom, _EmailReplySubject, _EmailReplyBody, _FormHeader, _FormText, _ThankYouHeaderText, _ThankYouMessageText, _PhoneNumber;

public string PhoneNumber
            {
                get
                {
                    return _PhoneNumber;
                }
                set
                {
                    _PhoneNumber = value;
                }
            }

Hi Ian,
The contact form doesnt really need that unless your passing in a value to PhoneNumber. For example all those variables are being passed into the macro from the current node with propertes on the document type to set the email to, email from, the email message etc...

Warren

Ah my mistake then, just thought that was the only difference between our code.

@Dirk De Grave - yes I think I have maintained the casing. I've followed Warren's original fields and placeholders faithfully and cross referenced with against all the files.

@Warren Buckley - no I haven't created a dll - it didn't look like it was compliled code (I'm very inexperienced in .net so tell me otherwise) - I couldn't find any references in /data/form_log.xslt other than when it calls <xsl:for-each select="field">, which are defined in your control.

@ianhoughton - those strings are referencing the properties from Warren's document types. They're so the site admin can content manage the generated customer reply email - rather than the one created by the customer using the site enquiry form - if you folllow me?

Cheers guys, appreciate your help. Hopefully some light will be thrown on this little nudget :)

OK yes you won't need to update the XSLT then. It would be worth checking the XML file to see if your new field is being added at all.

Yeh going from memory I dont think this is compiled code as I wanted easy for beginners to pickup/hack/learn.
So replacing the .ascx and .ascx.cs should be ok for it to work.

Warren

Tested form and got your auto email reply sent to me so the rest of the logic is fine.

@Warren Buckley well I didn't check the XML and you're quite right it is being written to the XML, so it's just not being written to the email body- that narrows it down a bit !

Also I guess I need to look at your dashboard control to display the telNumber within the Umbraco UI.

Seems we're getting warmer! :)

Ah ok, yes of course, the [TelNumber] placeholder needs to be on the content node!

Looks like I need to amend the log_script.js and ContactForm_logs.ascx to get the logs to display on the dashboard.

This great help. Many thanks!

Not a problem dont forget to mark one of the replies as the solution. So everyone knows that is thread is solved/closed.

Warren :)

is working on a reply...