From: Steve Singer <ssinger@ca.afilias.info>
Date: Fri, 10 Jun 2011 12:13:24 +0000 (-0400)
Subject: Fix for bug220.
X-Git-Tag: REL_2_1_0_B3~8^2
X-Git-Url: http://git.postgresql.org/gitweb/static/gitweb.js?a=commitdiff_plain;h=46faa6b75f2fddcab98d317b91584cfffad749b4;p=slony1-engine.git

Fix for bug220.
The parameters being passed to reshapeSubscription are in the wrong order.
Report/proposed fix from Glyn Astill <glynastill@yahoo.co.uk>
---

diff --git a/RELEASE b/RELEASE
index 46add0a1..a2c18dcc 100644
--- a/RELEASE
+++ b/RELEASE
@@ -7,6 +7,9 @@
 
 - Bug 218 - obtain a lock on sl_config_lock before calling storePath_int
 
+- Bug 220 -  Fix issue where changing the provider of an existing subscriber
+             passed parameters in the wrong order to the stored procedure.
+
 - Fixed slonik memory corruption when it tries to get the last known
   event_id from a node that doesn't yet have slony installed.
 
diff --git a/src/slonik/slonik.c b/src/slonik/slonik.c
index 165145d4..83435c9d 100644
--- a/src/slonik/slonik.c
+++ b/src/slonik/slonik.c
@@ -4199,7 +4199,8 @@ slonik_subscribe_set(SlonikStmt_subscribe_set * stmt)
 		slon_mkquery(&query,
 					 "select \"_%s\".reshapeSubscription(%d,%d,%d);",
 					 stmt->hdr.script->clustername,
-					 stmt->sub_provider,stmt->sub_setid,
+					 stmt->sub_setid,
+					 stmt->sub_provider,
 					 stmt->sub_receiver);	
 		if (db_exec_command((SlonikStmt *) stmt, adminfo2, &query) < 0)
 		{