Noget hjælp til min side functions.php

Linket hedder selvfølgelig http://lerduen.h4f.dk/simpleauth/join.php !!!

Jeg vil hvertfald gerne se på det, du kan sende scriptet til mig på ->
alexander_hv@hotmail.com

i din include-fil:

if (noget) {

$fejl = "hovsa.. bla bla"; }


og på selve siden (et sted mellem de røde bjælker):

if ($fejl) { echo $fejl; }



du burde lave en tabel istedet for at bruge <br> til mellemrum-justeringen

:)

hov.. du HAR brugt en tabel, bare på en lidt suspekt måde. jeg troede det var <br> da jeg omringede hele teksten med musen.

behandlingen af formen kunne være:

if ($submit) {

$status=0;
if (!$brugernavn) {
$fejl1 = "Brugernavn skal udfyldes før du kan blive oprettet<br>"; $status++; }
elseif (strlen($brugernavn)<4) {
$fejl1 = "Brugernavn er ugyldig, minimumslængden er 4 tegn<br>"; $status++; }

if (!$kodeord) {
$fejl2 = "Kodeord skal udfyldes før du kan blive oprettet<br>"; $status++; }
elseif (strlen($kodeord)<4) {
$fejl2 = "Kodeord er ugyldig, minimumslængden er 4 tegn<br>"; $status++; }

if (!$gentag) {
$fejl2 = "Gentag kodeord skal udfyldes før du kan blive oprettet<br>"; $status++; }
elseif ($gentag!=$kodeord) {
$fejl3 = "Dine passwords var ikke identiske<br>"; }

if ($status>0) { $fejl = $fejl1.$fejl2.$fejl3; } else { $fejl=""; }
}

$fejl3 mangler lige en $status++; til sidst.

og så vil jeg råde dig til at tilføje noget i stil med:

elseif (!eregi("^[ a-zæøå0-9_@,\.-]+$",$brugernavn)) { $fejl1="ugyldig(e) tegn i brugernavnet<br>"; $status++; }

Jeg er lige kommet på en ide til hvordan man kan omgås problemet. Hvis meddelsen popper op på en lille advarsels-side ville det være perfekt. Så bliver der jo ikke rykket ved noget når der advares og brugeren skal bare lukke vinduet igen. Men hvordan laver man sådan et lille pop-up vindue? Den må gerne komme frem lige som de belastende reklamer der nogen gange dukker op på siderne ;-)

lerduen ->
Hvilket lyder som en dårlig ide, da nogle har popupkillers og vil derfor ikke få beskeden, jeg tilbyder stadig at gøre det for dig hvis du sender scriptet til mig, på nævnte mail.

har du prøvet min løsning? det er såre simpelt..

ahv koden er nu sendt til dig :-)

he he ohmish... Det som måske er såre simplet for dig er temmelig hårdkogt for mig :-(

Lad mig lige stille nogle indledende spørgsmål. Du vil altså flytte checks'ne fra functions.php til join.php og kontrollere dem med det samme for at få meddelserne til at dukke op lad os sige oppe over login-boksen?

Det jeg har at arbejde med i functions.php ser sådan her ud ...


function doCSS() {
    /*
    CSS Output:
    This function simply outputs some cascading style sheet data for markup by the
    user's browser.
    */
    ?>
<style type="text/css">
body{font-family: Arial, Helvetica; font-size: 10pt}
h1{font-size: 12pt}
</style>
    <?php
} // end func doCSS()

# function validates HTML form field data passed to it:
function field_validator($field_descr, $field_data, $field_type, $min_length="", $max_length="", $field_required=1) {
    /*
    Field validator:
    This is a handy function for validating the data passed to us from a user's <form> fields.
    Using this function we can check a certain type of data was passed to us (email, digit, number, etc)
    and that the data was of a certain length.
    */
    # array for storing error messages
    global $messages;
   
    # first, if no data and field is not required, just return now:
    if(!$field_data && !$field_required){ return; }

    # initialize a flag variable - used to flag whether data is valid or not
    $field_ok=false;

    # this is the regexp for email validation:
    $email_regexp="^([a-zA-Z0-9_\-\.]+)@((\[[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\.)|";
    $email_regexp.="(([a-zA-Z0-9\-]+\.)+))([a-zA-Z]{2,4}|[0-9]{1,3})(\]?)$";

    # a hash array of "types of data" pointing to "regexps" used to validate the data:
    $data_types=array(
        "email"=>$email_regexp,
        "digit"=>"^[0-9]$",
        "number"=>"^[0-9]+$",
        "alpha"=>"^[a-zA-Z]+$",
        "alpha_space"=>"^[a-zA-Z ]+$",
        "alphanumeric"=>"^[a-zA-Z0-9]+$",
        "alphanumeric_space"=>"^[a-zA-Z0-9 ]+$",
        "string"=>""
    );
   
    # check for required fields
    if ($field_required && empty($field_data)) {
        $messages[] = "$field_descr skal udfyldes før du kan blive oprettet";
        return;
    }
   
    # if field type is a string, no need to check regexp:
    if ($field_type == "string") {
        $field_ok = true;
    } else {
        # Check the field data against the regexp pattern:
        $field_ok = ereg($data_types[$field_type], $field_data);       
    }
   
    # if field data is bad, add message:
    if (!$field_ok) {
        $messages[] = "Vær venlig at indtaste et gyldigt $field_descr.";
        return;
    }
   
    # field data min length checking:
    if ($field_ok && $min_length) {
        if (strlen($field_data) < $min_length) {
            $messages[] = "$field_descr er ugyldig, minimumslængden er $min_length tegn.";
            return;
        }
    }
   
    # field data max length checking:
    if ($field_ok && $max_length) {
        if (strlen($field_data) > $max_length) {
            $messages[] = "$field_descr er ugyldig, maximumlængden er $max_length tegn.";
            return;
        }
    }
}

Jeg vender tilbage så snart jeg hr set på det, har ikke modatget mailen endnu.

lerduen >> Sikker på du sendte til ->
alexander_hv@hotmail.com
har nemlig ikke modtaget scriptet endnu.

>>> Ohmish

Hør hov, det ser sgu da ud til at join.php netop er kodet som du der foreslår, men at jeg bare ikke helt har været med på hvad der var galt (jf. at jeg er TOTAL nybegynder) . Jeg nusser lige lidt rundt med tingene og vende evt. tilbage med point

ahv... min fejl :-( Den lange tekst var åbenbart ikke så villig til at forlade udbakken :-(

Sorry nu skulle den være der :-)

Jeg har modtaget, det ser blot ud til at du skal flytte:

<?php

//Check if $message is set, and output it if it is:

if(!empty($messages)){

            displayErrors($messages);

}

?>

som lægger i join.php, alle de linier flytter du så dertil hvor du vil have de,

Så hvis vi nu tager hele tabellen på join.php (Laaang post) Jeg har indsat php lininer det rigtig sted nu skulle jeg mene:




<table border="0" cellpadding="0" cellspacing="0" style="border-collapse: collapse" width="100%" id="AutoNumber1">

  <tr>

    <td width="25%">

    <p align="center">&nbsp;</td>

    <td width="75%" bgcolor="#CC0000" colspan="3" align="center">

    <font face="Impact" size="5" color="#FFFFFF">Opret bruger</font><p>&nbsp;</td>

  </tr>

  <tr>



  </tr>

  <tr>

    <td width="25%">

    <p align="center">&nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

  </tr>

  <tr>

    <td width="25%">

    <p align="center">&nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

  </tr>

  <tr>

    <td width="25%">

    <p align="center">&nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

  </tr>

  <tr>

    <td width="25%">

    <p align="center">&nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

  </tr>

  <tr>

    <td width="25%">

    <p align="center">&nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">
    <?php

    //Check if $message is set, and output it if it is:

    if(!empty($messages)){
        displayErrors($messages);
    }

?>

                        <form action="<?=$_SERVER["PHP_SELF"]?>" method="POST">

                        <table>

                        <tr><td>Brugernavn:</td><td><input type="text" name="login" value="<?php print $_POST["login"] ?>"



                        maxlength="15"></td></tr>

                        <tr><td>Kodeord:</td><td><input type="password" name="password" value="" maxlength="15"></td></tr>

                        <tr><td>Gentag kodeord:</td><td><input type="password" name="password2" value="" maxlength="15"></td></tr>

                        <tr><td>&nbsp;</td><td><input name="submit" type="submit" value="Opret"></td></tr>

                        <tr><td>&nbsp;</td><td><input type="reset" name="reset" value="Slet"></td></tr>

                        </table>

                        </form>

    <p>&nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

  </tr>

  <tr>

    <td width="25%">

    <p align="center">&nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

  </tr>

  <tr>

    <td width="25%">

    <p align="center">&nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

  </tr>

  <tr>

    <td width="25%">

    <p align="center">&nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

  </tr>

  <tr>

    <td width="25%">

    <p align="center">&nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

  </tr>

  <tr>

    <td width="25%">

    <p align="center">&nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

  </tr>

  <tr>

    <td width="25%">

    <p align="center">&nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

  </tr>

  <tr>

    <td width="25%">

    <p align="center">&nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

  </tr>

  <tr>

    <td width="25%">

    <p align="center">&nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

  </tr>

  <tr>

    <td width="25%">

    <p align="center">&nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

    <td width="25%" bgcolor="#FFFFFF">

    &nbsp;</td>

  </tr>

  <tr>

    <td width="25%">

    &nbsp;</td>

    <td width="75%" bgcolor="#CC0000" colspan="3">

    &nbsp;</td>

  </tr>

  <tr>

    <td width="25%">

    &nbsp;</td>

    <td width="75%" bgcolor="#CC0000" colspan="3">

    <p align="center">&nbsp;</td>

  </tr>

</table>

Det var satans! Du havde sgu ret... meget logisk egentligt når nu jeg ser på det med oplyste øjne :-)

Takker for hjælpen

Nu bliver jeg forvirret, havde jeg ret eller var det ohmish?

Skulle den næsten ikke mindst have været lige over? Efter som jeg kom med løsningen, og allerede helt fra start tilbød min hjælp, men skidt.

jah, i har jo begge fuldstændig ret! Ohmish kom først men det var muligvis kun fordi koden blev sendt så sent.

Jeg har delt point ud en anelse skævt, men håber at i begge føler jeg retfærdigt behandlet :-)

Takker MANGE gange for hjælpen

Jeg er sgu stadig imponeret over at Ohmish kunne gætte løsningen uden at se koden.

og jeg er også imponeret over ahv's promte svar efter koden blev sendt.

I er sgu begge nogle dygtige programmører (efter mine lave standarder)

Håber ikke der er nogen sure miner :-)