ZombieReport::action_delete() - Code Metrics - Inspection of "Minor - update from 1.11.x" - chamilo/chamilo-lms - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 2c1297...2ffb1a )

by Julito

created 2018-11-26 12:00 UTC

ZombieReport::action_delete() A

↳ Parent: ZombieReport::get_data()

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	1
nc	1
nop	1
dl	0
loc	3
rs	10
c	0
b	0
f	0

<?php

/**
 * Description of zombie_report.
 *
 * @copyright (c) 2012 University of Geneva
 * @license GNU General Public License - http://www.gnu.org/copyleft/gpl.html
 * @author Laurent Opprecht <[email protected]>
 */
class ZombieReport implements Countable
{
    protected $additional_parameters = [];

    protected $parameters_form = null;

    public function __construct($additional_parameters = [])
    {
        $this->additional_parameters = $additional_parameters;
    }

    /**
     * @return ZombieReport
     */
    public static function create($additional_parameters = [])
    {
        return new self($additional_parameters);
    }

    public function get_additional_parameters()
    {
        return $this->additional_parameters;
    }

    public function get_parameters()
    {
        $result = [
            'items' => [
                [
                    'name' => 'ceiling',
                    'label' => get_lang('LastAccess'),
                    'type' => 'date_picker',
                    'default' => $this->get_ceiling('Y-m-d'),
                    'rules' => [
                        [
                            'type' => 'date',
                            'message' => get_lang('Date'),
                        ],
                    ],
                ],
                [
                    'name' => 'active_only',
                    'label' => get_lang('ActiveOnly'),
                    'type' => 'checkbox',
                    'default' => $this->get_active_only(),
                ],
                [
                    'name' => 'submit_button',
                    'type' => 'button',
                    'value' => get_lang('Search'),
                    'attributes' => ['class' => 'search'],
                ],
            ],
        ];

        return $result;
    }

    /**
     * @return FormValidator
     */
    public function get_parameters_form()
    {
        $form = new FormValidator(
            'zombie_report_parameters',
            'get',
                null,
                null,
            ['class' => 'well form-horizontal form-search']
            );

        $form->addDatePicker('ceiling', get_lang('LastAccess'));
        $form->addCheckBox('active_only', get_lang('ActiveOnly'));
        $form->addButtonSearch(get_lang('Search'));

        $params = [
            'active_only' => $this->get_active_only(),
            'ceiling' => $this->get_ceiling('Y-m-d'),
        ];
        $form->setDefaults($params);
        $additional = $this->get_additional_parameters();
        foreach ($additional as $key => $value) {
            $value = Security::remove_XSS($value);
            $form->addHidden($key, $value);
        }

        return $form;
    }

    public function display_parameters($return = false)
    {
        $form = $this->get_parameters_form();
        $result = $form->returnForm();

        if ($return) {
            return $result;
        } else {
            echo $result;
        }
    }

    public function is_valid()
    {
        $form = $this->get_parameters_form();

        return $form->isSubmitted() == false || $form->validate();
    }

    public function get_ceiling($format = null)
    {
        $result = Request::get('ceiling');
        $result = $result ? $result : ZombieManager::last_year();

        $result = is_array($result) && count($result) == 1 ? reset($result) : $result;
        $result = is_array($result) ? mktime(0, 0, 0, $result['F'], $result['d'], $result['Y']) : $result;
        $result = is_numeric($result) ? (int) $result : $result;
        $result = is_string($result) ? strtotime($result) : $result;
        if ($format) {
            $result = date($format, $result);
        }

        return $result;
    }

    public function get_active_only()
    {
        $result = Request::get('active_only', false);
        $result = $result === 'true' ? true : $result;
        $result = $result === 'false' ? false : $result;
        $result = (bool) $result;

        return $result;
    }

    public function get_action()
    {
        /**
         * todo check token.
         */
        $check = Security::check_token('post');
        Security::clear_token();
        if (!$check) {
            return 'display';
        }

        return Request::post('action', 'display');
    }

    public function perform_action()
    {
        $ids = Request::post('id');
        if (empty($ids)) {
            return $ids;
        }

        $action = $this->get_action();
        switch ($action) {
            case 'activate':
                return UserManager::activate_users($ids);
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
            case 'deactivate':
                return UserManager::deactivate_users($ids);
                break;
            case 'delete':
                return UserManager::delete_users($ids);
        }

        return false;
    }

    public function count()
    {
        $ceiling = $this->get_ceiling();
        $active_only = $this->get_active_only();
        $items = ZombieManager::listZombies($ceiling, $active_only, null, null);

        return count($items);
    }

    public function get_data($from, $count, $column, $direction)
    {
        $ceiling = $this->get_ceiling();
        $active_only = $this->get_active_only();
        $items = ZombieManager::listZombies($ceiling, $active_only, $from, $count, $column, $direction);
        $result = [];
        foreach ($items as $item) {
            $row = [];
            $row[] = $item['user_id'];
            $row[] = $item['official_code'];
            $row[] = $item['firstname'];
            $row[] = $item['lastname'];
            $row[] = $item['username'];
            $row[] = $item['email'];
            $row[] = $item['status'];
            $row[] = $item['auth_source'];
            $row[] = api_format_date($item['registration_date'], DATE_FORMAT_SHORT);
            $row[] = api_format_date($item['login_date'], DATE_FORMAT_SHORT);
            $row[] = $item['active'];
            $result[] = $row;
        }

        return $result;
    }

    public function display_data($return = false)
    {
        $count = [$this, 'count'];
        $data = [$this, 'get_data'];

        $parameters = [];
        $parameters['sec_token'] = Security::get_token();
        $parameters['ceiling'] = $this->get_ceiling();
        $parameters['active_only'] = $this->get_active_only() ? 'true' : 'false';
        $additional_parameters = $this->get_additional_parameters();
        $parameters = array_merge($additional_parameters, $parameters);

        $table = new SortableTable('zombie_users', $count, $data, 1, 50);
        $table->set_additional_parameters($parameters);

        $col = 0;
        $table->set_header($col++, '', false);
        $table->set_header($col++, get_lang('OfficialCode'));
        $table->set_header($col++, get_lang('FirstName'));
        $table->set_header($col++, get_lang('LastName'));
        $table->set_header($col++, get_lang('LoginName'));
        $table->set_header($col++, get_lang('Email'));
        $table->set_header($col++, get_lang('Profile'));
        $table->set_header($col++, get_lang('AuthenticationSource'));
        $table->set_header($col++, get_lang('RegisteredDate'));
        $table->set_header($col++, get_lang('LastAccess'), false);
        $table->set_header($col, get_lang('Active'), false);

        $table->set_column_filter(5, [$this, 'format_email']);
        $table->set_column_filter(6, [$this, 'format_status']);
        $table->set_column_filter(10, [$this, 'format_active']);

        $table->set_form_actions([
            'activate' => get_lang('Activate'),
            'deactivate' => get_lang('Deactivate'),
            'delete' => get_lang('Delete'),
        ]);

        if ($return) {
            return $table->return_table();
        } else {
            echo $table->return_table();
        }
    }

    /**
     * Table formatter for the active column.
     *
     * @param string $active
     *
     * @return string
     */
    public function format_active($active)
    {
        $active = $active == '1';
        if ($active) {
            $image = 'accept';
            $text = get_lang('Yes');
        } else {
            $image = 'error';
            $text = get_lang('No');
        }

        $result = Display::return_icon($image.'.png', $text);

        return $result;
    }

    public function format_status($status)
    {
        $statusname = api_get_status_langvars();

        return $statusname[$status];
    }

    public function format_email($email)
    {
        return Display::encrypted_mailto_link($email, $email);
    }

    public function display($return = false)
    {
        $result = $this->display_parameters($return);
        $valid = $this->perform_action();

        if ($valid) {
            echo Display::return_message(get_lang('Updated'), 'confirmation');
        }

        $result .= $this->display_data($return);

        if ($return) {
            return $result;
        }
    }
}


1			<?php
2
3			/**
4			* Description of zombie_report.
5			*
6			* @copyright (c) 2012 University of Geneva
7			* @license GNU General Public License - http://www.gnu.org/copyleft/gpl.html
8			* @author Laurent Opprecht <[email protected]>
9			*/
10			class ZombieReport implements Countable
11			{
12			protected $additional_parameters = [];
13
14			protected $parameters_form = null;
15
16			public function __construct($additional_parameters = [])
17			{
18			$this->additional_parameters = $additional_parameters;
19			}
20
21			/**
22			* @return ZombieReport
23			*/
24			public static function create($additional_parameters = [])
25			{
26			return new self($additional_parameters);
27			}
28
29			public function get_additional_parameters()
30			{
31			return $this->additional_parameters;
32			}
33
34			public function get_parameters()
35			{
36			$result = [
37			'items' => [
38			[
39			'name' => 'ceiling',
40			'label' => get_lang('LastAccess'),
41			'type' => 'date_picker',
42			'default' => $this->get_ceiling('Y-m-d'),
43			'rules' => [
44			[
45			'type' => 'date',
46			'message' => get_lang('Date'),
47			],
48			],
49			],
50			[
51			'name' => 'active_only',
52			'label' => get_lang('ActiveOnly'),
53			'type' => 'checkbox',
54			'default' => $this->get_active_only(),
55			],
56			[
57			'name' => 'submit_button',
58			'type' => 'button',
59			'value' => get_lang('Search'),
60			'attributes' => ['class' => 'search'],
61			],
62			],
63			];
64
65			return $result;
66			}
67
68			/**
69			* @return FormValidator
70			*/
71			public function get_parameters_form()
72			{
73			$form = new FormValidator(
74			'zombie_report_parameters',
75			'get',
76			null,
77			null,
78			['class' => 'well form-horizontal form-search']
79			);
80
81			$form->addDatePicker('ceiling', get_lang('LastAccess'));
82			$form->addCheckBox('active_only', get_lang('ActiveOnly'));
83			$form->addButtonSearch(get_lang('Search'));
84
85			$params = [
86			'active_only' => $this->get_active_only(),
87			'ceiling' => $this->get_ceiling('Y-m-d'),
88			];
89			$form->setDefaults($params);
90			$additional = $this->get_additional_parameters();
91			foreach ($additional as $key => $value) {
92			$value = Security::remove_XSS($value);
93			$form->addHidden($key, $value);
94			}
95
96			return $form;
97			}
98
99			public function display_parameters($return = false)
100			{
101			$form = $this->get_parameters_form();
102			$result = $form->returnForm();
103
104			if ($return) {
105			return $result;
106			} else {
107			echo $result;
108			}
109			}
110
111			public function is_valid()
112			{
113			$form = $this->get_parameters_form();
114
115			return $form->isSubmitted() == false \|\| $form->validate();
116			}
117
118			public function get_ceiling($format = null)
119			{
120			$result = Request::get('ceiling');
121			$result = $result ? $result : ZombieManager::last_year();
122
123			$result = is_array($result) && count($result) == 1 ? reset($result) : $result;
124			$result = is_array($result) ? mktime(0, 0, 0, $result['F'], $result['d'], $result['Y']) : $result;
125			$result = is_numeric($result) ? (int) $result : $result;
126			$result = is_string($result) ? strtotime($result) : $result;
127			if ($format) {
128			$result = date($format, $result);
129			}
130
131			return $result;
132			}
133
134			public function get_active_only()
135			{
136			$result = Request::get('active_only', false);
137			$result = $result === 'true' ? true : $result;
138			$result = $result === 'false' ? false : $result;
139			$result = (bool) $result;
140
141			return $result;
142			}
143
144			public function get_action()
145			{
146			/**
147			* todo check token.
148			*/
149			$check = Security::check_token('post');
150			Security::clear_token();
151			if (!$check) {
152			return 'display';
153			}
154
155			return Request::post('action', 'display');
156			}
157
158			public function perform_action()
159			{
160			$ids = Request::post('id');
161			if (empty($ids)) {
162			return $ids;
163			}
164
165			$action = $this->get_action();
166			switch ($action) {
167			case 'activate':
168			return UserManager::activate_users($ids);
169			break;
			0 ignored issues – show Unused Code introduced 2018-11-22 09:29 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
170			case 'deactivate':
171			return UserManager::deactivate_users($ids);
172			break;
173			case 'delete':
174			return UserManager::delete_users($ids);
175			}
176
177			return false;
178			}
179
180			public function count()
181			{
182			$ceiling = $this->get_ceiling();
183			$active_only = $this->get_active_only();
184			$items = ZombieManager::listZombies($ceiling, $active_only, null, null);
185
186			return count($items);
187			}
188
189			public function get_data($from, $count, $column, $direction)
190			{
191			$ceiling = $this->get_ceiling();
192			$active_only = $this->get_active_only();
193			$items = ZombieManager::listZombies($ceiling, $active_only, $from, $count, $column, $direction);
194			$result = [];
195			foreach ($items as $item) {
196			$row = [];
197			$row[] = $item['user_id'];
198			$row[] = $item['official_code'];
199			$row[] = $item['firstname'];
200			$row[] = $item['lastname'];
201			$row[] = $item['username'];
202			$row[] = $item['email'];
203			$row[] = $item['status'];
204			$row[] = $item['auth_source'];
205			$row[] = api_format_date($item['registration_date'], DATE_FORMAT_SHORT);
206			$row[] = api_format_date($item['login_date'], DATE_FORMAT_SHORT);
207			$row[] = $item['active'];
208			$result[] = $row;
209			}
210
211			return $result;
212			}
213
214			public function display_data($return = false)
215			{
216			$count = [$this, 'count'];
217			$data = [$this, 'get_data'];
218
219			$parameters = [];
220			$parameters['sec_token'] = Security::get_token();
221			$parameters['ceiling'] = $this->get_ceiling();
222			$parameters['active_only'] = $this->get_active_only() ? 'true' : 'false';
223			$additional_parameters = $this->get_additional_parameters();
224			$parameters = array_merge($additional_parameters, $parameters);
225
226			$table = new SortableTable('zombie_users', $count, $data, 1, 50);
227			$table->set_additional_parameters($parameters);
228
229			$col = 0;
230			$table->set_header($col++, '', false);
231			$table->set_header($col++, get_lang('OfficialCode'));
232			$table->set_header($col++, get_lang('FirstName'));
233			$table->set_header($col++, get_lang('LastName'));
234			$table->set_header($col++, get_lang('LoginName'));
235			$table->set_header($col++, get_lang('Email'));
236			$table->set_header($col++, get_lang('Profile'));
237			$table->set_header($col++, get_lang('AuthenticationSource'));
238			$table->set_header($col++, get_lang('RegisteredDate'));
239			$table->set_header($col++, get_lang('LastAccess'), false);
240			$table->set_header($col, get_lang('Active'), false);
241
242			$table->set_column_filter(5, [$this, 'format_email']);
243			$table->set_column_filter(6, [$this, 'format_status']);
244			$table->set_column_filter(10, [$this, 'format_active']);
245
246			$table->set_form_actions([
247			'activate' => get_lang('Activate'),
248			'deactivate' => get_lang('Deactivate'),
249			'delete' => get_lang('Delete'),
250			]);
251
252			if ($return) {
253			return $table->return_table();
254			} else {
255			echo $table->return_table();
256			}
257			}
258
259			/**
260			* Table formatter for the active column.
261			*
262			* @param string $active
263			*
264			* @return string
265			*/
266			public function format_active($active)
267			{
268			$active = $active == '1';
269			if ($active) {
270			$image = 'accept';
271			$text = get_lang('Yes');
272			} else {
273			$image = 'error';
274			$text = get_lang('No');
275			}
276
277			$result = Display::return_icon($image.'.png', $text);
278
279			return $result;
280			}
281
282			public function format_status($status)
283			{
284			$statusname = api_get_status_langvars();
285
286			return $statusname[$status];
287			}
288
289			public function format_email($email)
290			{
291			return Display::encrypted_mailto_link($email, $email);
292			}
293
294			public function display($return = false)
295			{
296			$result = $this->display_parameters($return);
297			$valid = $this->perform_action();
298
299			if ($valid) {
300			echo Display::return_message(get_lang('Updated'), 'confirmation');
301			}
302
303			$result .= $this->display_data($return);
304
305			if ($return) {
306			return $result;
307			}
308			}
309			}
310

chamilo / chamilo-lms

Push — master ( 2c1297...2ffb1a )

ZombieReport::action_delete() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like