Patch #7664
openAllowing plugins to add an option for the default gravatar
0%
Description
Now redmine administrator can set the default gravatar for users which has no gravatar. I would like to use my own default gravatar.
The attached patch allows plugins to add an option for the default gravatar. https://github.com/yugui/redmine_fox_gravatar is an example plugin that uses this feature.
Files
Updated by Yuki Sonoda almost 14 years ago
Updated by Yuki Sonoda almost 14 years ago
Oops, I attached a wrong patch. Please use adding-gravater-options2.patch.
Updated by Yuki Sonoda almost 14 years ago
Felix Schäfer suggested that developing a plugin just for a default gravatar is overdoing. https://www.chiliproject.org/issues/212
So I rewrote the patch. The new patch scans public/gravatars/ and adds the found files to options for the default gravatar, just like themes. I am glad if you merge the patch.
Updated by Toshi MARUYAMA almost 14 years ago
Updated by Toshi MARUYAMA almost 14 years ago
Updated by Yuki Sonoda almost 14 years ago
I wrote the gravatar management based on the theme management. So it is an derivative of Redmine itself, particulary lib/redmine/themes.rb and test/unit/lib/redmine/thmes_test.rb.
I believe the copyright is correct. It should inherit JPL's copyright entry. Do you mean I should have added my own copyright entry?
And I believe .gitignore is also correct. This is because an additional gravatar is not a part of redmine. It is correct unless Redmine itself will have its own additional gravatar.
Updated by Jean-Baptiste Barth almost 14 years ago
The license looks good to me, since 1) Redmine project is globally copyrighted by Jean-Philippe, and 2) many files would need a license update anyway. But we could include copyright from other people too, depends on Jean-Philippe thoughts on this point.
The .gitignore looks correct too, but I'll put it in the main .gitignore file as not everybody is familiar with Git here, it will be easier to maintain.
Any other blocker to include this ?
Updated by Jean-Philippe Lang over 13 years ago
This feature doesn't seem to be needed by a lot of people, so if you don't mind, I'd prefer to commit the plugin based solution. Is it OK for you?