fix: suggested changes

erpnext/regional/report/vat_audit_report/vat_audit_report.py

@@ -40,7 +40,7 @@ class VATAuditReport(object):
 
 	def get_sa_vat_accounts(self):
 		self.sa_vat_accounts = frappe.get_list("South Africa VAT Account",
-			filters = {"parent":self.filters.company}, pluck="account")
+			filters = {"parent": self.filters.company}, pluck="account")
 		if not self.sa_vat_accounts and not frappe.flags.in_test and not frappe.flags.in_migrate:
 			frappe.throw(_("Please set VAT Accounts in South Africa VAT Settings"))
 
@@ -144,7 +144,7 @@ class VATAuditReport(object):
 
 		return conditions
 
-	def get_data(self,doctype):
+	def get_data(self, doctype):
 		consolidated_data = self.get_consolidated_data()
 		section_name = _("Purchases") if doctype == "Purchase Invoice" else _("Sales")
 
@@ -173,6 +173,7 @@ class VATAuditReport(object):
 	def get_consolidated_data(self):
 		consolidated_data_map={}
 		for inv, inv_data in self.invoices.items():
+			if self.items_based_on_tax_rate.get(inv):
 				for rate, items in self.items_based_on_tax_rate.get(inv).items():
 					consolidated_data_map.setdefault(rate, {"data": []})
 					for item in items:

erpnext/regional/south_africa/setup.py

@@ -6,7 +6,7 @@ from __future__ import unicode_literals
 # import frappe, os, json
 from frappe.permissions import add_permission, update_permission_property
 
-def setup():
+def setup(company=None, patch=True):
 	add_permissions()
 
 def add_permissions():