Klick auf Löschen

[xHBBo]

Guten Abend,

auf meiner Seite gibt es einen Löschlink

  
<a href="my.php?id=<?php echo $myAccountID;?>">Löschen</a>  

Code gibt es natürlich auch

  
$id = $_GET["id"];  
......  
Code für das Löschen  
......  
header("Location: http://www.$host$uri/$extra");  

Nun hab ich einen Fehler der Code wird aufgerufen wenn auch die Seite aufgerufen wird, ist natürlich nicht gut ;) wie kann ich es erreichen dass der Code nur aufgerufen wird wenn ich auch auf den Link geklickt habe.

[tk]

Hallo,

Nun hab ich einen Fehler der Code wird aufgerufen wenn auch die Seite aufgerufen wird, ist natürlich nicht gut ;) wie kann ich es erreichen dass der Code nur aufgerufen wird wenn ich auch auf den Link geklickt habe.

Du darfst den Code zum Löschen nur ausführen wenn $_GET['id'] gesetzt ist bzw. wenn $_POST['id'] gesetzt ist: das Ändern von Daten sollte niemals per get (also über Links) erfolgen sondern immer per post (also Formulare).

Gruß,
Tobias

[xHBBo]

Hallo,

Du darfst den Code zum Löschen nur ausführen wenn $_GET['id'] gesetzt ist bzw. wenn $_POST['id'] gesetzt ist: das Ändern von Daten sollte niemals per get (also über Links) erfolgen sondern immer per post (also Formulare).

du meinst also so? if(isset($_GET["id"])){....} ich würde es auch gerne über ein Formular machen aber ich weiß nicht wirklich wie ich das machen kann bzw. soll. Hab eine Auflistung mit aller Einträge. Hinter jedem Eintrag gibt es ein Link mit Löschen und Deaktivieren. Ich kann pro Formular nur ein Button setzten oder?

[tk]

Hallo,

Hab eine Auflistung mit aller Einträge. Hinter jedem Eintrag gibt es ein Link mit Löschen und Deaktivieren. Ich kann pro Formular nur ein Button setzten oder?

Nein, ein Formular darf auch mehrere Buttons haben - eine Seite darf aber auch mehrere Formulare enthalten.

Gruß,
Tobias

[xHBBo]

Hallo,

Nein, ein Formular darf auch mehrere Buttons haben - eine Seite darf aber auch mehrere Formulare enthalten.

ist mir neu. Also meinst du dieses so?

  
<form name="form1" method="post" action="">  
  <p>Eintrag 1 -  
    <input type="submit" name="loeschen" id="loeschen" value="Löschen">  
  </p>  
  <p>Eintrag 2 -  
  <input type="submit" name="loeschen" id="loeschen" value="Löschen">  
  </p>  
  <p>Eintrag 3 -  
  <input type="submit" name="loeschen" id="loeschen" value="Löschen">  
  </p>  
</form>  

Wenn ja, wo bringe ich meine jeweilige ID unter?

[Tabellenkalk]

Hallo,

<input type="submit" name="loeschen" id="loeschen" value="Löschen">

beachte, dass eine id nicht zweimal auf der Seite vorkommen darf.

Gruß
Kalk

[Felix Riesterer]

Liebe(r) xHBBo,

ist mir neu. Also meinst du dieses so?

<form name="form1" method="post" action="">

<p>Eintrag 1 -
    <input type="submit" name="loeschen" id="loeschen" value="Löschen">
  </p>
  <p>Eintrag 2 -
  <input type="submit" name="loeschen" id="loeschen" value="Löschen">
  </p>
  <p>Eintrag 3 -
  <input type="submit" name="loeschen" id="loeschen" value="Löschen">
  </p>
</form>

  
eher so:  
  
~~~html
<form action="" method="post" id="user-management">  
    <fieldset>  
        <legend>User bearbeiten</legend>  
        <ul>  
            <li>  
                <label for="user-login">User-Login</label>  
                <input id="user-login" name="user-login" type="text" value="xHBBo" />  
                <input name="user-id" type="hidden" value="4192" />  
            </li>  
            <li>  
                <label for="user-type">User-Typ</label>  
                <select id="user-type" name="user-type">  
                    <option value="user">einfacher User</option>  
                    <option value="mod">Moderator</option>  
                    <option value="admin" selected="selected">Administrator</option>  
                </select>  
            </li>  
            <li>  
                <label for="user-pw">User-Passwort</label>  
                <input id="user-pw" name="user-pw" type="text" value="changeme" />  
            </li>  
            <li>  
                <label for="user-pw2">User-Passwort (2x)</label>  
                <input id="user-pw2" name="user-pw2" type="text" value="changme" />  
            </li>  
        </ul>  
        <p>  
            <input type="submit" name="delete-button" value="User entfernen" />  
            <input type="submit" name="update-button" value="Userdaten aktualisieren" />  
        </p>  
    </fieldset>  
</form>

Wie Du sehen kannst, bietet dieses Formular zwei Buttons, die beide verschiedene Namen(!) haben. Diese name-Attributwerte werden im $_POST-Array als Schlüssel geführt werden. Allerdings taucht nur der Schlüssel in $_POST auf, dessen Button auch betätigt wurde. Folgende zwei Fälle an $_POST-Daten kannst Du mit dem vorliegenden Formular haben, wenn es exact so abgeschickt wird:

// wenn "User entfernen" betätigt wurde:  
$_POST = array(  
    'delete-button' => 'User entfernen',  
    'user-login'    => 'xHBBo',  
    'user-pw'       => 'changeme',  
    'user-pw2'      => 'changme',  
    'user-type'     => 'admin',  
);  
  
// wenn "Userdaten aktualisieren" betätigt wurde:  
$_POST = array(  
    'update-button' => 'Userdaten aktualisieren',  
    'user-login'    => 'xHBBo',  
    'user-pw'       => 'changeme',  
    'user-pw2'      => 'changme',  
    'user-type'     => 'admin',  
);

Nun willst Du eigentlich zwei Dinge prüfen:

1.) Was ist zu tun?
2.) Sind die Daten OK?

Zu der ersten Frage gehst Du vielleicht so vor:

$task = '';  
  
if (array_key_exists('delete-button', $_POST)) {  
    $task = 'delete';  
}  
  
if (array_key_exists('update-button', $_POST)) {  
    $task = 'update';  
}  
  
// password ok?  
if (array_key_exists('user-pw', $_POST)  
    && !empty($_POST['user-pw'])  
    && array_key_exists('user-pw2', $_POST)  
    && $_POST['user-pw'] === $_POST['user-pw2']  
) {  
    // password ok  
    ...  
}

Du kannst natürlich völlig andere Wege gehen, um Deine Userdaten handzuhaben, aber als Anregung, wie man verschiedene Submitbuttons einsetzen kann, sollte obiges Beispiel genügen.

Liebe Grüße,

Felix Riesterer.

[xHBBo]

Hallo Felix ,

eher so:

<form action="" method="post" id="user-management">

<fieldset>
        <legend>User bearbeiten</legend>
        <ul>
            <li>
                <label for="user-login">User-Login</label>
                <input id="user-login" name="user-login" type="text" value="xHBBo" />
                <input name="user-id" type="hidden" value="4192" />
            </li>
            <li>
                <label for="user-type">User-Typ</label>
                <select id="user-type" name="user-type">
                    <option value="user">einfacher User</option>
                    <option value="mod">Moderator</option>
                    <option value="admin" selected="selected">Administrator</option>
                </select>
            </li>
            <li>
                <label for="user-pw">User-Passwort</label>
                <input id="user-pw" name="user-pw" type="text" value="changeme" />
            </li>
            <li>
                <label for="user-pw2">User-Passwort (2x)</label>
                <input id="user-pw2" name="user-pw2" type="text" value="changme" />
            </li>
        </ul>
        <p>
            <input type="submit" name="delete-button" value="User entfernen" />
            <input type="submit" name="update-button" value="Userdaten aktualisieren" />
        </p>
    </fieldset>
</form>

>   
> Wie Du sehen kannst, bietet dieses Formular zwei Buttons, die beide verschiedene Namen(!) haben. Diese name-Attributwerte werden im $\_POST-Array als Schlüssel geführt werden. Allerdings taucht nur der Schlüssel in $\_POST auf, dessen Button auch betätigt wurde. Folgende zwei Fälle an $\_POST-Daten kannst Du mit dem vorliegenden Formular haben, wenn es exact so abgeschickt wird:  
>   
> ~~~php

// wenn "User entfernen" betätigt wurde:  

> $_POST = array(  
>     'delete-button' => 'User entfernen',  
>     'user-login'    => 'xHBBo',  
>     'user-pw'       => 'changeme',  
>     'user-pw2'      => 'changme',  
>     'user-type'     => 'admin',  
> );  
>   
> // wenn "Userdaten aktualisieren" betätigt wurde:  
> $_POST = array(  
>     'update-button' => 'Userdaten aktualisieren',  
>     'user-login'    => 'xHBBo',  
>     'user-pw'       => 'changeme',  
>     'user-pw2'      => 'changme',  
>     'user-type'     => 'admin',  
> );

Nun willst Du eigentlich zwei Dinge prüfen:

1.) Was ist zu tun?
2.) Sind die Daten OK?

Zu der ersten Frage gehst Du vielleicht so vor:

$task = '';

if (array_key_exists('delete-button', $_POST)) {
    $task = 'delete';
}

if (array_key_exists('update-button', $_POST)) {
    $task = 'update';
}

// password ok?
if (array_key_exists('user-pw', $_POST)
    && !empty($_POST['user-pw'])
    && array_key_exists('user-pw2', $_POST)
    && $_POST['user-pw'] === $_POST['user-pw2']
) {
    // password ok
    ...
}

>   
> Du kannst natürlich völlig andere Wege gehen, um Deine Userdaten handzuhaben, aber als Anregung, wie man verschiedene Submitbuttons einsetzen kann, sollte obiges Beispiel genügen.  
  
vielen dank für deine tolle Erklärung aber ich hab leider nicht diesen Aufbau. Mein User sieht eine Seite dort findet sich alle seine Einträge. Hinter jedem Eintrag soll nun ein Lösch Link / Button sein um einen Eintrag direkt zu löschen. Wenn ich dein Beispiel richtig verstanden habe wäre ich hier bereits in einem Eintrag.  

[Felix Riesterer]

Liebe(r) xHBBo,

Mein User sieht eine Seite dort findet sich alle seine Einträge. Hinter jedem Eintrag soll nun ein Lösch Link / Button sein um einen Eintrag direkt zu löschen. Wenn ich dein Beispiel richtig verstanden habe wäre ich hier bereits in einem Eintrag.

hmm. Wärst Du das? Du kannst ja beliebig viele Formulare auf einer Seite definieren. Jeder Eintrag steht in seinem eigenen <form>-Element, welches einen individuellen Button (und ein verstecktes <input>-Element für die zugehörige ID) hat.

Würde das umsetzbar sein?

Liebe Grüße,

Felix Riesterer.

[xHBBo]

Guten Morgen Felix,

hmm. Wärst Du das? Du kannst ja beliebig viele Formulare auf einer Seite definieren. Jeder Eintrag steht in seinem eigenen <form>-Element, welches einen individuellen Button (und ein verstecktes <input>-Element für die zugehörige ID) hat.

Würde das umsetzbar sein?

das wäre natürlich machbar. Wir später dann nicht sehr viele unnötiger Quelltext erzeugt? Was ich nicht ganz verstehe warum ist es denn so schlimm wenn ich einen Link zum löschen und deaktivieren nehme?

[Felix Riesterer]

Lieber xHBBo,

das wäre natürlich machbar. Wir später dann nicht sehr viele unnötiger Quelltext erzeugt?

nein. Das passt schon.

Was ich nicht ganz verstehe warum ist es denn so schlimm wenn ich einen Link zum löschen und deaktivieren nehme?

Ein Link erzeugt einen HTTP-Request mit der GET-Methode. Ein Absenden eines Formulars mittels Submit-Button erzeugt einen HTTP-Request mit der POST-Methode. Üblicherweise ändern GET-Requests keine Daten, POST-Requests aber schon. Dahinter steckt eine Normierung oder Denkweise beim Erstellen von Web-Anwendungen, die Du unter REST nachlesen kannst.

Liebe Grüße,

Felix Riesterer.

[Der Martin]

Hallo,

Was ich nicht ganz verstehe warum ist es denn so schlimm wenn ich einen Link zum löschen und deaktivieren nehme?

neben dem eher dogmatischen Aspekt, zur bewussten Veränderung des Datenbestands POST zu verwenden, gibt es noch einen sehr pragmatischen:

Wenn mit einem GET-Request, also dadurch, dass man einem Link folgt, Daten gelöscht werden können, dann können auch Suchmaschinen, die deine Seiten indizieren, deine Daten schneller löschen als du "Hoppla!" sagen kannst.

Ciao,
 Martin

[Jörg Reinholz]

Was ich nicht ganz verstehe warum ist es denn so schlimm wenn ich einen Link zum löschen und deaktivieren nehme?

Ganz einfach, weil praktisch jeder Webseitenbetreiber etwas bauen kann wie:

<img src="http://deinhost/verz/skript.php?action=delete&itemtype=user&item=xHBBo" width=2 height=2>

Du siehst in der Webseite oder in dem Mail gar nichts und doch ist der User weg.... Darüber hinaus gibt's noch Trottel, die lassen sich externe Grafiken in HTML-Mails anzeigen und solche, die klicken bei einem Mail auf "hier" weil davor steht "Wenn dieses Mail nicht vollständig oder nicht ordnungsgemäß angezeigt wird, dann klicken Sie ".

Du meinst, das sei ungefährlich, weil eine Authentifizierung das verhindert?

Das tut sie nicht. Du meldest Dich an und der Angreifer schickt Dir zufällig (oder gekonnt) kurz danach den Link. Den kann er auch verbergen, wenn  er z.B. einen URL-Shortener benutzt. Dann sähe obiges so aus: http://goo.gl/LDlklq

Aus genau diesem Grund gibt es also die Grundregel, dass alles, was Daten auf dem Server verändert per POST gesendet werden soll wenn es darum geht, dem Server nur mitzuteilen, welche Daten man sehen/holen/hören will, dann GET

Jörg Reinholz

[tk]

Hallo,

Wenn ja, wo bringe ich meine jeweilige ID unter?

Im name-Attribut (z.B. name="loeschen[123]"), das muss sowieso unterschiedlich sein sonst kannst du serverseitig nicht unterscheiden welcher Button gedrückt wurde. Einfacher ist es aber evtl. wenn du mehrere Formulare nimmst und jeweils ein verstecktes-Formularfeld mit der ID einbaust.

Gruß,
Tobias

[Jörg Reinholz]

Kleiner Augenöffner: Du kannst das Formular mit css als Inline-Block definieren:

<!-- benutzerliste.html -->  
<style type="text/css">  
   .inline {  
   // oder td form {  
     display:inline-block;  
     padding:0;  
     margin:0;  
   }  
</style>  
  
<table>  
  <tr>  
    <th>Benutzer</th>  
    <th>Aktion</th>  
  </tr>  
  <tr>  
    <td>Willi</td>  
    <td><form method="post" class="inline" action="/formularverarbeiter.php"><input type="hidden" value="user-id 12345"><input type="submit" name="delete" value="löschen"><input type="submit" name="edit" value="bearbeiten"></form></td>  
  </tr>  
    <td>Fred</td>  
    <td><form method="post" class="inline" action="/formularverarbeiter.php"><input type="hidden" value="user-id 54321"><input type="submit" name="delete" value="löschen"><input type="submit" name="edit" value="bearbeiten"></form></td>  
  </tr>  
</table>

--

<pre><?php  
#/formularverarbeiter.php  
print_r($_POST);

Spiel damit mal herum, das sollte Deine Probleme beheben.

Jörg Reinholz

[Jörg Reinholz]

Ich mal wieder ...

<input type="hidden" value="user-id 12345">

  
Das Input-Feld braucht natürlich einen Name:  
  
~~~html
<input type="hidden" name="user_id" value="12345">  

sonst wird nichts gesendet...

Jörg Reinholz

[hotti]

hi,

auf meiner Seite gibt es einen Löschlink

<a href="my.php?id=<?php echo $myAccountID;?>">Löschen</a>

  
Als Parameter hast Du die id, mehr nicht. Du möchtest den richtigen Umgang mit Parametern lernen, d.h., setze entweder einen zusätzlichen Parameter welcher die auszuführende Aktion beschreibt oder setze den Parameter so, dass die Aktion daraus hervorgeht.  
  
1) action=delete&id=123  
2) delete=123  
  
2 macht die Kontrollstruktur einfacher, delete ist ein sog. Schlüsselparameter, so musst Du nur einen Parameter befragen. Eine diesbezügliche Kontrollstruktur fragt immer zwei Dinge: Ist der Parameter gesetzt, welchen Wert liefert er.  
  
Wenn Du das mal richtig geübt hast, kannst Du Dich an kompliziertere Dinge ranmachen, z.B. wenn verschiedene Request-Methoden im Spiel sind.  
  
MfG

-- 
[Action Parameter vs. Schlüsselparameter](http://rolfrost.de/action.html)  

[xHBBo]

Guten Abend,

passt zu diesem Thema deshalb schreibe ich es hier mit rein. Wenn ich AND status = 1 schreibe dann werde alle Einträge ausgelesen die eine 0 haben, wenn ich AND status = '1' schreibe dann werde alle Einträge ausgeben die eine 1 haben (was ich auch erreichen wollte / will).

Warum sind die  ' ' so wichtig? Ich dachte immer diese könnte man auch weg lassen.

[MudGuard]

Hi,

passt zu diesem Thema deshalb schreibe ich es hier mit rein. Wenn ich AND status = 1 schreibe dann werde alle Einträge ausgelesen die eine 0 haben, wenn ich AND status = '1' schreibe dann werde alle Einträge ausgeben die eine 1 haben (was ich auch erreichen wollte / will).

Warum sind die  ' ' so wichtig? Ich dachte immer diese könnte man auch weg lassen.

Die kann man genausogut weglassen wie sämtliche relevante Information zur Fragestellung.

Du packst das zwar in einen php-Codeblock, das einfache = deutet aber darauf hin, daß es sich um SQL handeln könnte (denn es soll ja wohl verglichen, nicht zugewiesen, werden).

Aber wie gesagt, das ist geraten, da Du alles relevante zur Fragestellung weggelassen hast.

cu,
Andreas

[xHBBo]

Hallo Andreas,

Die kann man genausogut weglassen wie sämtliche relevante Information zur Fragestellung.

Nein kann man wohl nicht denn dann bekomme ich andere Werte angezeigt wie mit diesen Zeichen.

Du packst das zwar in einen php-Codeblock, das einfache = deutet aber darauf hin, daß es sich um SQL handeln könnte (denn es soll ja wohl verglichen, nicht zugewiesen, werden).

Aber wie gesagt, das ist geraten, da Du alles relevante zur Fragestellung weggelassen hast.

es ist ein mysqli Code

  
FROM anzeigen  
WHERE rubrik = 'city' AND status = '1'");  

[M.]

Mahlzeit,

Hallo Andreas,

Die kann man genausogut weglassen wie sämtliche relevante Information zur Fragestellung.

Nein kann man wohl nicht denn dann bekomme ich andere Werte angezeigt wie mit diesen Zeichen.

Du hast wederdie Antwort selbst noch die Ironie darin verstanden ;)

es ist ein mysqli Code

FROM anzeigen
WHERE rubrik = 'city' AND status = '1'");

  
Nein, das ist Ein Teil eines Mysql-Statements und hat mit PHP erstmal gar nichts zu tun.  
Dazu wäre die Struktur deiner Datenbank relevant, denn die Spaltentypen wirken sich durchaus auf das Suchverhalten aus.  

-- 
42

[xHBBo]

Hallo,

Du hast wederdie Antwort selbst noch die Ironie darin verstanden ;)

dann war es bestimmt nicht so wichtig ;)

Nein, das ist Ein Teil eines Mysql-Statements und hat mit PHP erstmal gar nichts zu tun.
Dazu wäre die Struktur deiner Datenbank relevant, denn die Spaltentypen wirken sich durchaus auf das Suchverhalten aus.

Hab folgende Felder in meiner Datenbank

id     = int(11)
status = enum('0', '1', '2')
rubrik = varchar(10)

[M.]

Mahlzeit,

dann war es bestimmt nicht so wichtig ;)

Doch, war es. Denn dass du alle relevante Info lieferst, ist erforderlich um dir helfen zu können.

status = enum('0', '1', '2')

Hier definierst du status als String. Damit musst du auch als String prüfen. Ob enum auch numerisch sein kann, weiss ich nicht, das kannst du aber in der Doku nachlesen.
Grundsätzlich ist aber logisch, dass du in diesem Fall status = '1' prüfen musst.

[Tabellenkalk]

Hallo,

Du hast wederdie Antwort selbst noch die Ironie darin verstanden ;)

dann war es bestimmt nicht so wichtig ;)

ist alles was du nicht verstehst, nicht so wichtig? Interessanter Ansatz fürs nächste Schuljahr, mal sehn, was die Lehrer davon halten...

Gruß
Kalk

[xHBBo]

Guten Morgen,

ist alles was du nicht verstehst, nicht so wichtig? Interessanter Ansatz fürs nächste Schuljahr, mal sehn, was die Lehrer davon halten...

wenn man es nicht so schreibt dass ich es gleich verstehe dann ist es für mich schlicht weg nicht wichtig. War schon immer so in meinem Leben und bin damit sehr gut durchgekommen.

[M.]

Mahlzeit,

War schon immer so in meinem Leben und bin damit sehr gut durchgekommen.

Dann bist du in diesem Forum an einen Punkt gekommen, bei dem du, mit dieser Einstellung, ziemlich schnell auf die Fresse fallen wirst.
Gut gemeinter Rat von mir, wenn du was nicht verstehst, frag nach.

Du kennst den Spruch "Wer nicht fragt, bleibt dumm"?

[xHBBo]

Hallo,

Du kennst den Spruch "Wer nicht fragt, bleibt dumm"?

du kennst den Spruch "Wer zuviel fragt, nervt"? Also frag ich grundsätzlich nur was ich genau wissen muss.

[M.]

Mahlzeit,

du kennst den Spruch "Wer zuviel fragt, nervt"?

Nein, allerdings finde ich den völlig idiotisch. Wurde vermutlich von Eltern aufgebracht, die von ihren Kindern genervt waren.

Also frag ich grundsätzlich nur was ich genau wissen muss.

Wie gesagt, deine Entscheidung. Ich sehe das anders und handle entsprechend :)

[Felix Riesterer]

Lieber M.,

ich stimme Dir voll und ganz zu.

du kennst den Spruch "Wer zuviel fragt, nervt"?

Nein, allerdings finde ich den völlig idiotisch. Wurde vermutlich von Eltern aufgebracht, die von ihren Kindern genervt waren.

Das bedeutet aber auch, dass ein Kind solcher Eltern gewisse Benachteiligungen bei seiner menschlichen Reifung hatte - was natürlich und selbstverständlich nicht bedeuten muss, dass am Ende einer solchen Reifung nicht ein sehr gutes Ergebnis stehen könnte.

Also frag ich grundsätzlich nur was ich genau wissen muss.

Wie gesagt, deine Entscheidung. Ich sehe das anders und handle entsprechend :)

Eine solche Aussage (die von xHBBo!) wirft große Fragen hinsichtlich der eigenen Mündigkeit auf. Immerhin muss man wissen, was einem an Wissen fehlt, um gezielt nur danach zu fragen. Wenn ich aber in einer Sache Wissenslücken schließen will, ist es ja gerade das Problem, dass ich zu wenig weiß - und deshalb gerade eben _nicht_ wissen kann, was mir an Wissen noch alles fehlt. Daher ist die Aussage von xHBBo sehr problematisch. Sie wirkt dadurch sehr kindlich.

Liebe Grüße,

Felix Riesterer.

[Martin R.]

$id = $_GET["id"];
......
Code für das Löschen
......

  
„Code für das Löschen“ enthält hoffentlich eine Überprüfung der Zugriffsrechte, sonst löscht dir jeder x-beliebige Datenreisende mal eben deine Nutzer.  
  
\0