If you configured a shipping service with taxes and you disable the Tax module, you will end up with an inaccessible Drupal site because of a call to an undefined function "commerce_tax_rate_load()". This patch adds Commerce Tax to the dependency list so you are not able to disable the tax module without disabling the Flat Rate module.

CommentFileSizeAuthor
add_commerce_tax_as_dependency.patch344 byteselmotri
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Simon Georges’s picture

Status: Active » Needs review

Closing #1391418: Disabling commerce_tax breaks the site as a duplicate of this issue.
Changing status as there is a patch to review.

TuFrac’s picture

Assigned: Unassigned » TuFrac
Status: Needs review » Reviewed & tested by the community

Changing status to reviewed & tested by the community

rszrama’s picture

Assigned: TuFrac » Unassigned
Status: Reviewed & tested by the community » Fixed

Instead of adding the dependency, I've just wrapped this code in a check to see if the Tax module is installed. This is how we treat it on the tax rate edit form.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.