Python: How to solve the issue : 'badly formed hexadecimal UUID string' in Django

Question

I have created 'post' model in which I included 'post_id' as the primary key field. When I am trying to create a post, it is raising an error: 'badly formed hexadecimal UUID string'. I would appreciate helping me in solve this.

Here's my code.

Models.py:

class Post(models.Model):

    post_id = models.UUIDField(primary_key=True, default='uuid.uuid4', editable=False)
    user = models.ForeignKey(settings.AUTH_USER_MODEL, default=1)
    from1 = models.CharField(max_length=20)
    To = models.CharField(max_length=20)
    timestamp = models.DateTimeField(auto_now=False, auto_now_add=True)

    objects = PostManager()

    def __unicode__(self):
        return self.post_id

    def __str__(self):
        return self.post_id

    def get_absolute_url(self):
        return reverse("posts:detail", kwargs={"post_id": self.post_id})

    class Meta:
        ordering = ["-timestamp", "-Time"]

views.py:

def post_create(request):

    form = PostForm(request.POST or None)
    if form.is_valid():
        instance = form.save(commit=False)
        print(form.cleaned_data.get("post_id"))
        instance.user = request.user
        instance.save()
        return HttpResponseRedirect(instance.get_absolute_url())
    context = {
        "form": form,
    }
    return render(request, "loggedin_load/post_load.html", context)

Why do you want to create a custom `post_id`? By default, Django models contain an `id` and `pk` field. You can use either one of those to track a `post_id`. — jape, Jul 19 '16 at 13:29
@Rohan If I am removing quotes, it raising a name error: name 'uuid' isn't defined. — sumanth, Jul 19 '16 at 13:38
@jape in my def __str__(self): i wanted to return it as return self.post_id, I couldn't able to do it with one it was created by default. If possible, How can achieve this with by default auotcreated 'id'? — sumanth, Jul 19 '16 at 13:38
If uuid isn't found, then you should import it, obviously. That is fairly basic Python. — Daniel Roseman, Jul 19 '16 at 13:39
`uuid` is the module, the way your using it you just need `import uuid` — AncientSwordRage, Jul 19 '16 at 13:42

jape · Answer 1 · 2016-07-19T14:07:06.473

I would do something like this:

from django.utils.encoding import python_2_unicode_compatible

@python_2_unicode_compatible
class Post(models.Model):
    # Instead of default, maybe do null=True to take old entries into account?
    user = models.ForeignKey(settings.AUTH_USER_MODEL, default=1)
    from1 = models.CharField(max_length=20)
    # You may want to reconsider the naming of the "To" field
    # to avoid capital letters and built-in functions
    To = models.CharField(max_length=20)
    timestamp = models.DateTimeField(auto_now_add=True)

    objects = PostManager()

    # You can remove this with the decorator above
    # def __unicode__(self):
        # return self.id

    def __str__(self):
        return self.id  # acts as your post_id

    def get_absolute_url(self):
        return reverse("posts:detail", kwargs={"post_id": self.id})

    class Meta:
        ordering = ["-timestamp", "-Time"]

Whenever an object is created, it will automatically be assigned an id, which will populate your __unicode__, __str__, and get_absolute_url.

score 3 · Accepted Answer · answered Jul 19 '16 at 13:44

You need to import the module and not use quotes around 'uuid.uuid4'.

It should be somewhat like:

import uuid  # The uuid module
class Post(models.Model):

    post_id = models.UUIDField(primary_key=True, default=uuid.uuid4, editable=False)  # using the function uuid4 on the module
    user = models.ForeignKey(settings.AUTH_USER_MODEL, default=1)
    from1 = models.CharField(max_length=20)
    To = models.CharField(max_length=20)
    timestamp = models.DateTimeField(auto_now=False, auto_now_add=True)

    objects = PostManager()

    def __unicode__(self):
        return self.post_id

    def __str__(self):
        return self.post_id

    def get_absolute_url(self):
        return reverse("posts:detail", kwargs={"post_id": self.post_id})

    class Meta:
        ordering = ["-timestamp", "-Time"]

N.B I've not tested the above code, and I agree with some of the comments you shouldn't need a UUID for the post_id. Without knowing more I couldn't help further.

Python: How to solve the issue : 'badly formed hexadecimal UUID string' in Django

2 Answers2

Linked