Issues in about.php (1.11.x) - Issues in 1.11.x - chamilo/chamilo-lms - Measure and Improve Code Quality continuously with Scrutinizer

Issues (2160)

main/course_info/about.php (2 issues)

Labels

Bug 1

Severity

<?php

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

use Chamilo\CoreBundle\Entity\CourseRelUser;
use Chamilo\CoreBundle\Entity\ExtraField;
.
|-- OtherDir
|   |-- Bar.php
|   `-- Foo.php
`-- SomeDir
    `-- Foo.php
use Chamilo\CoreBundle\Entity\Repository\SequenceResourceRepository;
use Chamilo\CoreBundle\Entity\SequenceResource;
use Chamilo\CourseBundle\Entity\CCourseDescription;

/**
 * Course about page
 * Show information about a course.
 *
 * @author Alex Aragon Calixto <[email protected]>
 */
$cidReset = true;

require_once __DIR__.'/../inc/global.inc.php';

if ((api_get_setting('course_catalog_published') != 'true' && api_is_anonymous()) || api_get_configuration_value('course_about_block_all_access') == 'true') {
    api_not_allowed(true);
}

$courseId = isset($_GET['course_id']) ? (int) $_GET['course_id'] : 0;

if (empty($courseId)) {
    api_not_allowed(true);
}

$token = Security::get_existing_token();
$em = Database::getManager();

$userId = api_get_user_id();
$course = api_get_course_entity($courseId);

if (!$course) {

    api_not_allowed(true);
}

$userRepo = UserManager::getRepository();
$fieldsRepo = $em->getRepository('ChamiloCoreBundle:ExtraField');
$fieldTagsRepo = $em->getRepository('ChamiloCoreBundle:ExtraFieldRelTag');

/** @var CCourseDescription $courseDescription */
$courseDescriptionTools = $em->getRepository('ChamiloCourseBundle:CCourseDescription')
    ->findBy(
        [
            'cId' => $course->getId(),
            'sessionId' => 0,
        ],
        [
            'id' => 'DESC',
            'descriptionType' => 'ASC',
        ]
    );

$courseValues = new ExtraFieldValue('course');
$userValues = new ExtraFieldValue('user');

$urlCourse = api_get_path(WEB_PATH)."course/$courseId/about";
$courseTeachers = $course->getTeachers();
$teachersData = [];

/** @var CourseRelUser $teacherSubscription */
foreach ($courseTeachers as $teacherSubscription) {
    $teacher = $teacherSubscription->getUser();
    $userData = [
        'complete_name' => UserManager::formatUserFullName($teacher),
        'image' => UserManager::getUserPicture(
            $teacher->getId(),
            USER_IMAGE_SIZE_ORIGINAL
        ),
        'diploma' => $teacher->getDiplomas(),
        'openarea' => $teacher->getOpenarea(),
    ];

    $teachersData[] = $userData;
}

$tagField = $fieldsRepo->findOneBy([
    'extraFieldType' => ExtraField::COURSE_FIELD_TYPE,
    'variable' => 'tags',
]);

$courseTags = [];

if (!is_null($tagField)) {
    $courseTags = $fieldTagsRepo->getTags($tagField, $courseId);
}

$courseDescription = $courseObjectives = $courseTopics = $courseMethodology = $courseMaterial = $courseResources = $courseAssessment = '';
$courseCustom = [];
foreach ($courseDescriptionTools as $descriptionTool) {
    switch ($descriptionTool->getDescriptionType()) {
        case CCourseDescription::TYPE_DESCRIPTION:
            $courseDescription = Security::remove_XSS($descriptionTool->getContent());
            break;
        case CCourseDescription::TYPE_OBJECTIVES:
            $courseObjectives = $descriptionTool;
            break;
        case CCourseDescription::TYPE_TOPICS:
            $courseTopics = $descriptionTool;
            break;
        case CCourseDescription::TYPE_METHODOLOGY:
            $courseMethodology = $descriptionTool;
            break;
        case CCourseDescription::TYPE_COURSE_MATERIAL:
            $courseMaterial = $descriptionTool;
            break;
        case CCourseDescription::TYPE_RESOURCES:
            $courseResources = $descriptionTool;
            break;
        case CCourseDescription::TYPE_ASSESSMENT:
            $courseAssessment = $descriptionTool;
            break;
        case CCourseDescription::TYPE_CUSTOM:
            $courseCustom[] = $descriptionTool;
            break;
    }
}

$topics = [
    'objectives' => $courseObjectives,
    'topics' => $courseTopics,
    'methodology' => $courseMethodology,
    'material' => $courseMaterial,
    'resources' => $courseResources,
    'assessment' => $courseAssessment,
    'custom' => array_reverse($courseCustom),
];

$subscriptionUser = CourseManager::is_user_subscribed_in_course($userId, $course->getCode());

$allowSubscribe = false;
if ($course->getSubscribe() || api_is_platform_admin()) {
    $allowSubscribe = true;
}
$plugin = BuyCoursesPlugin::create();
$checker = $plugin->isEnabled();
$courseIsPremium = null;
if ($checker) {
    $courseIsPremium = $plugin->getItemByProduct(
        $courseId,
        BuyCoursesPlugin::PRODUCT_TYPE_COURSE
    );
}

$courseItem = [
    'code' => $course->getCode(),
    'visibility' => $course->getVisibility(),
    'title' => $course->getTitle(),
    'description' => $courseDescription,
    'image' => CourseManager::getPicturePath($course, true),
    'syllabus' => $topics,
    'tags' => $courseTags,
    'teachers' => $teachersData,
    'extra_fields' => $courseValues->getAllValuesForAnItem(
        $course->getId(),
        null,
        true
    ),
    'subscription' => $subscriptionUser,
];

$metaInfo = '<meta property="og:url" content="'.$urlCourse.'" />';
$metaInfo .= '<meta property="og:type" content="website" />';
$metaInfo .= '<meta property="og:title" content="'.$courseItem['title'].'" />';
$metaInfo .= '<meta property="og:description" content="'.strip_tags($courseDescription).'" />';
$metaInfo .= '<meta property="og:image" content="'.$courseItem['image'].'" />';

$htmlHeadXtra[] = $metaInfo;
$htmlHeadXtra[] = api_get_asset('readmore-js/readmore.js');

/** @var SequenceResourceRepository $sequenceResourceRepo */
$sequenceResourceRepo = $em->getRepository('ChamiloCoreBundle:SequenceResource');
$requirements = $sequenceResourceRepo->getRequirements(
    $course->getId(),
    SequenceResource::COURSE_TYPE
);

$hasRequirements = false;
foreach ($requirements as $sequence) {
    if (!empty($sequence['requirements'])) {
        $hasRequirements = true;
        break;
    }
}

if ($hasRequirements) {
    $sequenceList = $sequenceResourceRepo->checkRequirementsForUser($requirements, SequenceResource::COURSE_TYPE, $userId);
    $allowSubscribe = $sequenceResourceRepo->checkSequenceAreCompleted($sequenceList);
}

$template = new Template($course->getTitle(), true, true, false, true, false);

$template->assign('course', $courseItem);
$essence = Essence\Essence::instance();
$template->assign('essence', $essence);
$template->assign('is_premium', $courseIsPremium);
$template->assign('allow_subscribe', $allowSubscribe);
$template->assign('token', $token);
$template->assign('url', $urlCourse);
$template->assign(
    'subscribe_button',
    CoursesAndSessionsCatalog::getRequirements(
        $course->getId(),
        SequenceResource::COURSE_TYPE,
        true,
        true
    )
);
$template->assign('has_requirements', $hasRequirements);
$template->assign('sequences', $requirements);

$layout = $template->get_template('course_home/about.tpl');
$content = $template->fetch($layout);
$template->assign('content', $content);
$template->display_one_col_template();


1			<?php
2
3			/* For licensing terms, see /license.txt */
4
5			use Chamilo\CoreBundle\Entity\CourseRelUser;
6			use Chamilo\CoreBundle\Entity\ExtraField;
			0 ignored issues – show Bug introduced 2018-01-05 13:33 UTC by Report Bug Copy Issue Report Show Similar Issues like this This use statement conflicts with another class in this namespace, `ExtraField`. Consider defining an alias. Let?s assume that you have a directory layout like this: . \|-- OtherDir \| \|-- Bar.php \| `-- Foo.php `-- SomeDir `-- Foo.php and let?s assume the following content of `Bar.php`: // Bar.php namespace OtherDir; use SomeDir\Foo; // This now conflicts the class OtherDir\Foo If both files `OtherDir/Foo.php` and `SomeDir/Foo.php` are loaded in the same runtime, you will see a PHP error such as the following: PHP Fatal error: Cannot use SomeDir\Foo as Foo because the name is already in use in OtherDir/Foo.php However, as `OtherDir/Foo.php` does not necessarily have to be loaded and the error is only triggered if it is loaded before `OtherDir/Bar.php`, this problem might go unnoticed for a while. In order to prevent this error from surfacing, you must import the namespace with a different alias: // Bar.php namespace OtherDir; use SomeDir\Foo as SomeDirFoo; // There is no conflict anymore. Loading history...
7			use Chamilo\CoreBundle\Entity\Repository\SequenceResourceRepository;
8			use Chamilo\CoreBundle\Entity\SequenceResource;
9			use Chamilo\CourseBundle\Entity\CCourseDescription;
10
11			/**
12			* Course about page
13			* Show information about a course.
14			*
15			* @author Alex Aragon Calixto <[email protected]>
16			*/
17			$cidReset = true;
18
19			require_once __DIR__.'/../inc/global.inc.php';
20
21			if ((api_get_setting('course_catalog_published') != 'true' && api_is_anonymous()) \|\| api_get_configuration_value('course_about_block_all_access') == 'true') {
22			api_not_allowed(true);
23			}
24
25			$courseId = isset($_GET['course_id']) ? (int) $_GET['course_id'] : 0;
26
27			if (empty($courseId)) {
28			api_not_allowed(true);
29			}
30
31			$token = Security::get_existing_token();
32			$em = Database::getManager();
33
34			$userId = api_get_user_id();
35			$course = api_get_course_entity($courseId);
36
37			if (!$course) {
			0 ignored issues – show introduced 2020-02-12 13:06 UTC by Report Bug Copy Issue Report Show Similar Issues like this `$course` is of type `Chamilo\CoreBundle\Entity\Course`, thus it always evaluated to `true`. Loading history...
38			api_not_allowed(true);
39			}
40
41			$userRepo = UserManager::getRepository();
42			$fieldsRepo = $em->getRepository('ChamiloCoreBundle:ExtraField');
43			$fieldTagsRepo = $em->getRepository('ChamiloCoreBundle:ExtraFieldRelTag');
44
45			/** @var CCourseDescription $courseDescription */
46			$courseDescriptionTools = $em->getRepository('ChamiloCourseBundle:CCourseDescription')
47			->findBy(
48			[
49			'cId' => $course->getId(),
50			'sessionId' => 0,
51			],
52			[
53			'id' => 'DESC',
54			'descriptionType' => 'ASC',
55			]
56			);
57
58			$courseValues = new ExtraFieldValue('course');
59			$userValues = new ExtraFieldValue('user');
60
61			$urlCourse = api_get_path(WEB_PATH)."course/$courseId/about";
62			$courseTeachers = $course->getTeachers();
63			$teachersData = [];
64
65			/** @var CourseRelUser $teacherSubscription */
66			foreach ($courseTeachers as $teacherSubscription) {
67			$teacher = $teacherSubscription->getUser();
68			$userData = [
69			'complete_name' => UserManager::formatUserFullName($teacher),
70			'image' => UserManager::getUserPicture(
71			$teacher->getId(),
72			USER_IMAGE_SIZE_ORIGINAL
73			),
74			'diploma' => $teacher->getDiplomas(),
75			'openarea' => $teacher->getOpenarea(),
76			];
77
78			$teachersData[] = $userData;
79			}
80
81			$tagField = $fieldsRepo->findOneBy([
82			'extraFieldType' => ExtraField::COURSE_FIELD_TYPE,
83			'variable' => 'tags',
84			]);
85
86			$courseTags = [];
87
88			if (!is_null($tagField)) {
89			$courseTags = $fieldTagsRepo->getTags($tagField, $courseId);
90			}
91
92			$courseDescription = $courseObjectives = $courseTopics = $courseMethodology = $courseMaterial = $courseResources = $courseAssessment = '';
93			$courseCustom = [];
94			foreach ($courseDescriptionTools as $descriptionTool) {
95			switch ($descriptionTool->getDescriptionType()) {
96			case CCourseDescription::TYPE_DESCRIPTION:
97			$courseDescription = Security::remove_XSS($descriptionTool->getContent());
98			break;
99			case CCourseDescription::TYPE_OBJECTIVES:
100			$courseObjectives = $descriptionTool;
101			break;
102			case CCourseDescription::TYPE_TOPICS:
103			$courseTopics = $descriptionTool;
104			break;
105			case CCourseDescription::TYPE_METHODOLOGY:
106			$courseMethodology = $descriptionTool;
107			break;
108			case CCourseDescription::TYPE_COURSE_MATERIAL:
109			$courseMaterial = $descriptionTool;
110			break;
111			case CCourseDescription::TYPE_RESOURCES:
112			$courseResources = $descriptionTool;
113			break;
114			case CCourseDescription::TYPE_ASSESSMENT:
115			$courseAssessment = $descriptionTool;
116			break;
117			case CCourseDescription::TYPE_CUSTOM:
118			$courseCustom[] = $descriptionTool;
119			break;
120			}
121			}
122
123			$topics = [
124			'objectives' => $courseObjectives,
125			'topics' => $courseTopics,
126			'methodology' => $courseMethodology,
127			'material' => $courseMaterial,
128			'resources' => $courseResources,
129			'assessment' => $courseAssessment,
130			'custom' => array_reverse($courseCustom),
131			];
132
133			$subscriptionUser = CourseManager::is_user_subscribed_in_course($userId, $course->getCode());
134
135			$allowSubscribe = false;
136			if ($course->getSubscribe() \|\| api_is_platform_admin()) {
137			$allowSubscribe = true;
138			}
139			$plugin = BuyCoursesPlugin::create();
140			$checker = $plugin->isEnabled();
141			$courseIsPremium = null;
142			if ($checker) {
143			$courseIsPremium = $plugin->getItemByProduct(
144			$courseId,
145			BuyCoursesPlugin::PRODUCT_TYPE_COURSE
146			);
147			}
148
149			$courseItem = [
150			'code' => $course->getCode(),
151			'visibility' => $course->getVisibility(),
152			'title' => $course->getTitle(),
153			'description' => $courseDescription,
154			'image' => CourseManager::getPicturePath($course, true),
155			'syllabus' => $topics,
156			'tags' => $courseTags,
157			'teachers' => $teachersData,
158			'extra_fields' => $courseValues->getAllValuesForAnItem(
159			$course->getId(),
160			null,
161			true
162			),
163			'subscription' => $subscriptionUser,
164			];
165
166			$metaInfo = '<meta property="og:url" content="'.$urlCourse.'" />';
167			$metaInfo .= '<meta property="og:type" content="website" />';
168			$metaInfo .= '<meta property="og:title" content="'.$courseItem['title'].'" />';
169			$metaInfo .= '<meta property="og:description" content="'.strip_tags($courseDescription).'" />';
170			$metaInfo .= '<meta property="og:image" content="'.$courseItem['image'].'" />';
171
172			$htmlHeadXtra[] = $metaInfo;
173			$htmlHeadXtra[] = api_get_asset('readmore-js/readmore.js');
174
175			/** @var SequenceResourceRepository $sequenceResourceRepo */
176			$sequenceResourceRepo = $em->getRepository('ChamiloCoreBundle:SequenceResource');
177			$requirements = $sequenceResourceRepo->getRequirements(
178			$course->getId(),
179			SequenceResource::COURSE_TYPE
180			);
181
182			$hasRequirements = false;
183			foreach ($requirements as $sequence) {
184			if (!empty($sequence['requirements'])) {
185			$hasRequirements = true;
186			break;
187			}
188			}
189
190			if ($hasRequirements) {
191			$sequenceList = $sequenceResourceRepo->checkRequirementsForUser($requirements, SequenceResource::COURSE_TYPE, $userId);
192			$allowSubscribe = $sequenceResourceRepo->checkSequenceAreCompleted($sequenceList);
193			}
194
195			$template = new Template($course->getTitle(), true, true, false, true, false);
196
197			$template->assign('course', $courseItem);
198			$essence = Essence\Essence::instance();
199			$template->assign('essence', $essence);
200			$template->assign('is_premium', $courseIsPremium);
201			$template->assign('allow_subscribe', $allowSubscribe);
202			$template->assign('token', $token);
203			$template->assign('url', $urlCourse);
204			$template->assign(
205			'subscribe_button',
206			CoursesAndSessionsCatalog::getRequirements(
207			$course->getId(),
208			SequenceResource::COURSE_TYPE,
209			true,
210			true
211			)
212			);
213			$template->assign('has_requirements', $hasRequirements);
214			$template->assign('sequences', $requirements);
215
216			$layout = $template->get_template('course_home/about.tpl');
217			$content = $template->fetch($layout);
218			$template->assign('content', $content);
219			$template->display_one_col_template();
220

chamilo / chamilo-lms

Issues (2160)

main/course_info/about.php (2 issues)

Labels

Severity

Introduced By

Duplication Side-by-Side

Filter issues like