From 841ddbd06427875e75e6c4a9173317f5fc42f210 Mon Sep 17 00:00:00 2001
From: Faris Ansari <netchamp.faris@gmail.com>
Date: Wed, 23 Aug 2017 19:20:28 +0530
Subject: [PATCH] Fix success url

---
 .../doctype/bounty/templates/bounty.html            |  2 +-
 .../doctype/bounty_backer/bounty_backer.py          | 13 ++++++++-----
 .../web_form/back_this_bounty/back_this_bounty.json |  4 ++--
 3 files changed, 11 insertions(+), 8 deletions(-)

diff --git a/foundation/erpnext_foundation/doctype/bounty/templates/bounty.html b/foundation/erpnext_foundation/doctype/bounty/templates/bounty.html
index 4ccf4933..5518ee08 100644
--- a/foundation/erpnext_foundation/doctype/bounty/templates/bounty.html
+++ b/foundation/erpnext_foundation/doctype/bounty/templates/bounty.html
@@ -47,7 +47,7 @@
 <script>
 	frappe.ready(function() {
 		$('[data-action="primary"]').click(function() {
-			var url = "/bounty/back-this-bounty?bounty_name={{doc.name}}&user=" + frappe.session.user;
+			var url = "/back-this-bounty?bounty_name={{doc.name}}&user=" + frappe.session.user;
 			var amount = $('[data-field="amount"]').val();
 			if(amount > 0) {
 				url += '&amount=' + amount;
diff --git a/foundation/erpnext_foundation/doctype/bounty_backer/bounty_backer.py b/foundation/erpnext_foundation/doctype/bounty_backer/bounty_backer.py
index d2950e5d..49e80ade 100644
--- a/foundation/erpnext_foundation/doctype/bounty_backer/bounty_backer.py
+++ b/foundation/erpnext_foundation/doctype/bounty_backer/bounty_backer.py
@@ -15,9 +15,12 @@ class BountyBacker(Document):
 		doc = frappe.get_doc('Bounty', self.bounty_name)
 		self.idx = len(doc.bounty_backer)
 
-	def on_payment_authorized(self, *args, **kwargs):
-		self.db_set('paid', 1)
-		# to trigger form events of Bounty doctype
-		doc = frappe.get_doc('Bounty', self.bounty_name)
-		doc.save()
+	def on_payment_authorized(self, status_changed_to=None):
+		if status_changed_to in ("Completed", "Authorized"):
+			self.db_set('paid', 1)
+			# to trigger form events of Bounty doctype
+			doc = frappe.get_doc('Bounty', self.bounty_name)
+			doc.save()
+			return '/' + doc.route
+		return '/bounties'
 
diff --git a/foundation/erpnext_foundation/web_form/back_this_bounty/back_this_bounty.json b/foundation/erpnext_foundation/web_form/back_this_bounty/back_this_bounty.json
index cd25f895..bf7d7152 100644
--- a/foundation/erpnext_foundation/web_form/back_this_bounty/back_this_bounty.json
+++ b/foundation/erpnext_foundation/web_form/back_this_bounty/back_this_bounty.json
@@ -19,7 +19,7 @@
  "is_standard": 1, 
  "login_required": 1, 
  "max_attachment_size": 0, 
- "modified": "2017-08-23 18:05:50.695856", 
+ "modified": "2017-08-23 18:21:28.733120", 
  "modified_by": "faris@erpnext.com", 
  "module": "ERPNext Foundation", 
  "name": "back-this-bounty", 
@@ -27,7 +27,7 @@
  "payment_button_label": "Pay now", 
  "payment_gateway": "Razorpay", 
  "published": 1, 
- "route": "bounty/back-this-bounty", 
+ "route": "back-this-bounty", 
  "show_sidebar": 1, 
  "sidebar_items": [], 
  "success_url": "/bounties",