Issue 510: Enhanced User Profile Page & Avatar - Issue tracking, code review, source browser for git, mercurial, subversion and monotone.

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. :-)

indefero-amberdms-userpage.patch - 37.87 kB - view
indefero-amberdms-userpage_example.png - 21.86 kB

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!

Labels: Type:Enhancement Priority:High ~~Type:Defect~~ ~~Priority:Medium~~
Status: Accepted
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?

Owner: loic

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.

Owner: tommyd

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

Labels: Milestone:Release1.1

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.

Status: Fixed