Skip to content

Commit 7cefb4c

Browse files
2.12.24: «Direct use of $_REQUEST Superglobal detected»: mage2pro#30
1 parent dac5ec1 commit 7cefb4c

File tree

4 files changed

+9
-5
lines changed

4 files changed

+9
-5
lines changed

Framework/Request.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ class Request {
1111
* @return array(string => mixed)|mixed|null
1212
*/
1313
static function clean($k = null, $d = null) {return dfak(function() {return
14-
dfa_unset($_REQUEST, self::extraKeysRaw())
14+
dfa_unset(df_request(), self::extraKeysRaw())
1515
;}, $k, $d);}
1616

1717
/**
@@ -25,14 +25,14 @@ static function clean($k = null, $d = null) {return dfak(function() {return
2525
static function extra($k = null, $d = null) {return dfak(function() {return
2626
dfa_key_transform(function($k) {return
2727
df_trim_text_left($k, 'df-')
28-
;}, dfa_select($_REQUEST, self::extraKeysRaw()))
28+
;}, dfa_select(df_request(), self::extraKeysRaw()))
2929
;}, $k, $d);}
3030

3131
/**
3232
* 2017-01-01
3333
* @return array(string => mixed)
3434
*/
3535
private static function extraKeysRaw() {return dfcf(function() {return array_filter(
36-
array_keys($_REQUEST), function($k) {return df_starts_with($k, 'df-');}
36+
array_keys(df_request()), function($k) {return df_starts_with($k, 'df-');}
3737
);});}
3838
}

Framework/lib/http.php

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -131,12 +131,16 @@ function df_http_json_c($urlBase, array $params = [], $timeout = null) {return d
131131
* @used-by df_store()
132132
* @used-by \Df\Backend\Model\Auth::loginByEmail()
133133
* @used-by \Df\Framework\Plugin\Data\Form\Element\Fieldset::beforeAddField()
134+
* @used-by \Df\Framework\Request::clean()
135+
* @used-by \Df\Framework\Request::extra()
136+
* @used-by \Df\Framework\Request::extraKeysRaw()
134137
* @used-by \Df\GoogleFont\Controller\Index\Index::execute()
135138
* @used-by \Df\GoogleFont\Controller\Index\Preview::familyA()
136139
* @used-by \Df\GoogleFont\Font\Variant\Preview\Params::fromRequest()
137140
* @used-by \Df\OAuth\App::getAndSaveTheRefreshToken()
138141
* @used-by \Df\OAuth\App::state()
139142
* @used-by \Df\OAuth\ReturnT::redirectUrl()
143+
* @used-by \Df\Payment\CustomerReturn::execute()
140144
* @used-by \Df\Payment\CustomerReturn::isSuccess()
141145
* @used-by \Dfe\AllPay\Controller\CustomerReturn\Index::message()
142146
* @used-by \Dfe\AmazonLogin\Customer::url()

Payment/CustomerReturn.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ class CustomerReturn extends Action {
3535
*/
3636
function execute() {
3737
if ($this->needLog()) {
38-
dfp_report($this->m(), $_REQUEST, 'customerReturn');
38+
dfp_report($this->m(), df_request(), 'customerReturn');
3939
}
4040
$ss = df_checkout_session(); /** @var Session $ss */
4141
/** @var O|DFO|null $o */ /** @var Redirect $result */

composer.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "mage2pro/core"
3-
,"version": "2.12.23"
3+
,"version": "2.12.24"
44
,"description": "Mage2.PRO core package."
55
,"type": "magento2-module"
66
,"homepage": "https://mage2.pro"

0 commit comments

Comments
 (0)