From e5c53eb7f4c198b2c67ce7ee44fbd734ea78d6dc Mon Sep 17 00:00:00 2001 From: Kentaro Ohkouchi Date: Wed, 20 May 2020 09:18:21 +0900 Subject: [PATCH 1/2] =?UTF-8?q?get=5Fmagic=5Fquotes=5Fgpc()=20=E3=81=AF=20?= =?UTF-8?q?PHP5.4=E4=BB=A5=E9=99=8D,=20=E5=B8=B8=E3=81=AB=20false=20?= =?UTF-8?q?=E3=81=AE=E3=81=9F=E3=82=81=E3=81=93=E3=81=AE=E5=87=A6=E7=90=86?= =?UTF-8?q?=E3=81=AF=E9=80=9A=E3=82=89=E3=81=AA=E3=81=84?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - see https://www.php.net/get_magic_quotes_gpc --- data/class/SC_Initial.php | 33 --------------------------------- 1 file changed, 33 deletions(-) diff --git a/data/class/SC_Initial.php b/data/class/SC_Initial.php index 6d0422be45..191f78e683 100644 --- a/data/class/SC_Initial.php +++ b/data/class/SC_Initial.php @@ -54,7 +54,6 @@ public function init() $this->complementParameter(); // defineConstants メソッドより後で実行 $this->phpconfigInit(); // defineConstants メソッドより後で実行 $this->createCacheDir(); // defineConstants メソッドより後で実行 - $this->stripslashesDeepGpc(); $this->resetSuperglobalsRequest(); // stripslashesDeepGpc メソッドより後で実行 $this->setTimezone(); // 本当はエラーハンドラーより先に読みたい気も $this->normalizeHostname(); // defineConstants メソッドより後で実行 @@ -447,38 +446,6 @@ public function defineConstants() define('UNLIMITED_FLG_LIMITED', '0'); } - /** - * クォートされた文字列のクォート部分を再帰的に取り除く. - * - * {@link http://jp2.php.net/manual/ja/function.get-magic-quotes-gpc.php PHP Manual} の記事を参考に実装。 - * $_REQUEST は後続の処理で再構成されるため、本処理では外している。 - * この関数は, PHP5以上を対象とし, PHP4 の場合は何もしない. - * - * @return void - */ - public function stripslashesDeepGpc() - { - // Strip magic quotes from request data. - if (get_magic_quotes_gpc() - && version_compare(PHP_VERSION, '5.0.0', '>=')) { - // Create lamba style unescaping function (for portability) - $quotes_sybase = strtolower(ini_get('magic_quotes_sybase')); - $unescape_function = (empty($quotes_sybase) || $quotes_sybase === 'off') ? 'stripslashes($value)' : 'str_replace("\'\'","\'",$value)'; - $stripslashes_deep = create_function('&$value, $fn', ' - if (is_string($value)) { - $value = ' . $unescape_function . '; - } elseif (is_array($value)) { - foreach ($value as &$v) $fn($v, $fn); - } - '); - - // Unescape data - $stripslashes_deep($_POST, $stripslashes_deep); - $stripslashes_deep($_GET, $stripslashes_deep); - $stripslashes_deep($_COOKIE, $stripslashes_deep); - } - } - /** * スーパーグローバル変数「$_REQUEST」を再セット * From eecf8927a3817d942968ab496cbdedd3aed11cdb Mon Sep 17 00:00:00 2001 From: Kentaro Ohkouchi Date: Wed, 20 May 2020 11:15:58 +0900 Subject: [PATCH 2/2] =?UTF-8?q?=E3=82=B0=E3=83=A9=E3=83=95=E8=A1=A8?= =?UTF-8?q?=E7=A4=BA=E3=81=8C=E3=82=A8=E3=83=A9=E3=83=BC=E3=81=AB=E3=81=AA?= =?UTF-8?q?=E3=82=8B=E3=81=9F=E3=82=81=20waitForElement=20=E3=81=AB?= =?UTF-8?q?=E4=BF=AE=E6=AD=A3?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../acceptance/admin/total/AdminTotalCept.php | 72 +++++++++---------- 1 file changed, 36 insertions(+), 36 deletions(-) diff --git a/ctests/acceptance/admin/total/AdminTotalCept.php b/ctests/acceptance/admin/total/AdminTotalCept.php index d8fde68bf9..64ad98ddb0 100644 --- a/ctests/acceptance/admin/total/AdminTotalCept.php +++ b/ctests/acceptance/admin/total/AdminTotalCept.php @@ -31,50 +31,50 @@ $I->click('月度で集計する'); $I->expect('グラフの表示を確認する'); -$I->seeElement(['css' => '#graph-image > img']); +$I->waitForElement(['css' => '#graph-image > img']); $I->expect('表の表示を確認する'); -$I->seeElement(['id' => 'total-term']); +$I->waitForElement(['id' => 'total-term']); $I->amGoingTo('売上集計>期間別集計>期間集計'); $I->selectOption('select[name=search_startyear]', date('Y', strtotime('-1 year'))); $I->click('期間で集計する'); $I->expect('グラフの表示を確認する'); -$I->seeElement(['css' => '#graph-image > img']); +$I->waitForElement(['css' => '#graph-image > img']); $I->expect('表の表示を確認する'); -$I->seeElement(['id' => 'total-term']); +$I->waitForElement(['id' => 'total-term']); $I->amGoingTo('売上集計>期間別集計>期間集計>月別'); $I->click('月別'); $I->expect('グラフの表示を確認する'); -$I->seeElement(['css' => '#graph-image > img']); +$I->waitForElement(['css' => '#graph-image > img']); $I->expect('表の表示を確認する'); -$I->seeElement(['id' => 'total-term']); +$I->waitForElement(['id' => 'total-term']); $I->amGoingTo('売上集計>期間別集計>期間集計>年別'); $I->click('年別'); $I->expect('グラフの表示を確認する'); -$I->seeElement(['css' => '#graph-image > img']); +$I->waitForElement(['css' => '#graph-image > img']); $I->expect('表の表示を確認する'); -$I->seeElement(['id' => 'total-term']); +$I->waitForElement(['id' => 'total-term']); $I->amGoingTo('売上集計>期間別集計>期間集計>曜日別'); $I->click('曜日別'); $I->expect('グラフの表示を確認する'); -$I->seeElement(['css' => '#graph-image > img']); +$I->waitForElement(['css' => '#graph-image > img']); $I->expect('表の表示を確認する'); -$I->seeElement(['id' => 'total-term']); +$I->waitForElement(['id' => 'total-term']); $I->amGoingTo('売上集計>期間別集計>期間集計>時間別'); $I->click('時間別'); $I->expect('グラフの表示を確認する'); -$I->seeElement(['css' => '#graph-image > img']); +$I->waitForElement(['css' => '#graph-image > img']); $I->expect('表の表示を確認する'); -$I->seeElement(['id' => 'total-term']); +$I->waitForElement(['id' => 'total-term']); $I->click('CSVダウンロード'); $file = $I->getLastDownloadFile('/^total\d{12}\.csv$/'); @@ -99,34 +99,34 @@ $I->click('月度で集計する'); $I->expect('グラフの表示を確認する'); -$I->seeElement(['css' => '#graph-image > img']); +$I->waitForElement(['css' => '#graph-image > img']); $I->expect('表の表示を確認する'); -$I->seeElement(['id' => 'total-products']); +$I->waitForElement(['id' => 'total-products']); $I->amGoingTo('売上集計>商品集計>期間集計'); $I->selectOption('select[name=search_startyear]', date('Y', strtotime('-1 year'))); $I->click('期間で集計する'); $I->expect('グラフの表示を確認する'); -$I->seeElement(['css' => '#graph-image > img']); +$I->waitForElement(['css' => '#graph-image > img']); $I->expect('表の表示を確認する'); -$I->seeElement(['id' => 'total-products']); +$I->waitForElement(['id' => 'total-products']); $I->amGoingTo('売上集計>商品集計>期間集計>会員'); $I->click('会員'); $I->expect('グラフの表示を確認する'); -$I->seeElement(['css' => '#graph-image > img']); +$I->waitForElement(['css' => '#graph-image > img']); $I->expect('表の表示を確認する'); -$I->seeElement(['id' => 'total-products']); +$I->waitForElement(['id' => 'total-products']); $I->amGoingTo('売上集計>商品集計>期間集計>非会員'); $I->click('非会員'); $I->expect('グラフの表示を確認する'); -$I->seeElement(['css' => '#graph-image > img']); +$I->waitForElement(['css' => '#graph-image > img']); $I->expect('表の表示を確認する'); -$I->seeElement(['id' => 'total-products']); +$I->waitForElement(['id' => 'total-products']); $I->click('CSVダウンロード'); $file = $I->getLastDownloadFile('/^total\d{12}\.csv$/'); @@ -151,34 +151,34 @@ $I->click('月度で集計する'); $I->expect('グラフの表示を確認する'); -$I->seeElement(['css' => '#graph-image > img']); +$I->waitForElement(['css' => '#graph-image > img']); $I->expect('表の表示を確認する'); -$I->seeElement(['id' => 'total-age']); +$I->waitForElement(['id' => 'total-age']); $I->amGoingTo('売上集計>年代別集計>期間集計'); $I->selectOption('select[name=search_startyear]', date('Y', strtotime('-1 year'))); $I->click('期間で集計する'); $I->expect('グラフの表示を確認する'); -$I->seeElement(['css' => '#graph-image > img']); +$I->waitForElement(['css' => '#graph-image > img']); $I->expect('表の表示を確認する'); -$I->seeElement(['id' => 'total-age']); +$I->waitForElement(['id' => 'total-age']); $I->amGoingTo('売上集計>年代別集計>期間集計>会員'); $I->click('会員'); $I->expect('グラフの表示を確認する'); -$I->seeElement(['css' => '#graph-image > img']); +$I->waitForElement(['css' => '#graph-image > img']); $I->expect('表の表示を確認する'); -$I->seeElement(['id' => 'total-age']); +$I->waitForElement(['id' => 'total-age']); $I->amGoingTo('売上集計>年代別集計>期間集計>非会員'); $I->click('非会員'); $I->expect('グラフの表示を確認する'); -$I->seeElement(['css' => '#graph-image > img']); +$I->waitForElement(['css' => '#graph-image > img']); $I->expect('表の表示を確認する'); -$I->seeElement(['id' => 'total-age']); +$I->waitForElement(['id' => 'total-age']); $I->click('CSVダウンロード'); $file = $I->getLastDownloadFile('/^total\d{12}\.csv$/'); @@ -203,18 +203,18 @@ $I->click('月度で集計する'); $I->expect('グラフの表示を確認する'); -$I->seeElement(['css' => '#graph-image > img']); +$I->waitForElement(['css' => '#graph-image > img']); $I->expect('表の表示を確認する'); -$I->seeElement(['id' => 'total-job']); +$I->waitForElement(['id' => 'total-job']); $I->amGoingTo('売上集計>職業別集計>期間集計'); $I->selectOption('select[name=search_startyear]', date('Y', strtotime('-1 year'))); $I->click('期間で集計する'); $I->expect('グラフの表示を確認する'); -$I->seeElement(['css' => '#graph-image > img']); +$I->waitForElement(['css' => '#graph-image > img']); $I->expect('表の表示を確認する'); -$I->seeElement(['id' => 'total-job']); +$I->waitForElement(['id' => 'total-job']); $I->click('CSVダウンロード'); $file = $I->getLastDownloadFile('/^total\d{12}\.csv$/'); @@ -239,18 +239,18 @@ $I->click('月度で集計する'); $I->expect('グラフの表示を確認する'); -$I->seeElement(['css' => '#graph-image > img']); +$I->waitForElement(['css' => '#graph-image > img']); $I->expect('表の表示を確認する'); -$I->seeElement(['id' => 'total-member']); +$I->waitForElement(['id' => 'total-member']); $I->amGoingTo('売上集計会員別集計>期間集計'); $I->selectOption('select[name=search_startyear]', date('Y', strtotime('-1 year'))); $I->click('期間で集計する'); $I->expect('グラフの表示を確認する'); -$I->seeElement(['css' => '#graph-image > img']); +$I->waitForElement(['css' => '#graph-image > img']); $I->expect('表の表示を確認する'); -$I->seeElement(['id' => 'total-member']); +$I->waitForElement(['id' => 'total-member']); $I->click('CSVダウンロード'); $file = $I->getLastDownloadFile('/^total\d{12}\.csv$/');