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

Hi

Dne Wed, 17 Apr 2013 18:17:43 +0200 Benjamin Talic b.ttalic@gmail.com napsal(a):

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.

The mysql.user_info table is used by MySQL Administrator and intention of this feature was to be able to use same data. So the table would have exactly same structure as this tool uses.