2011-08-04 Javier Lanz <javier.lanz@artica.es>

* godmode/modules/manage_nc_groups.php: Fixed creating &
        * updating
        component gruops with blank names

        Fixes: #3384668



git-svn-id: https://svn.code.sf.net/p/pandora/code/trunk@4665 c3f86ba8-e40f-0410-aaad-9ba5e7f4b01f
This commit is contained in:
javilanz 2011-08-04 11:25:02 +00:00
parent 4ab536af57
commit 15cd275722
2 changed files with 31 additions and 14 deletions

View File

@ -1,3 +1,10 @@
2011-08-04 Javier Lanz <javier.lanz@artica.es>
* godmode/modules/manage_nc_groups.php: Fixed creating & updating
component gruops with blank names
Fixes: #3384668
2011-08-02 Javier Lanz <javier.lanz@artica.es> 2011-08-02 Javier Lanz <javier.lanz@artica.es>
* images/skin/skin_default/include/styles/pandora.css: Removed old * images/skin/skin_default/include/styles/pandora.css: Removed old

View File

@ -41,6 +41,11 @@ if ($create) {
$name = (string) get_parameter ('name'); $name = (string) get_parameter ('name');
$parent = (int) get_parameter ('parent'); $parent = (int) get_parameter ('parent');
if ($name == '') {
ui_print_error_message (__('Could not be created. Blank name'));
require_once ('manage_nc_groups_form.php');
return;
} else {
$result = db_process_sql_insert ('tnetwork_component_group', $result = db_process_sql_insert ('tnetwork_component_group',
array ('name' => $name, array ('name' => $name,
'parent' => $parent)); 'parent' => $parent));
@ -48,11 +53,15 @@ if ($create) {
__('Successfully created'), __('Successfully created'),
__('Could not be created')); __('Could not be created'));
} }
}
if ($update) { if ($update) {
$name = (string) get_parameter ('name'); $name = (string) get_parameter ('name');
$parent = (int) get_parameter ('parent'); $parent = (int) get_parameter ('parent');
if ($name == '') {
ui_print_error_message (__('Not updated. Blank name'));
} else {
$result = db_process_sql_update ('tnetwork_component_group', $result = db_process_sql_update ('tnetwork_component_group',
array ('name' => $name, array ('name' => $name,
'parent' => $parent), 'parent' => $parent),
@ -61,6 +70,7 @@ if ($update) {
__('Successfully updated'), __('Successfully updated'),
__('Not updated. Error updating data')); __('Not updated. Error updating data'));
} }
}
if ($delete) { if ($delete) {
$result = db_process_sql_delete ('tnetwork_component_group', $result = db_process_sql_delete ('tnetwork_component_group',
@ -70,7 +80,7 @@ if ($delete) {
__('Not deleted. Error deleting data')); __('Not deleted. Error deleting data'));
} }
if ($id || $new) { if (($id || $new) && !$delete) {
require_once ('manage_nc_groups_form.php'); require_once ('manage_nc_groups_form.php');
return; return;
} }