Removed redundant digit check loop

[marcosonmachine]

Removed redundant digit check loop, optimizing phone number validation.

[adamsitnik]

Hi @marcosonmachine

From https://github.com/microsoft/referencesource?tab=readme-ov-file#net-reference-source:

Please note that the referencesource repository is read-only

If you want to contribute to .NET 9, you need to send a PR to https://github.com/dotnet/runtime/blob/main/src/libraries/System.ComponentModel.Annotations/src/System/ComponentModel/DataAnnotations/PhoneAttribute.cs

Thanks!

[KalleOlaviNiemitalo]

The loop isn't redundant anyway. The first loop verifies that the phone number has at least one digit. The second loop verifies that the phone number contains only digits and some other allowed characters. So if you deleted the first loop, then the method would also accept e.g. () as a phone number. This does not contain any digits and should be rejected.

[marcosonmachine]

The loop isn't redundant anyway. The first loop verifies that the phone number has at least one digit. The second loop verifies that the phone number contains only digits and some other allowed characters. So if you deleted the first loop, then the method would also accept e.g. () as a phone number. This does not contain any digits and should be rejected.

Yes, you are right. This is from last year, sorry I forgot to close it.

Again I am sorry for it caused inconvenience.