holm/egroupware
1
1
Fork 0
forked from extern/egroupware
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Use egw_customfields::update() instead of so->save(), since it handles the arrays and cache already.

Browse Source
This commit is contained in:
Nathan Gray 2014-10-27 16:31:24 +00:00
parent ea5ec6b787
commit 629c66d00e
1 changed files with 9 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
admin/inc/class.customfields.inc.php
View File

@ -271,8 +271,6 @@ class customfields
break; break;
case 'save': case 'save':
case 'apply': case 'apply':
$content['cf_type2'] = implode(',',(array)$content['cf_type2']);
$content['cf_private'] = implode(',',(array)$content['cf_private']);
if (!empty($content['cf_values'])) if (!empty($content['cf_values']))
{ {
$values = array(); $values = array();
@ -284,8 +282,15 @@ class customfields
} }
$content['cf_values'] = $values; $content['cf_values'] = $values;
} }
$content['cf_values'] = json_encode($content['cf_values']); $update_content = array();
$this->so->save($content); foreach($content as $key => $value)
{
if(substr($key,0,3) == 'cf_')
{
$update_content[substr($key,3)] = $value;
}
}
egw_customfields::update($update_content);
egw_framework::refresh_opener('Saved', 'admin', $cf_id, 'edit'); egw_framework::refresh_opener('Saved', 'admin', $cf_id, 'edit');
if ($action != 'save') if ($action != 'save')
{ {