problem med textarea input

Hvordan gør du, når du henter det ud igen?

Står det sådan her i databasen eller først, når du henter det ud:

Linje 1<br />\r\nLinje 2<br />\r\n

?

det står sådan i databasen. den indsætter <br/> men fjerner ikke rigtigt linjeskiftene

Er det ikke ligsom fordi du tilføjer ekstra linjeskift:
$Nyheder = $Nyheder.nl2br($PostNyhed[$i]).'\r\n';

Og jeg må indrømme at jeg udover det, slet ikke forstår hvordan din kode hænger sammen.
Hvis jeg, og sikker andre, skal forstå det bedre bliver du nok nød til at vise lidt mere kode, bl.a. kan du vise din form, hvordan du henter data og viser dem samt hvordan de data er gemt i din database.

hele admin delen:

// Admins nyhedsredigering
function AdminRedigerNyhederForm(){
    global $MySQL;
   
   
    // Redigering og oprettelse af nyheder.
    function RedigerNyheder($PostNyhed){
        global $MySQL;
       
        // Lav arrayet om til en streng.
        foreach ($PostNyhed as $i => $Nyhed){
       
            if(trim($Nyhed) == null)
                unset($PostNyhed[$i]);
            else
                $Nyheder = $Nyheder.nl2br($PostNyhed[$i]).'\r\n';
       
        }
       
        $Nyheder = str_replace('\r\n\r\n', '', $Nyheder);
       
        // Send afsted til databasen.
        Send('UPDATE `'.$MySQL[Praefix].'Indstillinger` SET `Vaerdi` = "'.$Nyheder.'" WHERE `Variabel` = "Nyheder"');
   
    }
   
    // Til sletning af valgte nyheder.
    function SletNyheder($SkalFjernes){
        global $MySQL;
       
       
        // Hent alle nyheder.
        $HentedeNyheder = Hent('SELECT `Vaerdi` FROM `'.$MySQL[Praefix].'Indstillinger` WHERE `Variabel` = "Nyheder"');
       
        $Nyheder = $HentedeNyheder[Vaerdi];
       
        // Giv os to variabler da vi skal manipulere den ene og vi skal bruge en med det hele i.
        $AlleNyheder = $Nyheder;
       
        // Lav hele massen om til et array.
        $Nyheder = explode("\n", str_replace("\r", '', trim(addslashes($Nyheder))));
       
        for ($i = 0; count($Nyheder) > $i; $i++){
   
            $Nyheder[$i] = stripslashes(trim($Nyheder[$i]));
           
        }
       
        // Og så skal vi søge og fjerne.
        for($i = 0; count($SkalFjernes) > $i; $i++){
       
        if(trim($Nyheder[$i]) == null)
            continue;
       
        $AlleNyheder = preg_replace('#'.$Nyheder[$SkalFjernes[$i]].'\r\n'.'#', '', $AlleNyheder);
        }
       
        $AlleNyheder = str_replace('\r\n\r\n', '', $AlleNyheder);
       
        // Send så den opdaterede streng til databasen
        Send('UPDATE `'.$MySQL[Praefix].'Indstillinger` SET `Vaerdi` = "'.$AlleNyheder.'" WHERE `Variabel` = "Nyheder"');
   
    }
   
    // Skal vi opdatere noget, slette noget, eller skal vi bare komme i gang?
    if(isset($_POST[RedigerNyhederOK]))
        RedigerNyheder($_POST[Nyhed]);
    elseif(isset($_POST[SletValgte]))
        SletNyheder($_POST[Fjern]);
   
    ?>
    <form action="<?php echo $_SERVER[REQUEST_URI]; ?>" method="post">
    <p>Du skal kun indtaste en nyhed i hvert felt, klik på linket under for at tilføje flere bokse. Tomme bokse ignoreres når der gemmes.</p>
    <table width="85%" cellpadding="5" cellspacing="0" border="0" align="center">
        <tr>
            <th width="50%"></th>
            <th align="left" width="45%">Gennemse</th>
            <th align="center" width="5%"><input type="checkbox" onclick="markerAlle(this, this.form);" /></th>
        </tr>
        <?php
       
        // Hent nyheder og tryl det om til et array.
        $Nyheder = Hent('SELECT `Vaerdi` FROM `'.$MySQL[Praefix].'Indstillinger` WHERE `Variabel` = "Nyheder"');
       
        $Nyheder = explode("\n", str_replace("\r", '', trim(addslashes($Nyheder[Vaerdi]))));
       
        $BG_Farve = false;
       
        for ($i = 0; count($Nyheder) > $i; $i++){
       
            if (trim($Nyheder[$i]) == '')
                continue;
           
            $BG_Farve = !$BG_Farve;
           
            // Ryd lidt op.
            $Nyheder[$i] = stripslashes(trim($Nyheder[$i]));
   
        ?>
        <tr <?php if($BG_Farve == true) echo 'class="BG_True"'; else echo 'class="BG_False"' ?>>
            <td align="center">
                <div style="margin-bottom: 2ex;"><textarea rows="3" cols="30" name="Nyhed[]" style="width: 85%;"><?php echo stripslashes(trim($Nyheder[$i])) ?></textarea></div>
            </td><td align="left" valign="top">
                <div style="overflow: auto; width: 100%; height: 10ex;"><?php echo stripslashes(stripslashes(trim($Nyheder[$i]))); ?></div>
            </td><td align="center">
                <input type="checkbox" name="Fjern[]" value="<?php echo $i; ?>" />
            </td>
        </tr>
        <?php
       
        }
       
        ?>
        <tr <?php if($BG_Farve == true) echo 'class="BG_False"'; else echo 'class="BG_True"' ?>>
            <td align="center">
                <div id="ekstraNyhederVoid"></div><div id="ekstraNyheder_Link" style="display: none;">[<a href="java script:void(0);" onclick="ekstraNyhed();">Tilføj ekstra nyhed</a>]</div>
                <script language="JavaScript" type="text/javascript">
                    <!--Lånt fra SMF, må se om jeg kan lave noget lignende selv.-->
                    document.getElementById("ekstraNyheder_Link").style.display = "";
                   
                    function markerAlle(headerfield, checkform, mask){
                   
                        for (var i = 0; i < checkform.length; i++){
                       
                            if (typeof(checkform[i].name) == "undefined" || (typeof(mask) != "undefined" && checkform[i].name.substr(0, mask.length) != mask))
                                continue;
                   
                            if (!checkform[i].disabled)
                                checkform[i].checked = headerfield.checked;
                       
                        }
                    }
                   
                    function ydreHTML(element, toValue){
                       
                        if (typeof(element.outerHTML) != 'undefined')
                            element.outerHTML = toValue;
                       
                        else{
                            var range = document.createRange();
                            range.setStartBefore(element);
                            element.parentNode.replaceChild(range.createContextualFragment(toValue), element);
                       
                        }
                    }
                   
                    function ekstraNyhed(){
                       
                            ydreHTML(document.getElementById("ekstraNyhederVoid"), '<div style="margin-bottom: 2ex;"><textarea rows="3" cols="30" name="Nyhed[]" style="width: 85%;"></textarea></div><div id="ekstraNyhederVoid"></div>');
                       
                        }
                </script>
                <noscript>
                    <div style="margin-bottom: 2ex;"><textarea rows="3" cols="30" style="width: 85%;" name="Nyhed[]"></textarea></div>
                </noscript>
            </td>
            <td colspan="2" valign="bottom" align="right" style="padding: 1ex;">
                <input type="submit" name="RedigerNyhederOK" value="Gem" /> <input type="submit" name="SletValgte" value="Slet valgte" />
            </td>
        </tr>
    </table>
</form>
    <?php
   
}

Hent() og Send() er bare mysql_query()

og så hvor nyheden kommer til at stå

function Nyheder(){
global $MySQL;
   
    $HentetNyheder = Hent('SELECT `Vaerdi` FROM `'.$MySQL[Praefix].'Indstillinger` WHERE `Variabel` = "Nyheder"');
   
    $Nyheder = explode("\n", str_replace("\r", '', trim(addslashes($HentetNyheder[Vaerdi]))));
    for ($i = 0; count($Nyheder) > $i; $i++)
    {
        if (trim($Nyheder[$i]) == '')
            continue;

        // Ryd lidt op.
        $Nyheder[$i] = stripslashes(trim($Nyheder[$i]));

    }
   
    echo stripslashes($Nyheder[mt_rand(0, count($Nyheder)-1)]);

}


det er meningen at den bare skal hente en tilfældig nyhed og ikke vise alle på samme tid da det bare skal stå i et lille felt i toppen af siderne.


tabellen Indstillinger:
Variabel(text), Vaerdi(text)
"Nyheder", "Nyhed 1\r\nNyhed 2\r\nNyhed 3\r\nNyhed 4";

Fik idéen da jeg snuste rundt i SMF's databaser og så hvordan de håndterede et mindre nyhedsscript. og har allerede en Indstillinger tabel der håndterer andre indstillinger som jeg kan ændre efter behov.

jeg indsætter \r\n til slut så jeg så senere kan dele nyhederne op.

forhøjer points

Jeg vil på det kraftigste opfordre dig til at ændre din database struktur med det samme!
Den måde du bruger databasen på at intet med database at gøre.
Med din struktur kan du foreksempel ikke hente den nyeste nyhed ud, men kun alle nyheder.
Flere data af samme type gemmes aldrig i samme felt i databasen, om de så er delt med \n\r, komma eller anden karakter.

De beste for dig vil være at lave en ekstra tabel som indeholder alle dine nyheder i hver sig række. Derved kan du sortere og ud kun et bestemt antal rækker efter dit eget valg, samt at redigere en nyhed uden at de andre bliver påvirket.

En anden ting som jeg lige falder over er, at du forhåbelig IKKE erklære nye funktioner inde i en anden funktion. Det er rigtig rigtig meget forkert brug af funktioner.

Synes bare det var rimelig meningsløst at lave en hel tabel hvis man ikke skal bruge mere end 3-4 nyheder. Der skal ikke stå dato på, der skal ikke være en titel. Det hele skal smides ind i en fader der (sjovt nok) fader mellem beskederne.

og mht. funktionerne, nej, det var bare for at få dem med... også. min funktionsfil er bare rodet.

Det er da langt mere simpelt at vedligeholde en rigtig opbygget tabel, end at have alle de problemer du har.
Om du så kun har to nyheder, hvor det nok vil være nemmere at have det gemt i en xml fil, så er det meget bedre kode og nemmere for dig selv hvis de ligger rigtigt.

Men til dit spørgsmål, så skyldes det at nl2br ikke fjerne \r\n fra linjerne, men derimod kun indsætter en <br /> hvergang den finder et \r\n.
Det betyder at du, i dit tilfælde skal fjerne \r\n efter du har brugt nl2br og så til sidst tilføje \r\n.
Det kunne se ud som følgende:
    $Nyheder .= str_replace( array("\r\n", "\n", "\r"), "", nl2br($PostNyhed[$i])) . '\r\n';


Så kommer bare problemet når du skal redigere nyhederne.
For så skal du først splitte nyhederne op, dernæst skal du udskifte alle <br /> med \r\n før du kan vise dem i de forskellige textarea's.

Ja, det er der noget om. Tror jeg ændrer det til en tabel kun til nyheder.

Men fik da også løst problemet således:

i rediger nyheder funktionen ændrede jeg $Nyheder = $Nyheder.nl2br($PostNyhed[$i]).'\r\n'; linjen til preg_replace("/(\r\n)+|(\n|\r)+/", "<br />", $PostNyhed[$i]).'\r\n';

og i selve formen ændrede jeg linjen $Nyheder[$i] = stripslashes(trim($Nyheder[$i]));
til $Nyheder[$i] = preg_replace('~<br( /)?' . '>~', "\n",(stripslashes(trim($Nyheder[$i]))));

så fungerer det hele efter hensigten.

Smid et svar og få dine points

Når du nu skifter om til en tabel, så søg for at du gemmer teksten som rå tekst, dvs du gemmer den ikke med <br>
<br> sætter du først ind når du viser teksten, det gør at det er nemmere at redigere teksten på et senere tidspunkt

yessir, den er jeg med på ;-)

yessir, den er jeg med på ;-)

Takker for point :-)