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

商品CSVアップロード時に商品画像に空文字が設定されていれば登録対象外とするように修正 #2161

Merged
merged 3 commits into from Mar 3, 2017

Conversation

k-yamamura
Copy link
Contributor

概要(Overview・Refs Issue)

#2160
管理画面「商品CSV登録」にて、「商品画像」の列の先頭や末尾にカンマがある等場合にfile_nameが空のProductImageエンティティが生成される対応

方針(Policy)

商品画像名がセットされていなければ対象外とし、ProductImageエンティティが生成されないように修正

@ryo-endo ryo-endo added this to the 3.0.14 milestone Mar 1, 2017
@ryo-endo
Copy link
Contributor

ryo-endo commented Mar 1, 2017

@t-nagahashi
Str::trimAllしているけど、スペースを含んだファイル名のときに問題ないか、動作確認時に検証お願いします。

Copy link
Contributor

@t-nagahashi t-nagahashi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ソースコードの修正はOKです。

@t-nagahashi
Copy link
Contributor

t-nagahashi commented Mar 2, 2017

動作確認も問題ありませんでした。

以下値を登録し、再現手順を実施しても、ディレクトリから削除されないことを確認済
""
" "
","
", "
"xxx.jpg, "
"xxx.jpg,"

Str::trimAllしているけど、スペースを含んだファイル名のときに問題ないか、動作確認時に検証お願いします。

ファイル名の中に半角スペースがあるファイル名がアップロードでき正常に表示されることも確認いたしました。

@ryo-endo
Copy link
Contributor

ryo-endo commented Mar 3, 2017

bug:Criticalですので、3.0.14に含めます。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants