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

#1054 ドキュメント添付の際、ドキュメントレコードは作成されるがファイルアップロードが出来ない問題の対応 #1055

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

hasesho28
Copy link
Collaborator

関連Issue / Related Issue

不具合の内容 / Bug

  1. 掲題の通り

原因 / Cause

  1. ファイルアップロードの際、validateImageFile()にてバリデーションを行っているが、その実体であるvtiger_Functions::validateImage()の戻り値の型がPHP8.0対応においてboolean型に変更されていた
  2. そのため、問題がないとtrue == 'false'という比較になり、比較結果はtrueになるため、バリデーションエラーとして動作してしまっていた

変更内容 / Details of Change

  1. ファイルのバリデーション結果を格納している変数をbooleanで扱うよう変更

スクリーンショット / Screenshot

image

影響範囲 / Affected Area

ファイルアップロード機能

チェックリスト / Check List

  • 自らテストを行った
  • 不必要な変更が無い
  • 影響範囲の検討を行った

備考 / Remarks

@hasesho28 hasesho28 changed the title refs #1054 ドキュメント添付の際、ドキュメントレコードは作成されるがファイルアップロードが出来ない問題の対応 #1054 ドキュメント添付の際、ドキュメントレコードは作成されるがファイルアップロードが出来ない問題の対応 Jul 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant