bitte um verbesserungsvorschläge : website / eigens modul

Seite 1 von 1 - Forum: Coding Stuff auf overclockers.at

URL: https://www.overclockers.at/coding-stuff/bitte_um_verbesserungsvorschlaege_website_eigens_m_120661/page_1 - zur Vollversion wechseln!

bin grad am fertigwerden von der webpage für meine eltern (phpnuke) und hab dabei auch als php neuling ein modul selbst schreiben / erweitern müssen, da es nix fertiges gegeben hat: http://www.gaertnerwirt.com

es ist das modul "tischreservierung" welches einfach die daten an ne email schickt:

als erster das file selbst, nacher das deutsche sprachfile, das die ausgabe "eindeutscht"

Code: PHP

<?php


if (!eregi("modules.php", $_SERVER['PHP_SELF'])) {
    die ("You can't access this file directly...");
}

require_once("mainfile.php");
$module_name = basename(dirname(__FILE__));
get_lang($module_name);


$index = 1;
$subject = "$sitename "._FEEDBACK."";


include("header.php");
$cookie[0] = intval($cookie[0]);
if ($cookie[1] != "") {
    $row = $db->sql_fetchrow($db->sql_query("SELECT name, username, user_email FROM ".$user_prefix."_users WHERE user_id='$cookie[0]'"));
    if ($row['name'] != "") {
	$sender_name = $row['name'];
    } else {
	$sender_name = $row['username'];
    }
    $sender_email = $row['user_email'];
}

$form_block = "
    <center><font class=\"title\"><b>$sitename: "._FEEDBACKTITLE."</b></font>
    <br><br><font class=\"content\">"._FEEDBACKNOTE."</font>
    <FORM METHOD=\"post\" ACTION=\"modules.php?name=$module_name\">
    <P><strong>"._YOURNAME.":</strong><br>
    <INPUT type=\"text\" NAME=\"sender_name\" VALUE=\"$sender_name\" SIZE=50></p>
    <P><strong>"._YOUREPHONE.":</strong><br>
    <INPUT type=\"text\" NAME=\"sender_phone\" VALUE=\"$sender_phone\" SIZE=20></p>
    <P><strong>"._YOUREMAIL.":</strong><br>
    <INPUT type=\"text\" NAME=\"sender_email\" VALUE=\"$sender_email\" SIZE=30></p>
    <P><strong>"._YOUREDATUM.":</strong><br>
    <INPUT type=\"text\" NAME=\"sender_datum\" VALUE=\"$sender_datum\" SIZE=15></p>
    <P><strong>"._YOUREZEIT.":</strong><br>
    <INPUT type=\"text\" NAME=\"sender_zeit\" VALUE=\"$sender_zeit\" SIZE=15></p>
    <P><strong>"._YOUREANZAHL.":</strong><br>
    <INPUT type=\"text\" NAME=\"sender_anzahl\" VALUE=\"$sender_anzahl\" SIZE=15></p>
    <P><strong>"._YOUREWUNSCH.":</strong><br>
    <INPUT type=\"text\" NAME=\"sender_wunsch\" VALUE=\"$sender_wunsch\" SIZE=50></p>
    <P><strong>"._MESSAGE.":</strong><br>
    <TEXTAREA NAME=\"message\" COLS=30 ROWS=5 WRAP=virtual>$message</TEXTAREA></p>
    <INPUT type=\"hidden\" name=\"opi\" value=\"ds\">
    <P><INPUT TYPE=\"submit\" NAME=\"submit\" VALUE=\""._SEND."\"></p>
    </FORM></center>
";

OpenTable();
if ($opi != "ds") {
    echo "$form_block";
} elseif ($opi == "ds") {
    if ($sender_name == "") {
	$name_err = "<center><font class=\"option\"><b><i>"._FBENTERNAME."</i></b></font></center><br>";
	$send = "no";
    } 
    if ($sender_phone == "") {
	$email_err = "<center><font class=\"option\"><b><i>"._FBENTEREMAIL."</i></b></font></center><br>";
	$send = "no";
    } 
    if ($sender_datum == "") {
    	$message_err = "<center><font class=\"option\"><b><i>"._FBENTERMESSAGE."</i></b></font></center><br>";
	$send = "no";
    } 
    if ($send != "no") {
	$sender_name = removecrlf($sender_name);
	$sender_email = removecrlf($sender_email);
	$msg = "$sitename\n\n";
	$msg .= ""._SENDERNAME.": $sender_name\n";
	$msg .= ""._SENDERPHONE.": $sender_phone\n";
	$msg .= ""._SENDEREMAIL.": $sender_email\n";
	$msg .= ""._SENDERDATUM.": $sender_datum\n";
	$msg .= ""._SENDERZEIT.": $sender_zeit\n";
	$msg .= ""._SENDERANZAHL.": $sender_anzahl\n";
	$msg .= ""._SENDERWUNSCH.": $sender_wunsch\n";
	$msg .= ""._MESSAGE.": $message\n\n";
	$to = $reservationmail;
	$mailheaders = "From: $sender_name <$sender_email>\n";
	$mailheaders .= "Reply-To: $sender_email\n\n";
	mail($to, $subject, $msg, $mailheaders);
	echo "<P><center>"._FBMAILSENT."</center></p>";
	echo "<P><center>"._FBTHANKSFORCONTACT."</center></p>";
    } elseif ($send == "no") {
	OpenTable2();
	echo "$name_err";
	echo "$email_err";
	echo "$message_err";
	CloseTable2();
	echo "<br><br>";
	echo "$form_block";  
    } 
}

CloseTable();   
include("footer.php");

?>

Code: PHP

<?php

define("_SEND","Senden");
define("_YOURNAME","Ihr Name & Firma");
define("_MESSAGE","Sonstiges");
define("_YOUREMAIL","Ihre eMailadresse");
define("_YOUREANZAHL","Personenanzahl");
define("_YOUREDATUM","Datum");
define("_YOUREZEIT","Uhrzeit");
define("_YOUREWUNSCH","Besondere Wünsche");
define("_YOUREPHONE","Ihre Telefonnummer");
define("_FEEDBACKNOTE","Bitte geben Sie uns Ihre Tischreservierung rechtzeitig bekannt (für den gleichen Tag bitte bis 10 Uhr) ");
define("_FEEDBACKTITLE","Reservierungs Formular");
define("_FEEDBACK","reservierung");
define("_FBENTERNAME","Fehler: Bitte geben Sie ihren Namen ein!");
define("_FBENTEREMAIL","Fehler: Bitte geben Sie ihre eMailadresse und Telefonnummer ein!");
define("_FBENTERMESSAGE","Fehler: Bitte geben Sie die Reservierung ein!");
define("_SENDEREMAIL","eMail des Absenders");
define("_SENDERPHONE","Telefon des Absenders");
define("_SENDERNAME","Name des Absenders");
define("_SENDERDATUM","Datum der Reservierung");
define("_SENDERZEIT","Uhrzeit der Reservierung");
define("_SENDERANZAHL","Personenanzahl");
define("_SENDERWUNSCH","Wünsche");
define("_FBMAILSENT","eMail wurde versendet!");
define("_FBTHANKSFORCONTACT","Danke, dass Sie uns kontaktierten.");

?>

zum code hab ich keine verbesserungsvorschläge(sollte passen ), aber du könntest die ganzen formulare usw. ganz links machen , sieht besser aus


es gibt ein modul auf http://www.warp-speed.de , da kann man ein eig. menü bauen, welches besser aus sieht als das standard menü

btw. wenn du eine version unter 6.0 verwendest, darfst du das © entfernen, bei allen versionen darüber darfst dus nicht entfernen

EDIT:


anfahrtsplan wär auch gut

Zitat

evtl. hab ich mir auch gedacht das ganze in zusätzlich in ner datenbank zu speichern, nur hab ich im moment keine ideen dazu

foto album hat fehler im mozilla, die schrift geht ins bild rein und wird somit unleserlich

nochwas...wer braucht private messages bei ner homepage eines gasthauses?

außerdem würde ich die "hauptgrafik" etwas kontrastreicher und kräftiger gestalten

ich finde ja CMS ganz nett und schön nur was ich einfach hässlich finde ist das man jedes CMS am ersten Blick erkennt!
Daher Menüaufbau & Co sind gleich!

ausserdem solltest du einige sachen überarbeiten
-PM
-Online
-Login
kannst du dir sparen denn diese sorgen nur zu verwirrung

-Vote kannst im prinzip auch weglassen denn immer die selbe Umfrage wird fad (ausser du änderst sie täglich auf (Wie hat ihnen unser Tagesmenü geschmeckt)

Das Tischreservierungsscript ist zwar ganz nett allerdings würde ich es schon an eine Datenbank anbinden. Um einfach leichter die übersicht zu behalten.

Den <center> tag für die Formulare würde ich auch kicken sieht nicht besonders toll aus

Verkleinere die Felder für Uhr und Datum zb auf

DD MM YYYY
und für die Uhr könntest du evtl. ein Pulldown machen!

Da ich selbst gerade so eine Programm geschrieben habe hatte ich anfangs die selben Fehler! Jetzt läuft es allerdings ganz gut!


Add 2

Würde ich das News template nochmals etwas überarbeiten
denn das genau Datum
- User
-Punkte
-.. sind glaube ich für diese Branche nicht wirklich wichtig und stören nur!


Ich kenne die Geschäftspolitk deiner Eltern nicht und habe auch keine ahnung für welche Zielgruppen das lokal ist aber

Anfahrtsplan
Öffnungszeiten
Speisekarte
Spezialitäten
Wochenmenükarte (wird wöchentlich aktualisiert und auf wunsch per mail verschickt)

würden sich ganz gut machen!

overclockers.at v4.thecommunity
© all rights reserved by overclockers.at 2000-2025