|
Tytuł: Formularz kontaktowy nie wysy?a wiadomo?ci Wiadomość wysłana przez: Sandra 10.07.2010 21:23:16 Witam,
Zainstalowa?am Advanced contact form (http://custom.simplemachines.org/mods/index.php?mod=1352) i mam perien problem: zg?asza b??d mimo poprawnie wpisanego adresu e-mail, nie umiem znale?? b??du w skrypcie :( Code: [select] <?php /********************************************************************************** * Contact.php * *********************************************************************************** * * * Execution file for "Contact Form" modification. * * * * Copyright (c) 2008 by Deprecated (at) Earthlink (dot) net. All rights reserved. * * Redistribution prohibited except at SimpleMachines.org * * * **********************************************************************************/ if (!defined('SMF')) die('Hacking attempt...'); function Contact() { global $context, $settings, $options, $scripturl, $boarddir, $sourcedir, $modSettings, $user_info, $txt, $webmaster_email; if($user_info['is_guest'] && empty($modSettings['contact_form_enable_guest_access'])) redirectexit(); if(!$user_info['is_guest'] && empty($modSettings['contact_form_enable_member_access'])) redirectexit(); $global_error = false; if (!empty($modSettings['reg_verification']) && $context['user']['is_guest']) { require_once($sourcedir . '/Subs-Editor.php'); $verificationOptions = array( 'id' => 'register', ); if (isset($_POST['submit'])) { $context['visual_verification'] = create_control_verification($verificationOptions, true); if (is_array($context['visual_verification'])) { loadLanguage('Errors'); foreach ($context['visual_verification'] as $error) { $global_error = $context['contact_form_error_visual_verification'] = true; $context['visual_verification_error_dscr'] = $txt['error_' . $error]; } } } $context['visual_verification'] = create_control_verification($verificationOptions); $context['visual_verification_id'] = $verificationOptions['id']; } else $context['visual_verification'] = false; $content = ''; $sender_ip = $_SERVER['REMOTE_ADDR']; $sender_rdns = gethostbyaddr($sender_ip); if ($context['user']['is_guest']) $sender = $txt['contact_form_sender_is_guest']; else { $sender = $context['user']['name']; if ($context['user']['name'] != $context['user']['username']) $sender .= ' (' . $context['user']['username'] . ')'; } $message = isset($_POST['message']) ? trim($_POST['message']) : ''; if (!$message) $global_error = $context['contact_form_error_no_message'] = true; $sender_name = isset($_POST['name']) ? trim($_POST['name']) : ''; if (!$sender_name && !empty($modSettings['contact_form_require_name'])) $global_error = $context['contact_form_error_no_name'] = true; if (!$sender_name || !empty($modSettings['contact_form_sender_name_dont_use_form'])) { $name = empty($modSettings['contact_form_default_name']) ? $txt['contact_form_default_name_default'] : $modSettings['contact_form_default_name']; if ($sender_name) $content .= $txt['contact_form_label_status_name'] . $sender_name . ' '; } else $name = $sender_name; $sender_email = isset($_POST['email']) ? trim($_POST['email']) : ''; if (!$sender_email && !empty($modSettings['contact_form_require_email'])) $global_error = $context['contact_form_error_no_email'] = true; if ($sender_email && !valid_email($sender_email)) $global_error = $context['contact_form_error_invalid_email'] = true; if (!$sender_email || !empty($modSettings['contact_form_sender_email_dont_use_form'])) { $email = empty($modSettings['contact_form_default_email_sender']) ? $webmaster_email : $modSettings['contact_form_default_email_sender']; if ($sender_email) $content .= $txt['contact_form_label_status_email'] . $sender_email . ' '; } else $email = $sender_email; $headers = 'From: ' . $name . ' <' . $email . '>'; $sender_subject = isset($_POST['subject']) ? trim($_POST['subject']) : ''; if (!$sender_subject && !empty($modSettings['contact_form_require_subject'])) $global_error = $context['contact_form_error_no_subject'] = true; if (!$sender_subject || !empty($modSettings['contact_form_sender_subject_dont_use_form'])) { $subject = empty($modSettings['contact_form_default_subject']) ? $txt['contact_form_default_subject_default'] : $modSettings['contact_form_default_subject']; if ($sender_subject) $content .= $txt['contact_form_label_status_subject'] . $sender_subject . ' '; } else $subject = $sender_subject; if (!empty($modSettings['contact_form_subject_prefix'])) $subject = $modSettings['contact_form_subject_prefix'] . ' ' . $subject; if ($content) $content .= $txt['contact_form_label_status_separator']; $content .= stripslashes($message); $content .= $txt['contact_form_label_ip'] . $sender_ip . $txt['contact_form_label_rdns'] . $sender_rdns . $txt['contact_form_label_sender'] . $sender; $dest_email_addr = empty($modSettings['contact_form_dest_email_addr']) ? $webmaster_email : $modSettings['contact_form_dest_email_addr']; $context['page_title'] = empty($modSettings['contact_form_display_title']) ? $txt['contact_form_title'] : $modSettings['contact_form_display_title']; if (!$global_error && isset($_POST['submit'])) { require_once($sourcedir . '/Subs-Post.php'); $context['contact_form_sendmail_override_headers'] = true; if (sendmail($dest_email_addr, $subject, $content, $headers)) { $context['contact_form_message_sent'] = true; $context['page_title'] .= ' - ' . $txt['contact_form_result_message_sent']; } else { $context['contact_form_message_failed'] = true; $context['page_title'] .= ' - ' . $txt['contact_form_result_send_mail_failed']; } } else { if ($global_error && isset($_POST['submit'])) $context['page_title'] .= ' - ' . $txt['contact_form_result_error']; } $context['linktree'][] = array( 'url' => $scripturl.'?action=contact', 'name' => $context['page_title'], ); loadTemplate('Contact'); } function valid_email($email) { global $modSettings; $at_ptr = strrpos($email, "@"); if (is_bool($at_ptr) && !$at_ptr) return false; $domain = substr($email, $at_ptr+1); $local = substr($email, 0, $at_ptr); $local_len = strlen($local); $domain_len = strlen($domain); if ($local_len < 1 || $local_len > 64 || $domain_len < 1 || $domain_len > 255) return false; // local or domain part length invalid else if ($local[0] == '.' || $local[$local_len-1] == '.') return false; // local part cannot start or end with dot else if (preg_match('/\\.\\./', $local)) return false; // local part cannot have two consecutive dots else if (!preg_match('/^[A-Za-z0-9\\-\\.]+$/', $domain)) return false; // domain part has invalid character else if (preg_match('/\\.\\./', $domain)) return false; // domain part cannot have two consecutive dots else if (!preg_match('/^(\\\\.|[A-Za-z0-9!#%&`_=\\/$\'*+?^{}|~.-])+$/', str_replace("\\\\", "", $local))) // exception for quoted if (!preg_match('/^"(\\\\"|[^"])+"$/', str_replace("\\\\", "", $local))) return false; // local part character invalid except when local part is quoted if (!empty($modSettings['contact_form_skip_dns_check'])) return true; // mod setting to skip the DNS A/MX record check if (function_exists('checkdnsrr')) if (!(checkdnsrr($domain, 'MX') || checkdnsrr($domain, 'A'))) return false; // domain does not have a valid A or MX record else if (!(win_checkdnsrr($domain, 'MX') || win_checkdnsrr($domain, 'A'))) return false; // domain does not have a valid A or MX record return true; } function win_checkdnsrr($host, $type) { if (!empty($host)) { exec('nslookup -type=' . $type . ' ' . escapeshellcmd($host), $result); foreach ($result as $line) if (eregi("^$hostName", $line)) return true; } return false; } ?> Czy kto? mi mo?e pomóc? Z góry dzi?kuj? i pozdrawiam - Sandra Tytuł: Odp: Formularz kontaktowy nie wysy?a wiadomo?ci Wiadomość wysłana przez: roco 12.07.2010 11:23:03 Witaj Sandro :)
Bardzo przepraszam, teraz to ju? w ogóle s? szkodliwe warunki pracy przy komie, a u mnie nie przybywa czasu na luzik i ostatnio w ogóle nie mog?em si? tu pojawia?... sorka :) Na szybko to co? mi w pod?wietlaniu sk?adni si? nie podoba, konkretnie chodzi mi o funkcj? sprawdzania maila: GeSHi -› Code: [select] Tu s? ró?ne warunki i nie podoba mi si? ta linijka, ( z funkcji sprawdzajki), linia ~148: GeSHi -› Code: [select] else if (!preg_match('/^(\\\\.|[A-Za-z0-9!#%&`_=\\/$\'*+?^{}|~.-])+$/', str_replace("\\\\", "", $local))) // exception for quoted hmm wygl?da jakby co najmniej brakowa?o ?rednika na ko?cu --> ; Dlatego przetestuj 3 opcje: 1 - ze ?rednikiem na ko?cu --> ; GeSHi -› Code: [select] else if (!preg_match('/^(\\\\.|[A-Za-z0-9!#%&`_=\\/$\'*+?^{}|~.-])+$/', str_replace("\\\\", "", $local))); // exception for quoted 2 - z return false; GeSHi -› Code: [select] else if (!preg_match('/^(\\\\.|[A-Za-z0-9!#%&`_=\\/$\'*+?^{}|~.-])+$/', str_replace("\\\\", "", $local))) return false; // exception for quoted 3 - i z return true; GeSHi -› Code: [select] else if (!preg_match('/^(\\\\.|[A-Za-z0-9!#%&`_=\\/$\'*+?^{}|~.-])+$/', str_replace("\\\\", "", $local))) return true; // exception for quoted Sprawd?, po kolei czy jest ok, czy s? jakie? b??dy? Funkcja sprawdza równie? rekordy DNS, wi?c mail powinien by? prawdziwy, ponadto wpisa?bym inny ni? mail admina. Na stronie smf.org, tam gdzie ten mod to klikn??em "parse" i nie podoba mi si? w template tego moda warto?? value="" w inpucie maila, poniewa? nie powinno tu by? ?adnych spacji, (w cudzys?owach value), a tym czasem widz? jedn? na ko?cu.. To jest ca?y ten input do wpisywania maila + opis itd: //-------------------------------------------------------------------------------------------------- // Email address //-------------------------------------------------------------------------------------------------- echo' <div style="padding-bottom: 6px;"> <label for="email"><span class="normaltext">', $txt['contact_form_label_email'], '<span class="smalltext"', empty($modSettings['contact_form_require_email']) ? '> ' . $txt['contact_form_field_optional_email'] : ' style="color: red;"> ' . $txt['contact_form_field_required'], '</span></span><br /> <input '; if (isset($context['contact_form_message_sent'])) echo 'readonly="readonly" style="background-color: #F0F0F0;" '; if (isset($_POST['email'])) echo 'value="', trim($_POST['email']), '" '; echo 'type="text" size="24" name="email" id="email"'; if (isset($context['contact_form_error_no_email']) && isset($_POST['submit'])) { echo ' style="border-style: solid; border-color: red;" /> </label> <span class="smalltext" style="color: red;">', $txt['contact_form_error_no_email'], '</span>'; } else if (isset($context['contact_form_error_invalid_email']) && isset($_POST['submit'])) { echo ' style="border-style: solid; border-color: red;" /> </label> <span class="smalltext" style="color: red;">', $txt['contact_form_error_invalid_email'], '</span>'; } else echo ' /> </label>'; echo ' </div>'; //-------------------------------------------------------------------------------------------------- // Subject line //-------------------------------------------------------------------------------------------------- To w linii oko?o ~60 zaczyna si? sam input, czyli ten kawa?ek kodu: GeSHi -› Code: [select] <input '; if (isset($context['contact_form_message_sent'])) echo 'readonly="readonly" style="background-color: #F0F0F0;" '; if (isset($_POST['email'])) echo 'value="', trim($_POST['email']), '" '; echo 'type="text" size="24" name="email" id="email"'; if (isset($context['contact_form_error_no_email']) && isset($_POST['submit'])) { echo ' style="border-style: solid; border-color: red;" /> Jak wida? w value= po ' ,trim($_POST['email']), ' -> w?a?nie tu jest spacja i dopiero cudzys?ów " to wytnij j?, ale daj spacj? linijk? poni?ej zaraz po echo i a'postofie, tam gdzie zaczyna si? type="text" Czyli zapisa?bym ten kawa?ek tak: GeSHi -› Code: [select] <input '; if (isset($context['contact_form_message_sent'])) echo 'readonly="readonly" style="background-color: #F0F0F0;" '; if (isset($_POST['email'])) echo 'value="', trim($_POST['email']), '"'; // wyci?ta spacja na ko?cu przed zamykaj?cym cudzys?owem echo ' type="text" size="24" name="email" id="email"'; // dodana spacja na pocz?tku, zaraz po echo ' if (isset($context['contact_form_error_no_email']) && isset($_POST['submit'])) { echo ' style="border-style: solid; border-color: red;" /> Po za tym podejrza?bym kod, - jestem ciekaw jak wygl?da ten input w przegl?darce w pogl?dzie ?ród?a? Sandro, sorry, na szybko tylko to przychodzi mi do g?owy.. W tym inpucie jest tak, ?e po przetworzeniu wpisanego emaila i tak zostaje tam spacja, przed zamykaj?cym value cudzys?owem " I chyba od tego zacznij, (a dopiero grzebnij w silniku tego moda), czyli zacznij w ./Themes/default/Contact.template.php Dopiero pó?niej! d?ubnij w silniku czyli w ./Sources/Contact.php Jest okrutnie gor?co, martwi? si? o kompy, a roboty i wydarze? nie ubywa.., dlatego we? prosz? na to poprawk?. Oczywi?cie w miar? mo?liwo?ci postaram si? tu zagl?da?., cz??ciej ni? ostatnio, okay? Mam nadziej?, ?e to bzdet i to co tu powypisywa?em, pomo?e :) Pozdrawiam :) roco Tytuł: Odp: Formularz kontaktowy nie wysy?a wiadomo?ci Wiadomość wysłana przez: Sandra 12.07.2010 12:28:53 Cze?? Roco :)
Upa?y faktycznie pioru?skie, ale lepsze (chyba) to ni? plucha ;) Co do formularza... ju? troch? b??dów wyeliminowa?am, ten brak ?rednika te? zauwa?y?am i wpisa?am tam return false;... Poma?u zacz??am eliminowa? potencjalnie z?a kawa?ki kodu (na razie analizuj? tylko contact.php) i zosta?o mi, ?e ?le dzia?a jeszcze to: Code: [select] if (!preg_match('/^"(\\\\"|[^"])+"$/', str_replace("\\\\", "", $local))) return false; i to: Code: [select] if (!(checkdnsrr($domain, 'MX') || checkdnsrr($domain, 'A'))) return false; else if (!(win_checkdnsrr($domain, 'MX') || win_checkdnsrr($domain, 'A'))) return false; Po zakomentowaniu tych linii niby jest ok, ale przepuszcza wiadomo?? z takim adresem e-mail: a@a Nie bardzo profesjonalnie to jest :( Poprawki w pliku contact.template.php nie maj? wp?ywu na dzia?anie formularza (przynajmniej ja ich nie widz?) Pozdrawiam - Sandra Tytuł: Odp: Formularz kontaktowy nie wysy?a wiadomo?ci Wiadomość wysłana przez: Sandra 14.07.2010 10:33:08 Póki co zast?pi?am ca?? tre?? funkcji takim kodem:
Code: [select] if ( !preg_match("#^[A-Za-z0-9](([_\.\-]?[a-zA-Z0-9]+)*)@([A-Za-z0-9]+)(([\.\-]?[a-zA-Z0-9]+)*)\.([A-Za-z]{2,})$#", $email) ) return false; return true; Teraz sprawdza adres e-mail jak nale?y i nie przepuszcza postaci a@a jak poprzednio. Czyli w zasadzie problem jest rozwi?zany, ale.... Chcia?abym jednak, uruchomi? t? funkcj?, skoro autor si? napracowa?, zeby wzbogaci? formularz o dodatkowe sprawdzanie, to warto skorzysta? (kolejne zabezpieczenie przed spamem :) )... generalnie podoba mi si? ten formularz, wi?c b?d? wdzi?czna za pomoc. Sandra Tytuł: Odp: Formularz kontaktowy nie wysy?a wiadomo?ci Wiadomość wysłana przez: roco 20.07.2010 05:36:21 Sandro :)
Podany przez Ciebie kod, jest jak najbardziej w porz?dku. :) To klasyczna sprawdzajka, mo?na by rzec uniwersalna.. Go?ciu od moda, ju? go dalej nie rozwija.. A s? jeszcze i inne mody do eMaili na smf.org To co zastosowa? autor w swoim modzie do wysy?ania poczty, to po za normalnymi mechanizmami, "odsiewczyczymi", zastosowa? autentyczn? sprawdzajk? samych adresów eMail w 2 bazach adresowych. To ju? powa?niejszy "dings", jednak sorry nie chc? Ci? martwi?, ale nie na wszystkich hostingach w PL, mo?na u?y? zastosowanych w modzie narz?dzi, po prostu nie wsz?dzie zadzia?a, nawet w hmm w "wi?kszej wi?kszo?ci", rzek?bym.. Oczywi?cie mo?na wyci?? w kodzie a i tak dostaniesz ca?kiem do rzeczy normalny mechanizm formularza. Moim zdaniem, powinna? "odptaszkowa?" sprawdzanie rekordów DNS czy jak tam jest.. w PA, i w formularzu mo?na wyci??/wy??czy? to: GeSHi -› Code: [select] if (function_exists('checkdnsrr')) if (!(checkdnsrr($domain, 'MX') || checkdnsrr($domain, 'A'))) return false; // domain does not have a valid A or MX record else if (!(win_checkdnsrr($domain, 'MX') || win_checkdnsrr($domain, 'A'))) return false; // domain does not have a valid A or MX record Mo?esz po jednej lini // --> wy??czona linia lub: /* wi?cej w??czonych linii */ Tylko, najlepiej to robi? za pomoc? edytora z pod?wietlan? sk?adni?, a nawet zdecydowanie nale?y! S? formularze i do 1.1.11 i do 2.0 RC3, przetestuj inne ;) Pozdrawiam :) roco Tytuł: Odp: Formularz kontaktowy nie wysy?a wiadomo?ci Wiadomość wysłana przez: Sandra 20.07.2010 08:44:52 Quote: [select] S? formularze i do 1.1.11 i do 2.0 RC3, przetestuj inne Nie jestem za ulepszaniem dobrego :) Tak jak pisa?am wcze?niej, formularz mi si? podoba, spe?nia swoje zadanie - nie b?d? testowa? czego? innego, przynajmniej na razie - dopóki nie ujawni?y si? inne b??dy :) Bardzo dzi?kuj? za pomoc :5: - Sandra |