Skip to content
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

Closed
wants to merge 9 commits into from

Conversation

brianteeman
Copy link
Contributor

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

  1. Create a user custom field
  2. Publish the user profile plugin
  3. Create a menu item to display a user profile
  4. Log in to the front end and go to the profile page
  5. Edit the profile by setting some but not all fields

Actual result BEFORE applying this Pull Request

image

Expected result AFTER applying this Pull Request

image

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

@brianteeman brianteeman changed the title [4.3] Incorrect message for user fields [4.4] Incorrect message for user fields Sep 14, 2023
@brianteeman brianteeman changed the base branch from 4.3-dev to 4.4-dev September 14, 2023 10:01
@brianteeman
Copy link
Contributor Author

rebased to 4.4

@laoneo laoneo removed the PR-4.3-dev label Sep 18, 2023
@Fedik Fedik added the bug label Sep 24, 2023
@joomleb
Copy link

joomleb commented Sep 24, 2023

@brianteeman
I updated the Test site
For me it is NOT running as expected.

Frontend

For all fields that can be left empty = just to don't show nothing, any value, left it empty.

Are you sure the rule is applying by default also on the Users Joomla Custom Fields ?
I created a Calendar (Birthday) Joomla Custom Field and "Website Default"is still shown.

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,...) = just to show the default value and not the written "site default".

Right now also here it is showing nothing as above, instead to show the default value.

So, for example, from:

  • Time Zone: Website default
  • Frontend Language: Website default

They should be shown directly as:

  • Time Zone: UTC-5
  • Frontend Language: Spanish

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 ?

@ceford
Copy link
Contributor

ceford commented Sep 26, 2023

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.

@joomleb
Copy link

joomleb commented Oct 18, 2023

Hi @ceford
on Frontend:

A - Are you sure the rule to write nothing is applying by default also on the Users Joomla Custom Fields ?
I tested creating some custom fields and I always have back "Website Default" as value.

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 ?

@ceford
Copy link
Contributor

ceford commented Oct 18, 2023

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.

@MacJoom MacJoom self-assigned this Nov 10, 2023
@MacJoom
Copy link
Contributor

MacJoom commented Nov 10, 2023

Hi @joomleb - what is the status here - does this PR solve your issue? Can you give a successful test? And would you close your PR #41435 in favour of this PR? Thank you

@joomleb
Copy link

joomleb commented Nov 10, 2023

Hi @MacJoom
Okay, I'm going to close the PR #41435
I'm still "stopped" to this point #41435 (comment) (backend/frontend).
For me, Joomla 4.4 frontend, is not running as expected:

A - Are you sure the rule to write nothing is applying by default also on the Users Joomla Custom Fields ?
I tested creating some custom fields and I always have back "Website Default" as value.

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 ?

@MacJoom
Copy link
Contributor

MacJoom commented Nov 14, 2023

@brianteeman - can you comment on joomlebs issues? Thank you.

@Quy
Copy link
Contributor

Quy commented Nov 17, 2023

Before PR:
41513-before

After PR:
41513-after

@MacJoom Regarding joomlebs: Issue A is solved with this PR. Issue B is a separate issue.

@joomleb
Copy link

joomleb commented Nov 17, 2023

Hi @Quy

A - As far as I understand, what is shown in your image are the fields of the (old) User Profile Joomla plugin.
What I asked is: Are you sure that the rule "to write nothing" is applying by default also on the (new) Users Joomla Custom Fields ?
I tested creating some new custom fields and I always have back "Website Default" as value...
Please, Have you tested it by creating a new Custom Field ?

B - @MacJoom - Please, Are you going to split it? Or, What should I do ?

@Quy
Copy link
Contributor

Quy commented Nov 17, 2023

No issues with custom fields. I did a calendar field as you mentioned in your previous comment.

41513-custom-fields

@joomleb
Copy link

joomleb commented Nov 18, 2023

@Quy

A - I re-tested it right now with success, thank you

B - I remain available...

Copy link

@joomleb joomleb left a 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 Quy added the Updates Requested Indicates that this pull request needs an update from the author and should not be tested. label Nov 19, 2023
@brianteeman
Copy link
Contributor Author

@Quy what are the updates that are requested

@Quy
Copy link
Contributor

Quy commented Nov 21, 2023

#41513 (comment)

Before PR, the About Me value is escaped.

@Quy
Copy link
Contributor

Quy commented Nov 21, 2023

B - @MacJoom - Please, Are you going to split it? Or, What should I do ?

Please open a new issue. Thanks.

@joomleb
Copy link

joomleb commented Nov 21, 2023

@Quy
Thank you
Done #42377

@brianteeman
Copy link
Contributor Author

@Quy sorry I didnt see anything in the comment you linked to that said the change was wrong. I have now reverted that line

@Quy
Copy link
Contributor

Quy commented Nov 21, 2023

Now no values are displayed except Website field.

41513-no-values

@brianteeman
Copy link
Contributor Author

isnt that the entire point?

@Quy
Copy link
Contributor

Quy commented Nov 21, 2023

Sorry I should have mentioned that these fields have values but are not displayed.

@brianteeman
Copy link
Contributor Author

Closing this for now

@joomleb
Copy link

joomleb commented Jan 2, 2024

Hi guys,
@brianteeman What do you mean with "closing this for now" ?
I'm asking to know what to do with its alter ego ticket #42377 I opened in November...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug PR-4.4-dev Updates Requested Indicates that this pull request needs an update from the author and should not be tested.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants