Skip to:

bp live notification plugin

  • @sbrajesh 🙂

    testing bp live notification 1.0.3 plugin, I’m not sure maybe this may vary depending on your PHP error reporting setting, btw it didn’t work for me so I checked the ajax response in Firebug and it started with some html (PHP warning):

    <br /><b>Warning</b>: Missing argument 2 for wpdb::prepare() … etc. and then the actual JSON started with {“change”:”yes” … etc.

    investigating a bit I came to this Andrew Nacin post:

    PHP Warning: Missing argument 2 for wpdb::prepare()

    so I just changed a line in bp-live-notification.php, inside function “bpln_get_notification_messages”

    this line:
    $notifications =$wpdb->get_results( $wpdb->prepare( “SELECT * FROM {$bp->core->table_name_notifications} WHERE id in {$list_ids} AND is_new = 1”) );

    changed in:

    $notifications =$wpdb->get_results( $wpdb->prepare( “SELECT * FROM {$bp->core->table_name_notifications} WHERE id in {%s} AND is_new = %d”), $list_ids, 1 );

    this suppressed the PHP warning and now seems everything is perfectly working.
    Would you consider the patch? 🙂
    thx very much, greetings from Italy

Viewing 8 replies - 1 through 8 (of 8 total)

  • Jigesh


    which version you were using? i had some problem with this plugin before one or two days but it is because of wp 3.6 Jquery version, i had to replicate it back to 3.5.2 , so this plugin don’t work with new jquery version.

    1.0.3 as I wrote above 🙂 after I modified that line, it’s working for me with WP 3.6 and BP 1.8

    sorry I was wrong, I was focused on the admin bar notifications and they work for me, but the “growl style” notifications… no no, they don’t :\



    @afercia , hello you found any solution ? i am talking with brajesh about it! may be he will give solution about it.




    @afercia, Hurray! finally i solved this,

    here is the solution on my blog:

    Hope this helps someone. thanks.

    Andrea Fercia


    @jigesh hi
    you should never remove $wpdb->prepare, is there for a reason 🙂
    The reason why the plugin doesn’t work with wordpress 3.6, 3.7 and later, it’s clearly explained here:
    see where they wrote:
    “Please note: As of 3.5, wpdb::prepare() enforces a minimum of 2 arguments.”
    more details here:

    PHP Warning: Missing argument 2 for wpdb::prepare()

    and the plugin code provides just 1 parameter, generating a PHP warning.
    Basically, the json returned doesn’t contain a valid json but contains the error message:
    <br /><b>Warning</b>: Missing argument 2 for wpdb::prepare()
    being not valid json, it can’t be rendered in the “Bubble”.
    The correct solution is to keep $wpdb->prepare but change the query in order to pass “1” as second argument:
    instead of
    AND is_new = 1″
    AND is_new = %d”, 1
    that’s because you *have* to pass a minimum of 2 arguments. So you have to re-factor the query in order to pass at least 1 second argument: first argument is the query, where you use placeholder %d (digit) instead of 1:
    “SELECT * FROM {$bp->core->table_name_notifications} WHERE id IN {$list_ids} AND is_new = %d”
    second argument is: 1
    If you have to pass a string instead of a digit, you would use %s as placeholder.



    @afercia, hi what i understand is we need DATA from database , The link you suggested me has also written this, YOU CAN USE: $myrows = $wpdb->get_results( “SELECT id, name FROM mytable” );

    same thing i did, and it works,

    i also understand that wpdb::prepare(); requires two parameters but after applying two parameters notification is updating but BUBBLE isn’t appearing , so i needed to remove it.

    Its working fine with my solution , do you mean removing wpdb::prepare(); would have security reason? i don’t think so, ultimately both the way we are fetching data from database.

    Correct me if i am wrong. thanks.

    Andrea Fercia


    hi @jigesh
    yes you’re right also @nacin says in his post “You should instead the query directly, as there are no inputs”.
    So as long as you don’t have user inputs to sanitize, you’re safe.

    Btw the latest version 1.0.4 of the plugin already solved this, keeping prepare and correctly passing at least 1 argument on both cases (there are 2 query with $wpdb->prepare).
    Note the plugin still passes $list_ids directly in the first case and “1” in the second case so that it’s just a workaround to avoid the PHP warning.
    If you want to keep $wpdb->prepare just to be super-safe, and pass all parameters to prepare, it’s doable with something like this:

    remove brackets from:

    $list_ids="(".join(",", $notification_ids).")";

    and change it in

    $list_ids = join( ",", $notification_ids);

    so the first query become:

    $wpdb->get_results( $wpdb->prepare( "SELECT * FROM {$bp->core->table_name_notifications} WHERE id in (%s) AND is_new = %d", $list_ids, 1)  );

    and change the second query:

    $wpdb->get_col( $wpdb->prepare( "SELECT id FROM {$bp->core->table_name_notifications} WHERE user_id = %d AND is_new = %d", $user_id, 1 ) );


Viewing 8 replies - 1 through 8 (of 8 total)
  • The topic ‘bp live notification plugin’ is closed to new replies.
Skip to toolbar