0002539: a zero value Double or Float Value type should return "0" when converted to String - Mantis

Mantis Bugtracker

Viewing Issue Simple Details [ Jump to Notes ]

[ View Advanced ] [ Issue History ] [ Print ]

ID

Category

Severity

Reproducibility

Date Submitted

Last Update

0002539

[Quercus]

minor

always

03-19-08 21:27

03-21-08 13:19

Reporter

ckchris

View Status

public

Assigned To

nam

Priority

normal

Resolution

fixed

Status

closed

Product Version

Summary

0002539: a zero value Double or Float Value type should return "0" when converted to String

Description

I ran into a problem with an unexpected behavior when running phpBB3.

There is a code inside phpBB3's includes/db/mysqli.php that does a float-based division. A floor() is then applied to it. The return value of a floor() is a float value, which is expected and is the proper behavior that Quercus exhibits.

However, the problem is that the floor'ed value is appended to a SQL query string for the "limit x, y" part of the query. Unfortunately, the toString() for the float value of 0 returned "0.0" when run under quercus. This turned the limit string into "limit 0.0, 10", which mysql driver throws an exception.

I recommend that the Double and Float and Decimal value types should check for 0, and if so, return "0" for the string instead of "0.0" to retain compatible behavior with PHP.

Additional Information

Relationships

Notes
(0002871) nam 03-20-08 19:36	What Quercus version are you using? I remember this issue being fixed for 3.1.4 or 3.1.5. You might want to give 3.1.5 a try.

(0002873) ckchris 03-20-08 20:05 edited on: 03-20-08 20:06	Running the latest quercus version compiled straight off the SVN that's checked out as of today. Revision 3969. I did a double confirmation and removed the (int) type conversion in mysqli.php in phpbb3. The line is originally this: $query .= "\n LIMIT " . ((!empty($offset)) ? $offset . ', ' . $total : $total); Running it with the newest SVN snapshot still gives me the error. By adding this a forced type conversion: $query .= "\n LIMIT " . ((!empty($offset)) ? (int) $offset . ', ' . $total : $total); Things worked after that. Previously, I was using 3.1.5. And before that I was using the version that came with Resin 3.1.5.

(0002875) nam 03-21-08 13:19	php/033k issue only seen if the float is on the left hand side of an append ('.') i.e. var_dump(10.0 . ", 10")

Issue History
Date Modified	Username	Field	Change
03-19-08 21:27	ckchris	New Issue
03-19-08 21:28	ckchris	Issue Monitored: ckchris
03-20-08 19:36	nam	Note Added: 0002871
03-20-08 20:05	ckchris	Note Added: 0002873
03-20-08 20:06	ckchris	Note Edited: 0002873
03-20-08 21:23	nam	Status	new => assigned
03-20-08 21:23	nam	Assigned To	=> nam
03-21-08 13:19	nam	Status	assigned => closed
03-21-08 13:19	nam	Note Added: 0002875
03-21-08 13:19	nam	Resolution	open => fixed
03-21-08 13:19	nam	Fixed in Version	=> 3.1.6

Mantis 1.0.0rc3[^]

Copyright © 2000 - 2005 Mantis Group

35 total queries executed.
28 unique queries executed.