User tests: Successful: Unsuccessful:
Pull Request for Issue #43242
don't process further if no user
Create a contact and select a 'Linked User', then delete that user.
error
no more errore
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
Status | New | ⇒ | Pending |
Category | ⇒ | Front End com_contact |
on Joomla 5.1 rc2, PHP 8.2.17
I just tried 3 times to create a new user, then create a new contact and link that contact to the new user and then I deleted the user in question and never received an error so therefore I was not able to test this PR for no error message [I have debug and error reporting to maximum].
I created a new user, then a new contact and linked that contact to the new user and then I deleted the user in question and never received an error.
using v5.1 RC1 - i also followed the instructions, "Create a contact and select a 'Linked User', then delete that user." But did not run into any issues deleting the user. going back to the contact, the ID is still listed as the linked user
I have tested this item ✅ successfully on 9979a19
There seem to be no errors present. All good.
I have tested this item ✅ successfully on 9979a19
strange that others in 5 could not reproduce it
I think the testers were not aware that the error occurs in frontend, not in backend
As a user I would expect the first message like "[contact_title] This contact is not found in the system. Please contact your system administrator" or something like that - native speaker required. The secind message should not appear at all.
Perhaps the contact form should not be generated if the user is not found
Perhaps the contact form should not be generated if the user is not found
That would be the most sensible behaviour. Just the same as when there is no email address in the contact information
not strange at all - your testing instructions were extremely vague!!
@brianteeman You're right. In my opinion this happens cause developers think different than a non-developer-tester.
To be honest i had given up to test this pr until i thought "maybe 'frontend' is missing in the instructions".
i apologize
Status | Pending | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2024-04-12 17:44:08 |
Closed_By | ⇒ | alikon | |
Labels |
Added:
PR-5.1-dev
|
PR test successfully but
Warning: Undefined property: stdClass::$value in /mnt/data/vhosts/casite-1466051.cloudaccess.net/httpdocs/components/com_contact/tmpl/contact/default_user_custom_fields.php on line 27
.