-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1
On Oct 5, 2007, at 3:51 PM, Mark Sapiro wrote:
We still need to make sure that we only create lists with lower case names in the first place. The API which is the Create() method needs to enforce this. So, I'm inclined to lower-case the name provided to Create(), but also lower-case the name provided to Utils.list_exists ().
I'm actually not convinced this is a bug, but a problem with the
client of Create() and list_exists(). The problem is that once you
lowercase the list name in those two methods, you might have to start
lowercasing them in an API that might get called via script, web, or
email. I really don't think we need viral .lower() calls all over
the place.
It's tempting to lowercase in Create() but I think it's better to
lowercase the list names at the edges of the system, in much the same
way that we should be converting strings to unicode at the edges of
the system. This means that Lawren's email and web extensions should
be doing the lowercasing there.
Another problem of course is that the constraints for internal API
functions like Create and list_exists() isn't really documented
anywhere. I'm trying to improve that situation for Mailman 3.
Cheers,
- -Barry
-----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.7 (Darwin)
iQCVAwUBRwapjnEjvBPtnXfVAQJZ4QP/RrFuMZ3FcwzCWFdDdq1xdBVvV6ZUcllB +J/nHS5vqY8Fw36JcbJ+BlmzDbwNcWk6FivcjQehktt2owkYJjEZPfvTXytpjgvr Um7UjX4Vbc+p9u2e3l2J3rQuwTt8HNZmU8yTgX7x/lzgLnvHOGujuqXbAkSv5ILv 3gpbHGaOqbM= =A+Ef -----END PGP SIGNATURE-----