Merge pull request #15997 from rohitwaghchaure/address_not_set_from_api
[Fix] Address not set from the API
This commit is contained in:
commit
0ccc05bdf7
@ -10,6 +10,7 @@ from erpnext.accounts.party import get_party_details
|
|||||||
from erpnext.stock.get_item_details import get_conversion_factor
|
from erpnext.stock.get_item_details import get_conversion_factor
|
||||||
from erpnext.buying.utils import validate_for_items, update_last_purchase_rate
|
from erpnext.buying.utils import validate_for_items, update_last_purchase_rate
|
||||||
from erpnext.stock.stock_ledger import get_valuation_rate
|
from erpnext.stock.stock_ledger import get_valuation_rate
|
||||||
|
from frappe.contacts.doctype.address.address import get_address_display
|
||||||
|
|
||||||
from erpnext.controllers.stock_controller import StockController
|
from erpnext.controllers.stock_controller import StockController
|
||||||
|
|
||||||
@ -39,6 +40,7 @@ class BuyingController(StockController):
|
|||||||
self.set_qty_as_per_stock_uom()
|
self.set_qty_as_per_stock_uom()
|
||||||
self.validate_stock_or_nonstock_items()
|
self.validate_stock_or_nonstock_items()
|
||||||
self.validate_warehouse()
|
self.validate_warehouse()
|
||||||
|
self.set_supplier_address()
|
||||||
|
|
||||||
if self.doctype=="Purchase Invoice":
|
if self.doctype=="Purchase Invoice":
|
||||||
self.validate_purchase_receipt_if_update_stock()
|
self.validate_purchase_receipt_if_update_stock()
|
||||||
@ -96,6 +98,16 @@ class BuyingController(StockController):
|
|||||||
if not d.cost_center and lc_voucher_data and lc_voucher_data[0][1]:
|
if not d.cost_center and lc_voucher_data and lc_voucher_data[0][1]:
|
||||||
d.db_set('cost_center', lc_voucher_data[0][1])
|
d.db_set('cost_center', lc_voucher_data[0][1])
|
||||||
|
|
||||||
|
def set_supplier_address(self):
|
||||||
|
address_dict = {
|
||||||
|
'supplier_address': 'address_display',
|
||||||
|
'shipping_address': 'shipping_address_display'
|
||||||
|
}
|
||||||
|
|
||||||
|
for address_field, address_display_field in address_dict.items():
|
||||||
|
if self.get(address_field):
|
||||||
|
self.set(address_display_field, get_address_display(self.get(address_field)))
|
||||||
|
|
||||||
def set_total_in_words(self):
|
def set_total_in_words(self):
|
||||||
from frappe.utils import money_in_words
|
from frappe.utils import money_in_words
|
||||||
if self.meta.get_field("base_in_words"):
|
if self.meta.get_field("base_in_words"):
|
||||||
|
@ -8,6 +8,7 @@ from frappe import _, throw
|
|||||||
from erpnext.stock.get_item_details import get_bin_details
|
from erpnext.stock.get_item_details import get_bin_details
|
||||||
from erpnext.stock.utils import get_incoming_rate
|
from erpnext.stock.utils import get_incoming_rate
|
||||||
from erpnext.stock.get_item_details import get_conversion_factor
|
from erpnext.stock.get_item_details import get_conversion_factor
|
||||||
|
from frappe.contacts.doctype.address.address import get_address_display
|
||||||
|
|
||||||
from erpnext.controllers.stock_controller import StockController
|
from erpnext.controllers.stock_controller import StockController
|
||||||
|
|
||||||
@ -40,6 +41,7 @@ class SellingController(StockController):
|
|||||||
self.validate_selling_price()
|
self.validate_selling_price()
|
||||||
self.set_qty_as_per_stock_uom()
|
self.set_qty_as_per_stock_uom()
|
||||||
self.set_po_nos()
|
self.set_po_nos()
|
||||||
|
self.set_customer_address()
|
||||||
check_active_sales_items(self)
|
check_active_sales_items(self)
|
||||||
|
|
||||||
def set_missing_values(self, for_validate=False):
|
def set_missing_values(self, for_validate=False):
|
||||||
@ -344,6 +346,17 @@ class SellingController(StockController):
|
|||||||
po_nos = frappe.get_all('Sales Order', 'po_no', filters = {'name': ('in', sales_orders)})
|
po_nos = frappe.get_all('Sales Order', 'po_no', filters = {'name': ('in', sales_orders)})
|
||||||
self.po_no = ', '.join(list(set([d.po_no for d in po_nos if d.po_no])))
|
self.po_no = ', '.join(list(set([d.po_no for d in po_nos if d.po_no])))
|
||||||
|
|
||||||
|
def set_customer_address(self):
|
||||||
|
address_dict = {
|
||||||
|
'customer_address': 'address_display',
|
||||||
|
'shipping_address_name': 'shipping_address',
|
||||||
|
'company_address': 'company_address_display'
|
||||||
|
}
|
||||||
|
|
||||||
|
for address_field, address_display_field in address_dict.items():
|
||||||
|
if self.get(address_field):
|
||||||
|
self.set(address_display_field, get_address_display(self.get(address_field)))
|
||||||
|
|
||||||
def validate_items(self):
|
def validate_items(self):
|
||||||
# validate items to see if they have is_sales_item enabled
|
# validate items to see if they have is_sales_item enabled
|
||||||
from erpnext.controllers.buying_controller import validate_item_type
|
from erpnext.controllers.buying_controller import validate_item_type
|
||||||
|
Loading…
Reference in New Issue
Block a user