Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix #2189 BcValidation::checkDateの調査 #2902

Merged
merged 1 commit into from
Dec 1, 2023

Conversation

seto1
Copy link
Collaborator

@seto1 seto1 commented Nov 30, 2023

issue: #2189

cakeのdateTimeバリデーションと重複しているBcValidation::checkDateの削除を行いました。
また、このルールへの対応のためだけに必要な日時の変換処理も削除しています。

# 例
if (!empty($postData['posted'])) $postData['posted'] = new FrozenTime($postData['posted']);

ご確認をお願いします。

@ryuring
Copy link
Collaborator

ryuring commented Dec 1, 2023

@seto1 おお、変換処理がいらなくなるんですね、確認しました。ありがとうございます。

@ryuring ryuring merged commit fe66b59 into baserproject:dev-5 Dec 1, 2023
1 check passed
@ryuring ryuring added this to the 5.0.8 milestone Dec 1, 2023
@ryuring ryuring added the Todo 実行確定タスク label Dec 1, 2023
@ryuring ryuring self-requested a review December 1, 2023 00:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Todo 実行確定タスク
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants