[Svn-src-all:1454] [version-2_5-dev 20346] #1031 商品CSVアップロードリファクタリング
adachi
admin ＠ mail.ec-cube.net
2011年 2月 23日 (水) 17:21:51 JST
Subversion committed to /home/svn/open 20346
http://svn.ec-cube.net/open_trac/changeset/20346
┌────────────────────────────┐
│更新者 : adachi │
│更新日時: 2011-02-23 17:21:50 +0900 (水, 23 2月 2011)│
└────────────────────────────┘
Log:
--------------------------------------------------------
#1031 商品CSVアップロードリファクタリング
Changed: [U:修正，A:追加，D:削除]
--------------------------------------------------------
U branches/version-2_5-dev/data/Smarty/templates/admin/products/upload_csv.tpl
A branches/version-2_5-dev/data/Smarty/templates/admin/products/upload_csv_complete.tpl
U branches/version-2_5-dev/data/class/pages/admin/products/LC_Page_Admin_Products_UploadCSV.php
変更: branches/version-2_5-dev/data/Smarty/templates/admin/products/upload_csv.tpl
===================================================================
--- branches/version-2_5-dev/data/Smarty/templates/admin/products/upload_csv.tpl 2011-02-23 07:54:00 UTC (rev 20345)
+++ branches/version-2_5-dev/data/Smarty/templates/admin/products/upload_csv.tpl 2011-02-23 08:21:50 UTC (rev 20346)
@@ -21,20 +21,7 @@
* Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.
*/
*}-->
-<script type="text/javascript">
-<!--
-self.moveTo(20,20);self.focus();
-
-function lfPopwinSubmit(formName) {
- win02('about:blank','csv_upload','500','400');
- document[formName].target = "csv_upload";
- document[formName].submit();
- return false;
-}
-//-->
-</script>
-
-<form name="form1" id="form1" method="post" action="?" enctype="multipart/form-data" onSubmit="lfPopwinSubmit('form1');">
+<form name="form1" id="form1" method="post" action="?" enctype="multipart/form-data"">
<input type="hidden" name="mode" value="csv_upload" />
<div id="products" class="contents-main">
<!--{if $tpl_errtitle != ""}-->
@@ -73,7 +60,7 @@
<!--▲登録テーブルここまで-->
<div class="btn-area">
<ul>
- <li><a class="btn-action" href="javascript:;" onclick="lfPopwinSubmit('form1'); return false;"><span class="btn-next">この内容で登録する</span></a></li>
+ <li><a class="btn-action" href="javascript:;" onclick="fnFormModeSubmit('form1', 'csv_upload', '', ''); return false;"><span class="btn-next">この内容で登録する</span></a></li>
</ul>
</div>
</div>
追加: branches/version-2_5-dev/data/Smarty/templates/admin/products/upload_csv_complete.tpl
===================================================================
--- branches/version-2_5-dev/data/Smarty/templates/admin/products/upload_csv_complete.tpl (rev 0)
+++ branches/version-2_5-dev/data/Smarty/templates/admin/products/upload_csv_complete.tpl 2011-02-23 08:21:50 UTC (rev 20346)
@@ -0,0 +1,55 @@
+<!--{*
+/*
+ * This file is part of EC-CUBE
+ *
+ * Copyright(c) 2000-2010 LOCKON CO.,LTD. All Rights Reserved.
+ *
+ * http://www.lockon.co.jp/
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License
+ * as published by the Free Software Foundation; either version 2
+ * of the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.
+ */
+*}-->
+<div id="products" class="contents-main">
+ <div class="message">
+ <span>CSV登録を実行しました。</span>
+ </div>
+ <!--{if $arrRowErr}-->
+ <table class="form">
+ <tr>
+ <td>
+ <!--{foreach item=err from=$arrRowErr}-->
+ <span class="attention"><!--{$err}--></span>
+ <!--{/foreach}-->
+ </td>
+ </tr>
+ </table>
+ <!--{/if}-->
+ <!--{if $arrRowResult}-->
+ <table class="form">
+ <tr>
+ <td>
+ <!--{foreach item=result from=$arrRowResult}-->
+ <span><!--{$result}--><br/></span>
+ <!--{/foreach}-->
+ </td>
+ </tr>
+ </table>
+ <!--{/if}-->
+ <div class="btn-area">
+ <ul>
+ <li><a class="btn-action" href="./upload_csv.php"><span class="btn-prev">戻る</span></a></li>
+ </ul>
+ </div>
+</div>
Property changes on: branches/version-2_5-dev/data/Smarty/templates/admin/products/upload_csv_complete.tpl
___________________________________________________________________
追加: svn:mime-type
+ text/html
追加: svn:eol-style
+ LF
変更: branches/version-2_5-dev/data/class/pages/admin/products/LC_Page_Admin_Products_UploadCSV.php
===================================================================
--- branches/version-2_5-dev/data/class/pages/admin/products/LC_Page_Admin_Products_UploadCSV.php 2011-02-23 07:54:00 UTC (rev 20345)
+++ branches/version-2_5-dev/data/class/pages/admin/products/LC_Page_Admin_Products_UploadCSV.php 2011-02-23 08:21:50 UTC (rev 20346)
@@ -57,6 +57,10 @@
/** 登録フォームカラム情報 **/
var $arrFormKeyList;
+ var $arrRowErr;
+
+ var $arrRowResult;
+
/**
* Page を初期化する.
*
@@ -99,21 +103,20 @@
*/
function action() {
$this->objDb = new SC_Helper_DB_Ex();
- $objView = new SC_SiteView_Ex();
// ファイル管理クラス
$this->objUpFile = new SC_UploadFile(IMAGE_TEMP_REALDIR, IMAGE_SAVE_REALDIR);
// サイト基本情報 (ポイントレート初期値用)
$this->arrInfo = $this->objDb->sfGetBasisData();
// CSV管理ヘルパー
- $this->objCSV = new SC_Helper_CSV();
+ $this->objCSV = new SC_Helper_CSV_Ex();
// CSV構造読み込み
$arrCSVFrame = $this->objCSV->sfGetCsvOutput($this->csv_id);
// CSV構造がインポート可能かのチェック
if( !$this->objCSV->sfIsImportCSVFrame($arrCSVFrame) ) {
// 無効なフォーマットなので初期状態に強制変更
- $arrCSVFram = $this->objCSV->sfGetCsvOutput($this->csv_id, '', array(), $order ='no');
+ $arrCSVFrame = $this->objCSV->sfGetCsvOutput($this->csv_id, '', array(), 'no');
$this->tpl_is_format_default = true;
}
// CSV構造は更新可能なフォーマットかのフラグ取得
@@ -125,141 +128,149 @@
$this->objFormParam = new SC_FormParam();
// パラメータ情報の初期化
$this->lfInitParam($arrCSVFrame);
- // 現在のフォーマットにおける列数を取得
- $col_count = $this->objFormParam->getCount();
+
$this->objFormParam->setHtmlDispNameArray();
$this->arrTitle = $this->objFormParam->getHtmlDispNameArray();
switch($this->getMode()) {
case 'csv_upload':
- // 登録先テーブル カラム情報の初期化
- $this->lfInitTableInfo();
- // 登録フォーム カラム情報
- $this->arrFormKeyList = $this->objFormParam->getKeyList();
+ $this->doUploadCsv();
+ break;
+ default:
+ break;
+ }
+ }
- $err = false;
- // CSVファイルアップロード エラーチェック
- $arrErr['csv_file'] = $this->objUpFile->makeTempFile('csv_file');
+ /**
+ * 登録/編集結果のメッセージをプロパティへ追加する
+ *
+ * @param integer $line_count 行数
+ * @param stirng $message メッセージ
+ * @return void
+ */
+ function addRowResult($line_count, $message) {
+ $this->arrRowResult[] = $line_count . "行目：" . $message;
+ }
- if($arrErr['csv_file'] == "") {
- $arrErr = $this->objUpFile->checkEXISTS();
- }
+ /**
+ * 登録/編集結果のエラーメッセージをプロパティへ追加する
+ *
+ * @param integer $line_count 行数
+ * @param stirng $message メッセージ
+ * @return void
+ */
+ function addRowErr($line_count, $message) {
+ $this->arrRowErr[] = $line_count . "行目：" . $message;
+ }
- $objView->assignobj($this);
- $objView->display('admin_popup_header.tpl');
+ /**
+ * CSVアップロードを実行します.
+ *
+ * @return void
+ */
+ function doUploadCsv() {
+ // ファイルアップロードのチェック
+ $this->objUpFile->makeTempFile('csv_file');
+ $arrErr = $this->objUpFile->checkExists();
+ if (count($arrErr) > 0) {
+ $this->arrErr = $arrErr;
+ return;
+ }
+ // 一時ファイル名の取得
+ $filepath = $this->objUpFile->getTempFilePath('csv_file');
+ // CSVファイルの文字コード変換
+ $enc_filepath = SC_Utils_Ex::sfEncodeFile($filepath, CHAR_CODE, CSV_TEMP_REALDIR);
+ // CSVファイルのオープン
+ $fp = fopen($enc_filepath, "r");
+ // 失敗した場合はエラー表示
+ if (!$fp) {
+ SC_Utils_Ex::sfDispError("");
+ }
+
+ // 登録先テーブル カラム情報の初期化
+ $this->lfInitTableInfo();
+
+ // 登録フォーム カラム情報
+ $this->arrFormKeyList = $this->objFormParam->getKeyList();
- // 実行時間を制限しない
- set_time_limit(0);
+ $err = false;
- // 出力をバッファリングしない(==日本語自動変換もしない)
- ob_end_flush();
+ // レコード行数を得る
+ $record_count = $this->objCSV->sfGetCSVRecordCount($fp);
+ // ファイルが無効な場合はエラー
+ if($record_count === FALSE) {
+ SC_Utils_Ex::sfDispError("");
+ }
- // IEのために256バイト空文字出力
- echo str_pad('',256);
+ // 登録対象の列数
+ $col_max_count = $this->objFormParam->getCount();
+ // 行数
+ $line_count = 0;
- if(empty($arrErr['csv_file'])) {
- // 一時ファイル名の取得
- $filepath = $this->objUpFile->getTempFilePath('csv_file');
- // エンコード
- $enc_filepath = SC_Utils_Ex::sfEncodeFile($filepath,
- CHAR_CODE, CSV_TEMP_REALDIR);
- $fp = fopen($enc_filepath, "r");
+ $objQuery =& SC_Query::getSingletonInstance();
+ $objQuery->begin();
- // 無効なファイルポインタが渡された場合はエラー表示
- if ($fp === false) {
- SC_Utils_Ex::sfDispError("");
- }
+ $errFlag = false;
- // レコード行数を得る
- $record_count = $this->objCSV->sfGetCSVRecordCount($fp);
- // ファイルが無効な場合はエラー
- if($record_count === FALSE) {
- SC_Utils_Ex::sfDispError("");
+ while (!feof($fp) && !$err) {
+ $arrCSV = fgetcsv($fp, CSV_LINE_MAX);
+ // 行カウント
+ $line_count++;
+ // ヘッダ行はスキップ
+ if ($line_count == 1) {
+ continue;
+ }
+ // 空行はスキップ
+ if (empty($arrCSV)) {
+ continue;
+ }
+ // 列数が異なる場合はエラー
+ if ($col_max_count != count($arrCSV)) {
+ $errFlag = true;
+ break;
+ }
+ // シーケンス配列を格納する。
+ $this->objFormParam->setParam($arrCSV, true);
+ $arrRet = $this->objFormParam->getHashArray();
+ $this->objFormParam->setParam($arrRet);
+ // 入力値の変換
+ $this->objFormParam->convParam();
+ // <br>なしでエラー取得する。
+ $arrCSVErr = $this->lfCheckError();
+
+ // 入力エラーチェック
+ if (count($arrCSVErr) > 0) {
+ foreach ($arrCSVErr as $err) {
+ $this->addRowErr($line_count, $err);
}
+ $errFlag = true;
+ break;
+ }
- $line = 0; // 現在行数
- $regist = 0; // 登録数
+ $this->lfRegistProduct($objQuery, $line_count);
+ $arrParam = $this->objFormParam->getHashArray();
- $objQuery =& SC_Query::getSingletonInstance();
- $objQuery->begin();
+ $this->addRowResult($line_count, "商品ID：".$arrParam['product_id'] . " / 商品名：" . $arrParam['name']);
+ }
- echo "■ CSV登録進捗状況 <br/><br/>\n";
+ // 実行結果画面を表示
+ $this->tpl_mainpage = 'products/upload_csv_complete.tpl';
- while(!feof($fp) && !$err) {
- $arrCSV = fgetcsv($fp, CSV_LINE_MAX);
- // 行カウント
- $line++;
- if($line <= 1) {
- continue;
- }
- // 現在行の項目数カウント
- $line_col_count = count($arrCSV);
- // 項目数が1以下の場合は無視する
- if($line_col_count <= 1) {
- continue;
- }
+ fclose($fp);
- // 項目数チェック
- if($line_col_count != $col_count) {
- echo "※ 項目数が" . $line_col_count . "個検出されました。項目数は" . $col_count . "個になります。</br>\n";
- $err = true;
- } else {
- // シーケンス配列を格納する。
- $this->objFormParam->setParam($arrCSV, true);
- $arrRet = $this->objFormParam->getHashArray();
- $this->objFormParam->setParam($arrRet);
- // 入力値の変換
- $this->objFormParam->convParam();
- // <br>なしでエラー取得する。
- $arrCSVErr = $this->lfCheckError();
- }
-
- // 入力エラーチェック
- if(count($arrCSVErr) > 0) {
- echo "<font color=\"red\">■ $line / $record_count 行目でエラーが発生しました。</font></br>\n";
- foreach($arrCSVErr as $val) {
- $this->printError($val);
- }
- $err = true;
- }
-
- if(!$err) {
- $this->lfRegistProduct($objQuery, $line);
- $regist++;
- }
- $arrParam = $this->objFormParam->getHashArray();
-
- if(!$err) echo $line. "行目 （商品ID：".$arrParam['product_id']." / 商品名：".$arrParam['name'].")\n<br />";
- flush();
- }
- fclose($fp);
-
- if(!$err) {
- $objQuery->commit();
- echo "■" . $regist . "件のレコードを登録しました。";
- // 商品件数カウント関数の実行
- $this->objDb->sfCountCategory($objQuery);
- $this->objDb->sfCountMaker($objQuery);
- } else {
- $objQuery->rollback();
- }
- } else {
- foreach($arrErr as $val) {
- $this->printError($val);
- }
- }
- echo "<br/><a href=\"javascript:window.close()\">→閉じる</a>";
- flush();
-
- $this->setTemplate('admin_popup_footer.tpl');
-
+ if ($errFlag) {
+ $objQuery->rollback();
return;
- break;
- default:
- break;
}
- }
+ $objQuery->commit();
+
+ // 商品件数カウント関数の実行
+ $this->objDb->sfCountCategory($objQuery);
+ $this->objDb->sfCountMaker($objQuery);
+ return;
+ }
+
/**
* デストラクタ.
*
@@ -275,8 +286,7 @@
* @return void
*/
function lfInitFile() {
- $this->objUpFile->addFile("CSVファイル", 'csv_file', array('csv'),
- CSV_SIZE, true, 0, 0, false);
+ $this->objUpFile->addFile("CSVファイル", 'csv_file', array('csv'), CSV_SIZE, true, 0, 0, false);
}
/**
@@ -709,7 +719,7 @@
function lfGetDbFormatTimeWithLine($line_no = '') {
$time = date("Y-m-d H:i:s");
// 秒以下を生成
- if($line != '') {
+ if($line_no != '') {
$microtime = sprintf("%06d", $line_no);
$time .= ".$microtime";
}
@@ -814,19 +824,4 @@
}
return true;
}
-
- /**
- * 引数の文字列をエラー出力する.
- *
- * 引数 $val の内容は, htmlspecialchars() によってサニタイズされる
- *
- * @param string $val 出力する文字列
- * @return void
- */
- function printError($val) {
- echo "<font color=\"red\">"
- . htmlspecialchars($val, ENT_QUOTES)
- . "</font><br />\n";
- }
}
-?>
Svn-src-all メーリングリストの案内