Thema: Datenbankabfrage, SQL Delete Problem

Hi,

Habe jetzt mal versucht über Joomla einen Datenbankeintrag zu löschen, mir wird zwar angezeigt, dass der SQL Befehl ausgeführt wurde allerdings wurde der Datensatz nicht gelöscht.

PHP-Code:

function delete_matchbericht()
{
  $id=$_POST["id"];
  $db  =& JFactory::getDBO();
  
  if(count($id)==0)
  {
    echo "Kein Matchbericht zum Löschen ausgewählt";
    matchbericht_anzeigen();
    return;
  }
  
  for($i=0;$i<count($id);$i++)
  {
    $sql="DELETE FROM matchberichte WHERE matchbericht_id=".$id[$i];
    $db->setQuery($sql);
  }
  
  if($db->getAffectedRows()>0)
    echo "Matchbericht(e) erfolgreich gelöscht";
  else
    echo "Matchbericht(e) nicht gelöscht";
  matchbericht_anzeigen();
} 


Der SQL Befehl funktioniert in phpmyadmin.
$id = Der Primärschlüsseleintrag in der Tabelle, hat auch den richtigen Wert.

Zeilen wo der Eintrag gelöscht werden soll:

PHP-Code:

    $sql="DELETE FROM matchberichte WHERE matchbericht_id=".$id[$i]; 
    $db->setQuery($sql); 


Mir wird angezeigt

PHP-Code:

  if($db->getAffectedRows()>0)
    echo "Matchbericht(e) erfolgreich gelöscht"; 


Obwohl nichts gelöscht wurde...
Mit mit .$id[$i] sollte die id ausgegeben werden, das ist doch korrekt oder?

Ja die ID ist die Richtige und es gibt auch einen Eintrag in der db mit dieser id, habe sie auch testweise ausgegeben.

Was ich nicht verstehe obwohl nichts gelöscht wird, affected_rows >0 ?!

Zitat von ricola

Und das: $_POST["id"] und damit die DB-Abfragen zu machen halte ich ja für den reinen Wahnsinn.

Sorry. MFG. Ricola

Was wäre deiner Meinung nach die saubere Lösung?

Zu der ID:

Oja es ist ein Array
<input type=checkbox name=id[] value=$row->matchbericht_id>

was rikola sicher sagen will, ist, dass du keine super globalen benutzen solltest.
benutz lieber die JRequest klasse von joomla...
zb
JRequest::getVar('id');
bzw
JRequest::getInt('id'); für integer...


edit...
zu deinem problem...
du schickst glaube ich den query garnicht ab...
du setzt den query "nur"

setz hinter setQuery noch ein
db->query()

PHP-Code:

function delete_matchbericht()
{
  global $mainframe;
  $db =& JFactory::getDBO();
  $id = JRequest::getVar('id', null, 'post');
  $id = JArrayHelper::toInteger($id);

if (count($id))
{
    $sql = 'DELETE FROM matchberichte ' .
             'WHERE matchbericht_id IN (' . implode(', ', $id). ')';
    $db->setQuery($sql);
    $db->query();
}
else
{
// hier wäre schon was schief gelaufen, das darf gar nicht sein!
// Formular vor dem Absenden überprüfen
$msg = 'Kein Matchbericht zum Löschen ausgewählt';
$mainframe->redirect('index2.php?option=com_hastenichtgesehen&task=show', $msg);
}
  
$msg = 'Matchbericht(e) nicht gelöscht';
if ($db->getAffectedRows()>0)
{
    $msg = 'Matchbericht(e) erfolgreich gelöscht';
}

$mainframe->redirect('index2.php?option=com_hastenichtgesehen&task=show', $msg);
} 


Danke für die Antworten, das Delete funktioniert zwar aber ein paar Probleme gibt es noch:

PHP-Code:

function delete_matchbericht()
{
  global $mainframe;
  $id = JRequest::getVar('id', null, 'post');
  $id = JArrayHelper::toInteger($id);
  $db  =& JFactory::getDBO();
  
  if(count($id))
  {
    $sql = 'DELETE FROM matchberichte WHERE matchbericht_id IN (' . implode(', ', $id). ')';
    $db->setQuery($sql);
    $db->query();
  }
  else
  {
    echo "Kein Matchbericht zum Löschen ausgewählt";
    echo var_dump($id);
    matchbericht_anzeigen();
    return;
  }
  
  if($db->getAffectedRows()>0)
    echo "Matchbericht(e) erfolgreich gelöscht";
  else
    echo "Matchbericht(e) nicht gelöscht";
  matchbericht_anzeigen();
} 


ID Wert = NULL...

Wenn ich mir per $_POST die id hole, dann funktioniert das Löschen!
$id = JRequest::getVar('id', null, $_POST);
$id = JArrayHelper::toInteger($id);

Liefert NULL als Ergebnis...
Für was steht das global $mainframe?

Und eine Frage habe ich auch noch:
Wenn ich in einer Funktion eine Variable mittels JRequest::setVar('mannschaft', $mannschaft); setze. Und sie später in einer anderen Funktion mit $mannschaft=JRequest::getVar('mannschaft'); hole dann erhalte ich die Variable. Wenn ich in einer Funktion nachher dann nochmal mittels getVar dieselbe Variable holen will, hat sie keinen Wert...