Page 1 of 1

What does this mean?

Posted: Fri Oct 24, 2008 3:20 pm
by fourd5dswb
What does this mean? I am getting this when I click on an entry on my site.

Warning: sprintf(): Too few arguments in /home/content/j/b/u/jbush45/html/plugins/serendipity_event_linktoolbar/serendipity_event_linktoolbar.php on line 103

Warning: sprintf(): Too few arguments in /home/content/j/b/u/jbush45/html/plugins/serendipity_event_linktoolbar/serendipity_event_linktoolbar.php on line 104

My site is www.prowlser.com.

Thanks.

Posted: Fri Oct 24, 2008 3:58 pm
by judebert
It means you have the HTML Link Metatags plugin installed, and it has been modified from the latest version. Line 103 should read

Code: Select all

$prevID = serendipity_db_query(sprintf($querystring, '<', 'DESC'));
And the $querystring, defined on the line above, should have exactly two % signs. Yours apparently has more than two % signs, messing up the plugin.

The only effect should be to make some of the metadata in your <HEAD> incorrect. You can remove the plugin if the metadata isn't important to you.

Where are you seeing it? I clicked an entry, but can't find the warning.

Posted: Fri Oct 24, 2008 4:01 pm
by judebert
Come to think of it, some other plugin could be modifying the 'join' or 'and', or your database prefix could include a % sign. What other plugins do you have installed? Are you comfortable with modifying a file on your server for debugging purposes?

Location

Posted: Fri Oct 24, 2008 7:48 pm
by fourd5dswb
When you click on an entry, at the top of the page under my ad banner you will see it. It is in black and it may be hard to spot because of the background.

Just tell me what you think I should do and I will let you know if I can do it.

Thanks.

Re: Location

Posted: Sat Oct 25, 2008 11:39 am
by garvinhicking
Hi!

I've patched the plugin.

you can download this file and replace your current /plugins/serendipity_event_linktoolbar/serendipity_event_linktoolbar.php file with it:

http://php-blog.cvs.sourceforge.net/vie ... vision=1.7

(Judebert:) However, now that I looked at the plugin, I believe it should best be merged with the entrypaging plugin. this one does a much better job on deducing the next entry (by date, not by ID)...

Regards,
Garvin

Posted: Tue Oct 28, 2008 2:13 pm
by judebert
Now, why didn't I think of that?

Do you see any performance differences between str_replace and sprintf?

I hope to be able to contribute again in a week or two. (I'm swamped with family, job, and volunteer work right now.) I'll add this to my issues in SourceForge.

Posted: Tue Oct 28, 2008 2:39 pm
by garvinhicking
Hi!
Do you see any performance differences between str_replace and sprintf?
Internally, they boil down to the same thing. But using "%" as the sprintf modifiere imposes too many problem, so I've resorted on customt okens.
I hope to be able to contribute again in a week or two. (I'm swamped with family, job, and volunteer work right now.) I'll add this to my issues in SourceForge.
If you have that time, of course it would be much appreciated. It's not that big of a thing,so don't let your family down because if it *g*

Regards,
Garvin

Thanks

Posted: Mon Nov 10, 2008 3:19 am
by fourd5dswb
Thanks for the help.