View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
15792 | Bug reports | Installation | public | 2020-01-28 17:11 | 2023-09-27 18:21 |
Reporter | tbart | Assigned To | c_schmitz | ||
Priority | normal | Severity | partial_block | ||
Status | closed | Resolution | reopened | ||
Product Version | 4.0.1 | ||||
Summary | 15792: Database upgrade errors DB version 359>425 (Incorrect string value, data too long for column) | ||||
Description | When executing the DB update, I get several errors. These characters seem to stem from copying texts from word. The next error, It seems we have longer "code"s than expected? The updater should fix both errors. | ||||
Steps To Reproduce | Update from 3.16.1 to 4.0.1 having mentioned code specifications and unicode characters in strings. | ||||
Tags | No tags attached. | ||||
Bug heat | 6 | ||||
Complete LimeSurvey version number (& build) | 4.0.1 | ||||
I will donate to the project if issue is resolved | No | ||||
Browser | |||||
Database type & version | mysql Ver 14.14 Distrib 5.7.29 | ||||
Server OS (if known) | Ubuntu 18.04.3 LTS | ||||
Webserver software & version (if known) | |||||
PHP Version | PHP 7.2.24 | ||||
Hey there, is it possible to test again the upgrade process and see which issues are popping up? Fixes were pushed in LS 4.4.x |
|
Sure. Tested again today, from 3.25.x to 4.4.4. I can confirm the special char encoding bug is gone! However, I still get CDbCommand konnte das SQL-Statement nicht ausführen: SQLSTATE[22001]: String data, right truncated: 1406 Data too long for column 'code' at row 291. The SQL statement executed was: INSERT INTO labels The respective table content looks like this: MariaDB [eval]> SELECT lid, code, min(sortorder), min(assessment_value) FROM labels_update400 GROUP BY lid, code limit 25; |
|
Thx - will get it in our backlog :) |
|
I cannot reproduce the issue. THe label code should never be longer than 5 chars and this limitation has been in the database forever. |
|
This is a vanilla install dating back to 2016 approx., no plugins whatsoever. No, the DB has never been touched, this must have been possible via the standard web GUI. I can adapt this for this installation, I guess (hope it does not need changes in other locations?). Still - this might not be the only installation where this happened and the fix should be in the upgrade process, i.e. shorten to 5 chars, see whether the label is still unique. Column "code" is defined as varchar(15), so the application should support that. If this is not intended, it should be reduced to varchar(5) so we do not end up with unexpectedly long data in the DB. Better throw an error right when the application tries to write beyond bounds not expected than write unsupported data to the DB! |
|
I checked back our source code to the beginning of LimeSurvey. The code never had a length of 15, it has been always 5 since LimeSurvey was created. This must have been manually modified. |
|
As a note to myself and anyone running into the same issue: https://github.com/LimeSurvey/LimeSurvey/blob/25f63d5758134ad5eca607453deb8a9e94b7f2bd/installer/create-database.php#L177 creates the column with string(20) now, but this happened only in DB version 483. |
|
Date Modified | Username | Field | Change |
---|---|---|---|
2020-01-28 17:11 | tbart | New Issue | |
2020-02-05 08:54 | cdorin | Assigned To | => cdorin |
2020-02-05 08:54 | cdorin | Status | new => assigned |
2021-02-04 10:13 | cdorin | Assigned To | cdorin => |
2021-02-04 10:13 | cdorin | Status | assigned => feedback |
2021-02-04 10:13 | cdorin | Note Added: 61931 | |
2021-02-09 16:05 | tbart | Note Added: 62083 | |
2021-02-09 16:05 | tbart | Status | feedback => new |
2021-02-09 16:39 | cdorin | Priority | none => normal |
2021-02-09 16:39 | cdorin | Status | new => confirmed |
2021-02-09 16:39 | cdorin | Sync to Zoho Project | => |Yes| |
2021-02-09 16:39 | cdorin | Note Added: 62085 | |
2021-02-15 13:48 | c_schmitz | Assigned To | => c_schmitz |
2021-02-15 13:48 | c_schmitz | Status | confirmed => feedback |
2021-02-15 13:48 | c_schmitz | Note Added: 62243 | |
2021-02-15 13:53 | c_schmitz | Status | feedback => closed |
2021-02-15 13:53 | c_schmitz | Resolution | open => unable to reproduce |
2021-02-16 12:23 | tbart | Status | closed => feedback |
2021-02-16 12:23 | tbart | Resolution | unable to reproduce => reopened |
2021-02-16 12:23 | tbart | Note Added: 62283 | |
2021-02-16 17:32 | c_schmitz | Status | feedback => closed |
2021-02-16 17:32 | c_schmitz | Note Added: 62294 | |
2021-02-18 14:17 | c_schmitz | Sync to Zoho Project | Yes => |
2023-09-27 18:21 | tbart | Note Added: 77387 |