Field::isLocked() - Code Metrics - Inspection of "Merge branch 'master' of https://github.com/XoopsM..." - mambax7/pedigree - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 5f51e9...d8eebf )

by Michael

created 2016-04-23 06:24 UTC

Field::isLocked() A

↳ Parent: Field

Complexity

Conditions	2
Paths	2

Size

Total Lines	9
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Metric	Value
dl	0
loc	9
rs	9.6666
cc	2
eloc	5
nc	2
nop	0

<?php

/**
 * Class Systemmessage
 */
class Systemmessage
namespace YourVendor;

class YourClass { }
{
    /**
     * @param $message
     */
    function Systemmessage($message)

    {
        echo '<span style="color: red;"><h3>' . $message . '</h3></span>';
    }
}

/**
 * Class Animal
 */
class Animal
namespace YourVendor;

class YourClass { }
{

    /**
     * @param int $animalnumber
     */
    function Animal($animalnumber = 0)

    {
        global $xoopsDB;
function myFunction($a, $b) {
    // Do something
}
        if ($animalnumber == 0) {

            $SQL = "SELECT * from " . $xoopsDB->prefix("pedigree_tree") . " WHERE ID = '1'";
        } else {
            $SQL = "SELECT * from " . $xoopsDB->prefix("pedigree_tree") . " WHERE ID = " . $animalnumber;
        }
        $result    = $xoopsDB->query($SQL);
        $row       = $xoopsDB->fetchArray($result);
        foreach ($row as $key => $value) {
            $this->$key = $value;
        }
    }

    /**
     * @return array
     */
    function numoffields()

    {
        global $xoopsDB;
function myFunction($a, $b) {
    // Do something
}
        $SQL    = "SELECT * from " . $xoopsDB->prefix("pedigree_fields") . " ORDER BY `order`";
        $fields = array();
        $result = $xoopsDB->query($SQL);
        $count  = 0;
        while ($row = $xoopsDB->fetchArray($result)) {
            $fields[] = $row['ID'];
            ++$count;
            $configvalues[] = $row;
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}

        }
        $this->configvalues = isset($configvalues) ? $configvalues : '';
class MyClass { }

$x = new MyClass();
$x->foo = true;
        //print_r ($this->configvalues); die();

        return $fields;
    }

    function getconfig()

    {
        return $this->configvalues;
    }
}

/**
 * Class Field
 */
class Field
namespace YourVendor;

class YourClass { }
{

    /**
     * @param $fieldnumber
     * @param $config
     */
    function Field($fieldnumber, $config)

    {
        //find key where ID = $fieldnumber;
        for ($x = 0; $x < count($config); ++$x) {
for ($i=0; $i<count($array); $i++) { // calls count() on each iteration
}

// Better
for ($i=0, $c=count($array); $i<$c; $i++) { // calls count() just once
}
            if ($config[$x]['ID'] == $fieldnumber) {
                foreach ($config[$x] as $key => $values) {
                    $this->$key = $values;
                }
            }

        }
        $this->id = $fieldnumber;
class MyClass { }

$x = new MyClass();
$x->foo = true;
    }

    /**
     * @return bool
     */
    function active()

    {
        $active = $this->getSetting("isActive");
        if ($active == '1') {
            return true;
        }

        return false;
    }

    /**
     * @return bool
     */
    function inadvanced()

    {
        $active = $this->getSetting("ViewInAdvanced");
        if ($active == '1') {
            return true;
        }

        return false;
    }

    /**
     * @return bool
     */
    function isLocked()

    {
        $active = $this->getSetting("locked");
        if ($active == '1') {
            return true;
        }

        return false;
    }

    /**
     * @return bool
     */
    function hassearch()

    {
        $active = $this->getSetting("HasSearch");
        if ($active == '1') {
            return true;
        }

        return false;
    }

    /**
     * @return bool
     */
    function addlitter()

    {
        $active = $this->getSetting("Litter");
        if ($active == '1') {
            return true;
        }

        return false;
    }

    /**
     * @return bool
     */
    function generallitter()

    {
        $active = $this->getSetting("Generallitter");
        if ($active == '1') {
            return true;
        }

        return false;
    }

    /**
     * @return bool
     */
    function haslookup()

    {
        $active = $this->getSetting("LookupTable");
        if ($active == '1') {
            return true;
        }

        return false;
    }

    /**
     * @return string
     */
    function getsearchstring()

    {
        return "&amp;o=naam&amp;p";
    }

    /**
     * @return bool
     */
    function inpie()

    {
        $active = $this->getSetting("ViewInPie");
        if ($active == '1') {
            return true;
        }

        return false;
    }

    /**
     * @return bool
     */
    function inpedigree()

    {
        $active = $this->getSetting("ViewInPedigree");
        if ($active == '1') {
            return true;
        }

        return false;
    }

    /**
     * @return bool
     */
    function inlist()

    {
        $active = $this->getSetting("ViewInList");
        if ($active == '1') {
            return true;
        }

        return false;
    }

    function getId()

    {
        return $this->id;
    }

    /**
     * @param $setting
     *
     * @return mixed
     */
    function getSetting($setting)

    {
        return $this->$setting;
    }

    /**
     * @param $fieldnumber
     *
     * @return array
     */
    function lookup($fieldnumber)

    {
        $ret = array();
        global $xoopsDB;
function myFunction($a, $b) {
    // Do something
}
        $SQL    = "SELECT * from " . $xoopsDB->prefix("pedigree_lookup" . $fieldnumber) . " ORDER BY 'order'";
        $result = $xoopsDB->query($SQL);
        while ($row = $xoopsDB->fetchArray($result)) {

            $ret[] = array('id' => $row['ID'], 'value' => $row['value']);
        }
        //array_multisort($ret,SORT_ASC);

        return $ret;
    }

    /**
     * @return XoopsFormLabel
     */
    function viewField()

    {
        $view = new XoopsFormLabel($this->fieldname, $this->value);
class MyClass { }

$x = new MyClass();
$x->foo = true;

        return $view;
    }

    /**
     * @return string
     */
    function showField()

    {
        return $this->fieldname . " : " . $this->value;
    }

    function showValue()

    {
        global $myts;
function myFunction($a, $b) {
    // Do something
}

        return $myts->displayTarea($this->value);
        //return $this->value;

    }

    /**
     * @return string
     */
    function searchfield()

    {
        return '<input type="text" name="query" size="20">';
    }
}

/**
 * Class radiobutton
 */
class radiobutton extends Field
namespace YourVendor;

class YourClass { }
{
    /**
     * @param $parentObject
     * @param $animalObject
     */
    function radiobutton($parentObject, $animalObject)

    {
        $this->fieldnumber = $parentObject->getId();
class MyClass { }

$x = new MyClass();
$x->foo = true;

        $this->fieldname    = $parentObject->FieldName;
class MyClass { }

$x = new MyClass();
$x->foo = true;
        $this->value        = $animalObject->{'user' . $this->fieldnumber};
class MyClass { }

$x = new MyClass();
$x->foo = true;
        $this->defaultvalue = $parentObject->DefaultValue;
class MyClass { }

$x = new MyClass();
$x->foo = true;
        $this->lookuptable  = $parentObject->LookupTable;
class MyClass { }

$x = new MyClass();
$x->foo = true;
        if ($this->lookuptable == '0') {
            new Systemmessage("A lookuptable must be specified for userfield" . $this->fieldnumber);
        }
    }

    /**
     * @return XoopsFormRadio
     */
    function editField()

    {
        $radio          = new XoopsFormRadio("<b>" . $this->fieldname . "</b>", 'user' . $this->fieldnumber, $value = $this->value);
        $lookupcontents = Field::lookup($this->fieldnumber);
        for ($i = 0; $i < count($lookupcontents); ++$i) {
for ($i=0; $i<count($array); $i++) { // calls count() on each iteration
}

// Better
for ($i=0, $c=count($array); $i<$c; $i++) { // calls count() just once
}
            $radio->addOption($lookupcontents[$i]['id'], $name = ($lookupcontents[$i]['value'] . "<br />"));
        }

        return $radio;
    }

    /**
     * @param string $name
     *
     * @return XoopsFormRadio
     */
    function newField($name = "")

    {
        $radio          = new XoopsFormRadio("<b>" . $this->fieldname . "</b>", $name . 'user' . $this->fieldnumber, $value = $this->defaultvalue);
        $lookupcontents = Field::lookup($this->fieldnumber);
        for ($i = 0; $i < count($lookupcontents); ++$i) {
for ($i=0; $i<count($array); $i++) { // calls count() on each iteration
}

// Better
for ($i=0, $c=count($array); $i<$c; $i++) { // calls count() just once
}
            $radio->addOption($lookupcontents[$i]['id'], $name = ($lookupcontents[$i]['value'] . "<br />"));
        }

        return $radio;
    }

    /**
     * @return XoopsFormLabel
     */
    function viewField()

    {
        $lookupcontents = Field::lookup($this->fieldnumber);
        for ($i = 0; $i < count($lookupcontents); ++$i) {
for ($i=0; $i<count($array); $i++) { // calls count() on each iteration
}

// Better
for ($i=0, $c=count($array); $i<$c; $i++) { // calls count() just once
}
            if ($lookupcontents[$i]['id'] == $this->value) {
                $choosenvalue = $lookupcontents[$i]['value'];
            }
        }
        $view = new XoopsFormLabel($this->fieldname, $choosenvalue);
function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}

        return $view;
    }

    /**
     * @return string
     */
    function showField()

    {
        $lookupcontents = Field::lookup($this->fieldnumber);
        for ($i = 0; $i < count($lookupcontents); ++$i) {
for ($i=0; $i<count($array); $i++) { // calls count() on each iteration
}

// Better
for ($i=0, $c=count($array); $i<$c; $i++) { // calls count() just once
}
            if ($lookupcontents[$i]['id'] == $this->value) {
                $choosenvalue = $lookupcontents[$i]['value'];
            }
        }

        return $this->fieldname . " : " . $choosenvalue;
function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}
    }

    function showValue()

    {
        $lookupcontents = Field::lookup($this->fieldnumber);
        for ($i = 0; $i < count($lookupcontents); ++$i) {
for ($i=0; $i<count($array); $i++) { // calls count() on each iteration
}

// Better
for ($i=0, $c=count($array); $i<$c; $i++) { // calls count() just once
}
            if ($lookupcontents[$i]['id'] == $this->value) {
                $choosenvalue = $lookupcontents[$i]['value'];
            }
        }

        return $choosenvalue;
function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}
    }

    /**
     * @return string
     */
    function searchfield()

    {
        $select         = '<select size="1" name="query" style="width: 140px;">';
        $lookupcontents = Field::lookup($this->fieldnumber);
        for ($i = 0; $i < count($lookupcontents); ++$i) {
for ($i=0; $i<count($array); $i++) { // calls count() on each iteration
}

// Better
for ($i=0, $c=count($array); $i<$c; $i++) { // calls count() just once
}
            $select .= '<option value="' . $lookupcontents[$i]['id'] . '">' . $lookupcontents[$i]['value'] . '</option>';
        }
        $select .= '</select>';

        return $select;
    }
}

/**
 * Class selectbox
 */
class selectbox extends Field
namespace YourVendor;

class YourClass { }
{
    /**
     * @param $parentObject
     * @param $animalObject
     */
    function selectbox($parentObject, $animalObject)

    {
        $this->fieldnumber  = $parentObject->getId();
class MyClass { }

$x = new MyClass();
$x->foo = true;
        $this->fieldname    = $parentObject->FieldName;
class MyClass { }

$x = new MyClass();
$x->foo = true;
        $this->value        = $animalObject->{'user' . $this->fieldnumber};
class MyClass { }

$x = new MyClass();
$x->foo = true;
        $this->defaultvalue = $parentObject->DefaultValue;
class MyClass { }

$x = new MyClass();
$x->foo = true;
        $this->lookuptable  = $parentObject->LookupTable;
class MyClass { }

$x = new MyClass();
$x->foo = true;
        if ($this->lookuptable == '0') {
            new Systemmessage("A lookuptable must be specified for userfield" . $this->fieldnumber);
        }
    }

    /**
     * @return XoopsFormSelect
     */
    function editField()

    {
        $select         = new XoopsFormSelect("<b>" . $this->fieldname . "</b>", 'user' . $this->fieldnumber, $value = $this->value, $size = 1, $multiple = false);
        $lookupcontents = Field::lookup($this->fieldnumber);
        for ($i = 0; $i < count($lookupcontents); ++$i) {
for ($i=0; $i<count($array); $i++) { // calls count() on each iteration
}

// Better
for ($i=0, $c=count($array); $i<$c; $i++) { // calls count() just once
}
            $select->addOption($lookupcontents[$i]['id'], $name = ($lookupcontents[$i]['value'] . "<br />"));
        }

        return $select;
    }

    /**
     * @param string $name
     *
     * @return XoopsFormSelect
     */
    function newField($name = "")

    {
        $select         = new XoopsFormSelect("<b>" . $this->fieldname . "</b>", $name . 'user' . $this->fieldnumber, $value = $this->defaultvalue, $size = 1, $multiple = false);
        $lookupcontents = Field::lookup($this->fieldnumber);
        for ($i = 0; $i < count($lookupcontents); ++$i) {
for ($i=0; $i<count($array); $i++) { // calls count() on each iteration
}

// Better
for ($i=0, $c=count($array); $i<$c; $i++) { // calls count() just once
}
            $select->addOption($lookupcontents[$i]['id'], $name = ($lookupcontents[$i]['value'] . "<br />"));
        }

        return $select;
    }

    /**
     * @return XoopsFormLabel
     */
    function viewField()

    {
        $lookupcontents = Field::lookup($this->fieldnumber);
        for ($i = 0; $i < count($lookupcontents); ++$i) {
for ($i=0; $i<count($array); $i++) { // calls count() on each iteration
}

// Better
for ($i=0, $c=count($array); $i<$c; $i++) { // calls count() just once
}
            if ($lookupcontents[$i]['id'] == $this->value) {
                $choosenvalue = $lookupcontents[$i]['value'];
            }
        }
        $view = new XoopsFormLabel($this->fieldname, $choosenvalue);
function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}

        return $view;
    }

    /**
     * @return string
     */
    function showField()

    {
        $choosenvalue   = '';
        $lookupcontents = Field::lookup($this->fieldnumber);
        for ($i = 0; $i < count($lookupcontents); ++$i) {
for ($i=0; $i<count($array); $i++) { // calls count() on each iteration
}

// Better
for ($i=0, $c=count($array); $i<$c; $i++) { // calls count() just once
}
            if ($lookupcontents[$i]['id'] == $this->value) {
                $choosenvalue = $lookupcontents[$i]['value'];
            }
        }

        return $this->fieldname . " : " . $choosenvalue;
    }

    /**
     * @return string
     */
    function showValue()

    {
        $choosenvalue   = '';
        $lookupcontents = Field::lookup($this->fieldnumber);
        for ($i = 0; $i < count($lookupcontents); ++$i) {
for ($i=0; $i<count($array); $i++) { // calls count() on each iteration
}

// Better
for ($i=0, $c=count($array); $i<$c; $i++) { // calls count() just once
}
            if ($lookupcontents[$i]['id'] == $this->value) {
                $choosenvalue = $lookupcontents[$i]['value'];
            }
        }

        return $choosenvalue;
    }

    /**
     * @return string
     */
    function searchfield()

    {
        $select         = '<select size="1" name="query" style="width: 140px;">';
        $lookupcontents = Field::lookup($this->fieldnumber);
        for ($i = 0; $i < count($lookupcontents); ++$i) {
for ($i=0; $i<count($array); $i++) { // calls count() on each iteration
}

// Better
for ($i=0, $c=count($array); $i<$c; $i++) { // calls count() just once
}
            $select .= '<option value="' . $lookupcontents[$i]['id'] . '">' . $lookupcontents[$i]['value'] . '</option>';
        }
        $select .= '</select>';

        return $select;
    }
}

/**
 * Class textbox
 */
class textbox extends Field
namespace YourVendor;

class YourClass { }
{
    /**
     * @param $parentObject
     * @param $animalObject
     */
    function textbox($parentObject, $animalObject)

    {
        $this->fieldnumber  = $parentObject->getId();
class MyClass { }

$x = new MyClass();
$x->foo = true;
        $this->fieldname    = $parentObject->FieldName;
class MyClass { }

$x = new MyClass();
$x->foo = true;
        $this->value        = $animalObject->{'user' . $this->fieldnumber};
class MyClass { }

$x = new MyClass();
$x->foo = true;
        $this->defaultvalue = $parentObject->DefaultValue;
class MyClass { }

$x = new MyClass();
$x->foo = true;
        $this->lookuptable  = $parentObject->LookupTable;
class MyClass { }

$x = new MyClass();
$x->foo = true;
        if ($this->lookuptable == '1') {
            new Systemmessage("No lookuptable may be specified for userfield" . $this->fieldnumber);
        }
        if ($parentObject->ViewInAdvanced == '1') {
            new Systemmessage("userfield" . $this->fieldnumber . " cannot be shown in advanced info");
        }
        if ($parentObject->ViewInPie == '1') {
            new Systemmessage("A Pie-chart cannot be specified for userfield" . $this->fieldnumber);
        }
    }

    /**
     * @return XoopsFormText
     */
    function editField()

    {
        $textbox = new XoopsFormText("<b>" . $this->fieldname . "</b>", 'user' . $this->fieldnumber, $size = 50, $maxsize = 50, $value = $this->value);

        return $textbox;
    }

    /**
     * @param string $name
     *
     * @return XoopsFormText
     */
    function newField($name = "")

    {
        $textbox = new XoopsFormText("<b>" . $this->fieldname . "</b>", $name . 'user' . $this->fieldnumber, $size = 50, $maxsize = 50, $value = $this->defaultvalue);

        return $textbox;
    }

    /**
     * @return string
     */
    function getsearchstring()

    {
        return "&amp;o=naam&amp;l=1";
    }
}

/**
 * Class textarea
 */
class textarea extends Field
namespace YourVendor;

class YourClass { }
{
    /**
     * @param $parentObject
     * @param $animalObject
     */
    function textarea($parentObject, $animalObject)

    {
        $this->fieldnumber  = $parentObject->getId();
class MyClass { }

$x = new MyClass();
$x->foo = true;
        $this->fieldname    = $parentObject->FieldName;
class MyClass { }

$x = new MyClass();
$x->foo = true;
        $this->value        = $animalObject->{'user' . $this->fieldnumber};
class MyClass { }

$x = new MyClass();
$x->foo = true;
        $this->defaultvalue = $parentObject->DefaultValue;
class MyClass { }

$x = new MyClass();
$x->foo = true;
        if ($parentObject->LookupTable == '1') {
            new Systemmessage("No lookuptable may be specified for userfield" . $this->fieldnumber);
        }
        if ($parentObject->ViewInAdvanced == '1') {
            new Systemmessage("userfield" . $this->fieldnumber . " cannot be shown in advanced info");
        }
        if ($parentObject->ViewInPie == '1') {
            new Systemmessage("A Pie-chart cannot be specified for userfield" . $this->fieldnumber);
        }
    }

    /**
     * @return XoopsFormTextArea
     */
    function editField()

    {
        $textarea = new XoopsFormTextArea("<b>" . $this->fieldname . "</b>", 'user' . $this->fieldnumber, $value = $this->value, $rows = 5, $cols = 50);

        return $textarea;
    }

    /**
     * @param string $name
     *
     * @return XoopsFormTextArea
     */
    function newField($name = "")

    {
        $textarea = new XoopsFormTextArea("<b>" . $this->fieldname . "</b>", $name . 'user' . $this->fieldnumber, $value = $this->defaultvalue, $rows = 5, $cols = 50);

        return $textarea;
    }

    /**
     * @return string
     */
    function getsearchstring()

    {
        return "&amp;o=naam&amp;l=1";
    }
}

/**
 * Class dateselect
 */
class dateselect extends Field
namespace YourVendor;

class YourClass { }
{
    /**
     * @param $parentObject
     * @param $animalObject
     */
    function dateselect($parentObject, $animalObject)

    {
        $this->fieldnumber  = $parentObject->getId();
class MyClass { }

$x = new MyClass();
$x->foo = true;
        $this->fieldname    = $parentObject->FieldName;
class MyClass { }

$x = new MyClass();
$x->foo = true;
        $this->value        = $animalObject->{'user' . $this->fieldnumber};
class MyClass { }

$x = new MyClass();
$x->foo = true;
        $this->defaultvalue = $parentObject->DefaultValue;
class MyClass { }

$x = new MyClass();
$x->foo = true;
        if ($parentObject->LookupTable == '1') {
            new Systemmessage("No lookuptable may be specified for userfield" . $this->fieldnumber);
        }
        if ($parentObject->ViewInAdvanced == '1') {
            new Systemmessage("userfield" . $this->fieldnumber . " cannot be shown in advanced info");
        }
        if ($parentObject->ViewInPie == '1') {
            new Systemmessage("A Pie-chart cannot be specified for userfield" . $this->fieldnumber);
        }
    }

    /**
     * @return XoopsFormTextDateSelect
     */
    function editField()

    {
        //$textarea = new XoopsFormFile("<b>".$this->fieldname."</b>", $this->fieldname, $maxfilesize = 2000);

        $textarea = new XoopsFormTextDateSelect("<b>" . $this->fieldname . "</b>", 'user' . $this->fieldnumber, $size = 15, $this->value);

        return $textarea;
    }

    /**
     * @param string $name
     *
     * @return XoopsFormTextDateSelect
     */
    function newField($name = "")

    {
        $textarea = new XoopsFormTextDateSelect("<b>" . $this->fieldname . "</b>", $name . 'user' . $this->fieldnumber, $size = 15, $this->defaultvalue);

        return $textarea;
    }

    /**
     * @return string
     */
    function getsearchstring()

    {
        return "&amp;o=naam&amp;l=1";
    }
}

/**
 * Class urlfield
 */
class urlfield extends Field
namespace YourVendor;

class YourClass { }
{
    /**
     * @param $parentObject
     * @param $animalObject
     */
    function urlfield($parentObject, $animalObject)

    {
        $this->fieldnumber  = $parentObject->getId();
class MyClass { }

$x = new MyClass();
$x->foo = true;
        $this->fieldname    = $parentObject->FieldName;
class MyClass { }

$x = new MyClass();
$x->foo = true;
        $this->value        = $animalObject->{'user' . $this->fieldnumber};
class MyClass { }

$x = new MyClass();
$x->foo = true;
        $this->defaultvalue = $parentObject->DefaultValue;
class MyClass { }

$x = new MyClass();
$x->foo = true;
        $this->lookuptable  = $parentObject->LookupTable;
class MyClass { }

$x = new MyClass();
$x->foo = true;
        if ($this->lookuptable == '1') {
            new Systemmessage("No lookuptable may be specified for userfield" . $this->fieldnumber);
        }
        if ($parentObject->ViewInAdvanced == '1') {
            new Systemmessage("userfield" . $this->fieldnumber . " cannot be shown in advanced info");
        }
        if ($parentObject->ViewInPie == '1') {
            new Systemmessage("A Pie-chart cannot be specified for userfield" . $this->fieldnumber);
        }
    }

    /**
     * @return XoopsFormText
     */
    function editField()

    {
        $textbox = new XoopsFormText("<b>" . $this->fieldname . "</b>", 'user' . $this->fieldnumber, $size = 50, $maxsize = 255, $value = $this->value);

        return $textbox;
    }

    /**
     * @param string $name
     *
     * @return XoopsFormText
     */
    function newField($name = "")

    {
        $textbox = new XoopsFormText("<b>" . $this->fieldname . "</b>", $name . 'user' . $this->fieldnumber, $size = 50, $maxsize = 255, $value = $this->defaultvalue);

        return $textbox;
    }

    /**
     * @return XoopsFormLabel
     */
    function viewField()

    {
        $view = new XoopsFormLabel($this->fieldname, '<a href="' . $this->value . '" target=\"_new\">' . $this->value . '</a>');

        return $view;
    }

    /**
     * @return string
     */
    function showField()

    {
        return $this->fieldname . " : <a href=\"" . $this->value . "\" target=\"_new\">" . $this->value . "</a>";
    }

    /**
     * @return string
     */
    function showValue()

    {
        return "<a href=\"" . $this->value . "\" target=\"_new\">" . $this->value . "</a>";
    }

    /**
     * @return string
     */
    function getsearchstring()

    {
        return "&amp;o=naam&amp;l=1";
    }
}

/**
 * Class Picture
 */
class Picture extends Field
namespace YourVendor;

class YourClass { }
{
    /**
     * @param $parentObject
     * @param $animalObject
     */
    function Picture($parentObject, $animalObject)

    {
        $this->fieldnumber  = $parentObject->getId();
class MyClass { }

$x = new MyClass();
$x->foo = true;
        $this->fieldname    = $parentObject->FieldName;
class MyClass { }

$x = new MyClass();
$x->foo = true;
        $this->value        = $animalObject->{'user' . $this->fieldnumber};
class MyClass { }

$x = new MyClass();
$x->foo = true;
        $this->defaultvalue = $parentObject->DefaultValue;
class MyClass { }

$x = new MyClass();
$x->foo = true;
        $this->lookuptable  = $parentObject->LookupTable;
class MyClass { }

$x = new MyClass();
$x->foo = true;
        if ($this->lookuptable == '1') {
            new Systemmessage("No lookuptable may be specified for userfield" . $this->fieldnumber);
        }
        if ($parentObject->ViewInAdvanced == '1') {
            new Systemmessage("userfield" . $this->fieldnumber . " cannot be shown in advanced info");
        }
        if ($parentObject->ViewInPie == '1') {
            new Systemmessage("A Pie-chart cannot be specified for userfield" . $this->fieldnumber);
        }
        if ($parentObject->ViewInList == '1') {
            new Systemmessage("userfield" . $this->fieldnumber . " cannot be included in listview");
        }
        if ($parentObject->HasSearch == '1') {
            new Systemmessage("Search cannot be defined for userfield" . $this->fieldnumber);
        }
    }

    /**
     * @return XoopsFormFile
     */
    function editField()

    {
        $picturefield = new XoopsFormFile($this->fieldname, 'user' . $this->fieldnumber, 1024000);
        $picturefield->setExtra("size ='50'");

        return $picturefield;
    }

    /**
     * @param string $name
     *
     * @return XoopsFormFile
     */
    function newField($name = "")

    {
        $picturefield = new XoopsFormFile($this->fieldname, $name . 'user' . $this->fieldnumber, 1024000);
        $picturefield->setExtra("size ='50'");

        return $picturefield;
    }

    /**
     * @return XoopsFormLabel
     */
    function viewField()

    {
        $view = new XoopsFormLabel($this->fieldname, "<img src=\"" . PEDIGREE_UPLOAD_URL . "/images/thumbnails/" . $this->value . "_400.jpeg\">");

        return $view;
    }

    /**
     * @return string
     */
    function showField()

    {
        return "<img src=" . PEDIGREE_UPLOAD_URL . "\"/images/thumbnails/" . $this->value . "_150.jpeg\">";
    }

    /**
     * @return string
     */
    function showValue()

    {
        return "<img src=" . PEDIGREE_UPLOAD_URL . "\"/images/thumbnails/" . $this->value . "_400.jpeg\">";
    }
}

/**
 * Class SISContext
 */
class SISContext
namespace YourVendor;

class YourClass { }
{
    var $_contexts;
class A {
    var $property;
}
    var $_depth;
class A {
    var $property;
}

    function SISContext()

    {
        $this->_contexts = array();
        $this->_depth    = 0;
    }

    /**
     * @param $url
     * @param $name
     */
    function mygoto($url, $name)

    {
        $keys = array_keys($this->_contexts);
        for ($i = 0; $i < $this->_depth; ++$i) {
            if ($keys[$i] == $name) {
                $this->_contexts[$name] = $url; // the url might be slightly different
                $this->_depth           = $i + 1;
                for ($x = count($this->_contexts); $x > $i + 1; $x--) {
                    array_pop($this->_contexts);
                }

                return;
            }
        }

        $this->_contexts[$name] = $url;
        $this->_depth++;

    }

    /**
     * @return array
     */
    function getAllContexts()

    {
        return $this->_contexts;
    }

    /**
     * @return array
     */
    function getAllContextNames()

    {
        return array_keys($this->_contexts);
    }
}


Push — master ( 5f51e9...d8eebf )

Field::isLocked() A

Complexity

Size

Duplication

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

Available Fixes

Available Fixes

Available Fixes

Available Fixes

1		<?php
2
3		/**
4		* Class Systemmessage
5		*/
6		class Systemmessage
		0 ignored issues – show Coding Style Compatibility introduced 2016-01-18 09:41 UTC by Report Bug Copy Issue Report PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions. You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
7		{
8		/**
9		* @param $message
10		*/
11		function Systemmessage($message)
		0 ignored issues – show Best Practice introduced 2016-01-18 09:41 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history... Coding Style Best Practice introduced 2016-01-18 09:41 UTC by Report Bug Copy Issue Report Please use `__construct()` instead of a PHP4-style constructor that is named after the class. Loading history...
12		{
13		echo '<span style="color: red;"><h3>' . $message . '</h3></span>';
14		}
15		}
16
17		/**
18		* Class Animal
19		*/
20		class Animal
		0 ignored issues – show Coding Style Compatibility introduced 2016-01-18 09:41 UTC by Report Bug Copy Issue Report PSR1 recommends that each class should be in its own file to aid autoloaders. Having each class in a dedicated file usually plays nice with PSR autoloaders and is therefore a well established practice. If you use other autoloaders, you might not want to follow this rule. Loading history... Coding Style Compatibility introduced 2016-01-18 09:41 UTC by Report Bug Copy Issue Report PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions. You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
21		{
22
23		/**
24		* @param int $animalnumber
25		*/
26		function Animal($animalnumber = 0)
		0 ignored issues – show Best Practice introduced 2016-01-18 09:41 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history... Coding Style Best Practice introduced 2016-01-18 09:41 UTC by Report Bug Copy Issue Report Please use `__construct()` instead of a PHP4-style constructor that is named after the class. Loading history...
27		{
28		global $xoopsDB;
		0 ignored issues – show Compatibility Best Practice introduced 2016-01-18 09:34 UTC by Report Bug Copy Issue Report Use of `global` functionality is not recommended; it makes your code harder to test, and less reusable. Instead of relying on `global` state, we recommend one of these alternatives: 1. Pass all data via parameters function myFunction($a, $b) { // Do something } 2. Create a class that maintains your state class MyClass { private $a; private $b; public function __construct($a, $b) { $this->a = $a; $this->b = $b; } public function myFunction() { // Do something } } Loading history...
29	View Code Duplication	if ($animalnumber == 0) {
		0 ignored issues – show Duplication introduced 2016-01-18 09:41 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
30		$SQL = "SELECT * from " . $xoopsDB->prefix("pedigree_tree") . " WHERE ID = '1'";
31		} else {
32		$SQL = "SELECT * from " . $xoopsDB->prefix("pedigree_tree") . " WHERE ID = " . $animalnumber;
33		}
34		$result = $xoopsDB->query($SQL);

mambax7 / pedigree

Push — master ( 5f51e9...d8eebf )

Field::isLocked() A

Complexity

Size

Duplication

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like