-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
[4.4] Incorrect message for user fields #41513
Conversation
rebased to 4.4 |
@brianteeman Frontend
Are you sure the rule is applying by default also on the Users Joomla Custom Fields ?
Right now also here it is showing nothing as above, instead to show the default value. So, for example, from:
They should be shown directly as:
Where "UTC-5" and "Spanish" are the default values. Do you want Super Admin credentials of the test site ? (Where I can send it to you ? |
I have tested this item ✅ successfully on 3db8c0b This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/41513. |
Hi @ceford A - Are you sure the rule to write nothing is applying by default also on the Users Joomla Custom Fields ? B - For all the fields that, when not yet setted, they redirect to a "site default value" like the "Basic Settings" fields (Time Zone, Frontend Language,...), they should just to show the default value and not the written "site default" (or nothing) Please, Can you reproduce them ? |
On A: I added a Textarea field and confirm that if I do not enter any data it appears in the front end without any data. That confirms the Test Pass result. On B: My test results show what the pull request required. If you think the Time Zone should show the actual time zone rather than Website default perhaps that is a separate issue. |
Hi @MacJoom A - Are you sure the rule to write nothing is applying by default also on the Users Joomla Custom Fields ? B - For all the fields that, when not yet setted, they redirect to a "site default value" like the "Basic Settings" fields (Time Zone, Frontend Language,...), they should just to show the default value and not the written "site default" (or nothing) Please, Can you reproduce them ? |
@brianteeman - can you comment on joomlebs issues? Thank you. |
@MacJoom Regarding joomlebs: Issue A is solved with this PR. Issue B is a separate issue. |
Hi @Quy A - As far as I understand, what is shown in your image are the fields of the (old) User Profile Joomla plugin. B - @MacJoom - Please, Are you going to split it? Or, What should I do ? |
A - I re-tested it right now with success, thank you B - I remain available... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Point A tested with success
@Quy what are the updates that are requested |
Before PR, the |
Please open a new issue. Thanks. |
@Quy sorry I didnt see anything in the comment you linked to that said the change was wrong. I have now reverted that line |
isnt that the entire point? |
Sorry I should have mentioned that these fields have values but are not displayed. |
Closing this for now |
Hi guys, |
Alternative for #41435
Summary of Changes
The user profile displays various fields. Some of them have a value, some of them have no value and some of them have no value but that means they use the site default value.
Originally the text was "No information entered" but this wasn't correct for the fields using a default value so I changed the text to "website default" in #25699
That was wrong as I only considered the basic params in a user profile and not any additional fields such as from the user profile plugin or a custom field.
This created a scenario where a users name and address in their profile would be displatyed as "website default" when it should have been blank.
This PR keeps the behaviour on the basic params to display "website default" but removes any default text being used for the other fields as its not appropriate.
Testing Instructions
Actual result BEFORE applying this Pull Request
Expected result AFTER applying this Pull Request
Link to documentations
Please select:
Documentation link for docs.joomla.org:
No documentation changes for docs.joomla.org needed
Pull Request link for manual.joomla.org:
No documentation changes for manual.joomla.org needed