? ? Pending

User tests: Successful: Unsuccessful:

avatar korenevskiy
korenevskiy
16 Aug 2021

image
image

In the Article settings. The RECORD HITS field is disabled.
But this value is displayed on the site.
The fact is that the disabled RECORD HITS field hides another HITS field.
I believe that it should be like this:

  • Either the RECORD HITS field will not hide the HITS field.
  • Either disabling the RECORD HITS field will automatically deactivate the HITS field.
    image
avatar korenevskiy korenevskiy - open - 16 Aug 2021
avatar korenevskiy korenevskiy - change - 16 Aug 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 16 Aug 2021
Category Administration com_content
avatar korenevskiy korenevskiy - change - 16 Aug 2021
The description was changed
avatar korenevskiy korenevskiy - edited - 16 Aug 2021
avatar korenevskiy
korenevskiy - comment - 16 Aug 2021
avatar brianteeman brianteeman - test_item - 16 Aug 2021 - Tested successfully
avatar brianteeman
brianteeman - comment - 16 Aug 2021

I have tested this item successfully on 9feee70


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

avatar richard67
richard67 - comment - 16 Aug 2021

@korenevskiy Does this PR solve your issue #35043 ? If yes then please mention the issue in the dexcription of your PR like it should be and like it is suggested by the PR template when creating a new PR:

Pull Request for Issue #35043 .

And the headings for testing instructions we also don't have without reason in our PR templates.

And finally, please close your issue when having made a PR which claims to solve it.

Thanks in advance.

avatar richard67
richard67 - comment - 16 Aug 2021

In general we have that problem everywhere where we use showon because we normally don't use js to reset fields to their default value when they are hidden. It always can happen you hide a field by changing an option but that field still has its value. That's a general issue with showon.

avatar brianteeman
brianteeman - comment - 16 Aug 2021

@richard67 that is true but its rarely an issue - here it is

avatar richard67
richard67 - comment - 16 Aug 2021

@richard67 that is true but its rarely an issue - here it is

@brianteeman Yes, I agree, and it can be a valid scenario for some site owner to stop recording hits but still show the old count and so have these 2 options independently.

avatar richard67
richard67 - comment - 16 Aug 2021

@brianteeman What's funny: It was you who implemented it with PR #32920 ;-) But well, things happen.

avatar richard67 richard67 - test_item - 16 Aug 2021 - Tested successfully
avatar richard67
richard67 - comment - 16 Aug 2021

I have tested this item successfully on 9feee70


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

avatar richard67 richard67 - change - 16 Aug 2021
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 16 Aug 2021

RTC


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

avatar korenevskiy
korenevskiy - comment - 17 Aug 2021

In general we have that problem everywhere where we use showon because we normally don't use js to reset fields to their default value when they are hidden. It always can happen you hide a field by changing an option but that field still has its value. That's a general issue with showon.

I think that JS is not needed ever. ShowOn needs to be checked in PHP as well. Just in PHP conditions, you need to do double checks of 2 fields. The fact is that there may be cases when you do not need to reset hidden fields to default values. For example, a hidden text field will simply store the text. But we will not use it. I think that we have no problems with hidden fields.

avatar bembelimen bembelimen - change - 23 Aug 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-08-23 01:08:49
Closed_By bembelimen
Labels Added: ? ?
avatar bembelimen bembelimen - close - 23 Aug 2021
avatar bembelimen bembelimen - merge - 23 Aug 2021
avatar bembelimen
bembelimen - comment - 23 Aug 2021

Thx

avatar korenevskiy
korenevskiy - comment - 26 Aug 2021

Can the patch be removed?.
Removing the patch will not affect the PR?

avatar Quy
Quy - comment - 29 Aug 2021

@korenevskiy Once merged, it is safe to remove the patch.

Add a Comment

Login with GitHub to post a comment