Completed
Push — master ( 604742...5b304f )
by Marcus
02:07
created

Shoppingcart::getNotification()   C

Complexity

Conditions 12
Paths 6

Size

Total Lines 37
Code Lines 27

Duplication

Lines 0
Ratio 0 %

Importance

Changes 3
Bugs 0 Features 1
Metric Value
c 3
b 0
f 1
dl 0
loc 37
rs 5.1612
cc 12
eloc 27
nc 6
nop 0

How to fix   Complexity   

Long Method

Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.

For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.

Commonly applied refactorings include:

1
<?php
2
3
/*
4
    HCSF - A multilingual CMS and Shopsystem
5
    Copyright (C) 2014  Marcus Haase - [email protected]
6
7
    This program is free software: you can redistribute it and/or modify
8
    it under the terms of the GNU General Public License as published by
9
    the Free Software Foundation, either version 3 of the License, or
10
    (at your option) any later version.
11
12
    This program is distributed in the hope that it will be useful,
13
    but WITHOUT ANY WARRANTY; without even the implied warranty of
14
    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
15
    GNU General Public License for more details.
16
17
    You should have received a copy of the GNU General Public License
18
    along with this program.  If not, see <http://www.gnu.org/licenses/>.
19
 */
20
21
namespace HaaseIT\HCSF\Controller\Shop;
22
23
use HaaseIT\HCSF\HelperConfig;
24
use HaaseIT\Toolbox\Tools;
25
use HaaseIT\HCSF\Helper;
26
use HaaseIT\HCSF\Customer\Helper as CHelper;
27
use HaaseIT\HCSF\Shop\Helper as SHelper;
28
use HaaseIT\Toolbox\DBTools;
29
30
class Shoppingcart extends Base
31
{
32
    /**
33
     * @var \HaaseIT\Toolbox\Textcat
34
     */
35
    private $textcats;
36
37
    /**
38
     * Shoppingcart constructor.
39
     * @param \Zend\ServiceManager\ServiceManager $serviceManager
40
     */
41
    public function __construct(\Zend\ServiceManager\ServiceManager $serviceManager)
42
    {
43
        parent::__construct($serviceManager);
44
        $this->textcats = $this->serviceManager->get('textcats');
45
    }
46
47
    /**
48
     *
49
     */
50
    public function preparePage()
0 ignored issues
show
Coding Style introduced by
preparePage uses the super-global variable $_SESSION which is generally not recommended.

Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable:

// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

    public function __construct($host)
    {
        $this->host = $host;
    }

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
Loading history...
51
    {
52
        $this->P = new \HaaseIT\HCSF\CorePage($this->serviceManager);
53
        $this->P->cb_pagetype = 'contentnosubnav';
54
55
56
        if (HelperConfig::$shop['show_pricesonlytologgedin'] && !CHelper::getUserData()) {
57
            $this->P->oPayload->cl_html = $this->textcats->T('denied_notloggedin');
58
        } else {
59
            $this->P->cb_customcontenttemplate = 'shop/shoppingcart';
60
61
            // ----------------------------------------------------------------------------
62
            // Check if there is a message to display above the shoppingcart
63
            // ----------------------------------------------------------------------------
64
            $this->P->oPayload->cl_html = $this->getNotification();
65
66
            // ----------------------------------------------------------------------------
67
            // Display the shoppingcart
68
            // ----------------------------------------------------------------------------
69
            $aErr = [];
70
            if (isset($_SESSION['cart']) && count($_SESSION['cart']) >= 1) {
71
                if (filter_input(INPUT_POST, 'doCheckout') === 'yes') {
72
                    $aErr = CHelper::validateCustomerForm(HelperConfig::$lang, $aErr, true);
73 View Code Duplication
                    if (!CHelper::getUserData() && filter_input(INPUT_POST, 'tos') !== 'y') {
0 ignored issues
show
Duplication introduced by
This code seems to be duplicated across your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
74
                        $aErr['tos'] = true;
75
                    }
76 View Code Duplication
                    if (!CHelper::getUserData() && filter_input(INPUT_POST, 'cancellationdisclaimer') !== 'y') {
0 ignored issues
show
Duplication introduced by
This code seems to be duplicated across your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
77
                        $aErr['cancellationdisclaimer'] = true;
78
                    }
79
                    $postpaymentmethod = filter_input(INPUT_POST, 'paymentmethod');
80
                    if (
81
                        $postpaymentmethod === null
82
                        || in_array($postpaymentmethod, HelperConfig::$shop['paymentmethods'], true) === false
83
                    ) {
84
                        $aErr['paymentmethod'] = true;
85
                    }
86
                }
87
                $aShoppingcart = SHelper::buildShoppingCartTable($_SESSION['cart'], false, '', $aErr);
0 ignored issues
show
Documentation introduced by
$aErr is of type array, but the function expects a string.

It seems like the type of the argument is not accepted by the function/method which you are calling.

In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug.

We suggest to add an explicit type cast like in the following example:

function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
Loading history...
88
            }
89
90
            // ----------------------------------------------------------------------------
91
            // Checkout
92
            // ----------------------------------------------------------------------------
93
            if (!isset($aShoppingcart)) {
94
                $this->P->oPayload->cl_html .= $this->textcats->T('shoppingcart_empty');
95
            } else {
96
                if (filter_input(INPUT_POST, 'doCheckout') === 'yes' && count($aErr) === 0) {
97
                    $this->doCheckout();
98
                }
99
            }
100
101
            if (isset($aShoppingcart)) {
102
                $this->P->cb_customdata = $aShoppingcart;
103
            }
104
        }
105
    }
106
107
    /**
108
     * @param $aV
109
     * @return array
110
     */
111
    private function getItemImage($aV)
112
    {
113
        // base64 encode img and prepare for db
114
        // image/png image/jpeg image/gif
115
        // data:{mimetype};base64,XXXX
1 ignored issue
show
Unused Code Comprehensibility introduced by
45% of this comment could be valid code. Did you maybe forget this after debugging?

Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it.

The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production.

This check looks for comments that seem to be mostly valid code and reports them.

Loading history...
116
117
        $aImagesToSend = [];
118
        $base64Img = false;
119
        $binImg = false;
120
121
        if (HelperConfig::$shop['email_orderconfirmation_embed_itemimages_method'] === 'glide') {
122
            $sPathToImage = '/'.HelperConfig::$core['directory_images'].'/'.HelperConfig::$shop['directory_images_items'].'/';
123
            $sImageroot = PATH_BASEDIR . HelperConfig::$core['directory_glide_master'];
124
125
            if (
126
                is_file($sImageroot.substr($sPathToImage.$aV['img'], strlen(HelperConfig::$core['directory_images']) + 1))
127
                && $aImgInfo = getimagesize($sImageroot.substr($sPathToImage.$aV['img'], strlen(HelperConfig::$core['directory_images']) + 1))
128
            ) {
129
                $glideserver = \League\Glide\ServerFactory::create([
130
                    'source' => $sImageroot,
131
                    'cache' => PATH_GLIDECACHE,
132
                    'max_image_size' => HelperConfig::$core['glide_max_imagesize'],
133
                ]);
134
                $glideserver->setBaseUrl('/' . HelperConfig::$core['directory_images'] . '/');
135
                $base64Img = $glideserver->getImageAsBase64($sPathToImage.$aV['img'], HelperConfig::$shop['email_orderconfirmation_embed_itemimages_glideparams']);
136
                $TMP = explode(',', $base64Img);
137
                $binImg = base64_decode($TMP[1]);
138
                unset($TMP);
139
            }
140
        } else {
141
            $sPathToImage =
142
                PATH_DOCROOT.HelperConfig::$core['directory_images'].'/'
143
                .HelperConfig::$shop['directory_images_items'].'/'
144
                .HelperConfig::$shop['directory_images_items_email'].'/';
145
            if ($aImgInfo = getimagesize($sPathToImage.$aV['img'])) {
146
                $binImg = file_get_contents($sPathToImage.$aV['img']);
147
                $base64Img = 'data:' . $aImgInfo['mime'] . ';base64,';
148
                $base64Img .= base64_encode($binImg);
149
            }
150
        }
151
        if (HelperConfig::$shop['email_orderconfirmation_embed_itemimages']) {
152
            $aImagesToSend['binimg'] = $binImg;
153
        }
154
        if ($base64Img) {
0 ignored issues
show
Bug Best Practice introduced by
The expression $base64Img of type string|false is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== false instead.

In PHP, under loose comparison (like ==, or !=, or switch conditions), values of different types might be equal.

For string values, the empty string '' is a special case, in particular the following results might be unexpected:

''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
Loading history...
155
            $aImagesToSend['base64img'] = $base64Img;
156
        }
157
        return $aImagesToSend;
158
    }
159
160
    /**
161
     * @return array
162
     */
163
    private function prepareDataOrder()
0 ignored issues
show
Coding Style introduced by
prepareDataOrder uses the super-global variable $_SESSION which is generally not recommended.

Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable:

// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

    public function __construct($host)
    {
        $this->host = $host;
    }

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
Loading history...
Coding Style introduced by
prepareDataOrder uses the super-global variable $_POST which is generally not recommended.

Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable:

// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

    public function __construct($host)
    {
        $this->host = $host;
    }

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
Loading history...
164
    {
165
        $cartpricesums = $_SESSION['cartpricesums'];
166
        return [
167
            'o_custno' => filter_var(trim(Tools::getFormfield('custno')), FILTER_SANITIZE_STRING, FILTER_FLAG_STRIP_LOW),
168
            'o_email' => filter_var(trim(Tools::getFormfield('email')), FILTER_SANITIZE_EMAIL),
169
            'o_corpname' => filter_var(trim(Tools::getFormfield('corpname')), FILTER_SANITIZE_STRING, FILTER_FLAG_STRIP_LOW),
170
            'o_name' => filter_var(trim(Tools::getFormfield('name')), FILTER_SANITIZE_STRING, FILTER_FLAG_STRIP_LOW),
171
            'o_street' => filter_var(trim(Tools::getFormfield('street')), FILTER_SANITIZE_STRING, FILTER_FLAG_STRIP_LOW),
172
            'o_zip' => filter_var(trim(Tools::getFormfield('zip')), FILTER_SANITIZE_STRING, FILTER_FLAG_STRIP_LOW),
173
            'o_town' => filter_var(trim(Tools::getFormfield('town')), FILTER_SANITIZE_STRING, FILTER_FLAG_STRIP_LOW),
174
            'o_phone' => filter_var(trim(Tools::getFormfield('phone')), FILTER_SANITIZE_STRING, FILTER_FLAG_STRIP_LOW),
175
            'o_cellphone' => filter_var(trim(Tools::getFormfield('cellphone')), FILTER_SANITIZE_STRING, FILTER_FLAG_STRIP_LOW),
176
            'o_fax' => filter_var(trim(Tools::getFormfield('fax')), FILTER_SANITIZE_STRING, FILTER_FLAG_STRIP_LOW),
177
            'o_country' => filter_var(trim(Tools::getFormfield('country')), FILTER_SANITIZE_STRING, FILTER_FLAG_STRIP_LOW),
178
            'o_group' => trim(CHelper::getUserData('cust_group')),
179
            'o_remarks' => filter_var(trim(Tools::getFormfield('remarks')), FILTER_SANITIZE_STRING, FILTER_FLAG_STRIP_LOW),
180
            'o_tos' => (filter_input(INPUT_POST, 'tos') === 'y' || CHelper::getUserData()) ? 'y' : 'n',
181
            'o_cancellationdisclaimer' => (filter_input(INPUT_POST, 'cancellationdisclaimer') === 'y' || CHelper::getUserData()) ? 'y' : 'n',
182
            'o_paymentmethod' => filter_var(trim(Tools::getFormfield('paymentmethod')), FILTER_SANITIZE_STRING, FILTER_FLAG_STRIP_LOW),
183
            'o_sumvoll' => $cartpricesums['sumvoll'],
184
            'o_sumerm' => $cartpricesums['sumerm'],
185
            'o_sumnettoall' => $cartpricesums['sumnettoall'],
186
            'o_taxvoll' => $cartpricesums['taxvoll'],
187
            'o_taxerm' => $cartpricesums['taxerm'],
188
            'o_sumbruttoall' => $cartpricesums['sumbruttoall'],
189
            'o_mindermenge' => isset($cartpricesums['mindergebuehr']) ? $cartpricesums['mindergebuehr'] : '',
190
            'o_shippingcost' => SHelper::getShippingcost(),
191
            'o_orderdate' => date('Y-m-d'),
192
            'o_ordertimestamp' => time(),
193
            'o_authed' => CHelper::getUserData() ? 'y' : 'n',
194
            'o_sessiondata' => serialize($_SESSION),
195
            'o_postdata' => serialize($_POST),
196
            'o_remote_address' => filter_input(INPUT_SERVER, 'REMOTE_ADDR', FILTER_SANITIZE_STRING, FILTER_FLAG_STRIP_LOW),
197
            'o_ordercompleted' => 'n',
198
            'o_paymentcompleted' => 'n',
199
            'o_srv_hostname' => filter_input(INPUT_SERVER, 'SERVER_NAME', FILTER_SANITIZE_STRING, FILTER_FLAG_STRIP_LOW),
200
            'o_vatfull' => HelperConfig::$shop['vat']['full'],
201
            'o_vatreduced' => HelperConfig::$shop['vat']['reduced'],
202
        ];
203
    }
204
205
    private function doCheckout()
0 ignored issues
show
Coding Style introduced by
doCheckout uses the super-global variable $_SESSION which is generally not recommended.

Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable:

// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

    public function __construct($host)
    {
        $this->host = $host;
    }

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
Loading history...
206
    {
207
        if (empty($_SESSION['cart'])) {
208
            return false;
209
        }
210
211
        /** @var \PDO $db */
212
        $db = $this->serviceManager->get('db');
213
214
        try {
215
            $db->beginTransaction();
216
217
            $aDataOrder = $this->prepareDataOrder();
218
            $sql = DBTools::buildPSInsertQuery($aDataOrder, 'orders');
219
            $hResult = $db->prepare($sql);
220
            foreach ($aDataOrder as $sKey => $sValue) {
221
                $hResult->bindValue(':' . $sKey, $sValue);
222
            }
223
            $hResult->execute();
224
            $iInsertID = $db->lastInsertId();
225
226
            $aDataOrderItems = [];
227
            $aImagesToSend = [];
228
            foreach ($_SESSION['cart'] as $sK => $aV) {
229
230
                $aImagesToSend[$aV['img']] = $this->getItemImage($aV);
231
232
                $aDataOrderItems[] = [
233
                    'oi_o_id' => $iInsertID,
234
                    'oi_cartkey' => $sK,
235
                    'oi_amount' => $aV['amount'],
236
                    'oi_price_netto_list' => $aV['price']['netto_list'],
237
                    'oi_price_netto_use' => $aV['price']['netto_use'],
238
                    'oi_price_brutto_use' => $aV['price']['brutto_use'],
239
                    'oi_price_netto_sale' => isset($aV['price']['netto_sale']) ? $aV['price']['netto_sale'] : '',
240
                    'oi_price_netto_rebated' => isset($aV['price']['netto_rebated']) ? $aV['price']['netto_rebated'] : '',
241
                    'oi_vat' => HelperConfig::$shop['vat'][$aV['vat']],
242
                    'oi_rg' => $aV['rg'],
243
                    'oi_rg_rebate' => isset(
244
                        HelperConfig::$shop['rebate_groups'][$aV['rg']][trim(CHelper::getUserData('cust_group'))]
245
                    )
246
                        ? HelperConfig::$shop['rebate_groups'][$aV['rg']][trim(CHelper::getUserData('cust_group'))]
247
                        : '',
248
                    'oi_itemname' => $aV['name'],
249
                    'oi_img' => $aImagesToSend[$aV['img']]['base64img'],
250
                ];
251
            }
252
            foreach ($aDataOrderItems as $aV) {
253
                $sql = DBTools::buildPSInsertQuery($aV, 'orders_items');
254
                $hResult = $db->prepare($sql);
255
                foreach ($aV as $sKey => $sValue) {
256
                    $hResult->bindValue(':' . $sKey, $sValue);
257
                }
258
                $hResult->execute();
259
            }
260
            $db->commit();
261
        } catch (\Exception $e) {
262
            // If something raised an exception in our transaction block of statements,
263
            // roll back any work performed in the transaction
264
            print '<p>Unable to complete transaction!</p>';
265
            error_log($e);
266
            $db->rollBack();
267
        }
268
        $sMailbody_us = $this->buildOrderMailBody(false, $iInsertID);
0 ignored issues
show
Bug introduced by
The variable $iInsertID does not seem to be defined for all execution paths leading up to this point.

If you define a variable conditionally, it can happen that it is not defined for all execution paths.

Let’s take a look at an example:

function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}

In the above example, the variable $x is defined if you pass “foo” or “bar” as argument for $a. However, since the switch statement has no default case statement, if you pass any other value, the variable $x would be undefined.

Available Fixes

  1. Check for existence of the variable explicitly:

    function myFunction($a) {
        switch ($a) {
            case 'foo':
                $x = 1;
                break;
    
            case 'bar':
                $x = 2;
                break;
        }
    
        if (isset($x)) { // Make sure it's always set.
            echo $x;
        }
    }
    
  2. Define a default value for the variable:

    function myFunction($a) {
        $x = ''; // Set a default which gets overridden for certain paths.
        switch ($a) {
            case 'foo':
                $x = 1;
                break;
    
            case 'bar':
                $x = 2;
                break;
        }
    
        echo $x;
    }
    
  3. Add a value for the missing path:

    function myFunction($a) {
        switch ($a) {
            case 'foo':
                $x = 1;
                break;
    
            case 'bar':
                $x = 2;
                break;
    
            // We add support for the missing case.
            default:
                $x = '';
                break;
        }
    
        echo $x;
    }
    
Loading history...
269
        $sMailbody_they = $this->buildOrderMailBody(true, $iInsertID);
270
271
        // write to file
272
        $this->writeCheckoutToFile($sMailbody_us);
273
274
        // Send Mails
275
        $this->sendCheckoutMails($iInsertID, $sMailbody_us, $sMailbody_they, $aImagesToSend);
0 ignored issues
show
Bug introduced by
The variable $aImagesToSend does not seem to be defined for all execution paths leading up to this point.

If you define a variable conditionally, it can happen that it is not defined for all execution paths.

Let’s take a look at an example:

function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}

In the above example, the variable $x is defined if you pass “foo” or “bar” as argument for $a. However, since the switch statement has no default case statement, if you pass any other value, the variable $x would be undefined.

Available Fixes

  1. Check for existence of the variable explicitly:

    function myFunction($a) {
        switch ($a) {
            case 'foo':
                $x = 1;
                break;
    
            case 'bar':
                $x = 2;
                break;
        }
    
        if (isset($x)) { // Make sure it's always set.
            echo $x;
        }
    }
    
  2. Define a default value for the variable:

    function myFunction($a) {
        $x = ''; // Set a default which gets overridden for certain paths.
        switch ($a) {
            case 'foo':
                $x = 1;
                break;
    
            case 'bar':
                $x = 2;
                break;
        }
    
        echo $x;
    }
    
  3. Add a value for the missing path:

    function myFunction($a) {
        switch ($a) {
            case 'foo':
                $x = 1;
                break;
    
            case 'bar':
                $x = 2;
                break;
    
            // We add support for the missing case.
            default:
                $x = '';
                break;
        }
    
        echo $x;
    }
    
Loading history...
276
277
        unset($_SESSION['cart'], $_SESSION['cartpricesums'], $_SESSION['sondercart']);
278
279
        $postpaymentmethod = filter_input(INPUT_POST, 'paymentmethod');
280
        if (
281
            $postpaymentmethod !== null
282
            && isset(HelperConfig::$shop['paypal_interactive'])
283
            && $postpaymentmethod === 'paypal'
284
            && in_array('paypal', HelperConfig::$shop['paymentmethods'], true) !== false
285
            && HelperConfig::$shop['paypal_interactive']
286
        ) {
287
            $redirectto = '/_misc/paypal.html?id=' . $iInsertID;
288
        } elseif (
289
            $postpaymentmethod !== null
290
            && $postpaymentmethod === 'sofortueberweisung'
291
            && in_array('sofortueberweisung', HelperConfig::$shop['paymentmethods'], true) !== false
292
        ) {
293
            $redirectto = '/_misc/sofortueberweisung.html?id=' . $iInsertID;
294
        } else {
295
            $redirectto = '/_misc/checkedout.html?id=' . $iInsertID;
296
        }
297
        \HaaseIT\HCSF\Helper::redirectToPage($redirectto);
298
    }
299
300
    /**
301
     * @param int $iInsertID
302
     * @param string $sMailbody_us
303
     * @param string $sMailbody_they
304
     * @param array $aImagesToSend
305
     */
306
    private function sendCheckoutMails($iInsertID, $sMailbody_us, $sMailbody_they, $aImagesToSend)
307
    {
308
        if (
309
            isset(HelperConfig::$shop['email_orderconfirmation_attachment_cancellationform_' .HelperConfig::$lang])
310
            && file_exists(
311
                PATH_DOCROOT.HelperConfig::$core['directory_emailattachments']
312
                .'/'.HelperConfig::$shop['email_orderconfirmation_attachment_cancellationform_'
313
                .HelperConfig::$lang]
314
            )
315
        ) {
316
            $aFilesToSend[] =
0 ignored issues
show
Coding Style Comprehensibility introduced by
$aFilesToSend was never initialized. Although not strictly required by PHP, it is generally a good practice to add $aFilesToSend = array(); before regardless.

Adding an explicit array definition is generally preferable to implicit array definition as it guarantees a stable state of the code.

Let’s take a look at an example:

foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}

As you can see in this example, the array $myArray is initialized the first time when the foreach loop is entered. You can also see that the value of the bar key is only written conditionally; thus, its value might result from a previous iteration.

This might or might not be intended. To make your intention clear, your code more readible and to avoid accidental bugs, we recommend to add an explicit initialization $myArray = array() either outside or inside the foreach loop.

Loading history...
317
                PATH_DOCROOT.HelperConfig::$core['directory_emailattachments'].'/'
318
                .HelperConfig::$shop['email_orderconfirmation_attachment_cancellationform_' .HelperConfig::$lang];
319
        } else {
320
            $aFilesToSend = [];
321
        }
322
323
        Helper::mailWrapper(
324
            filter_input(INPUT_POST, 'email', FILTER_SANITIZE_EMAIL),
325
            $this->textcats->T('shoppingcart_mail_subject') . ' ' . $iInsertID,
326
            $sMailbody_they,
327
            $aImagesToSend,
328
            $aFilesToSend
329
        );
330
        Helper::mailWrapper(
331
            HelperConfig::$core['email_sender'],
332
            'Bestellung im Webshop Nr: ' . $iInsertID,
333
            $sMailbody_us,
334
            $aImagesToSend
335
        );
336
    }
337
338
    /**
339
     * @param string $sMailbody_us
340
     */
341
    private function writeCheckoutToFile($sMailbody_us)
342
    {
343
        $fp = fopen(PATH_LOGS . 'shoplog_' . date('Y-m-d') . '.html', 'a');
344
        // Write $somecontent to our opened file.
345
        fwrite($fp, $sMailbody_us . "\n\n-------------------------------------------------------------------------\n\n");
346
        fclose($fp);
347
    }
348
349
    /**
350
     * @param string $field
351
     * @return string
352
     */
353
    private function getPostValue($field)
354
    {
355
        $postvalue = filter_input(INPUT_POST, $field);
356
        return (!empty($postvalue) ? $postvalue : '');
357
    }
358
359
    /**
360
     * @param bool $bCust
361
     * @param int $iId
362
     * @return mixed
363
     */
364
    private function buildOrderMailBody($bCust = true, $iId = 0)
0 ignored issues
show
Coding Style introduced by
buildOrderMailBody uses the super-global variable $_SESSION which is generally not recommended.

Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable:

// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

    public function __construct($host)
    {
        $this->host = $host;
    }

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
Loading history...
Coding Style introduced by
buildOrderMailBody uses the super-global variable $_POST which is generally not recommended.

Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable:

// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

    public function __construct($host)
    {
        $this->host = $host;
    }

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
Loading history...
365
    {
366
        $aSHC = SHelper::buildShoppingCartTable($_SESSION['cart'], true);
367
368
        $postcustno = trim(filter_input(INPUT_POST, 'custno', FILTER_SANITIZE_STRING, FILTER_FLAG_STRIP_LOW));
369
        $postcountry = trim(filter_input(INPUT_POST, 'country', FILTER_SANITIZE_STRING, FILTER_FLAG_STRIP_LOW));
370
        $postpaymentmethod = filter_input(INPUT_POST, 'paymentmethod', FILTER_SANITIZE_STRING, FILTER_FLAG_STRIP_LOW);
371
        $serverservername = filter_input(INPUT_SERVER, 'SERVER_NAME', FILTER_SANITIZE_URL);
372
        $aData = [
373
            'customerversion' => $bCust,
374
            //'shc_css' => file_get_contents(PATH_DOCROOT.'screen-shc.css'),
0 ignored issues
show
Unused Code Comprehensibility introduced by
55% of this comment could be valid code. Did you maybe forget this after debugging?

Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it.

The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production.

This check looks for comments that seem to be mostly valid code and reports them.

Loading history...
375
            'datetime' => date('d.m.Y - H:i'),
376
            'custno' => $postcustno !== null && strlen($postcustno) >= HelperConfig::$customer['minimum_length_custno'] ? $postcustno : '',
377
            'corpname' => $this->getPostValue('corpname'),
378
            'name' => $this->getPostValue('name'),
379
            'street' => $this->getPostValue('street'),
380
            'zip' => $this->getPostValue('zip'),
381
            'town' => $this->getPostValue('town'),
382
            'phone' => $this->getPostValue('phone'),
383
            'cellphone' => $this->getPostValue('cellphone'),
384
            'fax' => $this->getPostValue('fax'),
385
            'email' => $this->getPostValue('email'),
386
            'country' => !empty($postcountry) ?
387
            (
388
                isset(
389
                    HelperConfig::$countries['countries_' .HelperConfig::$lang][$postcountry]
390
                )
391
                    ? HelperConfig::$countries['countries_' .HelperConfig::$lang][$postcountry]
392
                    : $postcountry)
393
            : '',
394
            'remarks' => $this->getPostValue('remarks'),
395
            'tos' => $this->getPostValue('tos'),
396
            'cancellationdisclaimer' => $this->getPostValue('cancellationdisclaimer'),
397
            'paymentmethod' => $this->getPostValue('paymentmethod'),
398
            'shippingcost' => empty($_SESSION['shippingcost']) ? false : $_SESSION['shippingcost'],
399
            'paypallink' => $postpaymentmethod === 'paypal' ? $serverservername.'/_misc/paypal.html?id='.$iId : '',
400
            'sofortueberweisunglink' => $postpaymentmethod === 'sofortueberweisung' ?  $serverservername.'/_misc/sofortueberweisung.html?id='.$iId : '',
401
            'SESSION' => !$bCust ? Tools::debug($_SESSION, '$_SESSION', true, true) : '',
402
            'POST' => !$bCust ? Tools::debug($_POST, '$_POST', true, true) : '',
403
            'orderid' => $iId,
404
        ];
405
406
        $aM['customdata'] = $aSHC;
0 ignored issues
show
Coding Style Comprehensibility introduced by
$aM was never initialized. Although not strictly required by PHP, it is generally a good practice to add $aM = array(); before regardless.

Adding an explicit array definition is generally preferable to implicit array definition as it guarantees a stable state of the code.

Let’s take a look at an example:

foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}

As you can see in this example, the array $myArray is initialized the first time when the foreach loop is entered. You can also see that the value of the bar key is only written conditionally; thus, its value might result from a previous iteration.

This might or might not be intended. To make your intention clear, your code more readible and to avoid accidental bugs, we recommend to add an explicit initialization $myArray = array() either outside or inside the foreach loop.

Loading history...
407
        $aM['currency'] = HelperConfig::$shop['waehrungssymbol'];
408
        if (isset(HelperConfig::$shop['custom_order_fields'])) {
409
            $aM['custom_order_fields'] = HelperConfig::$shop['custom_order_fields'];
410
        }
411
        $aM['customdata']['mail'] = $aData;
412
413
        return $this->serviceManager->get('twig')->render('shop/mail-order-html.twig', $aM);
414
415
    }
416
417
    /**
418
     * @return string
419
     */
420
    private function getNotification()
421
    {
422
        $return = '';
423
        $getmsg = filter_input(INPUT_GET, 'msg');
424
        $getcartkey = filter_input(INPUT_GET, 'cartkey', FILTER_SANITIZE_STRING, FILTER_FLAG_STRIP_LOW);
425
        $getamount = filter_input(INPUT_GET, 'cartkey', FILTER_SANITIZE_NUMBER_INT);
426
        if (!empty($getmsg)) {
427
            if (
428
                ($getmsg === 'updated' && !empty($getcartkey) && !empty($getamount))
429
                || ($getmsg === 'removed' && !empty($getcartkey))
430
            ) {
431
                $return .= $this->textcats->T('shoppingcart_msg_' . $getmsg . '_1') . ' ';
432
                if (isset(HelperConfig::$shop['custom_order_fields']) && mb_strpos($getcartkey, '|') !== false) {
433
                    $mCartkeys = explode('|', $getcartkey);
434
                    foreach ($mCartkeys as $sKey => $sValue) {
435
                        if ($sKey == 0) {
436
                            $return .= $sValue . ', ';
437
                        } else {
438
                            $TMP = explode(':', $sValue);
439
                            $return .= $this->textcats->T('shoppingcart_item_' . $TMP[0]) . ' ' . $TMP[1] . ', ';
440
                            unset($TMP);
441
                        }
442
                    }
443
                    $return = Tools::cutStringend($return, 2);
444
                } else {
445
                    $return .= $getcartkey;
446
                }
447
                $return.= ' ' . $this->textcats->T('shoppingcart_msg_'.$getmsg.'_2');
448
                if ($getmsg === 'updated') {
449
                    $return .= ' '.$getamount;
450
                }
451
                $return .= '<br><br>';
452
            }
453
        }
454
455
        return $return;
456
    }
457
}
458