-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[5.3] Revert #44161 #44462
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[5.3] Revert #44161 #44462
Conversation
Thanks @bembelimen! |
I have tested this item ✅ successfully on 2cba929 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44462. |
I have tested this item ✅ successfully on 2cba929 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44462. |
RTC This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44462. |
Thanks |
joomla/joomla-cms#41496 - (upmerge с 5.2х) joomla/joomla-cms#42530 + joomla/joomla-cms#43994 - (upmerge с 5.2х) joomla/joomla-cms#44009 - (upmerge с 5.2х) joomla/joomla-cms#44010 - (upmerge с 5.2х) joomla/joomla-cms#44161 + joomla/joomla-cms#44187 - (upmerge с 5.2х) joomla/joomla-cms#44207 - (upmerge с 5.2х) joomla/joomla-cms#44271 + joomla/joomla-cms#44273 + joomla/joomla-cms#44288 - (только для en-GB) joomla/joomla-cms#44348 - (upmerge с 5.2х) joomla/joomla-cms#44366 + joomla/joomla-cms#44367 - (upmerge с 5.2х) joomla/joomla-cms#44434 - (upmerge с 5.2х) joomla/joomla-cms#44448 - (upmerge с 5.2х) joomla/joomla-cms#44462 + joomla/joomla-cms#44487 - (upmerge с 5.2х) joomla/joomla-cms#44587 + joomla/joomla-cms#44600 + joomla/joomla-cms#44604 + joomla/joomla-cms#44621 - (upmerge с 5.2х) joomla/joomla-cms#44623 + joomla/joomla-cms#44632 + joomla/joomla-cms#44640 - (позже был REVERT joomla/joomla-cms#44845) joomla/joomla-cms#44714 - (upmerge с 5.2х) joomla/joomla-cms#44756 + joomla/joomla-cms#44768 - (upmerge с 5.2х) joomla/joomla-cms#44792 - (только для en-GB) joomla/joomla-cms#44813 + joomla/joomla-cms#44822 - (upmerge с 5.2х) joomla/joomla-cms#44839 + joomla/joomla-cms#44871 + joomla/joomla-cms#44954 + joomla/joomla-cms#45034 - (upmerge с 5.2х) joomla/joomla-cms#45058 - (только для en-GB) joomla/joomla-cms#45064 + joomla/joomla-cms#45078 - (только для en-GB) joomla/joomla-cms#45130 - (upmerge с 5.2х) joomla/joomla-cms#45240 - (upmerge с 5.2х) joomla/joomla-cms#45246 - (только для др. пакетов)
Summary of Changes
With #44161 the limitation that custom fields has to be unique system wide was removed. This could break when e.g. a plugin loads custom fields from another extension into a form.
To simulare the problem
components/com_users/tmpl/registration/default.php
line 23:Expected result
two "Foobar" fields visible
Actual problem
Only one "Foobar" field is loaded, the fields are conflicting. This has also different effect when e.g. saving etc.
Solution
I like in general the idea of the original PR, but it has to be in a Major version respecting the B/C rules.
@Hackwar