search

SylvainTI / phpliteadmin

Automatically exported from code.google.com/p/phpliteadmin
0 stars 0 forks source link

Clicking creating a database without a name fails with wrong error #152

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?
Click "Create" in the sidebar to create a db without entering a name.

What is the expected output? What do you see instead?

Expected: An error that I did not enter a name.
Instead: 
Warning: file_get_contents(./): failed to open stream: No such file or 
directory in .../phpliteadmin.php on line 780
There was a problem setting up your database, ./. An attempt will be made to 
find out what's going on so you can fix the problem more easily.
Checking supported SQLite PHP extensions...

Original issue reported on code.google.com by crazy4ch...@gmail.com on 21 Nov 2012 at 6:57

GoogleCodeExporter commented 9 years ago 
Ähm, affected version: current svn version of 1.9.4

Original comment by crazy4ch...@gmail.com on 21 Nov 2012 at 6:57

GoogleCodeExporter commented 9 years ago 
Okay, I catched the error. No error messages appears now. I think "creating" 
error messages up there is currently done in an not-perfect way. Maybe we 
should improve this someday while adding an error message for this here.

Original comment by crazy4ch...@gmail.com on 21 Nov 2012 at 7:46

GoogleCodeExporter commented 9 years ago 
This is almost fixed, having an error message is only a low-priority 
enhancement.

Original comment by crazy4ch...@gmail.com on 1 Mar 2013 at 12:18

GoogleCodeExporter commented 9 years ago 
I'd prefer the "create new database" form to have its own page, leaving just a 
link in the left column (like in PMA). It would then be easier to have a proper 
descriptive text and show error messages.

Original comment by dreadnaut on 2 Mar 2013 at 4:27

GoogleCodeExporter commented 9 years ago 
It could also be a testbed for a more structured action->result->output->page 
rendering pipeline :-p

Original comment by dreadnaut on 2 Mar 2013 at 5:31