slides.php ➔ b_presenter_slides_edit()   A
last analyzed

Complexity

Conditions 4
Paths 6

Size

Total Lines 27

Duplication

Lines 0
Ratio 0 %

Importance

Changes 0
Metric Value
cc 4
nc 6
nop 1
dl 0
loc 27
rs 9.488
c 0
b 0
f 0
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
 * presenter module for xoops
13
 *
14
 * @copyright       XOOPS Project (https://xoops.org)
15
 * @license         GPL 2.0 or later
16
 * @package         presenter
17
 * @since           2.5.5
18
 * @author          XOOPS Development Team <[email protected]> - <https://xoops.org>
19
 */
20
require_once XOOPS_ROOT_PATH . '/modules/presenter/include/functions.php';
21
/**
22
 * @param $options
23
 * @return array
24
 */
25
function b_presenter_slides_show($options)
26
{
27
    require_once XOOPS_ROOT_PATH . '/modules/presenter/class/slides.php';
28
    $myts = MyTextSanitizer::getInstance();
0 ignored issues
show
Unused Code introduced by
$myts is not used, you could remove the assignment.

This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently.

$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}

Both the $myVar assignment in line 1 and the $higher assignment in line 2 are dead. The first because $myVar is never used and the second because $higher is always overwritten for every possible time line.

Loading history...
29
30
    $slides       = [];
31
    $type_block   = $options[0];
32
    $nb_slides    = $options[1];
33
    $lenght_title = $options[2];
0 ignored issues
show
Unused Code introduced by
$lenght_title is not used, you could remove the assignment.

This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently.

$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}

Both the $myVar assignment in line 1 and the $higher assignment in line 2 are dead. The first because $myVar is never used and the second because $higher is always overwritten for every possible time line.

Loading history...
34
35
    $slidesHandler = xoops_getModuleHandler('slides', 'presenter');
36
    $criteria      = new CriteriaCompo();
37
    array_shift($options);
38
    array_shift($options);
39
    array_shift($options);
40
41
    if ($type_block) {
42
        $criteria->add(new Criteria('slides_id', 0, '!='));
43
        $criteria->setSort('slides_id');
44
        $criteria->setOrder('ASC');
45
    }
46
47
    $criteria->setLimit($nb_slides);
48
    $slides_arr = $slidesHandler->getAll($criteria);
49
    foreach (array_keys($slides_arr) as $i) {
50
        $slides[$i]['slides_id']      = $slides_arr[$i]->getVar('slides_id');
51
        $slides[$i]['slides_cid']     = $slides_arr[$i]->getVar('slides_cid');
52
        $slides[$i]['slides_uid']     = $slides_arr[$i]->getVar('slides_uid');
53
        $slides[$i]['slides_title']   = $slides_arr[$i]->getVar('slides_title');
54
        $slides[$i]['slides_content'] = $slides_arr[$i]->getVar('slides_content');
55
56
        $slides[$i]['css_id']    = $slides_arr[$i]->getVar('css_id');
57
        $slides[$i]['css_class'] = $slides_arr[$i]->getVar('css_class');
58
59
        $slides[$i]['slides_transition_x'] = $slides_arr[$i]->getVar('slides_transition_x');
60
        $slides[$i]['slides_transition_y'] = $slides_arr[$i]->getVar('slides_transition_y');
61
        $slides[$i]['slides_transition_z'] = $slides_arr[$i]->getVar('slides_transition_z');
62
        $slides[$i]['slides_rotation_x']   = $slides_arr[$i]->getVar('slides_rotation_x');
63
        $slides[$i]['slides_rotation_y']   = $slides_arr[$i]->getVar('slides_rotation_y');
64
        $slides[$i]['slides_rotation_z']   = $slides_arr[$i]->getVar('slides_rotation_z');
65
        $slides[$i]['slides_scale_x']      = $slides_arr[$i]->getVar('slides_scale_x');
66
        $slides[$i]['slides_scale_y']      = $slides_arr[$i]->getVar('slides_scale_y');
67
        $slides[$i]['slides_scale_z']      = $slides_arr[$i]->getVar('slides_scale_z');
68
        $slides[$i]['slides_created']      = $slides_arr[$i]->getVar('slides_created');
69
        $slides[$i]['slides_published']    = $slides_arr[$i]->getVar('slides_published');
70
        $slides[$i]['slides_position']     = $slides_arr[$i]->getVar('slides_position');
71
        $slides[$i]['slides_online']       = $slides_arr[$i]->getVar('slides_online');
72
        $slides[$i]['slides_type']         = $slides_arr[$i]->getVar('slides_type');
73
        $slides[$i]['slides_notes']        = $slides_arr[$i]->getVar('slides_notes');
74
        $slides[$i]['slides_mp3']          = $slides_arr[$i]->getVar('slides_mp3');
75
        $slides[$i]['slides_time']         = $slides_arr[$i]->getVar('slides_time');
76
        $slides[$i]['slides_status']       = $slides_arr[$i]->getVar('slides_status');
77
        $slides[$i]['slides_waiting']      = $slides_arr[$i]->getVar('slides_waiting');
78
        $slides[$i]['slides_online']       = $slides_arr[$i]->getVar('slides_online');
79
    }
80
81
    return $slides;
82
}
83
84
/**
85
 * @param $options
86
 * @return string
87
 */
88
function b_presenter_slides_edit($options)
89
{
90
    require_once XOOPS_ROOT_PATH . '/modules/presenter/class/presenter_slides.php';
91
92
    $form          = _MB_PRESENTER_DISPLAY . "\n";
93
    $form          .= "<input type='hidden' name='options[0]' value='" . $options[0] . "'>";
94
    $form          .= "<input name='options[1]' size='5' maxlength='255' value='" . $options[1] . "' type='text'>&nbsp;<br>";
95
    $form          .= _MB_PRESENTER_TITLELENGTH . " : <input name='options[2]' size='5' maxlength='255' value='" . $options[2] . "' type='text'><br><br>";
96
    $slidesHandler = xoops_getModuleHandler('slides', 'presenter');
97
    $criteria      = new CriteriaCompo();
98
    array_shift($options);
99
    array_shift($options);
100
    array_shift($options);
101
    $criteria->add(new Criteria('slides_id', 0, '!='));
102
    $criteria->setSort('slides_id');
103
    $criteria->setOrder('ASC');
104
    $slides_arr = $slidesHandler->getAll($criteria);
105
    $form       .= _MB_PRESENTER_CATTODISPLAY . "<br><select name='options[]' multiple='multiple' size='5'>";
106
    $form       .= "<option value='0' " . (false === array_search(0, $options) ? '' : 'selected') . '>' . _MB_PRESENTER_ALLCAT . '</option>';
107
    foreach (array_keys($slides_arr) as $i) {
108
        $slides_id = $slides_arr[$i]->getVar('slides_id');
109
        $form      .= "<option value='" . $slides_id . "' " . (false === array_search($slides_id, $options) ? '' : 'selected') . '>' . $slides_arr[$i]->getVar('slides_title') . '</option>';
110
    }
111
    $form .= '</select>';
112
113
    return $form;
114
}
115