mambax7 / pedigree

admin/pedigree.php

<?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.

 This program is distributed in the hope that it will be useful,
 but WITHOUT ANY WARRANTY; without even the implied warranty of
 MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
*/
/**
 * Pedigree module for XOOPS
 *
 * @package         \XoopsModules\Pedigree
 * @copyright       {@link http://sourceforge.net/projects/xoops/ The XOOPS Project}
 * @license         https://www.gnu.org/licenses/gpl-2.0.html GNU Public License
 * @author          XOOPS Module Dev Team (https://xoops.org)
 */

use Xmf\Request;
use XoopsModules\Pedigree;

require_once __DIR__ . '/admin_header.php';

xoops_cp_header();
/**
 * @var Xmf\Module\Admin $adminObject
 * @var XoopsModules\Pedigree\Helper $helper
 * @var XoopsModules\Pedigree\TreeHandler $treeHandler
 */
$helper     = Pedigree\Helper::getInstance();
$treeHandler = $helper->getHandler('Tree');

//It recovered the value of argument op in URL$
$op = Request::getCmd('op', 'list');
switch ($op) {
    case 'list':
    default:
        $adminObject->displayNavigation(basename(__FILE__));
        $adminObject->addItemButton(_AM_PEDIGREE_NEWPEDIGREE, $helper->url('admin/pedigree.php?op=new_pedigree'), 'add');
        $adminObject->displayButton('left');
        $criteria = new \CriteriaCompo();
        $criteria->setSort('id');
        $criteria->setOrder('ASC');
        $numrows      = $treeHandler->getCount();
        $pedigree_arr = $treeHandler->getAll($criteria);

        //Table view
        if ($numrows > 0) {
            echo "<table class=\"width100 outer\" cellspacing=\"1\">\n"
               . "  <tr>\n"
               . "    <th class=\"center\">" . _AM_PEDIGREE_PEDIGREE_NAAM . "</th>\n"
               . "    <th class=\"center\">" . _AM_PEDIGREE_PEDIGREE_ID_OWNER . "</th>\n"
               . "    <th class=\"center\">" . _AM_PEDIGREE_PEDIGREE_ID_BREEDER . "</th>\n"
               . "    <th class=\"center\">" . _AM_PEDIGREE_PEDIGREE_USER . "</th>\n"
               . "    <th class=\"center\">" . _AM_PEDIGREE_PEDIGREE_ROFT . "</th>\n"
               . "    <th class=\"center\">" . _AM_PEDIGREE_PEDIGREE_MOTHER . "</th>\n"
               . "    <th class=\"center\">" . _AM_PEDIGREE_PEDIGREE_FATHER . "</th>\n"
               . "    <th class=\"center\">" . _AM_PEDIGREE_PEDIGREE_FOTO . "</th>\n"
               . "    <th class=\"center\">" . _AM_PEDIGREE_PEDIGREE_COI . "</th>\n"
               . "    <th class=\"center width10\">" . _AM_PEDIGREE_FORMACTION . "</th>\n"
               . "  </tr>\n";

            $class = 'even';

            /** @var XoopsModules\Pedigree\Tree $tObj */
            foreach ($pedigree_arr as $i => $tObj) {
                if (0 == $tObj->getVar('pedigree_pid')) {
                    $class = ('even' === $class) ? 'odd' : 'even';
                    echo "  <tr class=\"{$class}\">\n"
                       . "    <td class=\"center\">" . $tObj->getVar('naam') . "</td>\n"
                       . "    <td class=\"center\">" . $tObj->getVar('id_owner') . "</td>\n"
                       . "    <td class=\"center\">" . $tObj->getVar('id_breeder') . "</td>\n"
                       . "    <td class=\"center\">" . $tObj->getVar('user') . "</td>\n"
                       . "    <td class=\"center\">" . $tObj->getVar('roft') . "</td>\n"
                       . "    <td class=\"center\">" . $tObj->getVar('mother') . "</td>\n"
                       . "    <td class=\"center\">" . $tObj->getVar('father') . "</td>\n"
                       . "    <td class=\"center\">" . $tObj->getVar('foto') . "</td>\n"
                       . "    <td class=\"center\">" . $tObj->getVar('coi') . "</td>\n"
                       . "    <td class=\"center width10\">\n"
                       . "      <a href=\"" . $_SERVER['SCRIPT_NAME'] . "?op=edit_pedigree&id=" . $tObj->getVar('id') . "\"><img src=\"{$pathIcon16}/edit.png\" alt=\"" . _EDIT . "\" title=\"" . _EDIT . "\"></a>\n"
                       . "      <a href=" . $helper->url('delete.php?id=' . $tObj->getVar('id')) . "><img src=\"{$pathIcon16}/delete.png\" alt=\"" . _DELETE . "\" title=\"" . _DELETE . "\"></a>\n"
                       . "    </td>\n"
                       . "  </tr>\n";
                }
            }
            echo "</table><br><br>\n";
        } //@todo should add 'else' here to display "nothing here" message

        break;

    case 'new_pedigree':
        $adminObject->displayNavigation(basename(__FILE__));
        $adminObject->addItemButton(_AM_PEDIGREE_PEDIGREELIST, $helper->url('pedigree.php?op=list'), 'list');
        $adminObject->displayButton('left');

        /**
         * @var XoopsModules\Pedigree\Tree $tObj
         * @var \XoopsThemeForm $form
        */
        $tObj = $treeHandler->create();
        $form = $tObj->getForm();
        $form->display();
        break;

    case 'save_pedigree':
        /** @var \XoopsSecurity $GLOBALS['xoopsSecurity'] */
        if (!$GLOBALS['xoopsSecurity']->check()) {
            $helper->redirect('admin/pedigree.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
        }

        /** @var XoopsModules\Pedigree\Tree $tObj */
        $id   = Request::getInt('id', null, 'post');
        $tObj = $treeHandler->get($id);

        $varArray = array(
                'naam' => Request::getString('naam', '', 'post'),
            'id_owner' => Request::getInt('id_owner', 0, 'post'),
          'id_breeder' => Request::getInt('id_breeder', 0, 'post'),
                'user' => Request::getString('user', '', 'post'),
                'roft' => Request::getInt('roft', 0, 'post'),
              'mother' => Request::getInt('mother', 0, 'post'),
              'father' => Request::getInt('father', 0, 'post'),
                'foto' => Request::getString('foto', '', 'post'),
                 'coi' => Request::getString('coi', '', 'post')
        );

        $tObj->setVars($varArray);
        if ($treeHandler->insert($tObj)) {
            $helper->redirect('admin/pedigree.php?op=list', 2, _AM_PEDIGREE_FORMOK);
        }

        echo $tObj->getHtmlErrors();
        /** @var \XoopsThemeForm $form */
        $form = $tObj->getForm();

The method getForm() does not exist on XoopsObject. It seems like you code against a sub-type of XoopsObject such as XoopsModules\Pedigree\Temp or SystemSmilies or XoopsModules\Pedigree\Owner or SystemBanner or SystemBannerclient or XoopsModules\Pedigree\Tree or XoopsModules\Pedigree\Trash or XoopsModules\Pedigree\Fields or ProfileCategory or SystemUserrank or SystemGroup or SystemBlock or SystemAvatar or SystemUsers.

        $form->display();
        break;

    case 'edit_pedigree':
        $adminObject->displayNavigation(basename(__FILE__));
        $adminObject->addItemButton(_AM_PEDIGREE_NEWPEDIGREE, $helper->url('admin/pedigree.php?op=new_pedigree'), 'add');
        $adminObject->addItemButton(_AM_PEDIGREE_PEDIGREELIST, $helper->url('admin/pedigree.php?op=list'), 'list');
        $adminObject->displayButton('left');

        /**
         * @var XoopsModules\Pedigree\Tree $tObj
         * @var XoopsThemeForm $form
         */
        $id   = Request::getInt('id', 0);
        $tObj = $treeHandler->get($id);
        $form = $tObj->getForm();
        $form->display();
        break;

    case 'delete_pedigree':
        /** @var XoopsModules\Pedigree\Tree $tObj */
        $id  = Request::getInt('id', 0);
        $tObj = $treeHandler->get($id);
        if (isset($_REQUEST['ok']) && 1 == $_REQUEST['ok']) {
            if (!$GLOBALS['xoopsSecurity']->check()) {
                $helper->redirect('admin/pedigree.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
            }
            if ($treeHandler->delete($obj)) {
                $helper->redirect('admin/pedigree.php', 3, _AM_PEDIGREE_FORMDELOK);
            } else {
                echo $tObj->getHtmlErrors();
            }
        } else {
            xoops_confirm(['ok' => 1, 'id' => $id, 'op' => 'delete_pedigree'], $_SERVER['REQUEST_URI'], sprintf(_AM_PEDIGREE_FORMSUREDEL, $tObj->getVar('pedigree')));

It seems like $tObj->getVar('pedigree') can also be of type array and array; however, parameter $args of sprintf() does only seem to accept string, maybe add an additional type check?

        }
        break;
}
require_once __DIR__ . '/admin_footer.php';