X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=inc%2Fmodules%2Fadmin%2Foverview-inc.php;h=db1e154849cc5becddc63b7ae12d1a588e46a43a;hb=22f132c8d6314f76c69dff58ec0dac21f5ca3bf2;hp=870c5b277eff6ef2760f305b5096b4dcf91014cc;hpb=b64f960124452301abd69ae3bd7ac4daee420550;p=mailer.git diff --git a/inc/modules/admin/overview-inc.php b/inc/modules/admin/overview-inc.php index 870c5b277e..db1e154849 100644 --- a/inc/modules/admin/overview-inc.php +++ b/inc/modules/admin/overview-inc.php @@ -40,7 +40,7 @@ // Some security stuff... if ((!defined('__SECURITY')) || (!isAdmin())) { die(); -} +} // END - if // @TODO This function does also check for uncompleted tasks function outputStandardOverview (&$result_tasks) { @@ -133,7 +133,7 @@ function outputSeletectedTasks ($postData, $result_tasks) { if ((isset($postData['assign'])) && (count($postData['sel']) > 0)) { // Assign / do tasks $OUT = ''; - foreach ($postData['sel'] as $id => $sel) { + foreach ($postData['sel'] as $taskId => $sel) { $result_task = SQL_QUERY_ESC("SELECT `id`, `userid`, `task_type`, `subject`, `text`, `task_created`, `status`, `assigned_admin` FROM @@ -142,7 +142,7 @@ WHERE `id`=%s AND (`assigned_admin`=%s OR (`assigned_admin`=0 AND `status`='NEW')) LIMIT 1", array( - bigintval($id), + bigintval($taskId), getCurrentAdminId() ), __FILE__, __LINE__); @@ -162,7 +162,7 @@ LIMIT 1", $add = ''; // @TODO Rewrite this to a filter - if ($taskData['task_type'] == 'SUPPORT_MEMBER') { + if ($taskData['task_type'] == 'MEMBER_SUPPORT') { $mode = substr($taskData['text'], 0, strpos($taskData['text'], ':')); $taskData['text'] = substr($taskData['text'], strpos($taskData['text'], ':') + 1); $add = '
  • {--ADMIN_TASK_SUPPORT_MODE--}: ' . $mode . '
    • '; @@ -192,22 +192,22 @@ LIMIT 1", // Initialize variables (no title for SQL commands by default) $ext_name = ''; $ext_ver = ''; - $title = '{--TASK_NO_TITLE--}'; + $title = '{--ADMIN_TASK_NO_SQL_TITLE--}'; // Shall I list SQL commands assigned to an extension installation or update task? - if (isVerboseSqlEnabled()) { + if ((isVerboseSqlEnabled()) || (!isExtensionInstalled('sql_patches'))) { // Extract extension name from subject $ext_name = substr($taskData['subject'], 1, strpos($taskData['subject'], ':') - 1); // Update task or extension task? if (($taskData['task_type'] == 'EXTENSION') && (!isExtensionInstalled($ext_name))) { // Load SQL commands for registering in dry-run - registerExtension($ext_name, $id, true); + registerExtension($ext_name, $taskId, true); // Is this non-productive? if (isExtensionProductive() === false) { // Issue warning - $taskData['text'] .= loadTemplate('admin_settings_saved', true, getMaskedMessage('ADMIN_EXTENSION_IS_NON_PRODUCTIVE', $ext_name)); + $taskData['text'] = loadTemplate('admin_settings_saved', true, getMaskedMessage('ADMIN_EXTENSION_IS_NON_PRODUCTIVE', $ext_name)) . $taskData['text']; } // END - if // Set current extension name @@ -245,7 +245,7 @@ LIMIT 1", // Add SQLs to a table if ((!empty($ext_name)) && (isVerboseSqlEnabled())) { // Add verbose SQL table - $taskData['text'] .= addExtensionVerboseSqlTable($title, ' class="admin_table top left right"', true, '100%') . "
    \n"; + $taskData['text'] .= addExtensionVerboseSqlTable($title, ' class="admin_table top left right"', true, '100%') . '
    '; } // END - if } elseif ((!empty($ext_name)) && (!empty($ext_ver))) { // Run SQL commands in dry mode but only return the notes @@ -278,18 +278,14 @@ LIMIT 1", switch ($taskData['task_type']) { case 'EXTENSION': // Install new extensions $ext_name = substr($taskData['subject'], 1, strpos($taskData['subject'], ':') - 1); - $result_lines = SQL_QUERY_ESC("SELECT `id` FROM `{?_MYSQL_PREFIX?}_extensions` WHERE `ext_name`='%s' LIMIT 1", - array($ext_name), __FILE__, __LINE__); - $lines = SQL_NUMROWS($result_lines); - SQL_FREERESULT($result_lines); - if ($lines == '0') { - // New extension found + if (!isExtensionInstalled($ext_name)) { + // New (not yet installed) extension found $OUT .= loadTemplate('admin_extension_reg_form', true, array( - 'id' => bigintval($id), + 'id' => bigintval($taskId), 'ext_name' => $ext_name )); } else { - // Task is closed so nothing is todo + // Extension is already installed $OUT .= '
    {--ADMIN_EXTENSION_ALREADY_REGISTERED--}
    '; // Close task but not already closed, solved, deleted or update tasks @@ -311,7 +307,7 @@ LIMIT 1", } // END - if break; - case 'SUPPORT_MEMBER': // Assign on member's support request + case 'MEMBER_SUPPORT': // Assign on member's support request // @TODO This may also be rewritten to include files switch ($mode) { default: // @TODO Unknown support mode @@ -378,12 +374,12 @@ LIMIT 1", break; case 'MEMBER_ORDER': // Member mail orders - $OUT .= loadTemplate('admin_task_order', true, $id); + $OUT .= loadTemplate('admin_task_order', true, $taskId); break; default: // Unknown task type logDebugMessage(__FILE__, __LINE__, sprintf("Unknown task type %s detected.", $taskData['task_type'])); - $OUT .= '
    ' . sprintf(getMessage('ADMIN_UNKNOWN_TASK_TYPE'), $taskData['task_type'], $id) . '
    '; + $OUT .= '
    ' . sprintf(getMessage('ADMIN_UNKNOWN_TASK_TYPE'), $taskData['task_type'], $taskId) . '
    '; break; } $OUT .= ' @@ -402,15 +398,15 @@ LIMIT 1", // Only unassign / delete tasks when there are selected tasks posted if (!empty($postData['unassign'])) { // Unassign from tasks - foreach ($postData['sel'] as $id => $sel) { + foreach ($postData['sel'] as $taskId => $sel) { SQL_QUERY_ESC("UPDATE `{?_MYSQL_PREFIX?}_task_system` SET `assigned_admin`=0 WHERE `id`=%s AND `assigned_admin`=%s LIMIT 1", - array(bigintval($id), getCurrentAdminId()), __FILE__, __LINE__); + array(bigintval($taskId), getCurrentAdminId()), __FILE__, __LINE__); } } elseif (isset($postData['del'])) { // Delete tasks - foreach ($postData['sel'] as $id => $sel) { + foreach ($postData['sel'] as $taskId => $sel) { SQL_QUERY_ESC("DELETE LOW_PRIORITY FROM `{?_MYSQL_PREFIX?}_task_system` WHERE `id`=%s AND `assigned_admin` IN (%s,0) LIMIT 1", - array(bigintval($id), getCurrentAdminId()), __FILE__, __LINE__); + array(bigintval($taskId), getCurrentAdminId()), __FILE__, __LINE__); } } else { // Unknown action @@ -451,7 +447,7 @@ LIMIT 1", $content['task_created'] = generateDateTime($content['task_created'], 2); // Do we have extension task? - if (($content['task_type'] == 'EXTENSION') && (isExtensionNameValid($content['infos'])) && (!isExtensionInstalled($content['infos']))) { + if (isExtensionTask($content)) { // Load extension row template $OUT .= loadTemplate('admin_overview_list_ext_rows', true, $content); } else {