[Users] [Bug 3119] add plugin to display QR-Code for avatar

noreply at thewildbeast.co.uk noreply at thewildbeast.co.uk
Mon Apr 28 09:48:26 CEST 2014


--- Comment #12 from Ricardo Mones <mones at users.sourceforge.net> ---
(In reply to comment #10)
> Hi,
> I think that it would be better to have it inside libravatar plugin.

I have to disagree with that for several reasons:

 • Adds dependencies (QR library) not needed for libravatar operation.
 • QRs have nothing to do with libravatar: plugin name would be misleading.
 • Complicates the whole plugin logic, it's cleaner to have it separated.
 • The idea behind the avatar refactoring is precisely having separate plugins
   for separate avatar types, not a super-complicated avatar-hungry plugin for
 • People interested in libravatar are probably not interested in QR (since
   their goals are pretty different). This can change if multi-avatar display
   is implemented, but even in that case having it as a separate plugin works
   equally well, so no problem anyway.
 • It's already implemented as a separate plugin, why make more changes to make
   future changes/maintenance more complicated? :)

You are receiving this mail because:
You are the assignee for the bug.

More information about the Users mailing list