Skip to content

Commit b39e98c

Browse files
emsifaemsifa
committed
fix unexpected result by checking dot notation
1 parent 1a903e8 commit b39e98c

File tree

3 files changed

+64
-25
lines changed

3 files changed

+64
-25
lines changed

‎src/Attribute.php

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -124,6 +124,11 @@ public function isArrayAttribute()
124124
return count($this->getKeyIndexes()) > 0;
125125
}
126126

127+
public function isUsingDotNotation()
128+
{
129+
return strpos($this->getKey(), '.') !== false;
130+
}
131+
127132
public function resolveSiblingKey($key)
128133
{
129134
$indexes = $this->getKeyIndexes();
@@ -165,4 +170,4 @@ public function getAlias()
165170
return $this->alias;
166171
}
167172

168-
}
173+
}

‎src/Validation.php

Lines changed: 16 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ class Validation
2020
protected $aliases = [];
2121

2222
protected $messageSeparator = ':';
23-
23+
2424
protected $validData = [];
2525
protected $invalidData = [];
2626

@@ -72,7 +72,7 @@ protected function validateAttribute(Attribute $attribute)
7272
}
7373

7474
$attributeKey = $attribute->getKey();
75-
$rules = $attribute->getRules();
75+
$rules = $attribute->getRules();
7676

7777
$value = $this->getValue($attributeKey);
7878
$isEmptyValue = $this->isEmptyValue($value);
@@ -92,7 +92,7 @@ protected function validateAttribute(Attribute $attribute)
9292
if ($isEmptyValue AND $this->ruleIsOptional($attribute, $ruleValidator)) {
9393
continue;
9494
}
95-
95+
9696
if (!$valid) {
9797
$isValid = false;
9898
$this->addError($attribute, $value, $ruleValidator);
@@ -254,8 +254,8 @@ protected function isEmptyValue($value)
254254

255255
protected function ruleIsOptional(Attribute $attribute, Rule $rule)
256256
{
257-
return false === $attribute->isRequired() AND
258-
false === $rule->isImplicit() AND
257+
return false === $attribute->isRequired() AND
258+
false === $rule->isImplicit() AND
259259
false === $rule instanceof Required;
260260
}
261261

@@ -288,14 +288,14 @@ protected function resolveMessage(Attribute $attribute, $value, Rule $validator)
288288
];
289289

290290
if ($primaryAttribute) {
291-
// insert primaryAttribute keys
291+
// insert primaryAttribute keys
292292
// $message_keys = [
293293
// $attributeKey.$this->messageSeparator.$ruleKey,
294294
// >> here [1] <<
295295
// $attributeKey,
296296
// >> and here [3] <<
297297
// $ruleKey
298-
// ];
298+
// ];
299299
$primaryAttributeKey = $primaryAttribute->getKey();
300300
array_splice($message_keys, 1, 0, $primaryAttributeKey.$this->messageSeparator.$ruleKey);
301301
array_splice($message_keys, 3, 0, $primaryAttributeKey);
@@ -359,7 +359,7 @@ protected function resolveRules($rules)
359359
foreach($rules as $i => $rule) {
360360
if (empty($rule)) continue;
361361
$params = [];
362-
362+
363363
if (is_string($rule)) {
364364
list($rulename, $params) = $this->parseRule($rule);
365365
$validator = call_user_func_array($validatorFactory, array_merge([$rulename], $params));
@@ -446,7 +446,7 @@ protected function resolveInputAttributes(array $inputs)
446446
$resolvedInputs = [];
447447
foreach($inputs as $key => $rules) {
448448
$exp = explode(':', $key);
449-
449+
450450
if (count($exp) > 1) {
451451
// set attribute alias
452452
$this->aliases[$exp[0]] = $exp[1];
@@ -457,16 +457,17 @@ protected function resolveInputAttributes(array $inputs)
457457

458458
return $resolvedInputs;
459459
}
460-
460+
461461
public function getValidatedData() {
462462
return array_merge($this->validData, $this->invalidData);
463463
}
464464

465465
protected function setValidData(Attribute $attribute, $value)
466466
{
467467
$key = $attribute->getKey();
468-
if ($attribute->isArrayAttribute()) {
469-
Helper::arraySet($this->validData, $key, $value);
468+
if ($attribute->isArrayAttribute() || $attribute->isUsingDotNotation()) {
469+
Helper::arraySet($this->validData, $key, $value);
470+
Helper::arrayUnset($this->invalidData, $key);
470471
} else {
471472
$this->validData[$key] = $value;
472473
}
@@ -480,8 +481,9 @@ public function getValidData()
480481
protected function setInvalidData(Attribute $attribute, $value)
481482
{
482483
$key = $attribute->getKey();
483-
if ($attribute->isArrayAttribute()) {
484-
Helper::arraySet($this->invalidData, $key, $value);
484+
if ($attribute->isArrayAttribute() || $attribute->isUsingDotNotation()) {
485+
Helper::arraySet($this->invalidData, $key, $value);
486+
Helper::arrayUnset($this->validData, $key);
485487
} else {
486488
$this->invalidData[$key] = $value;
487489
}

‎tests/ValidatorTest.php

Lines changed: 42 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ public function testRequiredUploadedFile()
7676
$validation = $this->validator->validate([
7777
'file' => $empty_file
7878
], [
79-
'file' => 'required|uploaded_file'
79+
'file' => 'required|uploaded_file'
8080
]);
8181

8282
$errors = $validation->errors();
@@ -97,20 +97,20 @@ public function testOptionalUploadedFile()
9797
$validation = $this->validator->validate([
9898
'file' => $empty_file
9999
], [
100-
'file' => 'uploaded_file'
100+
'file' => 'uploaded_file'
101101
]);
102102
$this->assertTrue($validation->passes());
103103
}
104104

105105
/**
106106
* @dataProvider getSamplesMissingKeyFromUploadedFileValue
107-
*/
107+
*/
108108
public function testMissingKeyUploadedFile($uploaded_file)
109109
{
110110
$validation = $this->validator->validate([
111111
'file' => $uploaded_file
112112
], [
113-
'file' => 'required|uploaded_file'
113+
'file' => 'required|uploaded_file'
114114
]);
115115

116116
$errors = $validation->errors();
@@ -335,7 +335,7 @@ public function testAfterRule()
335335

336336
$this->assertFalse($validator2->passes());
337337
}
338-
338+
339339
public function testNewValidationRuleCanBeAdded()
340340
{
341341

@@ -735,7 +735,7 @@ public function testUsingDefaults()
735735
'is_enabled' => '1',
736736
'is_published' => 'invalid-value'
737737
]);
738-
738+
739739
// Getting only valid data
740740
$validData = $validation->getValidData();
741741
$this->assertEquals($validData, [
@@ -914,13 +914,22 @@ public function testGetValidData()
914914
'something',
915915
'[email protected]'
916916
],
917+
'stuffs' => [
918+
'one' => '1',
919+
'two' => '2',
920+
'three' => 'three',
921+
],
917922
'thing' => 'exists',
918923
], [
919924
'thing' => 'required',
920925
'items.*.product_id' => 'required|numeric',
921926
'emails.*' => 'required|email',
922927
'items.*.qty' => 'required|numeric',
923-
'something' => 'default:on|required|in:on,off'
928+
'something' => 'default:on|required|in:on,off',
929+
'stuffs' => 'required|array',
930+
'stuffs.one' => 'required|numeric',
931+
'stuffs.two' => 'required|numeric',
932+
'stuffs.three' => 'required|numeric',
924933
]);
925934

926935
$validData = $validation->getValidData();
@@ -936,8 +945,15 @@ public function testGetValidData()
936945
2 => '[email protected]'
937946
],
938947
'thing' => 'exists',
939-
'something' => 'on'
948+
'something' => 'on',
949+
'stuffs' => [
950+
'one' => '1',
951+
'two' => '2',
952+
]
940953
], $validData);
954+
955+
$stuffs = $validData['stuffs'];
956+
$this->assertFalse(isset($stuffs['three']));
941957
}
942958

943959
public function testGetInvalidData()
@@ -954,13 +970,22 @@ public function testGetInvalidData()
954970
'something',
955971
'[email protected]'
956972
],
973+
'stuffs' => [
974+
'one' => '1',
975+
'two' => '2',
976+
'three' => 'three',
977+
],
957978
'thing' => 'exists',
958979
], [
959980
'thing' => 'required',
960981
'items.*.product_id' => 'required|numeric',
961982
'emails.*' => 'required|email',
962983
'items.*.qty' => 'required|numeric',
963-
'something' => 'required|in:on,off'
984+
'something' => 'required|in:on,off',
985+
'stuffs' => 'required|array',
986+
'stuffs.one' => 'numeric',
987+
'stuffs.two' => 'numeric',
988+
'stuffs.three' => 'numeric',
964989
]);
965990

966991
$invalidData = $validation->getInvalidData();
@@ -974,8 +999,15 @@ public function testGetInvalidData()
974999
'emails' => [
9751000
1 => 'something'
9761001
],
977-
'something' => null
1002+
'something' => null,
1003+
'stuffs' => [
1004+
'three' => 'three',
1005+
]
9781006
], $invalidData);
1007+
1008+
$stuffs = $invalidData['stuffs'];
1009+
$this->assertFalse(isset($stuffs['one']));
1010+
$this->assertFalse(isset($stuffs['two']));
9791011
}
9801012

9811013
}

0 commit comments

Comments
 (0)