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

LC_Page_Admin_Products_Product::lfHasSameProductImage() 「$this->objUpFile」って存在しないかと #100

Closed
seasoftjapan opened this issue Aug 11, 2016 · 3 comments
Assignees
Labels
Milestone

Comments

@seasoftjapan
Copy link
Contributor

No description provided.

@seasoftjapan seasoftjapan changed the title LC_Page_Admin_Products_Product::lfHasSameProductImage 「$this->objUpFile」って存在しないかと LC_Page_Admin_Products_Product::lfHasSameProductImage() 「$this->objUpFile」って存在しないかと Aug 11, 2016
@Yangsin Yangsin added the bug label Aug 11, 2016
@ryo-endo ryo-endo added this to the 2.17.0 milestone Oct 26, 2017
@so-amuamu so-amuamu added bug:Low and removed bug labels Jul 31, 2018
@so-amuamu
Copy link

古いコードが残っている模様。
引数渡しに書き換えが必要。

@seasoftjapan seasoftjapan self-assigned this Jun 18, 2019
@seasoftjapan
Copy link
Contributor Author

テストデータを作成し、変更対象以外の画像項目での利用を検出できない (おそらく不具合) ことを確認した。

未検証だが、(「商品登録」画面で登録した商品では該当するデータは発生しないと思うが)「商品登録CSV」画面ではテストデータ同様のデータ状態を発生させられると思う。

seasoftjapan added a commit to seasoftjapan/eccube-2_13 that referenced this issue Jun 18, 2019
seasoftjapan added a commit to seasoftjapan/eccube-2_13 that referenced this issue Jun 19, 2019
@chihiro-adachi chihiro-adachi modified the milestones: 2.17.0, 2.17.1 Oct 31, 2019
seasoftjapan added a commit to seasoftjapan/eccube-2_13 that referenced this issue May 4, 2020
seasoftjapan added a commit that referenced this issue May 7, 2020
@okazy
Copy link
Contributor

okazy commented May 20, 2020

#278 こちらで修正されましたのでクローズします。

@okazy okazy closed this as completed May 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants