View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0010493 | mantisbt | code cleanup | public | 2009-05-18 02:51 | 2018-03-04 00:36 |
Reporter | dhx | Assigned To | vboctor | ||
Priority | normal | Severity | major | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Product Version | git trunk | ||||
Target Version | 2.12.0 | Fixed in Version | 2.12.0 | ||
Summary | 0010493: Non-existent duplicate_realname column is updated by various functions in user_api.php | ||||
Description | It seems that all references within user_api.php to a column "duplicate_realname" in mantis_user_table are invalid. This column does not appear to exist in the Mantis database schema (has it ever existed?). user_set_field will fail when it tries to update the non-existent column in mantis_user_table with the following error: APPLICATION ERROR 0000401 Database query failed. Error received from database was #1054: Unknown column 'duplicate_realname' in 'field list' for the query: UPDATE mantis_user_table | ||||
Tags | No tags attached. | ||||
To reproduce, simply attempt to set the real name for two different accounts to the same string. Bingo, you have an error! |
|
one mark.. Create one user with RealName: Test Name, second with Test Real. |
|
See admin/ugprades/0.18.inc.php |
|
Actually you're right, I accidentally had this setting to ON (which the default config strongly warns against). My bad doh I'm still confused why a function named "user_is_realname_unique" would go about updating the database. The function name implies it is performing a read-only operation. The documentation of $g_differentiate_duplicates is lacking... to be polite. It looks like this feature needs some major work to bring it up to the standards of the rest of Mantis... or it needs to be dropped completely (it doesn't sound like anyone can use it anyway at the moment). |
|
Part of the implementation was lost early in 1.0.0. I plan on adding it back in an incremental fashion. |
|
Code removed in PR https://github.com/mantisbt/mantisbt/pull/1277 |
|