SmallWorldForm::input_add() - Code Metrics - Inspection of "Merge pull request #8 from XoopsModules25x/revert-..." - XoopsModules25x/smallworld - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( fd0c8a...c024a6 )

unknown

created 2020-12-06 11:46 UTC

SmallWorldForm::input_add() A

↳ Parent: SmallWorldForm

Complexity

Conditions	3
Paths	4

Size

Total Lines

Duplication

Lines	7
Ratio	100 %

Importance

Changes

Metric	Value
cc	3
nc	4
nop	8
dl	7
loc	7
rs	10
c	0
b	0
f	0

How to fix Many Parameters

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

/**
 * SmallWorld
 *
 * @copyright    The XOOPS Project (https://xoops.org)
 * @copyright    2011 Culex
 * @license      GNU GPL (http://www.gnu.org/licenses/gpl-2.0.html/)
 * @package      SmallWorld
 * @since        1.0
 * @author       Michael Albertsen (http://culex.dk) <[email protected]>
 */
class SmallWorldForm

{
    /**
     * @create a dropdown select
     * @param string $name
     * @param array  $options
     * @param string $selected (optional)

     * @return string
     */
    public function dropdown($name, array $options, $selected = null)
    {
        $dropdown = '<select name="' . $name . '" id="' . $name . '">' . '<br>';
        $selected = $selected;

        foreach ($options as $key => $option) {
            $select   = $selected == $key ? ' selected="yes"' : '';
            $dropdown .= '<option value="' . $key . '"' . $select . '>' . $option . '</option>' . '<br>';
        }
        $dropdown .= '</select>' . '<br>';
        return $dropdown;
    }

    /**
     * @create a radio select
     * @param string $name
     * @param array  $options
     * @param string $selected (optional)

     * @return string
     */
    public function radio($name, array $options, $selected = null)
    {
        $selected = $selected;

        $form     = '';
        foreach ($options as $value => $option) {
            $select = $selected == $value ? ' checked' : ' checked="unchecked"';
            $form   .= '<input type="checkbox" name="' . $name . '[]" id="' . $name . '" value="' . $value . '" ' . $select . '> ' . $option . '<br>';
        }
        return $form;
    }

    /**
     * @param string $name
     * @param array  $options
     * @param array  $valuearray
     * @param string $selected (optional)

     * @return string
     */
    public function RetrieveRadio($name, $options, $valuearray, $selected = null)

    {
        $form = '';
        $a    = count($options) - 1;
        for ($i = 0; $i <= $a; ++$i) {
            if (in_array($i, $valuearray)) {
                $form .= '<input type="checkbox" id="' . $name . '-' . $i . '" name="' . $name . '[]" value="' . $i . '" checked>' . $options[$i] . '<br>';
            } else {
                $form .= '<input type="checkbox" id="' . $name . '-' . $i . '" name="' . $name . '[]" value="' . $i . '" >' . $options[$i] . '<br>';
            }
        }
        return $form;
    }

    /**
     * @param string $name
     * @param string $id
     * @param string $class
     * @param int    $size   (optional)

     * @param string $preset (optional)

     * @return string
     */
    public function input($name, $id, $class, $size = null, $preset = null)
    {
        $s    = $size ?: '50px';
        $data = "<input type='text' size='" . $s . "' name='" . $name . "' id='" . $id . "' class='" . $class . "' value='" . $preset . "' autocomplete='off'>";
        return $data;
    }

    /**
     * @param string $name
     * @param string $id
     * @param string $preset (optional)

     * @return string
     */
    public function hidden($name, $id, $preset = null)

    {
        $data = "<input type='hidden' name='" . $name . "' value='" . $preset . "' >";
        return $data;
    }

    /**
     * @param string $value
     * @return string
     */
    public function simpleText($value)
    {
        $data = $value;
        return $data;
    }

    /**
     *
     * @param string $class
     * @param string $name
     * @param         $name2
     * @param string $rel
     * @param array  $options
     * @param string $textmore
     * @param string $selected (optional)

     * @param string $preset   (optional)

     * @return as|string
     */
    public function dropdown_add($class, $name, $name2, $rel, array $options, $textmore, $selected = null, $preset = null)

    {
        $dropdown = '<span id="' . $name . '"><input type="text" name="' . $name2 . '[]" value="' . $preset . '">';
        $dropdown .= '<select class="smallworld_select" name="' . $name . '[]" id="' . $name . '"></p>' . '<br>';
        foreach ($options as $key => $option) {
            $select   = $selected == $key ? ' selected' : null;
            $dropdown .= '<option value="' . $key . '"' . $select . '>' . $option . '</option>' . '<br>';
        }
        $dropdown .= '</select></span>';
        return $dropdown;
    }

    /**
     *
     * @param string $class
     * @param string $name
     * @param         $name2
     * @param string $rel
     * @param int    $size
     * @param string $textmore
     * @param string $preset (optional)

     * @param string $id     (optional)

     * @return dynamic|string
     */
    public function input_add($class, $name, $name2, $rel, $size, $textmore, $preset = null, $id = null)

    {
        $s    = $size ?: '50px';
        $i    = $id ?: '';
        $data = "<span id='" . $name . "'><input type='text' size='" . $s . "' name='" . $name2 . "[]' value='" . $preset . "' id='" . $i . "'></span>";
        return $data;
    }

    /**
     * @return string|textarea
     * @param string $name
     * @param string $id
     * @param string $title
     * @param string $rows
     * @param string $cols
     * @param string $class
     * @param string $preset (optional)

     * @return string
     */
    public function textarea($name, $id, $title, $rows, $cols, $class, $preset = null)
    {
        return "<textarea name='" . $name . "' id='" . $id . "'  title='" . $title . "' rows='" . $rows . "' cols='" . $cols . "' class='" . $class . "'>" . $preset . '</textarea>';
    }

    /**
     * @return dynamic|string
     * @param string $class
     * @param string $name
     * @param string $name2
     * @param string $rel
     * @param array  $options
     * @param string $textmore
     * @param string $selected      (optional)

     * @param string $preset        (optional)

     * @param string $selectedstart (optional)

     * @param string $selectedstop  (optional)

     * @return string
     */
    public function school_add(
        $class,

        $name,
        $name2,
        $rel,

        array $options,
        $textmore,

        $selected = null,
        $preset = null,
        $selectedstart = null,
        $selectedstop = null
    ) {
        $dropdown = '<div id="' . $name . '"><p class="smallworld_clonebreaker">' . _SMALLWORLD_SCHOOLNAME . '<input class="school" type="text" value="' . $preset . '" name="' . $name2 . '[]">';
        $dropdown .= '<br><br>' . _SMALLWORLD_SCHOOLTYPE . '<select class="school" name="' . $name . '[]" id="' . $name . '"">' . '<br>';
        foreach ($options as $key => $option) {
            $select   = $selected == $key ? ' selected="selected"' : null;
            $dropdown .= '<option  class="school" value="' . $key . '"' . $select . '>' . $option . '</option>' . '<br>';
        }
        $dropdown .= '</select>';
        $dropdown .= '<br><br>';
        $dropdown .= _SMALLWORLD_START . '<select class="schooltime" name="schoolstart[]" id="schoolstart">';
        $array    = SmallworldGetTimestampsToForm();
        foreach ($array as $key => $option) {
            $selectstart = $selectedstart == $key ? ' selected="selected"' : null;
            $dropdown    .= '<option value="' . $key . '"' . $selectstart . '>' . $option . '</option>' . '<br>';
        }
        $dropdown .= '</select>';
        $dropdown .= '<br><br>';
        $dropdown .= _SMALLWORLD_STOP . '<select class="schooltime" name="schoolstop[]" id="schoolstop">';
        $array    = SmallworldGetTimestampsToForm();
        foreach ($array as $key => $option) {
            $selectstop = $selectedstop == $key ? ' selected="selected"' : null;
            $dropdown   .= '<option value="' . $key . '"' . $selectstop . '>' . $option . '</option>' . '<br>';
        }
        $dropdown .= '</select><br></p></div>';
        return $dropdown;
    }

    /**
     *
     * @param string $class
     * @param string $name
     * @param         $name2
     * @param string $rel
     * @param string $textmore
     * @param string $employer      (optional)

     * @param string $position      (optional)

     * @param string $selectedstart (optional)

     * @param string $selectedstop  (optional)

     * @param string $description   (optional)

     * @return dynamic|string
     */
    public function job(
        $class,

        $name,
        $name2,

        $rel,

        $textmore,

        $employer = null,
        $position = null,
        $selectedstart = null,
        $selectedstop = null,
        $description = null
    ) {
        $text = '<div id="' . $name . '"><p class="smallworld_clonebreaker">' . _SMALLWORLD_EMPLOYER . '<input class="job" id="job" value="' . $employer . '" type="text" name="employer[]">';
        $text .= '<br><br>' . _SMALLWORLD_POSITION . '<input class="job" type="text" value="' . $position . '" name="position[]">';
        $text .= '<br><br>' . _SMALLWORLD_JOBSTART . '<input class="jobstart" type="text" value="' . $selectedstart . '" name="jobstart[]">';
        $text .= '<br><br>' . _SMALLWORLD_JOBSTOP . '<input class="jobstop" value="' . $selectedstop . '" type="text" name="jobstop[]">';
        $text .= '<br><br><span class="jobdescText">' . _SMALLWORLD_DESCRIPTION . '</span><textarea class="jobdesc" name="description[]" rows="20" cols="20">' . $description . '</textarea><br></p></div>';
        $text .= '' . '<br>';
        return $text;
    }

    /**
     * @return file|string
     * @param int $userID
     * @return string
     */
    public function uploadform($userID)

    {
        $text = '<form action="imgupload.php" method="POST" enctype="multipart/form-data">';
        $text .= '<input type="file" name="file[]" multiple>';
        $text .= '<button type="submit">' . _SMALLWORLD_UPLOADTEXT . '</button>';
        $text .= '<span class="file_upload_label">' . _SMALLWORLD_UPLOADFILESTEXT . '</span>';
        $text .= '</form>';
        return $text;
    }

    /**
     * @return dynamic|string
     * @param int    $userID
     * @param string $imgurl
     * @param string $imgdesc
     * @param string $id
     * @return string
     */
    public function edit_images($userID, $imgurl, $imgdesc, $id)

    {
        global $xoopsUser;
        $text = '<p class="smallworld_clonebreaker"><br>';
        $text .= '<table class="smallworld_table" border="0" cellspacing="0" cellpadding="0">';
        $text .= '<tr>';
        $text .= '<td><img class="smallworld_edit_image" src="' . $imgurl . '" height="100px" width="80px;"></td>';
        $text .= '<td><span class="smallworld_editTextSpan">' . _SMALLWORLD_UPLOADDESC . '</span><br><br><textarea class="smallworld_edit_desc" name="imgdesc[]" rows="1" cols="1">' . stripslashes($imgdesc) . '</textarea><br><br></td>';
        $text .= '<input value="' . $id . '" type="hidden" name="id[]"></p>';
        $text .= '</tr></table>';
        return $text;
    }

    /**
     * @create a radio select
     * @param $userid
     * @return string
     */
    public function usersettings($userid)
    {
        global $xoopsUser, $xoopsDB;

        $form = "<div style='display:none'><div class='smallworld_usersetings'>";
        $form .= '<fieldset><legend>' . _SMALLWORLD_SHOWIFPUBLICORPRIVATE . '</legend>';
        $form .= "<form id='perset'>";
        if ($xoopsUser) {
            $sql    = 'SELECT value FROM ' . $xoopsDB->prefix('smallworld_settings') . ' WHERE userid = ' . (int)$userid;
            $result = $xoopsDB->queryF($sql);
            $i      = $xoopsDB->getRowsNum($result);
            $v      = [];
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
            if ($i >= 1) {
                while ($row = $xoopsDB->fetchArray($result)) {
                    $v    = unserialize(stripslashes($row['value']));
                    $pv   = ('1' == $v['posts']) ? ' checked' : '';
                    $cv   = ('1' == $v['comments']) ? ' checked' : '';
                    $nv   = ('1' == $v['notify']) ? ' checked' : '';
                    $form .= '<input type="checkbox" name="usersettings[]" id="posts" value="' . $v['posts'] . '" ' . $pv . '> ' . _SMALLWORLD_SHOWMYPOSTS . '<br>';
                    $form .= '<input type="checkbox" name="usersettings[]" id="comments" value="' . $v['comments'] . '" ' . $cv . '> ' . _SMALLWORLD_SHOWMYCOMMENTS . '<br>';
                    $form .= '<input type="checkbox" name="usersettings[]" id="notify" value="' . $v['notify'] . '" ' . $nv . '> ' . _SMALLWORLD_NOTIFYME . '<br>';
                }
            } else {
                $form .= '<input type="checkbox" name="usersettings[]" id="posts" value="0"> ' . _SMALLWORLD_SHOWMYPOSTS . '<br>';
                $form .= '<input type="checkbox" name="usersettings[]" id="comments" value="0"> ' . _SMALLWORLD_SHOWMYCOMMENTS . '<br>';
                $form .= '<input type="checkbox" name="usersettings[]" id="notify" value="0"> ' . _SMALLWORLD_NOTIFYME . '<br>';
            }
        }
        $form .= "<br><input type='submit' id='smallworld_privsave' value='" . _SMALLWORLD_SUBMIT . "' class='smallworld_finish'>";
        $form .= '</form></fieldset></div></div>';
        return $form;
    }
}


1		<?php
2		/**
3		* You may not change or alter any portion of this comment or credits
4		* of supporting developers from this source code or any supporting source code
5		* which is considered copyrighted (c) material of the original comment or credit authors.
6		*
7		* This program is distributed in the hope that it will be useful,
8		* but WITHOUT ANY WARRANTY; without even the implied warranty of
9		* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
10		*/
11
12		/**
13		* SmallWorld
14		*
15		* @copyright The XOOPS Project (https://xoops.org)
16		* @copyright 2011 Culex
17		* @license GNU GPL (http://www.gnu.org/licenses/gpl-2.0.html/)
18		* @package SmallWorld
19		* @since 1.0
20		* @author Michael Albertsen (http://culex.dk) <[email protected]>
21		*/
22	View Code Duplication	class SmallWorldForm
		0 ignored issues – show Duplication introduced 2020-04-14 12:42 UTC by Report Bug Copy Issue Report This class seems to be duplicated in 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...
23		{
24		/**
25		* @create a dropdown select
26		* @param string $name
27		* @param array $options
28		* @param string $selected (optional)
		0 ignored issues – show Documentation introduced 2016-01-18 10:30 UTC by Report Bug Copy Issue Report Should the type for parameter `$selected` not be `string\|null`? This check looks for `@param` annotations where the type inferred by our type inference engine differs from the declared type. It makes a suggestion as to what type it considers more descriptive. Most often this is a case of a parameter that can be null in addition to its declared types. Loading history...
29		* @return string
30		*/
31		public function dropdown($name, array $options, $selected = null)
32		{
33		$dropdown = '<select name="' . $name . '" id="' . $name . '">' . '<br>';
34		$selected = $selected;
		0 ignored issues – show Bug introduced 2016-01-18 10:30 UTC by Report Bug Copy Issue Report Why assign `$selected` to itself? This checks looks for cases where a variable has been assigned to itself. This assignement can be removed without consequences. Loading history...
35		foreach ($options as $key => $option) {
36		$select = $selected == $key ? ' selected="yes"' : '';
37		$dropdown .= '<option value="' . $key . '"' . $select . '>' . $option . '</option>' . '<br>';

XoopsModules25x / smallworld

Push — master ( fd0c8a...c024a6 )

SmallWorldForm::input_add() A

Complexity

Size

Duplication

Importance

How to fix Many Parameters

Many Parameters

Duplication Side-by-Side

Filter issues like