Changed raise Exception to frappe.throw
				
					
				
			This commit is contained in:
		
							parent
							
								
									5220b462fd
								
							
						
					
					
						commit
						f31221d0d3
					
				| @ -339,8 +339,8 @@ class SalesInvoice(SellingController): | |||||||
| 
 | 
 | ||||||
| 	def validate_pos(self): | 	def validate_pos(self): | ||||||
| 		if not self.cash_bank_account and flt(self.paid_amount): | 		if not self.cash_bank_account and flt(self.paid_amount): | ||||||
| 			msgprint(_("Cash or Bank Account is mandatory for making payment entry")) | 			frappe.throw(_("Cash or Bank Account is mandatory for making payment entry")) | ||||||
| 			raise Exception | 			 | ||||||
| 		if flt(self.paid_amount) + flt(self.write_off_amount) \ | 		if flt(self.paid_amount) + flt(self.write_off_amount) \ | ||||||
| 				- flt(self.grand_total) > 1/(10**(self.precision("grand_total") + 1)): | 				- flt(self.grand_total) > 1/(10**(self.precision("grand_total") + 1)): | ||||||
| 			frappe.throw(_("""Paid amount + Write Off Amount can not be greater than Grand Total""")) | 			frappe.throw(_("""Paid amount + Write Off Amount can not be greater than Grand Total""")) | ||||||
| @ -431,9 +431,8 @@ class SalesInvoice(SellingController): | |||||||
| 				submitted = frappe.db.sql("""select name from `tabSales Order` | 				submitted = frappe.db.sql("""select name from `tabSales Order` | ||||||
| 					where docstatus = 1 and name = %s""", d.sales_order) | 					where docstatus = 1 and name = %s""", d.sales_order) | ||||||
| 				if not submitted: | 				if not submitted: | ||||||
| 					msgprint(_("Sales Order {0} is not submitted").format(d.sales_order)) | 					frappe.throw(_("Sales Order {0} is not submitted").format(d.sales_order)) | ||||||
| 					raise Exception | 					 | ||||||
| 
 |  | ||||||
| 			if d.delivery_note: | 			if d.delivery_note: | ||||||
| 				submitted = frappe.db.sql("""select name from `tabDelivery Note` | 				submitted = frappe.db.sql("""select name from `tabDelivery Note` | ||||||
| 					where docstatus = 1 and name = %s""", d.delivery_note) | 					where docstatus = 1 and name = %s""", d.delivery_note) | ||||||
| @ -682,7 +681,7 @@ def manage_recurring_invoices(next_date=None, commit=True): | |||||||
| 
 | 
 | ||||||
| 	if exception_list: | 	if exception_list: | ||||||
| 		exception_message = "\n\n".join([cstr(d) for d in exception_list]) | 		exception_message = "\n\n".join([cstr(d) for d in exception_list]) | ||||||
| 		raise Exception, exception_message | 		frappe.throw(exception_message) | ||||||
| 
 | 
 | ||||||
| def make_new_invoice(ref_wrapper, posting_date): | def make_new_invoice(ref_wrapper, posting_date): | ||||||
| 	from erpnext.accounts.utils import get_fiscal_year | 	from erpnext.accounts.utils import get_fiscal_year | ||||||
|  | |||||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user