Как обновить количество товаров после сохранения формы?

Question

[Yura Khlyan]

Доброго времени суток, уважаемые знатоки)

Есть товар. Он продаётся в двух магазинах с одним складом. У модели есть поле "количество на складе", но нужно что бы оно было одинаковое для двух магазинов. В админке(Django-Oscar) я убрал возможность напрямую редактировать количество товара на складе для отдельного магазина. Вместо этого добавил новое поле, в котором нужно вписать сколько добавить товара.

Я думал решить это следующим образом:

• добавить новоый атрибут add_to_stock к модели продукта
  
• добавить метод update_stock, который бы делал num_in_stock += add_to_stock
  
• запускать метод для всех магазинов каждый раз, когда сохраняется товар
  

Застрял собственно на третьем пункте. Поможете решить? Или есть другие, лучшие решения?

Использую Django-Oscar 1.2 и Django 1.8

П. С. Извините за мой плохой русский

П. П. С. Вид админки:

Comments

[German Jet]

Хватит все время извиняться за плохой русский. Он у вас не плох

[Shmele]

Расскажите чуть больше о структуре моделей. Конкретно - как хранится продукт, как хранятся объекты магазинов и как они связаны. Тогда можно будет ответить. А пока не понятно.
Кажется, что вы храните количество каждого товара в двух местах в базе (для каждого магазина). Если это так, от этого нужно избавляться.

[Дмитрий Воронков]

Можно решить вопрос через триггеры в СУБД, ну а вообще правильно подсказывают выше, склад то один, и наличие должно быть одно. Структуру данных поменять.

[Yura Khlyan]

Дмитрий Воронков: Все уже сам решил. Но раз интересуются, напишу как сделал.

Answer 1

[Yura Khlyan]

Решение:

Как и советовали, количество товара на складе вынес в отделное поле для товара(а цены храняться в stockrecord). В stockrecord, количество сделал ссылкой на количество тпродукта.

class Product(AbstractProduct):
    ...
    num_in_stock_product = models.PositiveIntegerField(
        _("Number in stock(product)"), blank=True, null=True)
    stock_update = models.IntegerField(
        _("Stock update"), blank=True, null=True)

    def save(self, *args, **kwargs):
        if self.stock_update:
            if not self.num_in_stock_product:
                self.num_in_stock_product = 0
            self.num_in_stock_product += self.stock_update
        self.stock_update = None
        super(AbstractProduct, self).save(*args, **kwargs)
        self.attr.save()

class StockRecord(AbstractStockRecord):
     ...
    @property
    def num_in_stock_stockrecord(self):
        return self.product.num_in_stock_product
        
    @property
    def net_stock_level(self):
        """
        The effective number in stock (eg available to buy).

        This is correct property to show the customer, not the num_in_stock
        field as that doesn't account for allocations.  This can be negative in
        some unusual circumstances
        """
        all_allocations = 0
        for stockrecord in self.product.stockrecords.all():
            if stockrecord.num_allocated is not None:
                all_allocations += stockrecord.num_allocated
        if self.num_in_stock_stockrecord is None:
            return 0
        return self.num_in_stock_stockrecord - all_allocations

Ну и переделал формы/вьюшки/и т.п. для нового поля num_in_stock_stockrecord(было num_in_stock)