RTC bug PR-5.1-dev Pending

User tests: Successful: Unsuccessful:

avatar bembelimen
bembelimen
14 Jul 2024

Summary of Changes

Fix wrong field name for multilingual sites in user profile

Testing Instructions

  • Disable the "Behaviour - Backward Compatibility" plugin
  • Enable the "System - Language Filter" plugin
  • Check User profile (frontend/backend)

Actual result BEFORE applying this Pull Request

  • Language field is a text field

Expected result AFTER applying this Pull Request

  • Language field is a select
avatar bembelimen bembelimen - open - 14 Jul 2024
avatar bembelimen bembelimen - change - 14 Jul 2024
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 14 Jul 2024
Category Administration com_users Front End
avatar bembelimen bembelimen - change - 14 Jul 2024
The description was changed
avatar bembelimen bembelimen - edited - 14 Jul 2024
avatar brianteeman brianteeman - test_item - 14 Jul 2024 - Tested successfully
avatar brianteeman
brianteeman - comment - 14 Jul 2024

I have tested this item ✅ successfully on a1a927a


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43791.

avatar richard67
richard67 - comment - 14 Jul 2024
avatar dautrich dautrich - test_item - 14 Jul 2024 - Tested successfully
avatar dautrich
dautrich - comment - 14 Jul 2024

I have tested this item ✅ successfully on a1a927a


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43791.

avatar richard67 richard67 - change - 14 Jul 2024
Status Pending Ready to Commit
Labels Added: bug PR-5.1-dev
avatar richard67
richard67 - comment - 14 Jul 2024

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43791.

avatar bembelimen
bembelimen - comment - 14 Jul 2024

@bembelimen Do you know what this is good for, and is it somehow related to the issue? https://github.com/joomla/joomla-cms/blob/4.4-dev/libraries/src/Form/Field/FrontendlanguageField.php#L26-L32

Should not have any effect on it

avatar richard67
richard67 - comment - 14 Jul 2024

@bembelimen Do you know what this is good for, and is it somehow related to the issue? https://github.com/joomla/joomla-cms/blob/4.4-dev/libraries/src/Form/Field/FrontendlanguageField.php#L26-L32

Should not have any effect on it

@bembelimen Should it be changed to public $type = 'Frontendlanguage'; in 6.0-dev so it is like for the other fields? Or should that $type variable be deprecated for all fields because it has no effect at all?

avatar brianteeman
brianteeman - comment - 14 Jul 2024

@bembelimen Do you know what this is good for, and is it somehow related to the issue? https://github.com/joomla/joomla-cms/blob/4.4-dev/libraries/src/Form/Field/FrontendlanguageField.php#L26-L32

Should not have any effect on it

@bembelimen Should it be changed to public $type = 'Frontendlanguage'; in 6.0-dev so it is like for the other fields? Or should that $type variable be deprecated for all fields because it has no effect at all?

it confused me

avatar LadySolveig LadySolveig - change - 26 Jul 2024
Labels Added: RTC
avatar LadySolveig LadySolveig - change - 26 Jul 2024
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2024-07-26 16:54:42
Closed_By LadySolveig
avatar LadySolveig LadySolveig - close - 26 Jul 2024
avatar LadySolveig LadySolveig - merge - 26 Jul 2024
avatar LadySolveig
LadySolveig - comment - 26 Jul 2024

Thanks @bembelimen and also for testing and review @brianteeman @dautrich @richard67

Add a Comment

Login with GitHub to post a comment