The add buddy page returns an empty page after adding the buddy to the list.

Comments

jpwillms’s picture

I am having this problem as well. Could this be related to the fact that the maintainer doesn't use drupal_goto and drupal_get_destination?

..jordan
http://www.jordanwillms.com

Patrick Nelson’s picture

I'm having the same problem too - using 4.6 and buddylist version 1.39.2.3 with all recent patches. Is there likely to be a fix for this? It does somewhat render the module unuseable on a site...

Regards

Patrick

neale’s picture

I fixed it by applying the following patch:

--- buddylist.module~   2005-06-14 08:10:14.000000000 -0600
+++ buddylist.module    2005-08-08 20:47:45.000000000 -0600
@@ -355,11 +356,11 @@
   else {
     if (($_POST['op'] == t('add user')) && $_POST['edit']['confirm']) {
       buddylist_add($uid);
-      buddylist_goto_referrer();
+      print theme('page', '');
+    } else {
+      print theme('page', theme('confirm', t('Add user %name to your buddy list?', array('%name' => $buddy->name)), 'user',
+             t('%name will be be notified the next time s/he logs in.', array('%name' => $buddy->name)), t('add user')));
     }
-
-    print theme('page', theme('confirm', t('Add user %name to your buddy list?', array('%name' => $buddy->name)), 'user',
-           t('%name will be be notified the next time s/he logs in.', array('%name' => $buddy->name)), t('add user')));
   }
 }
 
@@ -376,7 +377,7 @@
   else {
     if (($_POST['op'] == t('remove user')) && $_POST['edit']['confirm']) {
       buddylist_remove($uid);
-      buddylist_goto_referrer();
+      print theme('page', '');
     }
     print theme('page', theme('confirm', t('Remove user %name from your buddy list?', array('%name' => $buddy->name)), 'user',
            t('%name will be not be notified the next time s/he logs in.', array('%name' => $buddy->name)), t('remove user')));
lennart’s picture

Status: Active » Reviewed & tested by the community

Could someone commit the patch? I am also having this problem but not that good with patching stuff.

Thanks,
Lennart

sami_k’s picture

Status: Reviewed & tested by the community » Closed (fixed)