Skip to content

Commit ba962d7

Browse files
cursoragentBaspa
andcommitted
Fix: Add checks for field name before applying validation rules
Co-authored-by: bspbaspa <[email protected]>
1 parent 31d1082 commit ba962d7

File tree

1 file changed

+24
-8
lines changed

1 file changed

+24
-8
lines changed

src/Fields/Validation/ValidationRuleApplier.php

Lines changed: 24 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -160,13 +160,17 @@ public static function applyValidationRule($input, array $rule, ?Field $field =
160160

161161
case 'different':
162162
$fieldName = FieldOptionsHelper::getFieldNameFromUlid($parameters['field'] ?? '', $field);
163-
$input->different($fieldName ?? '');
163+
if ($fieldName) {
164+
$input->different($fieldName);
165+
}
164166

165167
break;
166168

167169
case 'same':
168170
$fieldName = FieldOptionsHelper::getFieldNameFromUlid($parameters['field'] ?? '', $field);
169-
$input->same($fieldName ?? '');
171+
if ($fieldName) {
172+
$input->same($fieldName);
173+
}
170174

171175
break;
172176

@@ -255,7 +259,9 @@ public static function applyValidationRule($input, array $rule, ?Field $field =
255259

256260
case 'prohibits':
257261
$fieldName = FieldOptionsHelper::getFieldNameFromUlid($parameters['field'] ?? '', $field);
258-
$input->prohibits($fieldName ?? '');
262+
if ($fieldName) {
263+
$input->prohibits($fieldName);
264+
}
259265

260266
break;
261267

@@ -303,31 +309,41 @@ public static function applyValidationRule($input, array $rule, ?Field $field =
303309

304310
case 'required_if_accepted':
305311
$fieldName = FieldOptionsHelper::getFieldNameFromUlid($parameters['field'] ?? '', $field);
306-
$input->requiredIfAccepted($fieldName ?? '');
312+
if ($fieldName) {
313+
$input->requiredIfAccepted($fieldName);
314+
}
307315

308316
break;
309317

310318
case 'greater_than':
311319
$fieldName = FieldOptionsHelper::getFieldNameFromUlid($parameters['field'] ?? '', $field);
312-
$input->gt($fieldName ?? '');
320+
if ($fieldName) {
321+
$input->gt($fieldName);
322+
}
313323

314324
break;
315325

316326
case 'greater_than_or_equal':
317327
$fieldName = FieldOptionsHelper::getFieldNameFromUlid($parameters['field'] ?? '', $field);
318-
$input->gte($fieldName ?? '');
328+
if ($fieldName) {
329+
$input->gte($fieldName);
330+
}
319331

320332
break;
321333

322334
case 'less_than':
323335
$fieldName = FieldOptionsHelper::getFieldNameFromUlid($parameters['field'] ?? '', $field);
324-
$input->lt($fieldName ?? '');
336+
if ($fieldName) {
337+
$input->lt($fieldName);
338+
}
325339

326340
break;
327341

328342
case 'less_than_or_equal':
329343
$fieldName = FieldOptionsHelper::getFieldNameFromUlid($parameters['field'] ?? '', $field);
330-
$input->lte($fieldName ?? '');
344+
if ($fieldName) {
345+
$input->lte($fieldName);
346+
}
331347

332348
break;
333349

0 commit comments

Comments
 (0)