efqDataFieldHandler

Complexity

Total Complexity

26

Size/Duplication

Total Lines	131
Duplicated Lines	47.33 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	efqDataFieldHandler()	0	6	1
B	getDataFields()	23	36	6
D	getFieldValue()	39	81	19

<?php
// $Id: listing.php,v 1.1.0 2007/11/03 17:46:00 wtravel
//  ------------------------------------------------------------------------ //
//                				EFQ Directory			                     //
//                    Copyright (c) 2006 EFQ Consultancy                     //
//                       <http://www.efqdirectory.com/>                      //
//  ------------------------------------------------------------------------ //
//  This program is free software; you can redistribute it and/or modify     //
//  it under the terms of the GNU General Public License as published by     //
//  the Free Software Foundation; either version 2 of the License, or        //
//  (at your option) any later version.                                      //
//                                                                           //
//  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.  See the            //
//  GNU General Public License for more details.                             //
//                                                                           //
//  You should have received a copy of the GNU General Public License        //
//  along with this program; if not, write to the Free Software              //
//  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307 USA //
//  ------------------------------------------------------------------------ //
//	Part of the efqDirectory module provided by: wtravel					 //
// 	e-mail: info@efqdirectory.com											 //
//	Purpose: Create a business directory for xoops.		 	 				 //
//	Based upon the mylinks and the mxDirectory modules						 //
// ------------------------------------------------------------------------- //

/**
 * Class efqDataField
 * Manages operations for datafields
 *
 * @package   efqDirectory
 * @author    EFQ Consultancy <info@efqconsultancy.com>
 * @copyright EFQ Consultancy (c) 2007
 * @version   1.1.0
 */
class efqDataField extends XoopsObject

The type XoopsObject was not found. Maybe you did not declare it correctly or list all dependencies?

{

    /**
     * Constructor
     */
    public function efqDataField()
    {
        // class constructor;
    }
}

/**
 * Class efqDataFieldHandler
 * Manages database operations for data fields
 *
 * @package   efqDirectory
 * @author    EFQ Consultancy <info@efqconsultancy.com>
 * @copyright EFQ Consultancy (c) 2007
 * @version   1.1.0
 */
class efqDataFieldHandler extends XoopsObjectHandler

The type XoopsObjectHandler was not found. Maybe you did not declare it correctly or list all dependencies?

{
    public $errorhandler;

    public function efqDataFieldHandler()
    {
        //Instantiate class
        global $eh;
        $this->db           = XoopsDatabaseFactory::getDatabaseConnection();

The type XoopsDatabaseFactory was not found. Maybe you did not declare it correctly or list all dependencies?

        $this->errorhandler = $eh;
    }

    public function getDataFields($itemid, $show = 10, $min = 0)
    {
        $sql = "SELECT DISTINCT t.dtypeid, t.title, t.section, t.icon, f.typeid, f.fieldtype, f.ext, t.options, d.itemid, d.value, d.customtitle, t.custom ";
        $sql .= "FROM " . $this->db->prefix("efqdiralpha1_item_x_cat") . " ic, " . $this->db->prefix("efqdiralpha1_dtypes_x_cat") . " xc, " . $this->db->prefix("efqdiralpha1_fieldtypes") . " f, " . $this->db->prefix("efqdiralpha1_dtypes") . " t ";
        $sql .= "LEFT JOIN " . $this->db->prefix("efqdiralpha1_data") . " d ON (t.dtypeid=d.dtypeid AND d.itemid=" . $itemid . ") ";
        $sql .= "WHERE ic.cid=xc.cid AND ic.active='1' AND xc.dtypeid=t.dtypeid AND t.fieldtypeid=f.typeid AND t.activeyn='1' AND ic.itemid=" . $itemid . "";
        $data_result = $this->db->query($sql) or $this->errorhandler->show("0013");
        //$numrows = $this->db->getRowsNum($data_result);

62% of this comment could be valid code. Did you maybe forget this after debugging?

        $arr = array();
        while (list($dtypeid, $title, $section, $icon, $ftypeid, $fieldtype, $ext, $options, $itemid, $value, $customtitle, $custom) = $this->db->fetchRow($data_result)) {

This code seems to be duplicated across your project.

            $fieldvalue = $this->getFieldValue($fieldtype, $options, $value);
            if ($icon != '') {
                $iconurl = "<img src=\"uploads/$icon\" />";
            } else {
                $iconurl = "";
            }
            if ($custom != '0' && $customtitle != "") {
                $title = $customtitle;
            }
            $arr[] = array(
                'dtypeid'     => $dtypeid,
                'title'       => $title,
                'section'     => $section,
                'icon'        => $iconurl,
                'ftypeid'     => $ftypeid,
                'fieldtype'   => $fieldtype,
                'ext'         => $ext,
                'options'     => $options,
                'custom'      => $custom,
                'itemid'      => $itemid,
                'value'       => $fieldvalue,
                'customtitle' => $customtitle
            );
        }

        return $arr;
    }

    public function getFieldValue($fieldtype = "", $options = "", $value = 0)
    {
        global $myts, $moddir;
        switch ($fieldtype) {
            case "dhtml":
                return $myts->displayTarea($value);
                break;

break is not strictly necessary here and could be removed.

            //case "gmap":
            //			$gmapHandler = new efqGmapHandler();

46% of this comment could be valid code. Did you maybe forget this after debugging?

            //			$gmap = new efqGmap();

46% of this comment could be valid code. Did you maybe forget this after debugging?

            //			$gmap->setPointsJS($gmapHandler->getPointsJS($gmap));

77% of this comment could be valid code. Did you maybe forget this after debugging?

            //			$gmap->generateMap();

72% of this comment could be valid code. Did you maybe forget this after debugging?

            //			$ret = $gmap->showMap();

55% of this comment could be valid code. Did you maybe forget this after debugging?

            //			unset($gmap);

84% of this comment could be valid code. Did you maybe forget this after debugging?

            //			unset($gmapHandler);

84% of this comment could be valid code. Did you maybe forget this after debugging?

            //return $myts->makeTboxData4Show($value);

78% of this comment could be valid code. Did you maybe forget this after debugging?

            //break;
            case "radio":
                return $myts->makeTboxData4Show($value);
                break;
            case "rating":

This code seems to be duplicated across your project.

                $xoops_url = XOOPS_URL;

The constant XOOPS_URL was not found. Maybe you did not declare it correctly or list all dependencies?

                switch ($value) {
                    case 1:
                        $src = "$xoops_url/modules/$moddir/assets/images/rating_1.gif";
                        break;
                    case 2:
                        $src = "$xoops_url/modules/$moddir/assets/images/rating_2.gif";
                        break;
                    case 3:
                        $src = "$xoops_url/modules/$moddir/assets/images/rating_3.gif";
                        break;
                    case 4:
                        $src = "$xoops_url/modules/$moddir/assets/images/rating_4.gif";
                        break;
                    case 5:
                        $src = "$xoops_url/modules/$moddir/assets/images/rating_5.gif";
                        break;
                    case 6:
                        $src = "$xoops_url/modules/$moddir/assets/images/rating_6.gif";
                        break;
                    case 7:
                        $src = "$xoops_url/modules/$moddir/assets/images/rating_7.gif";
                        break;
                    case 8:
                        $src = "$xoops_url/modules/$moddir/assets/images/rating_8.gif";
                        break;
                    case 9:
                        $src = "$xoops_url/modules/$moddir/assets/images/rating_9.gif";
                        break;
                    case 10:
                        $src = "$xoops_url/modules/$moddir/assets/images/rating_10.gif";
                        break;
                    default:
                        $src = "";
                }
                $rating = "<img src=\"$src\" />";

                return $rating;
                break;
            case "select":
                return $myts->makeTboxData4Show($value);
                break;
            case "textbox":
                return $myts->makeTboxData4Show($value);
                break;
            case "url":
                $link = explode('|', $value);

                return '<a href="' . $myts->makeTboxData4Show($link[0]) . '" title="' . $myts->makeTboxData4Show($link[1]) . '">' . $myts->makeTboxData4Show($link[0]) . '</a>';
                break;
            case "yesno":
                if ($value == '1') {
                    return _YES;

The constant _YES was not found. Maybe you did not declare it correctly or list all dependencies?

                } else {
                    return _NO;

The constant _NO was not found. Maybe you did not declare it correctly or list all dependencies?

                }
                break;
            default:
                return $myts->makeTboxData4Show($value);
                break;
        }
    }
}