Conditions | 20 |
Paths | 4920 |
Total Lines | 282 |
Code Lines | 209 |
Lines | 0 |
Ratio | 0 % |
Changes | 0 |
Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.
For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.
Commonly applied refactorings include:
If many parameters/temporary variables are present:
1 | <?php declare(strict_types=1); |
||
321 | function modifyEmailTpl() |
||
322 | { |
||
323 | global $xoopsConfig, $session; |
||
324 | $helper = Helper::getInstance(); |
||
325 | |||
326 | if (is_dir(XOOPS_ROOT_PATH . '/modules/xhelp/language/' . $xoopsConfig['language'] . '/mail_template')) { |
||
327 | $opendir = opendir(XOOPS_ROOT_PATH . '/modules/xhelp/language/' . $xoopsConfig['language'] . '/mail_template/'); |
||
328 | $dir = XOOPS_ROOT_PATH . '/modules/xhelp/language/' . $xoopsConfig['language'] . '/mail_template/'; |
||
329 | $url = XOOPS_URL . '/modules/xhelp/language/' . $xoopsConfig['language'] . '/mail_template/'; |
||
330 | } else { |
||
331 | $opendir = opendir(XOOPS_ROOT_PATH . '/modules/xhelp/language/english/mail_template/'); |
||
332 | $dir = XOOPS_ROOT_PATH . '/modules/xhelp/language/english/mail_template/'; |
||
333 | $url = XOOPS_URL . '/modules/xhelp/language/english/mail_template/'; |
||
334 | } |
||
335 | |||
336 | $notNames = [ |
||
337 | _MI_XHELP_DEPT_NEWTICKET_NOTIFYTPL => [ |
||
338 | _MI_XHELP_DEPT_NEWTICKET_NOTIFY, |
||
339 | _MI_XHELP_DEPT_NEWTICKET_NOTIFYDSC, |
||
340 | _MI_XHELP_DEPT_NEWTICKET_NOTIFYTPL, |
||
341 | ], |
||
342 | _MI_XHELP_DEPT_REMOVEDTICKET_NOTIFYTPL => [ |
||
343 | _MI_XHELP_DEPT_REMOVEDTICKET_NOTIFY, |
||
344 | _MI_XHELP_DEPT_REMOVEDTICKET_NOTIFYDSC, |
||
345 | _MI_XHELP_DEPT_REMOVEDTICKET_NOTIFYTPL, |
||
346 | ], |
||
347 | _MI_XHELP_DEPT_NEWRESPONSE_NOTIFYTPL => [ |
||
348 | _MI_XHELP_DEPT_NEWRESPONSE_NOTIFY, |
||
349 | _MI_XHELP_DEPT_NEWRESPONSE_NOTIFYDSC, |
||
350 | _MI_XHELP_DEPT_NEWRESPONSE_NOTIFYTPL, |
||
351 | ], |
||
352 | _MI_XHELP_DEPT_MODIFIEDRESPONSE_NOTIFYTPL => [ |
||
353 | _MI_XHELP_DEPT_MODIFIEDRESPONSE_NOTIFY, |
||
354 | _MI_XHELP_DEPT_MODIFIEDRESPONSE_NOTIFYDSC, |
||
355 | _MI_XHELP_DEPT_MODIFIEDRESPONSE_NOTIFYTPL, |
||
356 | ], |
||
357 | _MI_XHELP_DEPT_MODIFIEDTICKET_NOTIFYTPL => [ |
||
358 | _MI_XHELP_DEPT_MODIFIEDTICKET_NOTIFY, |
||
359 | _MI_XHELP_DEPT_MODIFIEDTICKET_NOTIFYDSC, |
||
360 | _MI_XHELP_DEPT_MODIFIEDTICKET_NOTIFYTPL, |
||
361 | ], |
||
362 | _MI_XHELP_DEPT_CHANGEDSTATUS_NOTIFYTPL => [ |
||
363 | _MI_XHELP_DEPT_CHANGEDSTATUS_NOTIFY, |
||
364 | _MI_XHELP_DEPT_CHANGEDSTATUS_NOTIFYDSC, |
||
365 | _MI_XHELP_DEPT_CHANGEDSTATUS_NOTIFYTPL, |
||
366 | ], |
||
367 | _MI_XHELP_DEPT_CHANGEDPRIORITY_NOTIFYTPL => [ |
||
368 | _MI_XHELP_DEPT_CHANGEDPRIORITY_NOTIFY, |
||
369 | _MI_XHELP_DEPT_CHANGEDPRIORITY_NOTIFYDSC, |
||
370 | _MI_XHELP_DEPT_CHANGEDPRIORITY_NOTIFYTPL, |
||
371 | ], |
||
372 | _MI_XHELP_DEPT_NEWOWNER_NOTIFYTPL => [ |
||
373 | _MI_XHELP_DEPT_NEWOWNER_NOTIFY, |
||
374 | _MI_XHELP_DEPT_NEWOWNER_NOTIFYDSC, |
||
375 | _MI_XHELP_DEPT_NEWOWNER_NOTIFYTPL, |
||
376 | ], |
||
377 | _MI_XHELP_DEPT_CLOSETICKET_NOTIFYTPL => [ |
||
378 | _MI_XHELP_DEPT_CLOSETICKET_NOTIFY, |
||
379 | _MI_XHELP_DEPT_CLOSETICKET_NOTIFYDSC, |
||
380 | _MI_XHELP_DEPT_CLOSETICKET_NOTIFYTPL, |
||
381 | ], |
||
382 | _MI_XHELP_TICKET_NEWOWNER_NOTIFYTPL => [ |
||
383 | _MI_XHELP_TICKET_NEWOWNER_NOTIFY, |
||
384 | _MI_XHELP_TICKET_NEWOWNER_NOTIFYDSC, |
||
385 | _MI_XHELP_TICKET_NEWOWNER_NOTIFYTPL, |
||
386 | ], |
||
387 | _MI_XHELP_TICKET_REMOVEDTICKET_NOTIFYTPL => [ |
||
388 | _MI_XHELP_TICKET_REMOVEDTICKET_NOTIFY, |
||
389 | _MI_XHELP_TICKET_REMOVEDTICKET_NOTIFYDSC, |
||
390 | _MI_XHELP_TICKET_REMOVEDTICKET_NOTIFYTPL, |
||
391 | ], |
||
392 | _MI_XHELP_TICKET_MODIFIEDTICKET_NOTIFYTPL => [ |
||
393 | _MI_XHELP_TICKET_MODIFIEDTICKET_NOTIFY, |
||
394 | _MI_XHELP_TICKET_MODIFIEDTICKET_NOTIFYDSC, |
||
395 | _MI_XHELP_TICKET_MODIFIEDTICKET_NOTIFYTPL, |
||
396 | ], |
||
397 | _MI_XHELP_TICKET_NEWRESPONSE_NOTIFYTPL => [ |
||
398 | _MI_XHELP_TICKET_NEWRESPONSE_NOTIFY, |
||
399 | _MI_XHELP_TICKET_NEWRESPONSE_NOTIFYDSC, |
||
400 | _MI_XHELP_TICKET_NEWRESPONSE_NOTIFYTPL, |
||
401 | ], |
||
402 | _MI_XHELP_TICKET_MODIFIEDRESPONSE_NOTIFYTPL => [ |
||
403 | _MI_XHELP_TICKET_MODIFIEDRESPONSE_NOTIFY, |
||
404 | _MI_XHELP_TICKET_MODIFIEDRESPONSE_NOTIFYDSC, |
||
405 | _MI_XHELP_TICKET_MODIFIEDRESPONSE_NOTIFYTPL, |
||
406 | ], |
||
407 | _MI_XHELP_TICKET_CHANGEDSTATUS_NOTIFYTPL => [ |
||
408 | _MI_XHELP_TICKET_CHANGEDSTATUS_NOTIFY, |
||
409 | _MI_XHELP_TICKET_CHANGEDSTATUS_NOTIFYDSC, |
||
410 | _MI_XHELP_TICKET_CHANGEDSTATUS_NOTIFYTPL, |
||
411 | ], |
||
412 | _MI_XHELP_TICKET_CHANGEDPRIORITY_NOTIFYTPL => [ |
||
413 | _MI_XHELP_TICKET_CHANGEDPRIORITY_NOTIFY, |
||
414 | _MI_XHELP_TICKET_CHANGEDPRIORITY_NOTIFYDSC, |
||
415 | _MI_XHELP_TICKET_CHANGEDPRIORITY_NOTIFYTPL, |
||
416 | ], |
||
417 | _MI_XHELP_TICKET_NEWTICKET_NOTIFYTPL => [ |
||
418 | _MI_XHELP_TICKET_NEWTICKET_NOTIFY, |
||
419 | _MI_XHELP_TICKET_NEWTICKET_NOTIFYDSC, |
||
420 | _MI_XHELP_TICKET_NEWTICKET_NOTIFYTPL, |
||
421 | ], |
||
422 | _MI_XHELP_TICKET_NEWTICKET_EMAIL_NOTIFYTPL => [ |
||
423 | _MI_XHELP_TICKET_NEWTICKET_EMAIL_NOTIFY, |
||
424 | _MI_XHELP_TICKET_NEWTICKET_EMAIL_NOTIFYDSC, |
||
425 | _MI_XHELP_TICKET_NEWTICKET_EMAIL_NOTIFYTPL, |
||
426 | ], |
||
427 | _MI_XHELP_TICKET_CLOSETICKET_NOTIFYTPL => [ |
||
428 | _MI_XHELP_TICKET_CLOSETICKET_NOTIFY, |
||
429 | _MI_XHELP_TICKET_CLOSETICKET_NOTIFYDSC, |
||
430 | _MI_XHELP_TICKET_CLOSETICKET_NOTIFYTPL, |
||
431 | ], |
||
432 | _MI_XHELP_TICKET_NEWUSER_NOTIFYTPL => [ |
||
433 | _MI_XHELP_TICKET_NEWUSER_NOTIFY, |
||
434 | _MI_XHELP_TICKET_NEWUSER_NOTIFYDSC, |
||
435 | _MI_XHELP_TICKET_NEWUSER_NOTIFYTPL, |
||
436 | ], |
||
437 | _MI_XHELP_TICKET_NEWUSER_ACT1_NOTIFYTPL => [ |
||
438 | _MI_XHELP_TICKET_NEWUSER_ACT1_NOTIFY, |
||
439 | _MI_XHELP_TICKET_NEWUSER_ACT1_NOTIFYDSC, |
||
440 | _MI_XHELP_TICKET_NEWUSER_ACT1_NOTIFYTPL, |
||
441 | ], |
||
442 | _MI_XHELP_TICKET_NEWUSER_ACT2_NOTIFYTPL => [ |
||
443 | _MI_XHELP_TICKET_NEWUSER_ACT2_NOTIFY, |
||
444 | _MI_XHELP_TICKET_NEWUSER_ACT2_NOTIFYDSC, |
||
445 | _MI_XHELP_TICKET_NEWUSER_ACT2_NOTIFYTPL, |
||
446 | ], |
||
447 | _MI_XHELP_TICKET_EMAIL_ERROR_NOTIFYTPL => [ |
||
448 | _MI_XHELP_TICKET_EMAIL_ERROR_NOTIFY, |
||
449 | _MI_XHELP_TICKET_EMAIL_ERROR_NOTIFYDSC, |
||
450 | _MI_XHELP_TICKET_EMAIL_ERROR_NOTIFYTPL, |
||
451 | ], |
||
452 | _MI_XHELP_DEPT_MERGE_TICKET_NOTIFYTPL => [ |
||
453 | _MI_XHELP_DEPT_MERGE_TICKET_NOTIFY, |
||
454 | _MI_XHELP_DEPT_MERGE_TICKET_NOTIFYDSC, |
||
455 | _MI_XHELP_DEPT_MERGE_TICKET_NOTIFYTPL, |
||
456 | ], |
||
457 | _MI_XHELP_TICKET_MERGE_TICKET_NOTIFYTPL => [ |
||
458 | _MI_XHELP_TICKET_MERGE_TICKET_NOTIFY, |
||
459 | _MI_XHELP_TICKET_MERGE_TICKET_NOTIFYDSC, |
||
460 | _MI_XHELP_TICKET_MERGE_TICKET_NOTIFYTPL, |
||
461 | ], |
||
462 | ]; |
||
463 | |||
464 | $notKeys = array_keys($notNames); |
||
465 | |||
466 | while (false !== ($file = readdir($opendir))) { |
||
467 | //Do not Display . |
||
468 | if (is_dir($file)) { |
||
469 | continue; |
||
470 | } |
||
471 | |||
472 | if (!in_array($file, $notKeys)) { |
||
473 | continue; |
||
474 | } |
||
475 | |||
476 | $aFile = []; |
||
477 | $aFile['name'] = $notNames[$file][0]; |
||
478 | $aFile['desc'] = $notNames[$file][1]; |
||
479 | $aFile['filename'] = $notNames[$file][2]; |
||
480 | $aFile['url'] = "notifications.php?op=modifyEmailTpl&file=$file"; |
||
481 | $aFiles[] = $aFile; |
||
482 | } |
||
483 | |||
484 | if (isset($_GET['file'])) { |
||
485 | xoops_cp_header(); |
||
486 | //echo $oAdminButton->renderButtons('manNotify'); |
||
487 | $adminObject = Admin::getInstance(); |
||
488 | $adminObject->addItemButton(_AM_XHELP_TEXT_MANAGE_NOTIFICATIONS, 'notifications.php?op=manage', 'add'); |
||
489 | $adminObject->addItemButton(_AM_XHELP_MENU_MODIFY_EMLTPL, 'notifications.php?op=modifyEmailTpl', 'list'); |
||
490 | |||
491 | $adminObject->displayNavigation(basename(__FILE__)); |
||
492 | $adminObject->displayButton('left'); |
||
493 | |||
494 | foreach ($aFiles as $file) { |
||
495 | if (\Xmf\Request::getString('file', '', 'GET') == $file['filename']) { |
||
496 | $myFileName = $file['filename']; |
||
497 | $myFileDesc = $file['desc']; |
||
498 | $myName = $file['name']; |
||
499 | break; |
||
500 | } |
||
501 | } |
||
502 | if (!$has_write = is_writable($dir . $myFileName)) { |
||
503 | $message = _AM_XHELP_MESSAGE_FILE_READONLY; |
||
504 | $handle = fopen($dir . $myFileName, 'rb'); |
||
505 | $fileSize = filesize($dir . $myFileName); |
||
506 | } elseif (Request::hasVar('editTemplate', 'POST')) { |
||
507 | $handle = fopen($dir . $myFileName, 'wb+'); |
||
508 | } else { |
||
509 | $handle = fopen($dir . $myFileName, 'rb+'); |
||
510 | $fileSize = filesize($dir . $myFileName); |
||
511 | } |
||
512 | |||
513 | if (Request::hasVar('editTemplate', 'POST')) { |
||
514 | if (Request::hasVar('templateText', 'POST')) { |
||
515 | $text = $_POST['templateText']; // Get new text for template |
||
516 | } else { |
||
517 | $text = ''; |
||
518 | } |
||
519 | |||
520 | if (!$returnPage = $session->get('xhelp_return_page')) { |
||
521 | $returnPage = false; |
||
522 | } |
||
523 | |||
524 | if (fwrite($handle, $text)) { |
||
525 | $message = _AM_XHELP_MESSAGE_FILE_UPDATED; |
||
526 | $fileSize = filesize($dir . $myFileName); |
||
527 | fclose($handle); |
||
528 | if ($returnPage) { |
||
529 | $helper->redirect((string)$returnPage); |
||
530 | } else { |
||
531 | $helper->redirect('admin/notifications.php'); |
||
532 | } |
||
533 | } else { |
||
534 | $message = _AM_XHELP_MESSAGE_FILE_UPDATED_ERROR; |
||
535 | $fileSize = filesize($dir . $myFileName); |
||
536 | fclose($handle); |
||
537 | if ($returnPage) { |
||
538 | $helper->redirect((string)$returnPage, 3, $message); |
||
539 | } else { |
||
540 | $helper->redirect('admin/notifications.php', 3, $message); |
||
541 | } |
||
542 | } |
||
543 | } |
||
544 | if (!$has_write) { |
||
545 | echo "<div id='readOnly' class='errorMsg'>"; |
||
546 | echo $message; |
||
547 | echo '</div>'; |
||
548 | } |
||
549 | |||
550 | echo "<form action='" . XHELP_ADMIN_URL . '/notifications.php?op=modifyEmailTpl&file=' . $myFileName . "' method='post'>"; |
||
551 | echo $GLOBALS['xoopsSecurity']->getTokenHTML(); |
||
552 | echo "<table width='100%' border='0' cellspacing='1' class='outer'> |
||
553 | <tr><th colspan='2'>" . $myName . "</th></tr> |
||
554 | <tr><td colspan='2' class='head'>" . $myFileDesc . '</td></tr>'; |
||
555 | |||
556 | echo "<tr class='odd'> |
||
557 | <td><textarea name='templateText' cols='40' rows='40'>" . fread($handle, $fileSize) . "</textarea></td> |
||
558 | <td valign='top'> |
||
559 | <b>" . _AM_XHELP_TEXT_GENERAL_TAGS . '</b> |
||
560 | <ul> |
||
561 | <li>' . _AM_XHELP_TEXT_GENERAL_TAGS1 . '</li> |
||
562 | <li>' . _AM_XHELP_TEXT_GENERAL_TAGS2 . '</li> |
||
563 | <li>' . _AM_XHELP_TEXT_GENERAL_TAGS3 . '</li> |
||
564 | <li>' . _AM_XHELP_TEXT_GENERAL_TAGS4 . '</li> |
||
565 | <li>' . _AM_XHELP_TEXT_GENERAL_TAGS5 . '</li> |
||
566 | </ul> |
||
567 | <br> |
||
568 | <u>' . _AM_XHELP_TEXT_TAGS_NO_MODIFY . '</u> |
||
569 | </td> |
||
570 | </tr>'; |
||
571 | |||
572 | if ($has_write) { |
||
573 | echo "<tr><td class='foot' colspan='2'><input type='submit' name='editTemplate' value='" . _AM_XHELP_BUTTON_UPDATE . "' class='formButton'></td></tr>"; |
||
574 | } |
||
575 | echo '</table></form>'; |
||
576 | } else { |
||
577 | xoops_cp_header(); |
||
578 | //echo $oAdminButton->renderButtons('manNotify'); |
||
579 | $adminObject = Admin::getInstance(); |
||
580 | $adminObject->addItemButton(_AM_XHELP_TEXT_MANAGE_NOTIFICATIONS, 'notifications.php?op=manage', 'add'); |
||
581 | $adminObject->addItemButton(_AM_XHELP_MENU_MODIFY_EMLTPL, 'notifications.php?op=modifyEmailTpl', 'list'); |
||
582 | |||
583 | $adminObject->displayNavigation(basename(__FILE__)); |
||
584 | $adminObject->displayButton('left'); |
||
585 | |||
586 | echo "<table width='100%' border='0' cellspacing='1' class='outer'> |
||
587 | <tr><th colspan='2'><label>" . _AM_XHELP_MENU_MODIFY_EMLTPL . "</label></th></tr> |
||
588 | <tr class='head'><td>" . _AM_XHELP_TEXT_TEMPLATE_NAME . '</td><td>' . _AM_XHELP_TEXT_DESCRIPTION . '</td></tr>'; |
||
589 | |||
590 | static $rowSwitch = 0; |
||
591 | foreach ($aFiles as $file) { |
||
592 | if (0 == $rowSwitch) { |
||
593 | echo "<tr class='odd'><td><a href='" . $file['url'] . "'>" . $file['name'] . '</a></td><td>' . $file['desc'] . '</td></tr>'; |
||
594 | $rowSwitch = 1; |
||
595 | } else { |
||
596 | echo "<tr class='even'><td><a href='" . $file['url'] . "'>" . $file['name'] . '</a></td><td>' . $file['desc'] . '</td></tr>'; |
||
597 | $rowSwitch = 0; |
||
598 | } |
||
599 | } |
||
600 | echo '</table>'; |
||
601 | } |
||
602 | require_once __DIR__ . '/admin_footer.php'; |
||
603 | } |
||
604 |