Form::dropdown_add() - Code Metrics - Inspection of "namespaces, PHP7" - XoopsModules25x/smallworld - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#5)

by Michael

created 2020-03-30 11:20 UTC

Form::dropdown_add() A

↳ Parent: Form

Complexity

Conditions	3
Paths	3

Size

Total Lines

Duplication

Lines	4
Ratio	33.33 %

Importance

Changes

Metric	Value
cc	3
nc	3
nop	8
dl	4
loc	12
rs	9.8666
c	0
b	0
f	0

How to fix Many Parameters

<?php

namespace XoopsModules\Smallworld;

/**
 * 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 Form
{
    /**
     * @create a dropdown select
     * @param string $name
     * @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 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 . "' >";

        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 string  $textmore
     * @param string  $selected (optional)

     * @param string  $preset   (optional)

     * @return 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;
    }

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

     * @return string|textarea
     * @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>';
    }

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

     * @param string $preset        (optional)

     * @param string $selectedstart (optional)

     * @param string $selectedstop  (optional)

     * @return dynamic|string
     * @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;
    }

    /**
     * @param int $userID
     * @return file|string
     * @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;
    }

    /**
     * @param int    $userID
     * @param string $imgurl
     * @param string $imgdesc
     * @param string $id
     * @return dynamic|string
     * @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 (false !== ($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		namespace XoopsModules\Smallworld;
4
5		/**
6		* You may not change or alter any portion of this comment or credits
7		* of supporting developers from this source code or any supporting source code
8		* which is considered copyrighted (c) material of the original comment or credit authors.
9		*
10		* This program is distributed in the hope that it will be useful,
11		* but WITHOUT ANY WARRANTY; without even the implied warranty of
12		* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
13		*/
14
15		/**
16		* SmallWorld
17		*
18		* @copyright The XOOPS Project (https://xoops.org)
19		* @copyright 2011 Culex
20		* @license GNU GPL (http://www.gnu.org/licenses/gpl-2.0.html/)
21		* @package SmallWorld
22		* @since 1.0
23		* @author Michael Albertsen (http://culex.dk) <[email protected]>
24		*/
25		class Form
26		{
27		/**
28		* @create a dropdown select
29		* @param string $name
30		* @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...
31		* @return string
32		*/
33		public function dropdown($name, array $options, $selected = null)
34		{
35		$dropdown = '<select name="' . $name . '" id="' . $name . '">' . '<br>';
36		$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...
37	View Code Duplication	foreach ($options as $key => $option) {
		0 ignored issues – show Duplication introduced 2016-01-18 10:30 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...

XoopsModules25x / smallworld

Pull Request — master (#5)

Form::dropdown_add() A

Complexity

Size

Duplication

Importance

How to fix Many Parameters

Many Parameters

Duplication Side-by-Side

Filter issues like