Needs work
Project:
Address Phonenumber
Version:
8.x-1.1
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
15 Dec 2017 at 21:17 UTC
Updated:
27 Sep 2021 at 10:27 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
gouli commentedI have removed the code which was forcing the phone number to be mandatory.
The patch is attached.
Comment #3
gouli commentedIgnore my previous patch. Use this one.
Comment #4
gouli commentedComment #5
pelicani commentedIn addition to the required state of the field, the validation routine runs on it.
Even if the phone number is blank, it will get validated and fail.
We adjusted our file to check for the $telephone value.
i.e.
if (!empty($telephone)) {
try {
Not sure if that's what you would do, but wanted to mention this step to successfully remove the requirement.
peace,
michael
Comment #6
takki commentedOk added this into the patch.
Comment #7
pelicani commentedMarking this for review to get into the module.
peace,
Michael
Comment #8
brianwagner commentedReviewed #6. Works for me to prevent setting a default value.
Comment #9
dishabhadra commentedIf we are applying the above patch, the below criteria not working.
Steps:
1. Mark the phone number field as required while creating the field.
2. Create content and save the content without adding a phone number.
3. It will save the content without giving any error for the phone number field.
The field is required still no error. So above patch is failing when the phone number field is required.
Comment #10
webengr commentedthis is still broke 9/28/2023 with latest versions.