View Issue Details

This bug affects 1 person(s).
 6
IDProjectCategoryView StatusLast Update
18643Bug reportsDatabase designpublic2023-03-16 15:37
Reporterbismark Assigned Togabrieljenik  
PrioritynoneSeverityblock 
Status closedResolutionno change required 
Product Version5.6.x 
Summary18643: missing database column surveys.faxto
Description

when importing a survey there is a message regarding the faxto field in the surveys table. indeed the column is missing.

Steps To Reproduce

Steps to reproduce

(Replace this text with detailed step-by-step instructions on how to reproduce the issue)

Expected result

(Write here what you expected to happen)

Actual result

(Write here what happened instead)

TagsNo tags attached.
Bug heat6
Complete LimeSurvey version number (& build)5.5.x
I will donate to the project if issue is resolvedNo
Browser
Database type & versionMySQL
Server OS (if known)
Webserver software & version (if known)
PHP Version8.0

Users monitoring this issue

There are no users monitoring this issue.

Activities

bismark

bismark

2023-02-16 14:54

reporter   ~73850

we (mis)use this field in several projects, not for a fax number, but for other kind of data

Mazi

Mazi

2023-02-16 17:00

updater   ~73870

I think it is not a good idea to delete core survey fields. Fax is surely not used much any more. But for backwards compatibility, hiding it at the interface would be a starting point. Fully removing the field can break many things and should only be done at major version updates e.g. v 6.x.

c_schmitz

c_schmitz

2023-02-16 17:12

administrator   ~73871

I agree, this should have been done in a major version - I am really sorry for that.
However, this change has been done for months now and nobody else complained about it, so we won't roll it back.

bismark

bismark

2023-02-17 09:50

reporter   ~73873

I can see the commit https://github.com/LimeSurvey/LimeSurvey/commits/master/installer/create-database.php
but this important change has not been mentioned in the release notes https://github.com/LimeSurvey/LimeSurvey/blob/master/docs/release_notes.txt

I dont notice every commit, but I read the release notes on a regular basis

Mazi

Mazi

2023-03-07 22:44

updater   ~74066

I hope this only affects LS 5.x and not LS 3.x as well?

Issue History

Date Modified Username Field Change
2023-02-16 14:52 bismark New Issue
2023-02-16 14:54 bismark Note Added: 73850
2023-02-16 14:54 bismark Bug heat 0 => 2
2023-02-16 17:00 Mazi Note Added: 73870
2023-02-16 17:00 Mazi Bug heat 2 => 4
2023-02-16 17:12 c_schmitz Note Added: 73871
2023-02-16 17:12 c_schmitz Bug heat 4 => 6
2023-02-16 17:14 gabrieljenik Assigned To => gabrieljenik
2023-02-16 17:14 gabrieljenik Status new => assigned
2023-02-17 09:50 bismark Note Added: 73873
2023-03-07 22:44 Mazi Note Added: 74066
2023-03-16 15:37 gabrieljenik Status assigned => closed
2023-03-16 15:37 gabrieljenik Resolution open => no change required