Skip to content

Fix dLawyers gender field validation#2858

Merged
Armanul46 merged 1 commit into
sovware:developmentfrom
Arafat-plugins:fix-dlawyers-gender-field-validation
May 20, 2026
Merged

Fix dLawyers gender field validation#2858
Armanul46 merged 1 commit into
sovware:developmentfrom
Arafat-plugins:fix-dlawyers-gender-field-validation

Conversation

@Arafat-plugins
Copy link
Copy Markdown
Contributor

@Arafat-plugins Arafat-plugins commented May 18, 2026

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Security fix
  • Improvement
  • New Feature
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Text changes
  • Other... Please describe:

Description

How to reproduce the issue or how to test the changes

This fixes the dLawyers gender field validation error on the add listing form.

The dLawyers theme registers dlawyers_gender as a radio field and the submitted value is a string such as Male or Female. Directorist's theme custom field map incorrectly translated dlawyers_gender to the number field type, so the number validator rejected the submitted radio value and returned Gender: Invalid number.

This change maps dlawyers_gender to the radio field type so it is validated with the radio field validator instead of the number validator.

Steps to reproduce

  1. Use the dLawyers add listing form.
  2. Select a gender option in the Personal Information section.
  3. Continue to the finish step and submit/save the listing.
  4. Notice the form returns Gender: Invalid number. even though a radio option is selected.

Expected result

The dLawyers gender radio field should accept the selected radio option and should not be validated as a number.

How to test the fix

  1. Open the dLawyers add listing form.
  2. Select a gender option.
  3. Submit/save the listing.
  4. Confirm the Gender: Invalid number. validation error no longer appears.
  5. Confirm regular numeric fields such as dLawyers age still use number validation.

Any linked issues

Support ticket:

Related screenshot:

Screenshot

Checklist

@Armanul46 Armanul46 merged commit 1b71c9c into sovware:development May 20, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants