Yes I added that line and it was to fix an error involving round brackets.

So that really was a quick hack. I think you should have mentionned that in the code.

The problem is that readdump.php is set-up to try and handle too many different types of queries.

Well, I still think it's suitable like that. Indeed dealing with an uploaded dump and a dump that would have been pasted in the SQL query textarea is really the same and I think it really has to be treated the same way.

...

Actually, there is another big bug in this function that is here since the very begining of phpMyAdmin. Try that kind of manual SQL insert and it won't work: insert into table values ('\');insert into table values ('\');

This problem can be fixed with some quite complex code, but is will make the code so slow it will be absolutely useless for database/table restoration.

Well, it's already fixed in CVS. Go and have a look at the code. Honestly, I don't think it's as complex as that. If you find some bug, please tell me.

The only solution I can think of is to split the database restore code from the normal queries text box

So the choice is to either live with the current small bugs or rethink and redesign the whole section.

Now, the bug should be away. Could you test?

I have included a small test table containing lots of different symbols, this was the test data used to verify the current system.

Could you test the current CVS version with your test table, to see if my modification is ok? That would really help.

Benjamin