1
votes

I am building a project to place an order. When I run makemigrations command then it gives an error of

SystemCheckError: System check identified some issues and I have deleted the migration file from migrations

Traceback:

order.Order.price: (fields.E304) Reverse accessor for 'Order.price' clashes with reverse accessor for 'Order.product'.
  HINT: Add or change a related_name argument to the definition for 'Order.price' or 'Order.product'.

order.Order.product: (fields.E304) Reverse accessor for 'Order.product' clashes with reverse accessor for 'Order.price'.

    HINT: Add or change a related_name argument to the definition for 'Order.product' or 'Order.price'.

My models.py is as follows:

class Order(models.Model):
    company = models.ForeignKey(Company, on_delete=models.CASCADE)
    product = models.ForeignKey(Product, on_delete=models.CASCADE)
    price = models.ForeignKey(Product, on_delete=models.CASCADE)

    def __str__(self):
        return self.company + self.product + self.price
2

2 Answers

1
votes

Add related_name like:

    company = models.ForeignKey(Company, on_delete=models.CASCADE, related_name='company')
    product = models.ForeignKey(Product, on_delete=models.CASCADE, related_name='product')
    price = models.ForeignKey(Product, on_delete=models.CASCADE, related_name='price')

related_name is used to link the models. Especially, when you have more than 1 model linking to another model. It is compulsary as in your case. See_this_answer to understand properly about use of related_name.

1
votes

Both your foreign keys referrencing same model Product without defining different related names:

product = models.ForeignKey(Product, on_delete=models.CASCADE)

price = models.ForeignKey(Product, on_delete=models.CASCADE)

If that's not an error then simply add different related names