[php/mysql] ik kom er niet meer uit

[mph_rbi]

Het volgende heb ik in een functie zitten, maar ik krijg steeds een error.
Ik loop al een hele tijd te zoeken, maar ik kan er maar niet achter komen waar de fout zit. Ziet iemand toevallig in 1 oogopslag waar de fout in mijn query zit?

PHP:

            $sql = 'INSERT INTO db (db) VALUES ($dbname);
                    CREATE TABLE $dbname_changelog (
                        id int(11) NOT NULL auto_increment,
                        project_id int(11) NOT NULL default \'0\',
                        maintainer_id int(11) NOT NULL default \'0\',
                        date varchar(20) NOT NULL default \'0\',
                        comment text NOT NULL,
                        UNIQUE KEY id (id)
                        ) TYPE=MyISAM;");

                    CREATE TABLE $dbname_changelog (
                        id int(11) NOT NULL auto_increment,
                        project_id int(11) NOT NULL default \'0\',
                        maintainer_id int(11) NOT NULL default \'0\',
                        date varchar(20) NOT NULL default \'0\',
                        comment text NOT NULL,
                        UNIQUE KEY id (id)
                        ) TYPE=MyISAM;

                    CREATE TABLE $dbname_maintainers (
                        id int(11) NOT NULL auto_increment,
                        maintainer varchar(255) NOT NULL default \'\',
                        password varchar(250) NOT NULL default \'\',
                        UNIQUE KEY id (id)
                        ) TYPE=MyISAM;

                    INSERT INTO $dbname_maintainers VALUES (1,\'admin\', MD5(\'admin\'));

                    CREATE TABLE $dbname_projects (
                        id int(11) NOT NULL auto_increment,
                        project varchar(255) NOT NULL default \'\',
                        UNIQUE KEY id (id)
                        ) TYPE=MyISAM;
                     ';
            
        mysql_query($sql) or die ("werkt nie");

[whoami]

Mocht je nu eens zeggen welke fout je krijgt, en even de P&W quickstart kunt lezen, dan kunnen we je misschien beter helpen.

[mph_rbi]

de fout is waar ik die("werkt nie"); geef.. deze melding krijg ik dus ook keurig.
De fout zit hem dus daadwerkelijk in de query.. De connectie met de database e.d. werkt allemaal prima, want dit is slechts een onderdeel van een site.

[ Voor 52% gewijzigd door mph_rbi op 25-08-2003 16:18 ]

[Tuvow]

Op dit moment krijgt ie waarschijnlijk de error: "werkt nie"

[mph_rbi]

inderdaad

[Verwijderd]

maak daar eens van

code:

or die (mysql_error());

Dan krijg je een iets wat helderde foutmelding

[edit @ D2k] ff wat anders doen dan ASP en VBscript... kan ik zo weer fris aan de slag (tot 17:00)

[ Voor 35% gewijzigd door Verwijderd op 25-08-2003 16:20 ]

[D2k]

mysql_error()
.edit

ej Soet
ga es werken man

[ Voor 60% gewijzigd door D2k op 25-08-2003 16:19 ]

[thomaske]

verander
die('werkt niet')
eens in :
die(mysql_error())

[Brainstorm]

Verander het eens in:

PHP:

mysql_query($sql) or die mysql_error());

Welke fout kijg je nu?

[whoami]

Wel, zorg er eens voor dat je de fout die MySQL genereert te zien krijgt.

Ik ken geen php, maar meestal heb je 2 functies om queries uit te voeren:
1 om SELECT queries uit te voeren (en die dus een rowset returned)
en 1 om DELETE / INSERT / UPDATE / CREATE .... statements mee uit te voern.

[Tuvow]

Probeer eens:
mysql_query($sql) or die (mysql_error());

Dan zie je ook nog wát er fout is aan de query

[mph_rbi]

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 '; CREATE TABLE $dbname_changelog ( id int(11) N

okido.. dan is het nog vreemd .. Btw,. bedankt voor die tip.. mysql_error() kende ik nog niet.

[ Voor 15% gewijzigd door mph_rbi op 25-08-2003 16:20 ]

[Tuvow]

Uhm, hoezo we denken hetzelfde?

[thomaske]

NoLife schreef op 25 augustus 2003 @ 16:19:
Verander het eens in:

PHP:

mysql_query($sql) or die mysql_error());

Welke fout kijg je nu?

offtopic:
parse error, haakje te weinig

[Tuvow]

Misschien komt het doordat je probeert 2x dezelfde table aan te maken?
edit: die 1e Create table ziet er sowieso niet helemaal fris uit onderaan

[ Voor 35% gewijzigd door Tuvow op 25-08-2003 16:23 ]

[mph_rbi]

dacht ik ook .. was inderdaad een nogal grove fout .. is aangepast, maar ik krijg nog steeds dezelfde melding..

[Verwijderd]

De enkele quotes zijn het probleem denk ik. Nu worden nml de vars niet geparst en worden er tabelnamen die beginnen met $ aangemaakt en dat mag niet van MySQL

[edit]

dus sql = '...' wordt sql = "..."

[ Voor 13% gewijzigd door Verwijderd op 25-08-2003 16:25 ]

[whoami]

Tuvow schreef op 25 August 2003 @ 16:21:
Misschien komt het doordat je probeert 2x dezelfde table aan te maken?
edit: die 1e Create table ziet er sowieso niet helemaal fris uit onderaan

Dan zou je geen parse error krijgen.

[crisp]

kon mySQL niet toevallig maar 1 query tegelijk uitvoeren?
Verder: variabelen tussen enkele quotes worden niet geparsed...

[jurri@n]

Zelf gebruik ik voor iedere query een aparte mysql_query() dus ik weet het niet zeker, maar volgens mij vind PHP het niet leuk als je meerdere query's meegeeft (kan het antwoord dan niet parsen ofzo).

Maar zoals ik al zei: ik gebruik het nooit, dus het kan zijn dat ik uit m'n nek sta te kletsen...

[mph_rbi]

Scooterd : Je hebt gelijk .. ook dat heb ik nu aangepast.. ook de fout op #9 is verholpen, maar ik krijg nu dit :

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 '; CREATE TABLE ( id int(11) NOT NULL auto_incr

[ Voor 4% gewijzigd door mph_rbi op 25-08-2003 16:28 ]

[Tuvow]

whoami schreef op 25 August 2003 @ 16:26:
[...]


Dan zou je geen parse error krijgen.

Die krijgt hij toch ook niet? Volgens mij zegt MySQL dat hij een syntax error heeft

[Verwijderd]

ik mis al een tabel naam in jouw error-regel hierboven.... Komen alle variabele wel door???

[Tuvow]

mph_rbi schreef op 25 August 2003 @ 16:27:
Scooterd : Je hebt gelijk .. ook dat heb ik nu aangepast.. ook de fout op #9 is verholpen, maar ik krijg nu dit :

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 '; CREATE TABLE ( id int(11) NOT NULL auto_incr

De table moet een naam hebben

[Verwijderd]

dat bedoelde ik te zeggen

[Tuvow]

Verwijderd schreef op 25 August 2003 @ 16:31:
dat bedoelde ik te zeggen

Tsja, dit soort dingen krijg je als je tegelijkertijd een post zit te typen

[jurri@n]

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 '; CREATE TABLE ( id int(11) NOT NULL auto_incr

Waarschijnlijk wordt $dbname_changelog als hele variable-naam gezien. probeer eens

SQL:

mysql_query("CREATE TABLE ".$dbname."_changelog bla bla bla");

[ Voor 41% gewijzigd door jurri@n op 25-08-2003 16:34 ]

[whoami]

Tuvow schreef op 25 August 2003 @ 16:29:
[...]


Die krijgt hij toch ook niet? Volgens mij zegt MySQL dat hij een syntax error heeft

Toch wel, MySQL geeft een parse error dat er iets niet klopt met z'n sql syntax.

Mocht z'n query goed zijn (syntactisch), dan voert MySQL hem uit. Als dan blijkt dat hij een naam 2x gebruikt, dan gaat MySQL een ander type error geven.
Nu voert MySQL die query al niet uit omdat er een syntax-fout in zit: parse-error.

[Tuvow]

whoami schreef op 25 August 2003 @ 16:34:
[...]


Toch wel, MySQL geeft een parse error dat er iets niet klopt met z'n sql syntax.

Mocht z'n query goed zijn (syntactisch), dan voert MySQL hem uit. Als dan blijkt dat hij een naam 2x gebruikt, dan gaat MySQL een ander type error geven.
Nu voert MySQL die query al niet uit omdat er een syntax-fout in zit: parse-error.

Jij wint

[mph_rbi]

jurri@n: Heb ik geprobeerd.. hij parsed nu netjes de variabele, maar krijg nu dit :

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 '; CREATE TABLE testtest_changelog ( id int(11)

[whoami]

Er is hier al een aantal keer geopperd dat MySQL slechts 1 query tegelijk kan uitvoeren....
Misschien moet je dat eens van dichterbij gaan bekijken.

[Tuvow]

En misschien is het een optie om de tables gewoon even aan te maken met phpMyAdmin oid?

[djluc]

Tuvow schreef op 25 August 2003 @ 16:42:
En misschien is het een optie om de tables gewoon even aan te maken met phpMyAdmin oid?

Ik neem aan dat dit een soort van installatiescript wordt, anders lijkt dit me een beetje tijdsverspilling. Is het verder niet mogelijk om de map in het mysql data mapje te kopiëren? Of krijg je dan troubles met mysql. Uiteraard wel eerst mysql down halen.

[ Voor 10% gewijzigd door djluc op 25-08-2003 16:50 ]

[mph_rbi]

Tuvow schreef op 25 augustus 2003 @ 16:42:
En misschien is het een optie om de tables gewoon even aan te maken met phpMyAdmin oid?

dit is geen optie, omdat hij tabellen dynamisch creeert.
Ik ben nu de code aan het omhacken dat hij maar 1 query per keer uitvoert.

[ Voor 14% gewijzigd door mph_rbi op 25-08-2003 16:57 ]

[mph_rbi]

IK HEB HET!

PHP:

       mysql_query("INSERT INTO db (db) VALUES ('$dbname')") or die (mysql_error());
       mysql_query("CREATE TABLE ".$dbname."_changelog (
                        id int(11) NOT NULL auto_increment,
                        project_id int(11) NOT NULL default '0',
                        maintainer_id int(11) NOT NULL default '0',
                        date varchar(20) NOT NULL default '0',
                        comment text NOT NULL,
                        UNIQUE KEY id (id)
                        ) TYPE=MyISAM;") or die (mysql_error());

        mysql_query("CREATE TABLE ".$dbname."_maintainers (
                  id int(11) NOT NULL auto_increment,
                  maintainer varchar(255) NOT NULL default '',
                  password varchar(250) NOT NULL default '',
                  UNIQUE KEY id (id)
                ) TYPE=MyISAM;") or die (mysql_error());

        mysql_query("INSERT INTO ".$dbname."_maintainers VALUES (1,'admin', MD5('admin'))") or die (mysql_error());

        mysql_query("CREATE TABLE ".$dbname."_projects (
                  id int(11) NOT NULL auto_increment,
                  project varchar(255) NOT NULL default '',
                  UNIQUE KEY id (id)
                ) TYPE=MyISAM;") or die (mysql_error());

Dit werkt dus.. Bedankt !!!