Benjamin Talic a écrit :
Hi,
I am interested in applying for Google Summer of Code as a student. I would like to work on one of the phpMyAdim projects.
As suggested I wanted to provide a "fix" as part of my application. This http://sourceforge.net/p/phpmyadmin/feature-requests/785/?page=0 is the request I would like to implement.
Before I go on with it I wanted to get opinions about my approach. The idea is: -add another table to the mysql database, table: user_additional_info. I think this would be much better then adding columns to the original user table if for no other reason to prevent any issues people might get if they try to apply this fix.
Hi, why the "mysql" database? I am not in favor of this approach. Let's not mess with a database that does not belong to our sphere of influence.
The phpMyAdmin configuration storage is the place to go.
-add additional fields to the create user html -add a separate function to the code base, where this info would be saved to the new table, also add a new function for displaying this data. So basically I would try to add this feature without interfering with regular phpMyAdmin code except for those function calls. I would try and make the function calls executed only if certain parameters are met, in this approach it would probably be checking if the user_additional_info table exists. Does this sound like a good approach?
I communicated with Isaac about this and the biggest question so far is whether the first step (adding a new table) is appropriate.
Any suggestions are more than welcome.
Best, Ben