Serendipity Forums

Posted: **Fri Aug 26, 2005 6:52 am**

The insert into the table _pluginlist tries to insert an empty string instead of an integer when there is no local file for the corresponding plugin.
(the field last_modified). It might be a difference between postgresql and mysql.

I have fixed it by doing the following. I added the following code:

Code: Select all


[dmg@ti include]$ diff plugin_api.inc.php plugin_api.inc.php~
470,476d469
<
<         if (isset($pluginFile)) {
<           $lastModified = 0;filemtime($pluginFile);
<         } else {
<           $lastModified = 0;
<         }
<
492c485
<                 'last_modified'   => $lastModified
---
>                 'last_modified'   => filemtime($pluginFile)

Posted: **Fri Aug 26, 2005 3:38 pm**

Hi dmg!

Many thanks for spotting this! I've just committed your fix to SVN!

Best regards,
Garvin

Posted: **Sat Aug 27, 2005 1:32 am**

I just realized I made a mistake in the fix:

Code: Select all


[dmg@ti include]$ diff plugin_api.inc.php plugin_api.inc.php~
470,476d469
<
<         if (isset($pluginFile)) {
<           $lastModified = 0;filemtime($pluginFile);
                            ^^

please remove the 0;
What an ugly cut-and-paste bug! Sorry.

dmg

Posted: **Sat Aug 27, 2005 2:41 am**

dmg wrote:I just realized I made a mistake in the fix:
Code: Select all
[dmg@ti include]$ diff plugin_api.inc.php plugin_api.inc.php~
470,476d469
<
<         if (isset($pluginFile)) {
<           $lastModified = 0;filemtime($pluginFile);
                            ^^
please remove the 0;
What an ugly cut-and-paste bug! Sorry.

dmg

Well, that did not seem to fix it: the fix is just to add (int) in front of filemtime:
replace, in the original code:

Code: Select all

             last_modified'   => filemtime($pluginFile)

with
             'last_modified'   => (int)filemtime($pluginFile)

dmg

Serendipity Forums

bug in setPluginInfo and postgresql

bug in setPluginInfo and postgresql

Re: bug in setPluginInfo and postgresql

Re: bug in setPluginInfo and postgresql

Re: bug in setPluginInfo and postgresql