Don't add PHPSESSID parameter onto notice and conversation URIs if we save a notice...

This was being triggered by welcomebot messages created at account creation time, then propagated through replies.

diff --git a/classes/Conversation.php b/classes/Conversation.php
index ea8bd87b5636907893101255dccc67dac81942d4..f540004ef39b6f9de2c7299ad80fbcd492626aff 100755 (executable)
--- a/classes/Conversation.php
+++ b/classes/Conversation.php
@@ -63,7 +63,8 @@ class Conversation extends Memcached_DataObject
         }
 
         $orig = clone($conv);
-        $orig->uri = common_local_url('conversation', array('id' => $id));
+        $orig->uri = common_local_url('conversation', array('id' => $id),
+                                      null, null, false);
         $result = $orig->update($conv);
 
         if (empty($result)) {

diff --git a/lib/util.php b/lib/util.php
index 44ccc0deff546d7c15d107010911fbe706789e37..3d4ed087f9a74a42e78b7af926a5069a27af41dd 100644 (file)
--- a/lib/util.php
+++ b/lib/util.php
@@ -1521,7 +1521,8 @@ function common_user_uri(&$user)
 function common_notice_uri(&$notice)
 {
     return common_local_url('shownotice',
-                            array('notice' => $notice->id));
+                            array('notice' => $notice->id),
+                            null, null, false);
 }
 
 // 36 alphanums - lookalikes (0, O, 1, I) = 32 chars = 5 bits