Round ids with upper case letters bug
Review Request #72 - Created March 1, 2009 and submitted
Information | |
---|---|
Bogdan-Cristian Tătăroiu | |
infoarena | |
Reviewers | |
hackers | |
This bug is only visible when caching is enabled (tests/round.php fails), since the database is (apparently) case insensitive and the cache is case sensitive. All urls are normalized so admin/runda/tEst_Round becomes admin/runda/test_round. The round is stored in the database with id tEst_Round. The parameter cache uses id tEst_Round to get round parameters, but it uses test_round to update them, so the parameter cache is never updated the right way. I think the best solution is to just make all round ids lower case (this patch does that) to avoid problems somebody may have with normalized urls in the future.
Do we need to change the current round names in the database?
-
trunk/www/views/round_create.php (Diff revision 1) -
I think we should say that you can only use lower case letter.