Data
- Data Import Tool
+ Data Import Tool
Import data from spreadsheet (csv) files
- Global Defaults
+ Global Defaults
Set default values for entry
- Recycle Bin
+ Recycle Bin
Un-trash items
@@ -53,75 +57,75 @@
diff --git a/stock/doctype/item/item.py b/stock/doctype/item/item.py
index 5b00643843..38ff797291 100644
--- a/stock/doctype/item/item.py
+++ b/stock/doctype/item/item.py
@@ -128,16 +128,19 @@ class DocType:
else:
check_list.append(d.tax_type)
- def check_for_active_boms(self, check):
- if check in ['Is Active', 'Is Purchase Item']:
+ def check_for_active_boms(self, field_label):
+ if field_label in ['Is Active', 'Is Purchase Item']:
bom_mat = sql("select distinct t1.parent from `tabBOM Item` t1, `tabBOM` t2 where t1.item_code ='%s' and (t1.bom_no = '' or t1.bom_no is NULL) and t2.name = t1.parent and t2.is_active = 1 and t2.docstatus = 1 and t1.docstatus =1 " % self.doc.name )
if bom_mat and bom_mat[0][0]:
- msgprint("%s should be 'Yes'. As Item %s is present in one or many Active BOMs." % (cstr(check), cstr(self.doc.name)))
+ msgprint("%s should be 'Yes'. As Item %s is present in one or many Active BOMs." % (cstr(field_label), cstr(self.doc.name)))
raise Exception
- if check == 'Is Active' or ( check == 'Is Manufactured Item' and self.doc.is_sub_contracted_item != 'Yes') or (check == 'Is Sub Contracted Item' and self.doc.is_manufactured_item != 'Yes') :
+ if ((field_label == 'Allow Production Order'
+ and self.doc.is_sub_contracted_item != 'Yes')
+ or (field_label == 'Is Sub Contracted Item'
+ and self.doc.is_manufactured_item != 'Yes')):
bom = sql("select name from `tabBOM` where item = '%s' and is_active = 1" % cstr(self.doc.name))
if bom and bom[0][0]:
- msgprint("%s should be 'Yes'. As Item %s is present in one or many Active BOMs." % (cstr(check), cstr(self.doc.name)))
+ msgprint("%s should be 'Yes'. As Item %s is present in one or many Active BOMs." % (cstr(field_label), cstr(self.doc.name)))
raise Exception
def validate_barcode(self):
@@ -147,24 +150,21 @@ class DocType:
msgprint("Barcode: %s already used in item: %s" % (self.doc.barcode, cstr(duplicate[0][0])), raise_exception = 1)
def validate(self):
- fl = {'is_manufactured_item' :'Is Manufactured Item',
+ fl = {'is_manufactured_item' :'Allow Bill of Materials',
'is_sub_contracted_item':'Is Sub Contracted Item',
'is_purchase_item' :'Is Purchase Item',
- 'is_pro_applicable' :'Is Pro Applicable'}
+ 'is_pro_applicable' :'Allow Production Order'}
for d in fl:
if cstr(self.doc.fields.get(d)) != 'Yes':
- self.check_for_active_boms(check = fl[d])
+ self.check_for_active_boms(fl[d])
self.check_ref_rate_detail()
self.fill_customer_code()
self.check_item_tax()
self.validate_barcode()
- if not self.doc.min_order_qty:
- self.doc.min_order_qty = 0
self.check_non_asset_warehouse()
- if self.doc.is_pro_applicable and self.doc.is_pro_applicable == 'Yes' and self.doc.is_manufactured_item and self.doc.is_manufactured_item != 'Yes':
- msgprint("If making Production Order is allowed then, it should also allow to make Bill of Materials. Refer Manufacturing section.")
- raise Exception
+ if cstr(self.doc.is_manufactured_item) == "No":
+ self.doc.is_pro_applicable = "No"
if self.doc.is_pro_applicable == 'Yes' and self.doc.is_stock_item == 'No':
msgprint("As Production Order can be made for this Item, then Is Stock Item Should be 'Yes' as we maintain it's stock. Refer Manufacturing and Inventory section.", raise_exception=1)
diff --git a/support/doctype/support_ticket/__init__.py b/support/doctype/support_ticket/__init__.py
index 0295e32ec7..da1755feb9 100644
--- a/support/doctype/support_ticket/__init__.py
+++ b/support/doctype/support_ticket/__init__.py
@@ -89,7 +89,7 @@ class SupportMailbox(POP3Mailbox):
st.doc.status = 'Open'
st.doc.save()
- update_feed(st.doc, 'on_update')
+ update_feed(st, 'on_update')
# extract attachments
self.save_attachments(st.doc, mail.attachments)
return
@@ -109,21 +109,20 @@ class SupportMailbox(POP3Mailbox):
d.naming_series = opts and opts.split("\n")[0] or 'SUP'
try:
d.save(1)
+ try:
+ # extract attachments
+ self.save_attachments(d, mail.attachments)
+ except Exception, e:
+ self.description += "\n\n[Did not pull attachment]"
except:
d.description = 'Unable to extract message'
d.save(1)
-
else:
- # update feed
- update_feed(d, 'on_update')
-
# send auto reply
if cint(self.email_settings.send_autoreply):
if "mailer-daemon" not in d.raised_by.lower():
self.send_auto_reply(d)
- # extract attachments
- self.save_attachments(d, mail.attachments)
def save_attachments(self, doc, attachment_list=[]):
"""
diff --git a/support/doctype/support_ticket/support_ticket.py b/support/doctype/support_ticket/support_ticket.py
index 89a58a6860..0f4a25eeb7 100644
--- a/support/doctype/support_ticket/support_ticket.py
+++ b/support/doctype/support_ticket/support_ticket.py
@@ -50,11 +50,11 @@ class DocType(TransactionBase):
def close_ticket(self):
webnotes.conn.set(self.doc,'status','Closed')
- update_feed(self.doc)
+ update_feed(self)
def reopen_ticket(self):
webnotes.conn.set(self.doc,'status','Open')
- update_feed(self.doc)
+ update_feed(self)
def on_trash(self):
webnotes.conn.sql("""update `tabCommunication` set support_ticket=NULL