Как добавить одно поле для продукта в админке?

Question

[Yura Khlyan]

Доброго времени суток.

Я разрабатываю магазин на Django-Oscar и мне нужно что бы для каждого продукта вводить уровень налога на него.
Я сначала добавил поле tax_rate в самом продукте:

from django.utils.translation import ugettext_lazy as _
from django.db import models
from oscar.apps.partner.abstract_models import AbstractStockRecord
from django.core.validators import MinValueValidator

class StockRecord(AbstractStockRecord):

    tax_rate = models.DecimalField(
    	_("Tax rate"), decimal_places=2, max_digits=12,
    	blank=True, null=True, validators=[MinValueValidator(0)])

from oscar.apps.partner.models import *

Теперь нужно добавить это поле в админке. Этого можно достичь, если в ядре Oscar немного подправить apps/dashboard/catalogue/forms.py:

class StockRecordForm(forms.ModelForm):

    def __init__(self, product_class, user, *args, **kwargs):
        # The user kwarg is not used by stock StockRecordForm. We pass it
        # anyway in case one wishes to customise the partner queryset
        self.user = user
        super(StockRecordForm, self).__init__(*args, **kwargs)

        # If not tracking stock, we hide the fields
        if not product_class.track_stock:
            del self.fields['num_in_stock']
            del self.fields['low_stock_threshold']
        else:
            self.fields['price_excl_tax'].required = True
            self.fields['num_in_stock'].required = True
            self.fields['tax_rate'].required = True # добавил

    class Meta:
        model = StockRecord
        fields = ['partner', 'partner_sku', 'price_currency', 'price_excl_tax', 'tax_rate', # добавил
                  'price_retail', 'cost_price',  'num_in_stock', 'low_stock_threshold',]

Все прекрасно работает. Но это не хорошо изменять исходники.
Тогда я форкнул приложениеapps.dashboard.catalogue, потом в нем(локальном) создал файл forms.py, в которым добавил class StockRecordForm(forms.ModelForm)(уже подправленый), но всеравно не работает.

Кто-то может помочь мне? Буду очень благодарен за помощь.

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

Comments

[mukizu]

Отличный русский /offtop

[Yura Khlyan]

/offtop
mukizu: просто много отличных специалистов говорят на русском и не люблят, когда пишут на кривом русском. А так, как для мення русский не родной, то заранее извиняюсь за возможные косяки.

Answer 1

[Oscar Django]

Тогда я форкнул приложение

Точно форкнул? В настройках добавил?

from oscar import get_core_apps
INSTALLED_APPS += get_core_apps([
    'path.to.local.folder.dashboard.catalogue',
])

Comments

[Yura Khlyan]

Спасибо Вам. И сам выяснил, просто прежде добавил долько dashboard...
Кстати, локално нужно держать целый файл, или можно только ту часть, которая должна быть изменена? Оставил только class StockRecordForm(forms.ModelForm), BaseStockRecordFormSet = ... и class StockRecordFormSet(BaseStockRecordFormSet) и все работает, но не навредит ли это в будущем? Или можна оставить только StockRecordForm(в котором только и делались изменения)?

[Oscar Django]

По сути это обычное питоновское наследование. Т.е. можно просто написать

from oscar.apps.dashboard.catalogue.forms import *

class StockRecordForm(StockRecordForm):
    def __init__(self, product_class, user, *args, **kwargs):
        super(StockRecordForm, self).__init__(*args, **kwargs)
        if product_class.track_stock:
            self.fields['tax_rate'].required = True

[Oscar Django]

ошибся немного:

super(StockRecordForm, self).__init__(product_class, user, *args, **kwargs)

[Yura Khlyan]

Не работает так. Ну и на том спасибо, два класса не так страшно писать...