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 46add0a102b34ab24ad3292994c28d19c1272741..a2c18dcc420152551509d1d47c054ea9d56633e2 100644 (file)
--- 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 165145d40acc7fb49235caea6246bbc3e4920ef9..83435c9d7cc96a35c94c81570be85d3d1bd61508 100644 (file)
--- 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)
                {