Stop tax calculation on SalesOrder instead of SalesOrderItem

Setting `tax_rate1` to 0 didn't always seem to work in our case… sometimes taxes were generated + charged in NS.

This seems to be a more reliable way of stopping tax calculation.

Would this change cause issues for anybody?