mambax7/pedigree

File tools.php has 382 lines of code (exceeds 250 allowed). Consider refactoring.

Open

<?php
/*
 * You may not change or alter any portion of this comment or credits
 * of supporting developers from this source code or any supporting source code
 * which is considered copyrighted (c) material of the original comment or credit authors.

Method settings has 53 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function settings()
{
    global $xoopsTpl;
    $helper = Helper::getInstance();
    require_once XOOPS_ROOT_PATH . '/class/xoopsformloader.php';

Method lang has 46 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function lang()
{
    global $xoopsTpl;
    $helper = Helper::getInstance();
    require_once XOOPS_ROOT_PATH . '/class/xoopsformloader.php';

Method database_oa has 31 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function database_oa()
{
    global $xoopsTpl;
    $form   = _AM_PEDIGREE_DATABASE_CHECK_ANCESTORS;
    $sql    = 'SELECT d.id AS d_id, d.pname AS d_pname

Identical blocks of code found in 2 locations. Consider refactoring.

Open

    $sql    = 'SELECT d.id AS d_id, d.pname AS d_pname
            FROM ' . $GLOBALS['xoopsDB']->prefix('pedigree_registry') . ' d
            LEFT JOIN ' . $GLOBALS['xoopsDB']->prefix('pedigree_registry') . ' m ON m.id = d.mother
            LEFT JOIN ' . $GLOBALS['xoopsDB']->prefix('pedigree_registry') . ' f ON f.id = d.father
            LEFT JOIN ' . $GLOBALS['xoopsDB']->prefix('pedigree_registry') . ' mm ON mm.id = m.mother

• Read up
  Read up

Identical blocks of code found in 3 locations. Consider refactoring.

Open

    foreach ($_POST as $key => $values) {
        if (in_array($key, $settings)) {
            //          $query = 'UPDATE ' . $GLOBALS['xoopsDB']->prefix('config') . " SET conf_value = '" . $values . "' WHERE conf_name = '" . $key . "'";
            $query = 'UPDATE ' . $GLOBALS['xoopsDB']->prefix('config') . " SET conf_value = '" . $GLOBALS['xoopsDB']->escape($values) . "' WHERE conf_name = '" . $GLOBALS['xoopsDB']->escape($key) . "'";
            $GLOBALS['xoopsDB']->query($query);

• Read up
  Read up

Identical blocks of code found in 3 locations. Consider refactoring.

Open

    foreach ($_POST as $key => $values) {
        if (in_array($key, $settings)) {
            //          $query = 'UPDATE ' . $GLOBALS['xoopsDB']->prefix('config') . " SET conf_value = '" . $values . "' WHERE conf_name = '" . $key . "'";
            $query = 'UPDATE ' . $GLOBALS['xoopsDB']->prefix('config') . " SET conf_value = '" . $GLOBALS['xoopsDB']->escape($values) . "' WHERE conf_name = '" . $GLOBALS['xoopsDB']->escape($key) . "'";
            $GLOBALS['xoopsDB']->query($query);

• Read up
  Read up