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

Comments

simon georges’s picture

simon georges commented
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

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

Changing status to reviewed & tested by the community

rszrama’s picture

rszrama commented
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.