sql-Syntax nicht valide, warum

ich schiebe gerade ziemlichen Frust, ein SQL-String wird in meinem Blog nicht verarbeitet und ich finde partout nicht den Fehler:
Hier der String (mit echo ausgegeben):

SELECT * FROM  WHERE ort=’büro’ ORDER BY lastname firstname ASC

Hier wie er im skript angegeben ist:

$sql = "SELECT * FROM ".$tabellenname." WHERE ort='büro'";
if($_GET['masort'] == 1)
    {$sql.=" ORDER BY lastname firstname ASC";}
elseif($_GET['masort'] == 2)
    {$sql.=" ORDER BY firstname lastname ASC";}
else
    {$sql.=" ORDER BY lastname firstname ASC";}

Hier die Fehlermeldung:

Warning:  mysql_fetch_array(): supplied argument is not a valid MySQL result resource in /srv/www/wordpress/wp-content/plugins/exec-php/includes/runtime.php(42) : eval()’d code on line 32

Hier der Hinweis vom mysql_error():

Zitat

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ‘WHERE ort=’büro’ ORDER BY lastname firstname ASC’ at line 1

Die MySQL server version kann nicht der Fehler sein, genau die selbe Syntax funktioniert in einer anderen Anwendung auf dem selben Server, ausserdem:
Auf einer anderen Seite mit einer anderen Abfrage an die selbe DB-Tabelle gibt es keine Probleme.

Auf dem Blog läuft Exec-PHP

Kann mir irgendjemand sagen, wo das Problem liegt?

Danke im voraus,
diltigug

P.S. werde erst morgen reagieren, habe für heute die Nase voll

Guten Morgen,

Zitat von marX

Wo ist denn der Tabellenname abgeblieben?

SQL

SELECT * FROM  WHERE ort=’büro’ ORDER BY lastname firstname ASC

Ups, da habe ich nicht aufgepasst und hatte tatsächlich eine falschen Variablennamen erwischt.:oops:
Aber leider war das nicht die Lösung des Problems.

Die Variablen sind am Anfang des Skripts definiert.

Ich habe heute den sql-String direkt in phpmyadmin eingegeben und ich glaube das bringt mich der Lösung näher:

Zitat

MySQL meldet:
#1054 - Unknown column '’büro’' in 'where clause'

Es gibt anscheinend ein Problem mit den Umlauten in der where-Klausel, was ich allerdings nicht verstehe, weil ich, wie schon gesagt, die Klausel mit der selben column in einer anderen Anwendung mit der selben DB auch verwende. Und dort funkt sie ganz normal.
Die Datenbank hat als Kollation latin1_general_ci, ich habe sowohl das Feld, als auch die gesamte DB versuchsweise mal auf [FONT=Verdana]utf8_general_ci und auf [/FONT][FONT=Verdana]utf8_unicode_ci umgestellt.
Geholfen hat´s nicht.
:confused:

[/FONT]

Zitat von marX

Warum enthält der Spaltenname Umlaute? Welche Zeichencodierung ist für die MySQL-Verbindung vorgesehen?
Editing wp-config.php « WordPress Codex

Enthält er eigentlich nicht die colum heisst "ort" und aus der soll er mir alle Einträge rausholen die "Büro" enthalten. Oder steh´ da gerade auf dem Schlauch?

Das mit der Zeichencodierung ist interessant, z. Zt. sieht die wp-config bei mir so aus:

Zitat

define('DB_CHARSET', 'utf8');
define('DB_COLLATE', '');

Muss ich bei DB_COLLATE vielleicht was definieren?

Wenn ich mir die WP-Tabellen in php-myadmin anschaue so stehen unter Kollation unterschiedliche einträge: latin1_general_ci, latin1_german1_ci, utf8_general_ci und die letzte Zeile:

Zitat

23 Tabellen Gesamt 10.612 MyISAM latin1_german1_ci 1,3 MiB 18,4 KiB

Rein aus dem Bauch raus, sollte man das nicht vereinheitlichen?

Zitat von marX

Als erstes würde ich mal mit Prepared Statements arbeiten, statt irgendwelche Werte direkt in die Query zu tippen. Siehe hier:
WordPress Coding Standards « WordPress Codex
(Abschnitt "Formatting SQL statements")
Möglicherweise behebt das bereits das Problem, da hier Escaping und Quoting der Daten die WPDB-Klasse für dich übernimmt und so eine mögliche Fehlerquelle ausgeschlossen wird.

Prepared Statements ? in die Query zu tippen ? WP-DB-Klasse nutzen ?

So langsam komme ich da als Hobby-WPler doch ins schwitzen :???:.
Aber egal, ich werde trotz meiner beschränkten Englischkenntnisse mal mit dem Codex-Link beschäftigen. Auf den ersten Blick wird da der SQL-String irgendwie so übergeben (Bsp. aus dem WP Codex):

Zitat

$wpdb->query( $wpdb->prepare( "UPDATE $wpdb->posts SET post_title = %s WHERE ID = %d", $var, $id ) );

Das habe ich bisher nicht so gemacht. Demnach sähe die Abfrage ungefähr so aus:

$wpdb->query( $wpdb->prepare( "SELECT $wpdb-> * FROM mitarbeiter WHERE ort='büro' ORDER BY lastname firstname ASC ) );

richtig? Wenn ja, werde ich damit nachher mal rumspielen. Mein SQL-String wird ja auch noch durch eine if-Abfrage zusammengesetzt. Aber ich glaub, das bekomme ich hin.
Bin jetzt aber erstmal weg, melde mich auf jeden Fall nochmal, ob es klappt.
Jedenfalls schon jetzt mal danke für diesen Super-Support von Euch.
Ich glaube, ohne dieses Forum wären meine autodidaktischen Fähigkeiten schon lange an ihre Grenzen gestossen.

diltigug

Hallo marX,

mir fehlt einfach Grundwissen in Bezug auf Klassen und deren Anwendung. Und alles was ich dazu lese, verstehe ich im Moment nicht wirklich. Ziemlich frustrierend.

Ich habe es mal mit Deinem Code probiert, habe aber jetzt z. B. überhaupt keine Ahnung, wie ich die Daten denn damit ausgeben kann.

Mir ist klar, das das Forum hier kein PHP-Grundkurs sein kann, aber vielleicht kannst Du mir ja doch nochmal weiterhelfen?

Hier mal mein bisherigen Code im Zusammenhang:

<?php
$pass="xxx";  //Datenbankpasswort
$data="xxx";  //Datenbank
$user="xxx";  //Datenbanknutzer
$tabelle="mitarbeiter";
$link=mysql_connect("localhost", $user, $pass, true);
mysql_select_db($data, $link);


/*Hier der Code von marX*/
$ort_value = 'büro';
$wpdb->query(
   $wpdbprepare("
       SELECT * FROM mitarbeiter 
       WHERE ort = %s 
       ORDER BY lastname ASC, firstname ASC",
       $ort_value));
/*marX-Code ende, und dann... ? */


$ergebnis = mysql_fetch_array();
?>
<table>
    <tr>
        <td width="100px" ><a href="../LP67v2/include/memberpopup.php?id=<?php echo "$ergebnis[0]"; ?>" onclick="MADetailOeffnen(this.href); return false"><img src="../LP67v2/grafiken/info.png" alt="MA-Info komplett" border="0" /></a>&nbsp;<? echo "$ergebnis[1]"; ?> </td>
        <td width="80px"><?php echo "$ergebnis[2]"; ?> </td>
      <td width="50px"><div align="right">- <? echo "$ergebnis[3]"; ?></div></td>
         <td width="155px" ><a href="mailto:<?php echo $ergebnis['email']; ?>" title="Email senden"><?php echo "$ergebnis[4]"; ?></a></td>
      <td width="30px" ><div align="center"><?php echo "$ergebnis[5]"; ?></div></td>
      <td width="135px" ><?php echo "$ergebnis[6]"; ?></td>
    </tr>
</table>
<?php
mysql_close($link);
?>

Ich würde die Tabellen so gerne in dem Blog ausgeben, wäre ein echter Mehrwert für unser Büro-Intranet.

Gruß, diltigug

Zitat von marX

PHP

<?php
$pass="xxx";  //Datenbankpasswort
$data="xxx";  //Datenbank
$user="xxx";  //Datenbanknutzer
$tabelle="mitarbeiter";

Bis hierher okay.

PHP

$link=mysql_connect("localhost", $user, $pass, true);
mysql_select_db($data, $link);

So, hier legst du jetzt eine neue Datenbank-Verbindung mit Hilfe der WPDB-Klasse an.

Hier geht´s leider schon wieder los, wie genau mach ich das? Ich habe mich mit meine Englischkenntnissen versucht was in der Doku zu finden, leider ohne Erfolg. Darum einfach mal ins Blaue geraten:

$pass="xxx";
$data="xxx";
$user="xxx";
$tabelle="mitarbeiter";
$wpdb->selectdb($data, $wpdb->connect("localhost", $user, $pass, true));

Funktionert natürlich nicht:

Zitat

Fatal error: Call to a member function selectdb() on a non-object in /srv/www/wordpress/wp-content/plugins/exec-php/includes/runtime.php(42) : eval()'d code on line 17

ich vermute, weil selectdb und connect keine Objekte der Klasse sind? Ist das so richtig formuliert?

Zitat von marX

Besser wäre es allerdings, wenn du die bestehende Datenbank-verbindung nutzen könntest, d.h. die Mitarbeiter-Tabelle würde in der selben Datenbank liegen wie die wp_*-Tabellen. Dann sparst du dir eine weitere Datenbank-Verbindung, das spart Resourcen und schließt mögliche Fehlerquellen aus.

Ist in diesem Fall leider nicht möglich.

Zitat von marX

PHP

/*Hier der Code von marX*/
$ort_value = 'büro';
$wpdb->query(
   $wpdbprepare("
       SELECT * FROM mitarbeiter
       WHERE ort = %s
       ORDER BY lastname ASC, firstname ASC",
       $ort_value));
/*marX-Code ende, und dann... ? */

...

Alle Klarheiten beseitigt?

Wie man siehtst, leider nein. In dem Code oben, muss das $wpdbprepare oder $wpdb->prepare heissen. Ist im Zitat und Original unterschiedlich.

Gruss,
diltigug, oder die Hoffnung stirbt zuletzt.:?

so, jetzt hab´ich´s soweit und stehe, glaub ich, wieder am Anfang:
Folgende Meldung wird ausgegeben:

Zitat

Warning: mysql_fetch_array(): supplied argument is not a valid MySQL result resource in /srv/www/wordpress/wp-content/plugins/exec-php/includes/runtime.php(42) : eval()’d code on line 25

Das hatte ich irgendwann schon mal. Müsste sich eigentlich auf die folgende Zeile beziehen:

Zitat

while($ergebnis=mysql_fetch_array($ergebnis))

So schaut der Code jetzt aus:

<?php
$pass="xxx";
$data="xxx";
$user="xxx";
$tabelle="mitarbeiter";
$my_wpdb = new wpdb($user, $pass, $data, "localhost"); 
$ortvalue='Büro';
$my_wpdb->query($my_wpdb->prepare("SELECT * FROM mitarbeiter WHERE ort = %s ORDER BY lastname ASC, firstname ASC", $ortvalue));
$ergebnis = $my_wpdb->get_results(null, ARRAY_N);
?>
<table border="0">
<?php
echo mysql_error();
$i=0;
while($ergebnis=mysql_fetch_array($ergebnis))
{
$i++;
?>
<tr>
<td width="100"><a href="../LP67v2/include/memberpopup.php?id=<?php echo $ergebnis['id']; ?>" onclick="MADetailOeffnen(this.href); return false"><img src="../LP67v2/grafiken/info.png" border="0" alt="MA-Info komplett" /></a><?php echo " ".$ergebnis['lastname']; ?></td>
<td width="80"><?php echo $ergebnis['firstname']; ?></td>
<td width="50">
<div>-</div><?php echo $ergebnis['dw']; ?></td>
<td width="155"><a title="Email senden" href="mailto:<?php echo $ergebnis['email']; ?>"></a></td>
<td width="30"><?php echo $ergebnis['kz']; ?></td>
<td width="135"></td>
</tr>
<?php }
//$my_wpdb = close();
?>
</table>

Dann noch gleich die Frage, wie schliesse ich die DB-Verbindung wieder?

gebe ich

var_dump($ergebnis);

ein, steht da nur NULL.
:-? Toll, heisst das jetzt, ich habe immer noch keine Daten aus der DB erhalten?

so?

$my_wpdb->query($my_wpdb->prepare("SELECT * FROM mitarbeiter WHERE ort = %s ORDER BY lastname ASC, firstname ASC", $ortvalue));
echo mysql_error();
$ergebnis = $my_wpdb->get_results(null, ARRAY_N);
var_dump($ergebnis);

Da kommen keine weiteren Meldungen:

Zitat

NULL

Warning: Invalid argument supplied for foreach() in /srv/www/wordpress/wp-content/plugins/exec-php/includes/runtime.php(42) : eval()’d code on line 27