mambax7 / tdmcreate-1.91

class/Files/Includes/IncludeSearch.php

<?php namespace XoopsModules\Tdmcreate\Files\Includes;

use XoopsModules\Tdmcreate;
use XoopsModules\Tdmcreate\Files;

/*
 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.
 */
/**
 * tdmcreate module.
 *
 * @copyright       XOOPS Project (https://xoops.org)
 * @license         GNU GPL 2 (http://www.gnu.org/licenses/old-licenses/gpl-2.0.html)
 *
 * @since           2.5.0
 *
 * @author          Txmod Xoops http://www.txmodxoops.org
 *
 * @version         $Id: IncludeSearch.php 12258 2014-01-02 09:33:29Z timgno $
 */

/**
 * Class IncludeSearch.
 */
class IncludeSearch extends Files\CreateFile
{
    /**
    *  @public function constructor
    *  @param null
    */

    public function __construct()
    {
        parent::__construct();
        $this->tdmcfile = Tdmcreate\Files\CreateFile::getInstance();

The property tdmcfile does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

    }

    /**
    *  @static function getInstance
    *  @param null
     * @return IncludeSearch
     */
    public static function getInstance()
    {
        static $instance = false;
        if (!$instance) {
            $instance = new self();
        }

        return $instance;
    }

    /**
    *  @public function write
    *  @param string $module
    *  @param mixed $table
    *  @param string $filename
     */
    public function write($module, $table, $filename)
    {
        $this->setModule($module);
        $this->setFileName($filename);
        $this->setTable($table);
    }

    /**
    *  @static function getSearchField
    *  @param string $fieldSearch
    *  @param string $options
     *
     * @return string
     */
    public function getSearchField($fieldSearch, $options)
    {
        // fieldSearch = fields parameters search field
        $sql = '';
        if (isset($fieldSearch)) {
            $nb_fieldSearch = count($fieldSearch);

$fieldSearch of type string is incompatible with the type Countable|array expected by parameter $var of count().

            $sql .= '(';
            for ($i = 0; $i < $nb_fieldSearch; ++$i) {
                if ($i != $nb_fieldSearch - 1) {
                    $sql .= ''.$fieldSearch[$i].' LIKE %$queryarray['.$options.']% OR ';
                } else {
                    $sql .= ''.$fieldSearch[$i].' LIKE %$queryarray[0]%';
                }
            }
            $sql .= ')';
        }

        return $sql;
    }

    /**
    *  @static function getSearchFunction
     * @param $moduleDirname
     *
     * @return string
     */
    public function getSearchFunction($moduleDirname)
    {
        $table = $this->getTable();
        $tableName = $table->getVar('table_name');
        $tableFieldname = $table->getVar('table_fieldname');
        $fieldId = '';
        $fieldSearch = null;
        $fields = $this->getTableFields($table->getVar('table_mid'), $table->getVar('table_id'));
        foreach (array_keys($fields) as $f) {
            $fieldName = $fields[$f]->getVar('field_name');
            if ((0 == $f) && (1 == $table->getVar('table_autoincrement'))) {
                $fieldId = $fieldName;
            }
            if (1 == $fields[$f]->getVar('field_main')) {
                $fieldMain = $fieldName;
            }
            if (1 == $fields[$f]->getVar('field_search')) {
                $fieldSearch = $fieldName;
            }
        }
        $img_search = 'blank.gif';
        $ret = <<<EOT
\n
/**
 * search callback functions
 * @param \$queryarray
 * @param \$andor
 * @param \$limit
 * @param \$offset
 * @param \$userid
 */
function {$moduleDirname}_search(\$queryarray, \$andor, \$limit, \$offset, \$userid)
{
    global \$xoopsDB;
    \$sql = "SELECT '{$fieldId}', '{$fieldMain}' FROM " . \$xoopsDB->prefix('{$moduleDirname}_{$tableName}') . ' WHERE {$fieldId} != 0';

The variable $fieldMain does not seem to be defined for all execution paths leading up to this point.

    if ( \$userid != 0 ) {
        \$sql .= ' AND {$tableFieldname}_submitter='.(int)\$userid;
    }
    if ( is_array(\$queryarray) && \$count = count(\$queryarray) )
    {
        \$sql .= " AND (
EOT;
        $ret .= $this->getSearchField($fieldSearch, 0).'";';
        $ret .= <<<EOT

        for(\$i = 1; \$i < \$count; ++\$i)
        {
            \$sql .= " \$andor ";
            \$sql .= "{$this->getSearchField($fieldSearch, '$i')}";
        }
        \$sql .= ')';
    }
    \$sql .= " ORDER BY '{$fieldId}' DESC";
    \$result = \$xoopsDB->query(\$sql,\$limit,\$offset);
    \$ret = array();
    \$i = 0;
    while(\$myrow = \$xoopsDB->fetchArray(\$result))
    {
        \$ret[\$i]['image'] = 'assets/icons/32/{$img_search}';
        \$ret[\$i]['link'] = '{$tableName}.php?{$fieldId}='.\$myrow['{$fieldId}'];
        \$ret[\$i]['title'] = \$myrow['{$fieldMain}'];
        ++\$i;
    }
    unset(\$i);
}
EOT;

        return $ret;
    }

    /**
    *  @public function render
    *  @param null
     * @return bool|string
     */
    public function render()
    {
        $module = $this->getModule();
        $filename = $this->getFileName();
        $moduleDirname = $module->getVar('mod_dirname');
        $content = $this->getHeaderFilesComments($module, $filename);
        $content .= $this->getSearchFunction($moduleDirname);

        $this->tdmcfile->create($moduleDirname, 'include', $filename, $content, _AM_TDMCREATE_FILE_CREATED, _AM_TDMCREATE_FILE_NOTCREATED);

        return $this->tdmcfile->renderFile();
    }
}