https://bugzilla.wikimedia.org/show_bug.cgi?id=28419

--- Comment #37 from Tyler Romeo <tylerro...@gmail.com> 2012-03-31 08:00:51 UTC 
---
OK, now I understand the logic. In that case I'll give my support FWIW. The
only thing I would recommend is cleanup the structure a little bit. There's no
need for a PasswordType interface when there's a base abstract class that's
already required to be inherited by the primary Password class.

Also, I would change the cryptParams() function to defaultCryptParams() and
replace preferred format with current format.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

_______________________________________________
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l

Reply via email to