Skip to:
Content
Pages
Categories
Search
Top
Bottom

bp-templates/bp-legacy/buddypress/activity/index.php line 164

  • @mike80222

    Participant

    I’m having some trouble with line 164 of bp-templates/bp-legacy/buddypress/activity/index.php

    I’m getting PHP errors about printf() only getting one argument instead of two. It looks to me like the parentheses aren’t right on that line, but honestly I’m having a little bit of trouble parsing it myself. I would be very grateful if someone more knowledgeable could take a look!

Viewing 7 replies - 1 through 7 (of 7 total)
  • @venutius

    Moderator

    Hi Mike,

    It looks like we have repeated '%s new', and if you remove that is should work.

    Are you able to give that a try?

    @espellcaste

    Moderator

    Can you share the error and which version of BuddyPress you are using?

    @espellcaste

    Moderator

    I see the problem, btw. But just trying to get more data about the BP version.

    @mike80222

    Participant

    @venutius, yes that does appear to fix it. I guess I was reading the line wrong. This seems like a simple change. I can certainly patch it temporarily, but is there any chance of getting it fixed in the next release?


    @espellcaste
    , actually I’m running different versions of BP on different sites. But I don’t think that file has change recently (it is legacy after all). The exact error depends on what PHP you’re running. Older PHPs give:

    Warning: printf(): Too few arguments ... plugins/buddypress/bp-templates/bp-legacy/buddypress/activity/index.php on line 164

    Newer PHPs give:

    PHP Fatal error: Uncaught ArgumentCountError: 2 arguments are required, 1 given in ... plugins/buddypress/bp-templates/bp-legacy/buddypress/activity/index.php:164

    and then a stack trace.

    @espellcaste

    Moderator

    @mike80222

    Participant

    I see it. Thank you!

    @espellcaste

    Moderator

    This was fixed and will be released as part of BuddyPress 14.1. https://buddypress.trac.wordpress.org/milestone/14.1.0

Viewing 7 replies - 1 through 7 (of 7 total)
  • You must be logged in to reply to this topic.
Skip to toolbar