Reported by Jethro Carr, Jul 24, 2010
The current Indefero user profile page somewhat limiting - it's not clear to someone when clicking on the user who they are or how to contact them. There was also the dependency to use Gravatar to all user avatars rather than being able to upload one to the local server. Amberdms has made a number of changes against Indefero 1.0 and we have attached a patch to this issue with those changes. This patch adds: * Ability to upload a local avatar and fall back to gravatar if a local one has not been set. * New user profile page with additional fields including publicly shown email, website, twitter and description. See attachments for: * Patch against Indefero 1.0 * Example screenshot of new user profile page This is the first time we've worked on Indefero's codebase, constructive criticisms welcome. :-)
Comment 1 by Jethro Carr, Jul 24, 2010
There is a need to create a new SQL table for this feature, we weren't sure if it was best to modify the upgrade script and submit that or to just provide the SQL for the main developers to add. Anyway, here it is: CREATE TABLE IF NOT EXISTS `indefero_idf_user_data` ( `id` mediumint(9) unsigned NOT NULL auto_increment, `user` mediumint(9) unsigned NOT NULL default '0', `description` text NOT NULL, `twitter` varchar(255) NOT NULL, `public_email` varchar(255) NOT NULL, `website` varchar(255) NOT NULL, `avatar` varchar(255) NOT NULL, PRIMARY KEY (`id`), KEY `user_foreignkey_idx` (`user`) ) ENGINE=MyISAM DEFAULT CHARSET=utf8 AUTO_INCREMENT=1 ;
Comment 2 by Patrick Georgi, Jul 25, 2010
That's what the migrations are for: src/IDF/Migrations/* are executed in order, and the highest applied number is stored somewhere. That way, new schema modifications can be done incrementially. Where possible it can use the Pluf DB APIs, making things DB independent (see 15AddGConf.php). Where Pluf's API is not enough (eg. when changing column types), SQL queries are necessary (see 5DescToText.php)
Comment 3 by Loïc d'Anterroches, Sep 6, 2010
Great, I will reuse your code and use the general GConf object to store the data without the need to add a new table. Patrick, this will solve the gravatar issue!
Summary: Enhanced User Profile Page & Avatar
Comment 4 by Thomas Keller, Nov 25, 2010
I'd really like to see that feature in 1.1 - Loic, what do you think?
Comment 5 by Loïc d'Anterroches, Nov 29, 2010
Ok to include it. Either you are ready to update it with GConf, if not assign it back to me and I will do it.
Comment 6 by Thomas Keller, Nov 29, 2010
You said you wanted to do that - I just wanted to second the demand :) But if you don't have the time for it, then I could do it. But I have a pile of other work on my list as well.
Comment 7 by Jethro Carr, Nov 29, 2010
I could potentially take a look at helping out with this change, but realistically I won't have the free time till late Dec.
Comment 8 by Thomas Keller, Dec 2, 2010
Comment 9 by Thomas Keller, Dec 5, 2010
Thanks for the patch. It was included, but heavily modified in commit 784c971. Follow the link to see the full list of changes.