NOTE:: Before reporting an issue, make sure you are running the latest version, currently 3.3.1

View Issue Details Jump to Notes ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0002809phplist applicationAll Otherpublic30-03-05 13:1220-05-05 21:14
PlatformOSOS Version
Product Version 
Target VersionFixed in Version2.9.5 
Summary0002809: Sql_Affected_Rows missing explicit database link
DescriptionThe function Sql_Affected_Rows is missing the explicit database
connection handler:

function Sql_Affected_Rows() {
   return mysql_affected_rows();

When e.g. using custom database session save handlers, it's not
guaranteed that the last opened mysql link is the one needed by this

I've seen that Sql_Insert_id() has been fixed since, it would be nice if
Sql_Affected_Rows() could be fixed too.
TagsNo tags attached.
Attached Filesdiff file icon [^] (312 bytes) 30-03-05 13:12 [Show Content]

- Relationships Relation Graph ] Dependency Graph ]
related to 0002457resolvedmichiel PHPList v2.9.5 release 

-  Notes
michiel (manager)
30-03-05 14:19


patching file
Hunk #1 FAILED at 166.
1 out of 1 hunk FAILED -- saving rejects to file

in this case the patch is small enough to do manually, but it would be useful to diff against the CVS code instead
mfischer (reporter)
30-03-05 14:59

I'm sorry I'm unable to do the CVS checkout. This is all I can do for this bug report, this is actually only a one-line change so I hope you're capable of doing it yourself. Thanks.
michiel (manager)
30-03-05 15:05

Yes, in this case it's ok, but if you were to submit bigger patches, please use the cvs.
mfischer (reporter)
30-03-05 15:14

Thanks for your accommodation. I promised I will. Cheers.
DamienMcKenna (manager)
31-03-05 05:29

The new line should say:

return mysql_affected_rows($GLOBALS["database_connection"]);

Figured I'd save everyone the hassle.


Copyright © 2000 - 2017 MantisBT Team
Powered by Mantis Bugtracker