Support uuid input field

Hi there,

I have the model book:

class User(models.Model):
    id = models.UUIDField(default=uuid.uuid4, primary_key=True)
    ... 

class Book(models.Model):
    id = models.UUIDField(default=uuid.uuid4, primary_key=True)
   owner = models.foreignFeild(User)
    ...

class UserType(DjangoObjectType):
   model = User

class BookType(DjangoObjectType):
   model = Book

class BookInput(InputObjectType):
    id = graphene.String() # Could you support graphene.Uuid() 
    owner_id = graphene.String() # Could you support graphene.Uuid() 

class ChangeBookOwner(graphene.Mutation):
    class Input:
        book_input = graphene.Field(BookInput, required=True)
    
    @staticmethod
    def mutate(root, args, context, info):
       id = args.get('book_input').get('id')
       book = Book.objects.get(pk=id)
       if book.owner_id == args.get('book_input').get('owner_id'): # this is always false
          # notify to the new owner

The book.owner_id == args.get('book_input').get('owner_id') is always false because:

1. when querying the BookType it returns the owner_id with aa-bb-cc-dd format which includes dash symbol and I use this owner_id to set the owner_id of BookInput.

2. book.owner_id is always in aabbccdd format which doesn’t include dash symbol.

Could you help me figure out the best way to deal with it without hacks?

Thanks a lot.

[dopeboy]

@zbyte64, is correct. So long as you’ve defined your argument to be of type graphene.UUID(), that equality test mentioned by @quytang will work. Closing this down.

[quytang]

The problem is the comparison, not loading from DB.

Have you ever tried the comparision args.get('uuid') === user.uuid. Let do that and you will see the comparison produce False.

I don’t want to replace the dashes from the args.get('uuid') or do extra redundant work like load object from DB.

Do you know why args.get('uuid') contains dashes? Because the uuid field returns from the graphql query contains dashes. user.uuid loaded from DB has no dashes. I already mentioned them in the description of the issue.

Additionally, user.uuid loaded from DB is instance of uuid.UUID, but args.get(‘uuid’) is instance of String. So what I asked for is to have graphene.Uuid() field for the ideal solution.