[PHP] 1 form, 2 verschillende acties

[FlowDesign]

Ik zit met een probleem waar ik niet uit kom.
Ik heb één PHP-pagina met daarop een form met daarin allerlei tekstvelden die gevuld worden met waarden uit een MySQL DB.
Nu wil ik bij een groepje tekstvelden 2 buttons; eentje om de waarden te updaten en eentje om heel de regel te verwijderen a.d.h.v. een ID (id staat in een hidden veld).

Echter heb ik het eerst zo gemaakt dat er een extra hidden veld is met een naam (bijvoorbeeld updaterow) en in de php code gebruik ik dan de code if($updaterow) { ... } zodat de code die daar bij hoort uitgevoerd wordt.

Maar hoe krijg ik het nou voor elkaar dat er een bepaalde functie wordt uitgevoerd a.d.h.v. welke van de twee knoppen wordt geklikt (updaten of verwijderen) ?

PHP:

    /* 
     * Aan de hand van de geselecteerde titel de grafiek en in details de activiteiten weergeven.
     */
    
    if($submittitel)
    {   
        print("
            <p>
            <img src='http://www.xxxxx.nl/planning.php?titel=$selectedtitel'><br>
            <br>
            <font color='red'>U kunt 1 activiteit per keer updaten:</font>
            </p>
            <table width='500' border='0' cellspacing='0' cellpadding='5' align='left'>
            "); 

        $sql2 = mysql_query("SELECT * FROM planning WHERE titel = '$selectedtitel' ORDER BY positie ASC") or die(mysql_error());

        while($row2 = mysql_fetch_assoc($sql2))
        {   
            $id = stripslashes($row2[id]);
            $titel = stripslashes($row2[titel]);
            $positie = stripslashes($row2[positie]);
            $activiteit = stripslashes($row2[activiteit]);
            $start = stripslashes($row2[start]);
            $eind = stripslashes($row2[eind]);

            print("
                <form action='editplanning.php' method='post'>
                <input type='hidden' name='updaterow' value='yes'>
                
                <input type='hidden' name='id_e' value='$id'>
                <table>
                <tr>
                    <td>Titel: </td>
                    <td><input type='text' name='titel_e' value='$titel'></td>
                </tr>
                <tr>
                    <td>Positie: </td>
                    <td><input type='text' name='positie_e' value='$positie'></td>
                </tr>
                <tr>
                    <td>Activiteit: </td>
                    <td><input type='text' name='activiteit_e' value='$activiteit'></td>
                </tr>
                <tr>
                    <td>Startdatum: </td>
                    <td><input type='text' name='start_e' value='$start'></td>
                </tr>
                <tr>
                    <td>Einddatum: </td>
                    <td><input type='text' name='eind_e' value='$eind'></td>
                </tr>
                <tr>
                    <td></td>
                    <td><input type='submit' value='Activiteit opslaan'></td>
                </tr>
                </table>
                </form>
                ");

        } // Einde while
        
        print("
            </table>
            ");

    } // Einde if($submittitel)

    
    /*
     * Activiteit updaten
     */
    
    if($updaterow)
    {

        // Zet voor 'gevaarlijke' tekens (', ", en \) een extra \
        $id = addslashes($id_e);
        $positie = addslashes($positie_e);
        $titel = addslashes($titel_e);
        $activiteit = addslashes($activiteit_e);
        $start = addslashes($start_e);
        $eind = addslashes($eind_e);

        $sql3 = mysql_query("UPDATE `planning` SET `positie`='$positie', `titel`='$titel', `activiteit`='$activiteit', `start`='$start', `eind`='$eind' WHERE `id`='$id' LIMIT 1;") or die(mysql_error());

        $rows = mysql_affected_rows();
        if($rows == 1)
        {       
            echo "U heeft succesvol een activiteit geupdate.<br>
            U kunt direct naar een grafiek terugkeren door op [Ga verder...] te klikken.
            <br>
            <hr align='left' width='500'>
            <br>
            Bekijk de grafiek:<br><br>
            <img src='http://www.xxxxx.nl/planning.php?titel=$titel'><br>
            <br>
            Plak deze code in uw verhaal op de website om de planning zichtbaar te maken: <br>
            <font color='red'>
            <xmp><img src='http://www.xxxxx.nl/planning.php?titel=$titel'></xmp>
            </font>
            ";
        }
        else
        {
            echo "Er is iets fout gegaan bij het updaten. Ga <a href='JavaScript:history.go(-1);'>terug</a> en probeer het opnieuw.";
        }

    } //einde if($updaterow)

[Rowanov]

Javascript gebruiken om je formulier te submitten; dit is niet een probleem wat je server-side moet willen afhandelen.

Een tweede optie is om een extra var mee te geven in je post (hidden field), waarin je met javascript de waarde veranderd naar "update" of "delete".

[FlowDesign]

Rowanov schreef op vrijdag 25 augustus 2006 @ 17:04:
Javascript gebruiken om je formulier te submitten; dit is niet een probleem wat je server-side moet willen afhandelen.

Een tweede optie is om een extra var mee te geven in je post (hidden field), waarin je met javascript de waarde veranderd naar "update" of "delete".

Die tweede optie... hoe ziet zoiets eruit ongeveer?

[Verwijderd]

2 Buttons zeg je, maar in je code zie ik er maar 1? Wat je kunt doen (en ja, dat kan serverside) is je submit een name value meegeven. Dan kun je serverside je post (of get) waarde uitlezen. Er wordt altijd maar 1 van je submit buttons meegestuurd.
Nadeel hiervan is dat je dat altijd op 1 pagina moet uitlezen dus af en toe zie je wel eens van die constructies als:

PHP:

if(isset($_POST["update"]))
{}

if(isset($_POST["delete"]))
{}

Ik vind dat dat er persoonlijk niet overzichtelijker van wordt.

[FlowDesign]

Verwijderd schreef op vrijdag 25 augustus 2006 @ 22:52:
2 Buttons zeg je, maar in je code zie ik er maar 1? Wat je kunt doen (en ja, dat kan serverside) is je submit een name value meegeven. Dan kun je serverside je post (of get) waarde uitlezen. Er wordt altijd maar 1 van je submit buttons meegestuurd.
Nadeel hiervan is dat je dat altijd op 1 pagina moet uitlezen dus af en toe zie je wel eens van die constructies als:

PHP:

if(isset($_POST["update"]))
{}

if(isset($_POST["delete"]))
{}

Ik vind dat dat er persoonlijk niet overzichtelijker van wordt.

Dus regel 54 en 55 worden dan als volgt:

PHP:

<td><input type='submit' name='update' value='Activiteit opslaan'></td>
<td><input type='submit' name='delete' value='Activiteit verwijderen'></td>

en daaronder, moet ik dan als volgt doen?

PHP:

if($updaterow)
{
     if(isset($_POST["update"]))
     {}

     if(isset($_POST["delete"]))
     {}
}

[Verwijderd]

Het principe is dat je, als je:

PHP:

if(isset($_POST["update"]))

gebruikt, de code daarbij horend wordt uitgevoerd als je op de knop met de name value "update" klikt.

[Verwijderd]

Waarom 2 buttons? Waarom niet een selectbox of een setje radiobuttons?

[FlowDesign]

Verwijderd schreef op zaterdag 26 augustus 2006 @ 20:28:
Het principe is dat je, als je:

PHP:

if(isset($_POST["update"]))

gebruikt, de code daarbij horend wordt uitgevoerd als je op de knop met de name value "update" klikt.

En dan zou

PHP:

if($updaterow)
{...}

dus overbodig zijn, toch?

[FlowDesign]

Verwijderd schreef op zaterdag 26 augustus 2006 @ 20:43:
Waarom 2 buttons? Waarom niet een selectbox of een setje radiobuttons?

Omdat dat niet logisch zou zijn. Dat is net zo iets als 2 radioboxen maken met de keuzes OK en Cancel en dan 1 button eronder met OK ofzo.

[MueR]

Maak een mooie checkbox "Delete" bij de regel, en een knop met "OK". Wanneer de delete checkbox niet is aangevinkt, slaat ie de zooi op, anders mikt ie de regel uit de database. Jullie denken veel te moeilijk.

[seamus21]

MueR schreef op maandag 28 augustus 2006 @ 02:27:
Maak een mooie checkbox "Delete" bij de regel, en een knop met "OK". Wanneer de delete checkbox niet is aangevinkt, slaat ie de zooi op, anders mikt ie de regel uit de database. Jullie denken veel te moeilijk.

Op zich is het wel een oplossing die grafisch minder ruimte inneemt (elegant). Maar ook een die minder gebruikersvriendelijk is. De gebruiker kan niet direct zien of hij/zij nu iets gaat bewerken, verwijderen of eventueel iets anders. Daarnaast geeft de TS in de SP aan met 2 buttons te willen werken.

Twee submit buttons met een andere naam in je form. Dan kun je met een if constructie kijken welke is ingedrukt en kun je dus verschillende acties ondernemen. Die oplossing is hier indd al besproken.

[SinergyX]

Waarom moeilijk? Ik gebruik altijd een 'update' en 'delete' button bij zulke queries, gebruikers snappen het (fool-proof; dat is zo'n delete-checkbox dus niet) en het is best goed toe te passen in de code.

[ Voor 3% gewijzigd door SinergyX op 28-08-2006 02:42 ]

[JKVA]

Ik zou persoonlijk de submits allebei dezelfde naam geven. (action ofzo) Geef ze allebei een verschillende value, zoals de ene update en de andere delete. Serverside lees je de action parameter uit en je weet wat je moet doen.

Overigens is het niet lelijk dat acties naar één centrale plaats verstuurd worden, maar dan moet je wel zorgen dat alles daar overzichtelijk blijft. Dit is dan volgens de Model 2 aanpak, welke doorgaans als netter wordt gezien dan steeds naar een aparte pagina posten.

http://en.wikipedia.org/wiki/Model_2

In feite post je steeds naar een controller. Dit is gewoon een PHP pagina die request parameters kan uitlezen en aan de hand van deze parameters door kan delegeren naar de daadwerkelijke code.

[CH4OS]

FlowDesign schreef op maandag 28 augustus 2006 @ 00:04:
Omdat dat niet logisch zou zijn. Dat is net zo iets als 2 radioboxen maken met de keuzes OK en Cancel en dan 1 button eronder met OK ofzo.

Mij lijkt de oplossing van Cheetah anders heel erg logisch. Je geeft daarmee namelijk aan wat je écht wil gaan doen, namelijk deleten óf updaten. Ik zou dus hierbij kiezen voor een radio button, daar je een van de twee (schijnbaar) moet kiezen.

De waarde van je submit button geef je dan "doorvoeren" of iets dergelijks... Onderstaande code ter verduidelijking, met extra optie om toe te voegen:

HTML:

<input type="radio" name="actie" value="1" />Toevoegen<br />
<input type="radio" name="actie" value="2" />Updaten<br /> 
<input type="radio" name="actie" value="3" />Verwijderen<br /> 
<input type="submit" value="Doorvoeren" />

Eventueel kan je aan de hand van de gekozen optie bij 'option' via JS de value dan bepalen bij de submit button... Dat je dan dingen dubbel aan geeft, dat is niet erg, lijkt mij alleen maar duidelijker naar de gebruiker toe...

MueR schreef op maandag 28 augustus 2006 @ 02:27:
Maak een mooie checkbox "Delete" bij de regel, en een knop met "OK". Wanneer de delete checkbox niet is aangevinkt, slaat ie de zooi op, anders mikt ie de regel uit de database. Jullie denken veel te moeilijk.

Zie hierboven, mij lijkt het logischer om een radio button te gebruiken, omdat je dan ook niet 'per ongeluk' kan vergeten om de checkbox aan te vinken, mocht je een record willen deleten... Uiteraard moet je de action / option / hoe je het ook noemen wilt, verplicht stellen binnen je form...

Wat je natuurlijk ook kan doen is de te verrichten actie afvangen via een $_GET variabele...

[ Voor 24% gewijzigd door CH4OS op 28-08-2006 13:33 ]

[FlowDesign]

JKVA schreef op maandag 28 augustus 2006 @ 07:44:
Ik zou persoonlijk de submits allebei dezelfde naam geven. (action ofzo) Geef ze allebei een verschillende value, zoals de ene update en de andere delete. Serverside lees je de action parameter uit en je weet wat je moet doen.

Overigens is het niet lelijk dat acties naar één centrale plaats verstuurd worden, maar dan moet je wel zorgen dat alles daar overzichtelijk blijft. Dit is dan volgens de Model 2 aanpak, welke doorgaans als netter wordt gezien dan steeds naar een aparte pagina posten.

http://en.wikipedia.org/wiki/Model_2

In feite post je steeds naar een controller. Dit is gewoon een PHP pagina die request parameters kan uitlezen en aan de hand van deze parameters door kan delegeren naar de daadwerkelijke code.

Ik ken het MVC design pattern, maar dat is nogal een overkill voor deze (redelijk simpele) pagina.

[FlowDesign]

FlowDesign schreef op zaterdag 26 augustus 2006 @ 00:56:
[...]


Dus regel 54 en 55 worden dan als volgt:

PHP:

<td><input type='submit' name='update' value='Activiteit opslaan'></td>
<td><input type='submit' name='delete' value='Activiteit verwijderen'></td>

en daaronder, moet ik dan als volgt doen?

PHP:

if($updaterow)
{
     if(isset($_POST["update"]))
     {}

     if(isset($_POST["delete"]))
     {}
}

Bovenstaand werkt perfect. Bedankt AMDinc

[susscorfa]

zo ver ik weet kunnen multiple submits een probleem opleveren. je kunt daar door problemen krijgen dat het niet altijd goed gesubmit word mss moet je er ff op googlen. iig zelf heeft iemand mij dit laatst verteld dus doen we daar nu alles met javascript submits

[XWB]

Wat ook kan:

HTML:

<form method="post">
<input type='button' value='opslaan' onclick="this.form.action='url'; this.form.submit();">
<input type='button' value='verwijderen' onclick="this.form.action='andereurl'; this.form.submit();">
</form>

[mithras]

susscorfa schreef op dinsdag 29 augustus 2006 @ 13:03:
zo ver ik weet kunnen multiple submits een probleem opleveren. je kunt daar door problemen krijgen dat het niet altijd goed gesubmit word mss moet je er ff op googlen. iig zelf heeft iemand mij dit laatst verteld dus doen we daar nu alles met javascript submits

Waarom zou dat niet kunnen? Van w3.org:

quote: http://www.w3.org/TR/1999.../forms.html#submit-button

submit buttons: When activated, a submit button submits a form. A form may contain more than one submit button.

Waarbij het 'name' attribuut de control name toewijst:

quote: http://www.w3.org/TR/1999...t/forms.html#control-name

A control's "control name" is given by its name attribute. The scope of the name attribute for a control within a FORM element is the FORM element.

Dus je kan een submit type van een input element een name meegeven, die dus binnen dat form als control name wordt gebruikt.

Van een submit button is het geloof ik zo dat de waarde naar TRUE gaat als die verstuurd wordt, maar dat weet ik niet zeker (het kan net zo goed een '1' zijn).

[XWB]

De andere buttons worden niet meeverzonden wanneer je bv op "opslaan" klikt. Doe maar eens print_r($_POST) en klik op elke button. De waarde is geen true maar gewoon de value van die button. Dus je kan makkelijk checken:

PHP:

if (isset($_POST['opslaan']) && $_POST['opslaan'] != '')

[FlowDesign]

Wat mijns inziens eigenlijk ook wel een mooie oplossing is...
Eén checkbox neerzetten voor verwijderen en één submit button.

Dus als de checkbox niet aangevinkt is, dan doe je de update en als ie wel aangevinkt is dan verwijder je wat je wilt verwijderen.

Het voordeel hiervan is, dat een gebruiker niet per ongeluk op verwijderen zal klikken i.p.v. op opslaan. (Voor nog meer fool-proofness kun je uiteraard ook nog een pop-up box gebruiken.)

[mithras]

FlowDesign schreef op dinsdag 29 augustus 2006 @ 13:54:
Wat mijns inziens eigenlijk ook wel een mooie oplossing is...
Eén checkbox neerzetten voor verwijderen en één submit button.

Dus als de checkbox niet aangevinkt is, dan doe je de update en als ie wel aangevinkt is dan verwijder je wat je wilt verwijderen.

Het voordeel hiervan is, dat een gebruiker niet per ongeluk op verwijderen zal klikken i.p.v. op opslaan. (Voor nog meer fool-proofness kun je uiteraard ook nog een pop-up box gebruiken.)

Wat je dan vaak ziet is een combinatie van beide: een gewone 'update' button, en daarnaast zowel een checkbox als een verwijder button. Zo moet je 2 handelingen uitvoeren om een item te verwijderen. Natuurlijk kan je na het submitten van 'verwijder' nog een check uit laten voeren van 'weet je het zeker?', maar dat zorgt voor een extra request. Vandaar dat een checkbox+aparte button wel het veiligst is daarvoor.