View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0007690 | mantisbt | db mssql | public | 2006-12-31 00:45 | 2014-05-16 15:00 |
Reporter | rzg | Assigned To | vboctor | ||
Priority | normal | Severity | minor | Reproducibility | have not tried |
Status | closed | Resolution | fixed | ||
Product Version | 1.1.0a4 | ||||
Fixed in Version | 1.1.0rc1 | ||||
Summary | 0007690: number_format(); bad for SQL statments | ||||
Description | See file_api line 200. We do a number_format, then use the formatted number in a SQL statment on line 230. This leads to SQL syntax errors since '7,428' isn't an integer. Not MSSQL specific really. | ||||
Tags | No tags attached. | ||||
That should read 200 and 230 respectively |
|
This bug will happen in the following case:
The fix would be to replace number_format() with db_prepare_int(). This is in line 198 of the latest code. |
|
Hello, From victor's comment, it would appear that this issue is already fixed in the latest code. Paul |
|
Looking at CVS, it was never fixed... |
|
The fix is now applied. It seems that MySQL doesn't complain on the code before the fix. |
|
MantisBT currently supports Mysql and has support for other database engines. The support for other databases is known to be problematic. Having implemented the current database layer into Mantis 10 years ago, I'm currently working on replacing the current layer. If you are interested in using Mantis with non-mysql databases - for example, Oracle, PGSQL or MSSQL, and would be willing to help out testing the new database layer, please drop me an email at paul@mantisforge.org In the meantime, I'd advise running Mantis with Mysql Only to avoid issues. Thanks |
|