CMS MADE SIMPLE FORGE

Form Builder

 

[#6640] TextFieldExpandable: incorrect output when only one value

avatar
Created By: Lindsay Adams (dingonv)
Date Submitted: Fri Jul 08 13:13:51 -0400 2011

Assigned To:
Version: 0.7
CMSMS Version: None
Severity: Minor
Resolution: Fixed
State: Closed
Summary:
TextFieldExpandable: incorrect output when only one value
Detailed Description:
<< TextFieldExpandable.class.php:86
$thisRow->input = $mod->fbCreateInputText($id,
'fbrp__'.$this->Id.'[]',$this->Value[$i],$this->GetOption('length')<25?$this->GetOption('length'):25,
$this->GetOption('length'),$js.$this->GetCSSIdTag('_'.$i));

>>  TextFieldExpandable.class.php:86-90
if(is_array($this->Value)) {
                                $outputValue=$this->Value[$i];
                        }
                        else {
                                $outputValue=$this->Value;
                        }
$thisRow->input = $mod->fbCreateInputText($id,
'fbrp__'.$this->Id.'[]',$outputValue,$this->GetOption('length')<25?$this->GetOption('length'):25,
$this->GetOption('length'),$js.$this->GetCSSIdTag('_'.$i));

output was only spitting out the first letter of $this->Value because
$this->Value was not an array.
added a check for data type to change the outputValue depending on whether or
not it was an array.


History

Comments
avatar
Date: 2013-09-02 15:55
Posted By: Tapio Löytty (Stikki)

Will check this.
      
avatar
Date: 2014-10-24 03:52
Posted By: Rolf (rolf1)

We did some fixes here, could you please test it again in latest FB release.
Thanks
      
avatar
Date: 2014-11-18 16:33
Posted By: Jeff Bosch (ajprog)

I made a change so this should be fixed in 0.8.1, but I don't know how to
reproduce it so I can't check.
      
avatar
Date: 2015-01-21 10:44
Posted By: Rolf (rolf1)

FB 0.8.1 is released
      
Updates

Updated: 2015-01-21 10:44
state: Open => Closed

Updated: 2014-11-18 16:33
resolution_id: 10 => 7

Updated: 2014-10-24 03:52
resolution_id: 6 => 10
assigned_to_id: 11306 => 100

Updated: 2013-09-02 15:55
resolution_id: => 6
cmsms_version_id: => -1
assigned_to_id: 100 => 11306