diff --git a/main/badge/assign.php b/main/badge/assign.php index d54bac8324..42a8c77cc0 100644 --- a/main/badge/assign.php +++ b/main/badge/assign.php @@ -1,7 +1,9 @@ set_id($gradebookCategory->getId()); @@ -561,7 +560,7 @@ class Category implements GradebookItem if (isset($this->name) && isset($this->user_id)) { $em = Database::getManager(); - $category = new \Chamilo\CoreBundle\Entity\GradebookCategory(); + $category = new GradebookCategory(); $category->setName($this->name); $category->setDescription($this->description); $category->setUserId($this->user_id); @@ -580,7 +579,6 @@ class Category implements GradebookItem $em->flush(); $id = $category->getId(); - $this->set_id($id); if (!empty($id)) { diff --git a/main/inc/lib/auth.lib.php b/main/inc/lib/auth.lib.php index 33242a4ece..22f4eca7fa 100755 --- a/main/inc/lib/auth.lib.php +++ b/main/inc/lib/auth.lib.php @@ -1,6 +1,8 @@ $row['user_course_cat'] ); } - + return $courses; } @@ -130,7 +132,7 @@ class Auth $TABLE_COURSE_FIELD = Database::get_main_table(TABLE_EXTRA_FIELD); $TABLE_COURSE_FIELD_VALUE = Database::get_main_table(TABLE_EXTRA_FIELD_VALUES); - $extraFieldType = \Chamilo\CoreBundle\Entity\ExtraField::COURSE_FIELD_TYPE; + $extraFieldType = ExtraField::COURSE_FIELD_TYPE; // get course list auto-register $sql = "SELECT item_id @@ -417,7 +419,7 @@ class Auth WHERE tcf.variable = 'special_course' AND tcfv.value = 1 "; - + $special_course_result = Database::query($sql); if (Database::num_rows($special_course_result) > 0) { $special_course_list = array(); @@ -720,7 +722,7 @@ SQL; return $count; } - + /** * Search sessions by the tags in their courses * @param string $termTag Term for search in tags @@ -763,7 +765,7 @@ SQL; $qb->expr()->like('t.tag', ":tag") ) ->andWhere( - $qb->expr()->eq('f.extraFieldType', Chamilo\CoreBundle\Entity\ExtraField::COURSE_FIELD_TYPE) + $qb->expr()->eq('f.extraFieldType', ExtraField::COURSE_FIELD_TYPE) ) ->setFirstResult($limit['start']) ->setMaxResults($limit['length']) diff --git a/plugin/buycourses/src/buy_course_plugin.class.php b/plugin/buycourses/src/buy_course_plugin.class.php index 7598718f2c..4481482975 100644 --- a/plugin/buycourses/src/buy_course_plugin.class.php +++ b/plugin/buycourses/src/buy_course_plugin.class.php @@ -1,5 +1,9 @@ $this->get_lang('SaleStatusCompleted') ]; } - + /** * Get the statuses for Payouts * @return array @@ -1203,7 +1209,7 @@ class BuyCoursesPlugin extends Plugin ] ); } - + /** * Get a list of sales by the user id * @param int $id The user id @@ -1239,11 +1245,11 @@ class BuyCoursesPlugin extends Plugin /** * Convert the course info to array with necessary course data for save item - * @param \Chamilo\CoreBundle\Entity\Course $course + * @param Course $course * @param array $defaultCurrency Optional. Currency data * @return array */ - public function getCourseForConfiguration(\Chamilo\CoreBundle\Entity\Course $course, $defaultCurrency = null) + public function getCourseForConfiguration(Course $course, $defaultCurrency = null) { $courseItem = [ 'item_id' => null, @@ -1271,11 +1277,11 @@ class BuyCoursesPlugin extends Plugin /** * Convert the session info to array with necessary session data for save item - * @param Chamilo\CoreBundle\Entity\Session $session The session data + * @param Session $session The session data * @param array $defaultCurrency Optional. Currency data * @return array */ - public function getSessionForConfiguration(Chamilo\CoreBundle\Entity\Session $session, $defaultCurrency = null) + public function getSessionForConfiguration(Session $session, $defaultCurrency = null) { $buyItemTable = Database::get_main_table(BuyCoursesPlugin::TABLE_ITEM); $buyCurrencyTable = Database::get_main_table(BuyCoursesPlugin::TABLE_CURRENCY); @@ -1448,10 +1454,10 @@ class BuyCoursesPlugin extends Plugin /** * Check if a course is valid for sale - * @param Chamilo\CoreBundle\Entity\Course $course The course + * @param Course $course The course * @return boolean */ - public function isValidCourse(Chamilo\CoreBundle\Entity\Course $course) + public function isValidCourse(Course $course) { $courses = $this->getCourses(); @@ -1463,7 +1469,7 @@ class BuyCoursesPlugin extends Plugin return false; } - + /** * Gets the beneficiaries with commissions and current paypal accounts by sale * @param int $saleId The sale ID @@ -1471,17 +1477,17 @@ class BuyCoursesPlugin extends Plugin */ public function getBeneficiariesBySale($saleId) { - + $userTable = Database::get_main_table(TABLE_MAIN_USER); - + $beneficiaries = []; $sale = $this->getSale($saleId); $item = $this->getItemByProduct($sale['product_id'], $sale['product_type']); $itemBeneficiaries = $this->getItemBeneficiaries($item['id']); return $itemBeneficiaries; - + } - + /** * gets all payouts * @param int $status - default 0 - pending @@ -1499,7 +1505,7 @@ class BuyCoursesPlugin extends Plugin $userTable = Database::get_main_table(TABLE_MAIN_USER); $extraFieldTable = Database::get_main_table(TABLE_EXTRA_FIELD); $extraFieldValues = Database::get_main_table(TABLE_EXTRA_FIELD_VALUES); - + $paypalExtraField = Database::select( "*", $extraFieldTable, @@ -1508,11 +1514,11 @@ class BuyCoursesPlugin extends Plugin ], 'first' ); - + if (!$paypalExtraField) { return false; } - + $innerJoins = " INNER JOIN $userTable u ON p.user_id = u.id INNER JOIN $saleTable s ON s.id = p.sale_id @@ -1520,7 +1526,7 @@ class BuyCoursesPlugin extends Plugin LEFT JOIN $extraFieldValues efv ON p.user_id = efv.item_id AND field_id = " . intval($paypalExtraField['id']) . " "; - + $payouts = Database::select( "p.* , u.firstname, u.lastname, efv.value as paypal_account, s.reference as sale_reference, s.price as item_price, c.iso_code", "$payoutsTable p $innerJoins", @@ -1529,10 +1535,10 @@ class BuyCoursesPlugin extends Plugin ], $typeResult ); - + return $payouts; } - + /** * Verify if the beneficiary have a paypal account * @param int $userId @@ -1542,7 +1548,7 @@ class BuyCoursesPlugin extends Plugin { $extraFieldTable = Database::get_main_table(TABLE_EXTRA_FIELD); $extraFieldValues = Database::get_main_table(TABLE_EXTRA_FIELD_VALUES); - + $paypalExtraField = Database::select( "*", $extraFieldTable, @@ -1551,13 +1557,13 @@ class BuyCoursesPlugin extends Plugin ], 'first' ); - + if (!$paypalExtraField) { return false; } - + $paypalFieldId = $paypalExtraField['id']; - + $paypalAccount = Database::select( "value", $extraFieldValues, @@ -1566,18 +1572,18 @@ class BuyCoursesPlugin extends Plugin ], 'first' ); - + if (!$paypalAccount) { return false; } - + if ($paypalAccount['value'] === '') { return false; } - + return true; } - + /** * Register the users payouts * @param int $saleId The sale ID @@ -1590,8 +1596,8 @@ class BuyCoursesPlugin extends Plugin $sale = $this->getSale($saleId); $teachersCommission = number_format((floatval($sale['price']) * intval($platformCommission['commission']))/100, 2); - - + + $beneficiaries = $this->getBeneficiariesBySale($saleId); foreach ($beneficiaries as $beneficiary) { Database::insert( @@ -1607,7 +1613,7 @@ class BuyCoursesPlugin extends Plugin ); } } - + /** * Register the users payouts * @param int $payoutId The payout ID @@ -1617,15 +1623,15 @@ class BuyCoursesPlugin extends Plugin public function setStatusPayouts($payoutId, $status) { $payoutsTable = Database::get_main_table(BuyCoursesPlugin::TABLE_PAYPAL_PAYOUTS); - + Database::update( $payoutsTable, ['status' => intval($status)], ['id = ?' => intval($payoutId)] ); - + } - + /** * Gets the stored platform commission params * @return array @@ -1639,7 +1645,7 @@ class BuyCoursesPlugin extends Plugin 'first' ); } - + /** * Update the platform commission * @param int $params platform commission