Merge branch 'develop' into payment-terms

.travis.yml

@@ -57,8 +57,11 @@ jobs:
       script:
         - set -e
         - bench run-tests
+      env: Server Side Test
     - # stage
       script:
         - bench --verbose run-setup-wizard-ui-test
         - bench execute erpnext.setup.utils.enable_all_roles_and_domains
         - bench run-ui-tests --app erpnext
+      env: Client Side Test
+

erpnext/accounts/doctype/payment_entry/payment_entry.py

@@ -221,9 +221,9 @@ class PaymentEntry(AccountsController):
 						elif self.party_type=="Employee":
 							ref_party_account = ref_doc.payable_account
 
-					if ref_party_account != self.party_account:
-							frappe.throw(_("{0} {1} is associated with {2}, but Party Account is {3}")
-								.format(d.reference_doctype, d.reference_name, ref_party_account, self.party_account))
+						if ref_party_account != self.party_account:
+								frappe.throw(_("{0} {1} is associated with {2}, but Party Account is {3}")
+									.format(d.reference_doctype, d.reference_name, ref_party_account, self.party_account))
 
 					if ref_doc.docstatus != 1:
 						frappe.throw(_("{0} {1} must be submitted")

erpnext/schools/doctype/assessment_result/test_assessment_result.js

@@ -20,8 +20,9 @@ QUnit.test('Test: Assessment Result', function(assert){
 		() => cur_frm.refresh(),
 		() => frappe.timeout(1),
 		() => {
-			for(i = 0; i < $('tbody tr').size() * 4; i = (i + 4))
-				student_list.push($(`tbody td:eq("${i}")`).text());
+			$("tbody tr").each( function(i, input){
+				student_list.push($(input).data().student);
+			});
 		},
 
 		// Looping through each student in the list and setting up their score

erpnext/schools/doctype/fee_structure/test_records.json

@@ -14,8 +14,8 @@
 			}
 		],
 		"total_amount": 50000,
-		"debit_to": "_Test Receivable - _TC",
-		"against_income_account": "Sales - _TC",
+		"receivable_account": "_Test Receivable - _TC",
+		"income_account": "Sales - _TC",
 		"cost_center": "_Test Cost Center - _TC",
 		"company": "_Test Company"
 	},
@@ -34,8 +34,8 @@
 			}
 		],
 		"total_amount": 50000,
-		"debit_to": "_Test Receivable - _TC",
-		"against_income_account": "Sales - _TC",
+		"receivable_account": "_Test Receivable - _TC",
+		"income_account": "Sales - _TC",
 		"cost_center": "_Test Cost Center - _TC",
 		"company": "_Test Company"
 	}

erpnext/schools/doctype/fees/fees.json

@@ -93,7 +93,7 @@
    "label": "Student Name", 
    "length": 0, 
    "no_copy": 0, 
-   "options": "student.student_name", 
+   "options": "student.title", 
    "permlevel": 0, 
    "precision": "", 
    "print_hide": 0, 
@@ -1276,7 +1276,7 @@
  "istable": 0, 
  "max_attachments": 0, 
  "menu_index": 0, 
- "modified": "2017-09-19 16:45:14.313744", 
+ "modified": "2017-09-20 23:17:09.819606", 
  "modified_by": "Administrator", 
  "module": "Schools", 
  "name": "Fees", 

erpnext/schools/doctype/fees/fees.py

@@ -52,8 +52,13 @@ class Fees(AccountsController):
 			where g.name = sg.guardian and sg.parent = %s and sg.parenttype = 'Student'
 		""", self.student)
 
-		student_emails.append(frappe.db.get_value("Student", self.student, "student_email_id"))
-		return ", ".join(list(set(student_emails)))
+		student_email_id = frappe.db.get_value("Student", self.student, "student_email_id")
+		if student_email_id:
+			student_emails.append(student_email_id)
+		if student_emails:
+			return ", ".join(list(set(student_emails)))
+		else:
+			return None
 
 
 	def calculate_total(self):

erpnext/schools/doctype/fees/test_fees.py

@@ -19,8 +19,8 @@ class TestFees(unittest.TestCase):
 		fee.posting_date = nowdate()
 		fee.due_date = nowdate()
 		fee.student = student
-		fee.debit_to = "_Test Receivable - _TC"
-		fee.against_income_account = "Sales - _TC"
+		fee.receivable_account = "_Test Receivable - _TC"
+		fee.income_account = "Sales - _TC"
 		fee.cost_center = "_Test Cost Center - _TC"
 		fee.company = "_Test Company"
 

erpnext/setup/doctype/setup_progress/setup_progress.json

@@ -84,7 +84,7 @@
  "issingle": 1, 
  "istable": 0, 
  "max_attachments": 0, 
- "modified": "2017-08-28 17:44:43.100932", 
+ "modified": "2017-09-21 11:52:56.106659", 
  "modified_by": "Administrator", 
  "module": "Setup", 
  "name": "Setup Progress", 
@@ -95,8 +95,8 @@
    "amend": 0, 
    "apply_user_permissions": 0, 
    "cancel": 0, 
-   "create": 1, 
-   "delete": 1, 
+   "create": 0, 
+   "delete": 0, 
    "email": 1, 
    "export": 0, 
    "if_owner": 0, 
@@ -105,7 +105,7 @@
    "print": 1, 
    "read": 1, 
    "report": 0, 
-   "role": "System Manager", 
+   "role": "All", 
    "set_user_permissions": 0, 
    "share": 1, 
    "submit": 0, 

erpnext/setup/doctype/setup_progress/test_setup_progress.py

@@ -0,0 +1,9 @@
+# -*- coding: utf-8 -*-
+# Copyright (c) 2017, Frappe Technologies Pvt. Ltd. and Contributors
+# See license.txt
+from __future__ import unicode_literals
+
+import unittest
+
+class TestSetupProgress(unittest.TestCase):
+	pass