@@ -86,10 +86,22 @@ def add_charge(request,slugs=""): slugs = slugs[:-1] f = AddChargeForm(request.POST) snapshot = working_snapshot() + slug = slugify(request.POST['penal_code'] + " " + request.POST['label'])[:50] + # need to check for duplicate slugs and fix + try: + ct = Charge.objects.get(slug=slug) + # uh oh. there's already a charge with that slug + # need to come up with a relatively unique new one + # this is the most reasonable approach I can think of + slug = slug[:-3] + "%03d" % (Charge.objects.count() % 1000) + # any better ideas? + except Charge.DoesNotExist: + # that's good + pass c = Charge.objects.create(snapshot=snapshot, label=request.POST['label'], penal_code=request.POST['penal_code'], - name=slugify(request.POST['penal_code'] + " " + request.POST['label'])[:50]) + name=slug) slugs = slugs.split("/") description = "charge %s added" % str(c) if len(slugs) > 0 and slugs != ['']: law/views.py cb17534ccfa7447749dcc6c0484e98b325d1241d Anders Pearson anders@columbia.edu http://github.com/thraxil/ccdb/commit/843b744312b2e47c3af7f122eb30acc6d08e75fc 843b744312b2e47c3af7f122eb30acc6d08e75fc 2009-05-04T11:43:03-07:00 2009-05-04T11:43:03-07:00 attempt to prevent slug collisions on charges 6ea2cf223a3d3efe011b2eb040aa8602488b9a07 Anders Pearson anders@columbia.edu