src/CoreBundle/Controller/AccountController.php

@@ -61,13 +61,6 @@ class AccountController extends BaseController
                 }
             }
 
-            if ($form->has('password')) {
-                $password = $form['password']->getData();
-                if ($password) {
-                    $user->setPlainPassword($password);
-                }
-            }
-
             $showTermsIfProfileCompleted = ('true' === $settingsManager->getSetting('show_terms_if_profile_completed'));
             $user->setProfileCompleted($showTermsIfProfileCompleted);
 

src/CoreBundle/Entity/User.php

@@ -1220,7 +1220,7 @@ class User implements UserInterface, EquatableInterface, ResourceInterface, Reso
         return $this->plainPassword;
     }
 
-    public function setPlainPassword(?string $password): self
+    public function setPlainPassword(string $password): self
     {
         $this->plainPassword = $password;
         // forces the object to look "dirty" to Doctrine. Avoids
@@ -2210,17 +2210,6 @@ class User implements UserInterface, EquatableInterface, ResourceInterface, Reso
         return $this;
     }
 
-    public function getLogin(): string
-    {
-        return $this->username;
-    }
-
-    public function setLogin(string $login): self
-    {
-        $this->username = $login;
-        return $this;
-    }
-
     /**
      * @return Collection<int, TrackELogin>
      */

src/CoreBundle/Form/ProfileType.php

@@ -11,9 +11,8 @@ use Chamilo\CoreBundle\Form\Type\IllustrationType;
 use Chamilo\CoreBundle\Repository\LanguageRepository;
 use Chamilo\CoreBundle\Settings\SettingsManager;
 use Symfony\Component\Form\AbstractType;
-use Symfony\Component\Form\Extension\Core\Type\ChoiceType;
 use Symfony\Component\Form\Extension\Core\Type\EmailType;
-use Symfony\Component\Form\Extension\Core\Type\PasswordType;
+use Symfony\Component\Form\Extension\Core\Type\LocaleType;
 use Symfony\Component\Form\Extension\Core\Type\TextType;
 use Symfony\Component\Form\Extension\Core\Type\TimezoneType;
 use Symfony\Component\Form\FormBuilderInterface;
@@ -24,16 +23,20 @@ use Symfony\Component\OptionsResolver\OptionsResolver;
  */
 class ProfileType extends AbstractType
 {
+    private LanguageRepository $languageRepository;
+
     public function __construct(
-        private readonly LanguageRepository $languageRepository,
-        private readonly SettingsManager $settingsManager,
-    ) {}
+        LanguageRepository $languageRepository,
+        private readonly SettingsManager $settingsManager
+    ) {
+        $this->languageRepository = $languageRepository;
+    }
 
     public function buildForm(FormBuilderInterface $builder, array $options): void
     {
-        $changeableOptions = $this->settingsManager->getSetting('profile.changeable_options', true) ?? [];
-        $visibleOptions = $this->settingsManager->getSetting('profile.visible_options', true) ?? [];
-        $languages = array_flip($this->languageRepository->getAllAvailableToArray(true));
+        $changeableOptions = $this->settingsManager->getSetting('profile.changeable_options') ?? [];
+        $visibleOptions = $this->settingsManager->getSetting('profile.visible_options') ?? [];
+        $languages = array_flip($this->languageRepository->getAllAvailableToArray());
 
         $fieldsMap = [
             'name' => ['field' => 'firstname', 'type' => TextType::class, 'label' => 'Firstname'],
@@ -46,16 +49,10 @@ class ProfileType extends AbstractType
                 'mapped' => false,
             ],
             'login' => ['field' => 'login', 'type' => TextType::class, 'label' => 'Login'],
-            'password' => [
-                'field' => 'password',
-                'type' => PasswordType::class,
-                'label' => 'Password',
-                'mapped' => false,
-                'required' => false,
-            ],
+            'password' => ['field' => 'password', 'type' => TextType::class, 'label' => 'Password'],
             'language' => [
                 'field' => 'locale',
-                'type' => ChoiceType::class,
+                'type' => LocaleType::class,
                 'label' => 'Language',
                 'choices' => $languages,
             ],
@@ -72,8 +69,7 @@ class ProfileType extends AbstractType
                     array_merge(
                         [
                             'label' => $fieldConfig['label'],
-                            'required' => $fieldConfig['required'] ?? false,
-                            'mapped' => $fieldConfig['mapped'] ?? true,
+                            'required' => false,
                             'attr' => !$isEditable ? ['readonly' => true] : [],
                         ],
                         isset($fieldConfig['choices']) ? ['choices' => $fieldConfig['choices']] : []

src/CoreBundle/Repository/LanguageRepository.php

@@ -44,15 +44,9 @@ class LanguageRepository extends ServiceEntityRepository
         return $qb;
     }
 
-    public function getAllAvailableToArray(bool $onlyActive = false): array
+    public function getAllAvailableToArray(): array
     {
-        $queryBuilder = $this->getAllAvailable();
-
-        if (!$onlyActive) {
-            $queryBuilder->resetDQLPart('where');
-        }
-
-        $languages = $queryBuilder->getQuery()->getResult();
+        $languages = $this->getAllAvailable()->getQuery()->getResult();
 
         $list = [];