User tests: Successful: Unsuccessful:
Pull Request for Issue #35609, #36961, #36956.
This PR fixes issue #35609 by removing unnecessary code from view class:
Please note that after applying patch, you should closer your browser and re-open it to clear existing session data (or better, use a different browser to test the new behavior).
Status | New | ⇒ | Pending |
Category | ⇒ | Front End com_contact |
@GennaroMarra Each PR needs two successful tests to be merged. Please go to https://issues.joomla.org/tracker/joomla-cms/35763 and report your test result.
Then we will need to have one more test before it could be merged.
The test was successful. Thanks.
Hi,
I was testing this issue but I have not been able to reproduce the error.
I followed the testing instructions but each contact showed its own fields. There was no error I am afraid.
Cheers,
Berta
@BertaOctech Thanks for testing. When you access to first menu item, the system will display custom fields from first contact category properly.
However, when you access to the second menu item, you will see custom fields from first category (which is wrong - it should display custom fields from second category). Could you please try again one more time?
Thanks !
Yes, agree with @BertaOctech. Not able to show custom field value from one contact in the other one.
Thank you and best regards
@BertaOctech Thanks for testing. When you access to first menu item, the system will display custom fields from first contact category properly.
However, when you access to the second menu item, you will see custom fields from first category (which is wrong - it should display custom fields from second category). Could you please try again one more time?
Thanks !
Hi, thanks for your comment.
When I accessed the second item the custom fields from the first category disappeared and only the field from the second category were shown.
Maybe this issue was solved via another issue, sometimes this happen.
Cheers,
Berta
@BertaOctech Thanks for testing. Strange that you see it work properly. It is still not working, maybe someone you can test it and confirm later (I can still see it from current code by code reading)
In testing instructions is necessary to add another step: after create new contacts, fill each field with a different value, right?
I've created a Calendar field for Category 1 and Color for Category 2. Should the fields be the same type?
Thanks
@pabloarias This issue happens with custom fields display on contact form, so you do not need to enter data for these custom fields for contact. Maybe you can try to look at the original issue #35609 , it might describes the issue better than what I described here
Ok, I understand now. Custom fields are for mail form, not for contact.
I see the problem.
I'm really sorry but after applying the path, it continues failing.
One question: why these fields are shown as disabled in frontend form?
Thank you very much!
Ok, I understand now. Custom fields are for mail form, not for contact.
The testing instructions should be more clear. We need to create custom fields for Mail, not to Contact for this PR.
I'm really sorry but after applying the path, it continues failing.
That's strange to me. I don't see how it could still fail after the patch.
One question: why these fields are shown as disabled in frontend form?
It's something with permission settings. You need to edit the field, look at Permissions tab, select Public user group and set Edit Custom Field Value permission to Allowed. After that, all users should be allowed to enter value for that field.
Thanks for your help with testing.
Hey, even I tried to test the same but it didn't work out for me as well. I think that the test instructions are to be more clear.
I have tested this item
Before applying the patch, both contacts show the same custom field: mail field #1.
After applying the patch, both contacts show the same custom field: mail field #2.
The error is different but remains equal
I have tested this item
I could reproduce the problem but after applying the patch the problem gets worse.
What I did:
Before the patch:
When you click on the "Contact left" menu item the contact form shows with the "cfield-left" field. Next when you click on the "Contact right" menu item the field "cfield-left" remains. After clicking a second time or after a page refresh the custom fields updates to the correct one.
After the patch:
Only one of the fields is loaded even when switching menu items an refreshing / clicking multiple times.
Labels |
Added:
Conflicting Files
?
?
|
@RickR2H and others: I think I know why your tests were un-successful. The catid
data is still remembered from your previous tests (before patch applied)
So somehow, after applying patch. We need to clear existing session data to have the right behavior:
This works!. I have tested it in the following scenarios: A classic form and an appointment form with extra fields, and 2 forms with custom fields of type Mail. The forms are in different categories and have different menu items.
For those that don't work, you can clear your browser's cache and cookies or try it from another browser.
Works like expected, thanks Tuan.
I have tested it successfully.
Status | Pending | ⇒ | Ready to Commit |
RTC
Status | Ready to Commit | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2022-03-03 03:10:46 |
Closed_By | ⇒ | bembelimen | |
Labels |
Added:
?
Removed: Conflicting Files ? |
Thx
Good morning I had already communicated that I had tested the solution that turned out to be working. In the last release of Joomla, i.e. version 4.0.3, has the reported bug been fixed? Thank you