Ticket #822 (closed バグ指摘: 修正済)

Opened 10 years ago

Last modified 10 years ago

商品CSVダウンロードの文字列が""で囲まれない

Reported by: kim Owned by: somebody
Priority: Milestone: EC-CUBE2.5.0beta
Component: 管理画面 Version: 2.4.4
Keywords: Cc:
修正済み:

Description (last modified by kim) (diff)

商品マスタから検索結果をCSVダウンロードした際に、文字列が""で囲われず、ダウンロードしたCSVを利用して、CSVアップロードを行うと""で囲われていない文字列の1文字目が消えてしまう。

Windows環境では大丈夫でしたがLinux環境では1文字目が消えてしまいました。

#577 にたいして、 r18476 の修正のSC_Helper_CSV.phpのsfArrayToCsvの以下の箇所が、文字列中に"や,がなければ""で囲わないようになっているためのようです。

431 /* enclose a field that contains a delimiter, an enclosure character, or a newline */ 
432	            if ( 
433	                   is_string($field) 
434	                && preg_match('/[' . preg_quote($delimiter) . preg_quote($enclosure) . '\\s]/', $field) 
435	            ) { 
436	                $field = $enclosure . preg_replace('/' . preg_quote($enclosure) . '/', $enclosure . $enclosure, $field) . $enclosure; 
437	            } 

Change History

comment:1 Changed 10 years ago by Seasoft

  • Status changed from new to closed
  • Resolution set to 無効

バージョン「2.4.4」との事ですが、チェンジセット「r17799」はコミュニティ版のものです。

バージョンに付きまして「コミュニティ版」または、「2.5-dev」での確認をお願いします。

確認の結果、不具合の再現が確認されましたら、チケットの差し戻しをお願いいたします。

comment:2 follow-up: ↓ 3 Changed 10 years ago by kim

  • Status changed from closed to reopened
  • Resolution 無効 deleted
  • Description modified (diff)

r18476 でしたので、説明を修正させていただきました。 バージョンは2.4.4のままとさせていただいております。

商品規格周りの修正作業中のため、確認はおこなえておりませんが、 コミュニティ版の内容を反映したもののため、2.5-devでの再現の可能性も高いと思われます。

comment:3 in reply to: ↑ 2 Changed 10 years ago by Seasoft

kim への返信

商品規格周りの修正作業中のため、確認はおこなえておりませんが、 コミュニティ版の内容を反映したもののため、2.5-devでの再現の可能性も高いと思われます。

前述の通り動作確認はコミュニティ版で大丈夫です。

それによって、2.5系での状況は推測可能です。

comment:4 Changed 10 years ago by AMUAMU

  • Status changed from reopened to closed
  • Resolution set to 修正済

#657 (CSVダウンロードの改善)のRFC4180準拠(""で囲まれている必要性は無い)と矛盾している事と、再現が出来ない為、現マイルストーンでクローズにします。 再現性があればコミュニティフォーラムで環境などについて確認の上、reopenしてください。

comment:5 Changed 10 years ago by AMUAMU

  • Milestone set to EC-CUBE2.5.0beta
Note: See TracTickets for help on using tickets.