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

performance fix

Browse Source
This commit is contained in:
Cornelius Weiß 2007-06-09 22:23:29 +00:00
parent 349b8c521a
commit b6a28b8090
1 changed files with 8 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
importexport/inc/class.definition.inc.php
View File

@ -80,6 +80,8 @@ class definition implements iface_egw_record {
if ( !( in_array( $this->user, $this->get_allowed_users() ) || $this->definition['owner'] == $this->user || $this->is_admin)) { if ( !( in_array( $this->user, $this->get_allowed_users() ) || $this->definition['owner'] == $this->user || $this->is_admin)) {
throw new Exception('Error: User "'.$this->user.'" is not permitted to get definition with identifier "'.$_identifier.'"!'); throw new Exception('Error: User "'.$this->user.'" is not permitted to get definition with identifier "'.$_identifier.'"!');
} }
$options_data = arrayxml::xml2array( $this->definition['plugin_options'] );
$this->definition['plugin_options'] = $options_data['root'];
} }
} }
@ -150,8 +152,7 @@ class definition implements iface_egw_record {
* @return array * @return array
*/ */
private function get_options() { private function get_options() {
$options_data = arrayxml::xml2array( $this->definition['plugin_options'] ); return $this->definition['plugin_options'];
return $options_data['plugin_options'];
} }
/** /**
@ -160,7 +161,7 @@ class definition implements iface_egw_record {
* @param array $options * @param array $options
*/ */
private function set_options(array $_plugin_options) { private function set_options(array $_plugin_options) {
$this->definition['plugin_options'] = arrayxml::array2xml( $_plugin_options, 'plugin_options' ); $this->definition['plugin_options'] = $_plugin_options;
} }
/** /**
@ -204,8 +205,8 @@ class definition implements iface_egw_record {
} }
// convert plugin_options into internal representation // convert plugin_options into internal representation
$this->set_allowed_users($this->definition['allowed_users']); $this->set_allowed_users( $this->definition['allowed_users'] );
$this->set_options($this->definition['plugin_options']); $this->set_options( $this->definition['plugin_options'] );
} }
/** /**
@ -229,9 +230,11 @@ class definition implements iface_egw_record {
} }
$this->so_sql->data = $this->definition; $this->so_sql->data = $this->definition;
$this->so_sql->data['plugin_options'] = arrayxml::array2xml( $this->definition['plugin_options'] );
if ($this->so_sql->save( array( 'definition_id' => $_dst_identifier ))) { if ($this->so_sql->save( array( 'definition_id' => $_dst_identifier ))) {
throw new Exception('Error: so_sql was not able to save definition: '.$this->get_identifier()); throw new Exception('Error: so_sql was not able to save definition: '.$this->get_identifier());
} }
return $this->definition['definition_id']; return $this->definition['definition_id'];
} }