View Issue Details

Related ChangesetsJump to NotesJump to History
This issue affects 1 person(s).
 8
IDProjectCategoryView StatusDate SubmittedLast Update
15803Bug reportsExpression Managerpublic2020-01-31 08:462026-10-20 17:06
ReporterDenisChenu Assigned Toc_schmitz  
PrioritynormalSeverityminor 
Status closedResolutionfixed 
Product Version3.22.1 
Summary15803: No warning shown when compare numeric with string function
Description

When comparing a number with a string function : there are no warning

Steps To Reproduce

Import included lss : check logic

But clearly {3 lt str_pad("A", 3)} must be set as warning

Additional Information

Function return always NUMBER type
Must return STRING or NUMBER (and maybe BOOLEAN ?)

TagsNo tags attached.
Attached Files
Capture d’écran du 2020-01-31 08-46-33.png (20,001 bytes)   
Capture d’écran du 2020-01-31 08-46-33.png (20,001 bytes)   
limesurvey_survey_BadWarning.lss (15,142 bytes)
Bug heat8
Complete LimeSurvey version number (& build)3.22 and 4.10
I will donate to the project if issue is resolvedNo
Browsernot relevant
Database type & versionnot relevant
Server OS (if known)not relevant
Webserver software & version (if known)not relevant
PHP Versionphp7.3
Git branch / Pull Request URLhttps://github.com/LimeSurvey/LimeSurvey/pull/5303

Relationships

Relationship GraphDependency Graph
related to 15802 acknowledgedollehar Invalid warning string about numeric value 

Users monitoring this issue

There are no users monitoring this issue.

Activities

ollehar

ollehar

2020-01-31 12:55

reporter   ~55582

lt? Same as '<'?
DenisChenu

DenisChenu

2020-01-31 14:12

developer   ~55586

https://manual.limesurvey.org/Expression_Manager#Operators

< <= > >= lt le gt ge
DenisChenu

DenisChenu

2020-01-31 14:13

developer   ~55587

Oups : link where return a number : https://github.com/LimeSurvey/LimeSurvey/blob/eba81f2ab24d2b2484d7b2b1d730df191b573aed/application/helpers/expressions/em_core_helper.php#L2182
c_schmitz

c_schmitz

2026-10-20 13:02

administrator   ~86554

Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=40933
LimeBot

LimeBot

2026-10-20 17:06

administrator   ~86630

Fixed in Release 7.7.0+261020

Related Changesets

LimeSurvey: master 364c1632

2026-10-20 13:02

c_schmitz

Committer: GitHub


Details Diff		 Fixed issue 15803: No warning shown when comparing numeric result with string result (05303) Affected Issues
15803
mod - application/helpers/expressions/em_core_helper.php Diff File
mod - tests/unit/helpers/ExpressionWarningTest.php Diff File

Issue History

Date Modified Username Field Change
2020-01-31 08:46 DenisChenu New Issue
2020-01-31 08:46 DenisChenu File Added: Capture d’écran du 2020-01-31 08-46-33.png
2020-01-31 08:46 DenisChenu File Added: limesurvey_survey_BadWarning.lss
2020-01-31 08:47 DenisChenu Relationship added related to 15802
2020-01-31 12:55 ollehar Note Added: 55582
2020-01-31 12:55 ollehar Priority none => normal
2020-01-31 13:45 ollehar Status new => acknowledged
2020-01-31 13:45 ollehar Assigned To => ollehar
2020-01-31 13:45 ollehar Status acknowledged => assigned
2020-01-31 13:45 ollehar Status assigned => acknowledged
2020-01-31 14:12 DenisChenu Note Added: 55586
2020-01-31 14:13 DenisChenu Note Added: 55587
2026-09-14 23:22 c_schmitz Assigned To ollehar => c_schmitz
2026-09-14 23:22 c_schmitz Status acknowledged => ready for code review
2026-09-14 23:23 c_schmitz Story point estimate => 0
2026-09-14 23:23 c_schmitz Git branch / Pull Request URL => https://github.com/LimeSurvey/LimeSurvey/pull/5303
2026-10-20 13:02 c_schmitz Changeset attached => LimeSurvey master 364c1632
2026-10-20 13:02 c_schmitz Note Added: 86554
2026-10-20 13:02 c_schmitz Status ready for code review => resolved
2026-10-20 13:02 c_schmitz Resolution open => fixed
2026-10-20 13:02 c_schmitz Bug heat 4 => 6
2026-10-20 17:06 LimeBot Note Added: 86630
2026-10-20 17:06 LimeBot Status resolved => closed
2026-10-20 17:06 LimeBot Bug heat 6 => 8