Feature Request #1415
Instances array should use email as key
|Assignee:||Sam de Freyssinet||% Done:|
|Category:||Library::Gravatar||Estimated time:||2.00 hours|
|Target version:||0.1.1||Spent time:||-|
I think, all of the current avatars use the same configuration. So the one difference (by default) will be in email field.
In this case instance(), factory() and _construct() should take _$email as first parameter and $config option as second or add config($config) method. Changing of the configuration parameters can be tracked using the Gravatar::changed flag or something else (check it before render() calls).
Also Gravatar should support config names a-la Database profiles.
#1 Updated by Sam de Freyssinet almost 5 years ago
- Due date set to 04/20/2009
- Status changed from New to Assigned
- Priority changed from Normal to High
- Estimated time set to 2.00
Accepted. I like your thinking and will implement as much as I see appropriate. Instances should be available for each email address though I agree