Okay
  Public Ticket #774031
Warning: printf(): Too few arguments
Closed

Comments

  • Monty started the conversation

    Just an FYI at this point, but we were getting this error displayed in place of the names of all of our commenters: 

    Warning: printf(): Too few arguments in /home/content/.../html/wp-content/themes/mightymag/inc/template-tags.php on line 90

    It appears there was a second argument missing in that function. We corrected it by adding $output after the %s

  • Monty replied

    Actually, that did not fix the issue. Some -- not all -- commenters are having their names replaced with the warning. And now it appears the others who don't have that happening, just have "output" appearing after their names.

    We need this fixed. What do we do?

    You can see an example here: http://totalpackers.com/2016/05/green-bay-packers-ruining-football/

  •  93
    djwd replied

    Hi there,
    that is just a PHP warning that shouldn't affect any functionality. I don't see the warnings so you probably fixed it already,  btw you'd just need to turn WP_debug off which should be always disabled on a production site.

    Regards
    djwd

  • Monty replied

    Debug was never on. It was a code error and we did fix it. We had to rewrite it. 

    <?php?>
    <cite class="fn">
    <?php comment_author_link(); ?>
    </cite>

  •  93
    djwd replied

    Hello there,
    If you are seeing errors/warnings/notices on production site with WP_DEBUG disabled then it would mean you have set display_errors=1 in your php.ini file. It's enabled by default for development. But you'll want it off on production servers, just a personal suggestion :)

    Btw indeed something to fix, so thanks for the feedback! 

    Regards
    djwd