projects
/
fa-stable.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
submit_js_confirm() fixed to be used on button loadable via ajax.
[fa-stable.git]
/
gl
/
manage
/
gl_account_types.php
diff --git
a/gl/manage/gl_account_types.php
b/gl/manage/gl_account_types.php
index 171bf83f6504758b982d7352774830538fc6b953..3cef7a07540acdc4e051ae30c8c8665eae737ae3 100644
(file)
--- a/
gl/manage/gl_account_types.php
+++ b/
gl/manage/gl_account_types.php
@@
-19,27
+19,26
@@
include($path_to_root . "/gl/includes/gl_db.inc");
include($path_to_root . "/includes/ui.inc");
include($path_to_root . "/includes/ui.inc");
-simple_page_mode(
tru
e);
+simple_page_mode(
fals
e);
//-----------------------------------------------------------------------------------
function can_process()
{
//-----------------------------------------------------------------------------------
function can_process()
{
- global $selected_id;
-
- /*if (!input_num('id'))
+ if (strlen(trim($_POST['id'])) == 0)
{
{
- display_error( _("The account
id must be an integer an
d cannot be empty."));
+ display_error( _("The account
group i
d cannot be empty."));
set_focus('id');
return false;
set_focus('id');
return false;
- }
*/
- if (strlen(
$_POST['name']
) == 0)
+ }
+ if (strlen(
trim($_POST['name'])
) == 0)
{
display_error( _("The account group name cannot be empty."));
set_focus('name');
return false;
}
{
display_error( _("The account group name cannot be empty."));
set_focus('name');
return false;
}
- if (isset($selected_id) && ($selected_id == $_POST['parent']))
+ //if (strcmp($_POST['id'], $_POST['parent']) == 0)
+ if ($_POST['id'] === $_POST['parent'])
{
display_error(_("You cannot set an account group to be a subgroup of itself."));
return false;
{
display_error(_("You cannot set an account group to be a subgroup of itself."));
return false;
@@
-56,7
+55,7
@@
if ($Mode=='ADD_ITEM' || $Mode=='UPDATE_ITEM')
if (can_process())
{
if (can_process())
{
- if ($selected_id !=
-1
)
+ if ($selected_id !=
""
)
{
if (update_account_type($_POST['id'], $_POST['name'], $_POST['class_id'], $_POST['parent'], $_POST['old_id']))
display_notification(_('Selected account type has been updated'));
{
if (update_account_type($_POST['id'], $_POST['name'], $_POST['class_id'], $_POST['parent'], $_POST['old_id']))
display_notification(_('Selected account type has been updated'));
@@
-65,9
+64,9
@@
if ($Mode=='ADD_ITEM' || $Mode=='UPDATE_ITEM')
{
if (add_account_type($_POST['id'], $_POST['name'], $_POST['class_id'], $_POST['parent'])) {
display_notification(_('New account type has been added'));
{
if (add_account_type($_POST['id'], $_POST['name'], $_POST['class_id'], $_POST['parent'])) {
display_notification(_('New account type has been added'));
- $Mode = 'RESET';
}
}
}
}
+ $Mode = 'RESET';
}
}
}
}
@@
-75,7
+74,7
@@
if ($Mode=='ADD_ITEM' || $Mode=='UPDATE_ITEM')
function can_delete($selected_id)
{
function can_delete($selected_id)
{
- if ($selected_id ==
-1
)
+ if ($selected_id ==
""
)
return false;
$type = db_escape($selected_id);
return false;
$type = db_escape($selected_id);
@@
-109,7
+108,7
@@
if ($Mode == 'Delete')
}
if ($Mode == 'RESET')
{
}
if ($Mode == 'RESET')
{
- $selected_id =
-1
;
+ $selected_id =
""
;
$_POST['id'] = $_POST['name'] = '';
unset($_POST['parent']);
unset($_POST['class_id']);
$_POST['id'] = $_POST['name'] = '';
unset($_POST['parent']);
unset($_POST['class_id']);
@@
-119,7
+118,7
@@
if ($Mode == 'RESET')
$result = get_account_types(check_value('show_inactive'));
start_form();
$result = get_account_types(check_value('show_inactive'));
start_form();
-start_table(
$table_style
);
+start_table(
TABLESTYLE
);
$th = array(_("Group ID"), _("Group Name"), _("Subgroup Of"), _("Class Type"), "", "");
inactive_control_column($th);
table_header($th);
$th = array(_("Group ID"), _("Group Name"), _("Subgroup Of"), _("Class Type"), "", "");
inactive_control_column($th);
table_header($th);
@@
-132,7
+131,7
@@
while ($myrow = db_fetch($result))
$bs_text = get_account_class_name($myrow["class_id"]);
$bs_text = get_account_class_name($myrow["class_id"]);
- if ($myrow["parent"] == '
0' || $myrow["parent"] == '
-1')
+ if ($myrow["parent"] == '-1')
{
$parent_text = "";
}
{
$parent_text = "";
}
@@
-155,9
+154,9
@@
inactive_control_row($th);
end_table(1);
//-----------------------------------------------------------------------------------
end_table(1);
//-----------------------------------------------------------------------------------
-start_table(
$table_style
2);
+start_table(
TABLESTYLE
2);
-if ($selected_id !=
-1
)
+if ($selected_id !=
""
)
{
if ($Mode == 'Edit')
{
{
if ($Mode == 'Edit')
{
@@
-167,15
+166,19
@@
if ($selected_id != -1)
$_POST['id'] = $myrow["id"];
$_POST['name'] = $myrow["name"];
$_POST['parent'] = $myrow["parent"];
$_POST['id'] = $myrow["id"];
$_POST['name'] = $myrow["name"];
$_POST['parent'] = $myrow["parent"];
+ if ($_POST['parent'] == '-1')
+ $_POST['parent'] == "";
$_POST['class_id'] = $myrow["class_id"];
$_POST['class_id'] = $myrow["class_id"];
- hidden('selected_id', $
selected_id
);
+ hidden('selected_id', $
myrow['id']
);
hidden('old_id', $myrow["id"]);
}
hidden('old_id', $myrow["id"]);
}
- hidden('id');
- text_row_ex(_("ID:"), 'id', 10);
+ else
+ {
+ hidden('selected_id', $selected_id);
+ hidden('old_id', $_POST["old_id"]);
+ }
}
}
-else
- text_row_ex(_("ID:"), 'id', 10);
+text_row_ex(_("ID:"), 'id', 10);
text_row_ex(_("Name:"), 'name', 50);
gl_account_types_list_row(_("Subgroup Of:"), 'parent', null, _("None"), true);
text_row_ex(_("Name:"), 'name', 50);
gl_account_types_list_row(_("Subgroup Of:"), 'parent', null, _("None"), true);
@@
-184,7
+187,7
@@
class_list_row(_("Class Type:"), 'class_id', null);
end_table(1);
end_table(1);
-submit_add_or_update_center($selected_id ==
-1
, '', 'both');
+submit_add_or_update_center($selected_id ==
""
, '', 'both');
end_form();
end_form();