Skip to content

Commit 1f2c017

Browse files
authored
Merge pull request #1132 from CakeDC/hotifx/backport-validation-fix
fix tests failing
2 parents 54eea89 + e953c58 commit 1f2c017

File tree

3 files changed

+8
-9
lines changed

3 files changed

+8
-9
lines changed

src/Model/Behavior/RegisterBehavior.php

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -163,7 +163,7 @@ protected function _emailValidator(Validator $validator, $validateEmail)
163163
{
164164
$this->validateEmail = (bool)$validateEmail;
165165
$validator
166-
->add('email', 'valid', ['rule' => 'email'])
166+
->add('email', 'valid_email', ['rule' => 'email'])
167167
->notBlank('email', __d('cake_d_c/users', 'This field is required'), function ($context) {
168168
return $this->validateEmail;
169169
});
@@ -197,8 +197,7 @@ public function getRegisterValidators($options)
197197
$validateEmail = $options['validate_email'] ?? null;
198198
$useTos = $options['use_tos'] ?? null;
199199

200-
$validator = $this->_table->validationDefault(new Validator());
201-
$validator = $this->_table->validationRegister($validator);
200+
$validator = $this->_table->validationRegister(new Validator());
202201
if ($useTos) {
203202
$validator = $this->_tosValidator($validator);
204203
}

src/Model/Table/SocialAccountsTable.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ public function initialize(array $config): void
5959
public function validationDefault(Validator $validator): Validator
6060
{
6161
$validator
62-
->add('id', 'valid', ['rule' => 'uuid'])
62+
->add('id', 'valid_id', ['rule' => 'uuid'])
6363
->allowEmptyString('id', null, 'create');
6464

6565
$validator
@@ -91,11 +91,11 @@ public function validationDefault(Validator $validator): Validator
9191
->allowEmptyString('token_secret');
9292

9393
$validator
94-
->add('token_expires', 'valid', ['rule' => 'datetime'])
94+
->add('token_expires', 'valid_social_token_expires', ['rule' => 'datetime'])
9595
->allowEmptyString('token_expires');
9696

9797
$validator
98-
->add('active', 'valid', ['rule' => 'boolean'])
98+
->add('active', 'valid_active', ['rule' => 'boolean'])
9999
->requirePresence('active', 'create')
100100
->notBlank('active');
101101

src/Model/Table/UsersTable.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -158,18 +158,18 @@ public function validationDefault(Validator $validator): Validator
158158
->allowEmptyString('token');
159159

160160
$validator
161-
->add('token_expires', 'valid', ['rule' => 'datetime'])
161+
->add('token_expires', 'valid_user_token_expires', ['rule' => 'datetime'])
162162
->allowEmptyDateTime('token_expires');
163163

164164
$validator
165165
->allowEmptyString('api_token');
166166

167167
$validator
168-
->add('activation_date', 'valid', ['rule' => 'datetime'])
168+
->add('activation_date', 'valid_activation_date', ['rule' => 'datetime'])
169169
->allowEmptyDateTime('activation_date');
170170

171171
$validator
172-
->add('tos_date', 'valid', ['rule' => 'datetime'])
172+
->add('tos_date', 'valid_tos_date', ['rule' => 'datetime'])
173173
->allowEmptyDateTime('tos_date');
174174

175175
return $validator;

0 commit comments

Comments
 (0)