StudentFollowPage::getLpAcquisition() - Code Metrics - Inspection of "Add student follow page - refs BT#18671" - chamilo/chamilo-lms - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — 1.11.x (#3859)

by Angel Fernando Quiroz

created 2021-04-26 23:47 UTC

StudentFollowPage::getLpAcquisition() A

↳ Parent: StudentFollowPage

Complexity

Conditions	5
Paths	7

Size

Total Lines	60
Code Lines	35

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
cc	5
eloc	35
c	2
b	0
f	0
nc	7
nop	5
dl	0
loc	60
rs	9.0488

How to fix Long Method

<?php

/* For licensing terms, see /license.txt */

use Chamilo\CourseBundle\Entity\CItemProperty;

/**
 * Class StudentFollowPage.
 */
class StudentFollowPage
{
    const VARIABLE_ACQUISITION = 'acquisition';
    const VARIABLE_INVISIBLE = 'invisible';

    public static function getLpSubscription(
        array $lpInfo,
        int $studentId,
        int $courseId,
        int $sessionId = 0
    ): string {
        $em = Database::getManager();

        if ($lpInfo['subscribe_users']) {
            $itemRepo = $em->getRepository(CItemProperty::class);
            $itemProperty = $itemRepo->findByUserSuscribedToItem(
                'learnpath',
                $lpInfo['iid'],
                api_get_user_entity($studentId),
                api_get_course_entity($courseId),
                api_get_session_entity($sessionId)
            );

            if (null === $itemProperty) {
                return '-';
            }

            return "{$itemProperty->getInsertUser()->getCompleteName()}<br>"
                .Display::tag(
                    'small',
                    api_convert_and_format_date($itemProperty->getInsertDate(), DATE_TIME_FORMAT_LONG)
                );
        }

        $subscriptionEvent = Event::findUserSubscriptionToCourse($studentId, $courseId, $sessionId);
        $subscriptionEvent = Event::findUserSubscriptionToCourse($studentId, $courseId, $sessionId);

        if (empty($subscriptionEvent)) {
            return '-';
        }

        $creator = api_get_user_entity($subscriptionEvent['default_user_id']);

        return "{$creator->getCompleteName()}<br>"
            .Display::tag(
                'small',
                api_convert_and_format_date($subscriptionEvent['default_date'], DATE_TIME_FORMAT_LONG)
            );
    }

    public static function getLpAcquisition(
        array $lpInfo,
        int $studentId,
        int $courseId,
        int $sessionId = 0,
        bool $allowEdit = false
    ): string {
        $lpView = learnpath::findLastView($lpInfo['iid'], $studentId, $courseId, $sessionId);

        if (empty($lpView)) {
            return '-';
        }

        $extraField = new ExtraField('lp_view');
        $field = $extraField->get_handler_field_info_by_field_variable(self::VARIABLE_ACQUISITION);

        $extraFieldValue = new ExtraFieldValue('lp_view');
        $value = $extraFieldValue->get_values_by_handler_and_field_variable($lpView['iid'], self::VARIABLE_ACQUISITION);

        $return = '';

        if (empty($value)) {
            $return .= '-';
        } else {
            $optionSelected = array_filter(
                $field['options'],
                function (array $option) use ($value) {
                    return $option['option_value'] == $value['value'];
                }
            );

            if (empty($optionSelected)) {
                $return .= '-';
            } else {
                $optionSelected = current($optionSelected);
                $valueComment = json_decode($value['comment'], true);

                $register = api_get_user_entity($valueComment['user']);

                $return .= $optionSelected['display_text'].'<br>'
                    .Display::tag('small', $register->getCompleteName()).'<br>'
                    .Display::tag(
                        'small',
                        api_convert_and_format_date($valueComment['datetime'], DATE_TIME_FORMAT_LONG)
                    ).'<br>';
            }
        }

        $editUrl = api_get_path(WEB_AJAX_PATH).'student_follow_page.ajax.php?'
            .http_build_query(['lp_view' => $lpView['iid'], 'a' => 'form_adquisition']);

        if ($allowEdit) {
            $return .= Display::url(
                Display::return_icon('edit.png', get_lang('Edit'), [], ICON_SIZE_TINY),
                $editUrl,
                ['class' => 'ajax', 'data-title' => $lpInfo['lp_name']]
            );
        }

        return '<div id="acquisition-'.$lpView['iid'].'">'.$return.'</div>';
    }

    public static function getLpVisibleScript()
    {
        $url = api_get_path(WEB_AJAX_PATH).'student_follow_page.ajax.php?'.http_build_query(['a' => 'views_invisible']);

        return "<script>$(function () {
            var chkbView = $('[name=\"chkb_view[]\"]');

            function doRequest(element, state) {
                element.prop('disabled', true);

                var views = $.makeArray(element).map(function (input) { return input.value; });

                return $.post('$url', { 'chkb_view[]': views, 'state': state }, function () { element.prop('disabled', false); });
            }

            $('[name=\"chkb_category[]\"]').on('change', function () {
                var checked = this.checked;
                var chkbs = $(this).parents('table').find('td :checkbox').each(function () { this.checked = checked; });

                doRequest(chkbs, checked);
            }).prop('checked', true);

            chkbView.on('change', function () {
                doRequest($(this), this.checked);

                $(this).parents('table').find('th :checkbox').prop(
                    'checked',
                    $.makeArray($(this).parents('table').find('td :checkbox'))
                        .map(function (input) { return input.checked; })
                        .reduce(function (acc, cur) { return acc && cur; })
                );
            }).each(function () {
                if (!this.checked) {
                    $(this).parents('table').find('th :checkbox').prop('checked', false);
                }
            });
        });</script>";
    }

    public static function getLpVisibleField(array $lpInfo, int $studentId, int $courseId, int $sessionId = 0)
    {
        $attrs = [];

        $isVisible = self::isViewVisible($lpInfo['iid'], $studentId, $courseId, $sessionId);

        if (!$isVisible) {
            $attrs['checked'] = 'checked';
        }

        return Display::input(
            'checkbox',
            'chkb_view[]',
            implode('_', [$lpInfo['iid'], $studentId, $courseId, $sessionId]),
            $attrs
        );
    }

    public static function isViewVisible(int $lpId, int $studentId, int $courseId, int $sessionId): bool
    {
        $lpView = learnpath::findLastView($lpId, $studentId, $courseId, $sessionId);

        if (empty($lpView)) {
            return true;
        }

        $extraFieldValue = new ExtraFieldValue('lp_view');
        $value = $extraFieldValue->get_values_by_handler_and_field_variable($lpView['iid'], self::VARIABLE_INVISIBLE);

        return empty($value) || empty($value['value']);
    }
}


chamilo / chamilo-lms

Pull Request — 1.11.x (#3859)

StudentFollowPage::getLpAcquisition() A

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like