CMS MADE SIMPLE FORGE

Frontend Users

 
Back Back to List

[#6819] Error when submitting empty non-required multi-select field in change settings

avatar
Created By: Steven Hart (uksteve98)
Date Submitted: Tue Aug 23 12:29:59 -0400 2011

Assigned To: Robert Campbell (calguy1000)
Version: 1.15.1
CMSMS Version: 1.9.2
Severity: Minor
Resolution: Awaiting Response
State: Closed
Summary:
Error when submitting empty non-required multi-select field in change settings
Detailed Description:
There appears to be a problem when trying to submit a non-required multi-select
field with no value via the change settings template. The error is:

Warning: implode() [function.implode]: Invalid arguments passed in
/modules/FrontEndUsers/action.do_userchangesettings.php on line 176

Code around here is:

case '5': /* multiselect */
      if( $required && !isset($params['feu_input_'.$propname]) )
	{
	  $params['error'] = 1;
$params['message'] =
$this->Lang('error_requiredfield',$defnsbyname[$propname]['prompt']);
	  $this->Redirect( $id, 'changesettings', $returnid, $params );
	}
      // encode it into a comma separated list.
$params['feu_input_'.$propname] =
implode(',',$params['feu_input_'.$propname]);
      break;

...I'm guessing this is a bug with the validation?


History

Comments
avatar 
Date: 2011-08-26 22:00
Posted By: Robert Campbell (calguy1000)

fixed in svn.
avatar 
Date: 2011-09-20 19:53
Posted By: Geoff Hill (heffegg)

So where do we get the code to fix this problem?

CMSMS 1.9.4.2 with FEU 1.15.1 has this error - I need to fix it for a production
site.
avatar 
Date: 2011-09-28 05:22
Posted By: Steven Hart (uksteve98)

I'm also unable to access the SVN repository for frontendusers, it appears to be
missing. Or does it come under a different name?

Tried connecting via TortoiseSVN and the web-based source code browser just
shows "Unable to find repository "frontendusers"."
Updates

Updated: 2012-02-18 11:22
state: Open => Closed

Updated: 2011-09-28 05:22
resolution_id: 7 => 10

Updated: 2011-09-28 05:22
state: Closed => Open

Updated: 2011-09-15 08:24
state: Open => Closed

Updated: 2011-08-26 22:00
resolution_id: 5 => 7

Updated: 2011-08-23 12:30
resolution_id: => 5
severity_id: 2 => 3