'Area' tax module config option needs 'requires'

Issue #1478 closed
Darren Hollenbeck
created an issue

Somewhat related to Issue #626 - the TAX_CLASS config option on the area module should probably have the following arguments passed to the StringValue constructor:

         requires=TAX_MODULE,
         requiresvalue='tax.modules.area',

Does not make sense to show this option when the area module is not the active one.

New code:

config_register(
     StringValue(TAX_GROUP,
         'TAX_CLASS',
         description=_("TaxClass for shipping"),
         help_text=_("Select a TaxClass that should be applied for shipments."),
         requires=TAX_MODULE,
         requiresvalue='tax.modules.area',
         #TODO: [BJK] make this dynamic - doesn't work to have it be preloaded.
         default='Shipping'
     )
)

Comments (1)

  1. Log in to comment