Completed
Pull Request — master (#114)
by Florian
03:20
created
Helper/Toolkit.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -124,7 +124,7 @@
 block discarded – undo
124 124
      *
125 125
      * @param  string   $sText
126 126
      * @param  string   $aSubstitutionArray
127
-     * @param  int|bool $iMaxLength
127
+     * @param  integer $iMaxLength
128 128
      * @return string
129 129
      */
130 130
     public function handleSubstituteReplacement($sText, $aSubstitutionArray, $iMaxLength = false)
Please login to merge, or discard this patch.
Test/Unit/Controller/Onepage/RedirectTest.php 1 patch
Unused Use Statements   -2 removed lines patch added patch discarded remove patch
@@ -28,10 +28,8 @@
 block discarded – undo
28 28
 
29 29
 use Payone\Core\Controller\Onepage\Redirect as ClassToTest;
30 30
 use Magento\Framework\TestFramework\Unit\Helper\ObjectManager;
31
-use Magento\Sales\Model\Order as OrderCore;
32 31
 use Magento\Checkout\Model\Session;
33 32
 use Magento\Framework\App\Action\Context;
34
-use Magento\Customer\Model\Session as CustomerSession;
35 33
 use Magento\Store\App\Response\Redirect as RedirectResponse;
36 34
 use Magento\Framework\App\Console\Response;
37 35
 use Magento\Framework\UrlInterface;
Please login to merge, or discard this patch.
Test/Unit/Controller/Onepage/ReturnedTest.php 1 patch
Unused Use Statements   -2 removed lines patch added patch discarded remove patch
@@ -28,10 +28,8 @@
 block discarded – undo
28 28
 
29 29
 use Payone\Core\Controller\Onepage\Returned as ClassToTest;
30 30
 use Magento\Framework\TestFramework\Unit\Helper\ObjectManager;
31
-use Magento\Sales\Model\Order as OrderCore;
32 31
 use Magento\Checkout\Model\Session;
33 32
 use Magento\Framework\App\Action\Context;
34
-use Magento\Customer\Model\Session as CustomerSession;
35 33
 use Magento\Store\App\Response\Redirect as RedirectResponse;
36 34
 use Magento\Framework\App\Console\Response;
37 35
 use Magento\Framework\App\ResponseInterface;
Please login to merge, or discard this patch.
Test/Unit/Controller/Paypal/ExpressTest.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -28,7 +28,6 @@
 block discarded – undo
28 28
 
29 29
 use Payone\Core\Controller\Paypal\Express as ClassToTest;
30 30
 use Magento\Framework\TestFramework\Unit\Helper\ObjectManager;
31
-use Magento\Sales\Model\Order as OrderCore;
32 31
 use Magento\Framework\Controller\Result\Redirect;
33 32
 use Magento\Checkout\Model\Session;
34 33
 use Magento\Framework\Controller\ResultFactory;
Please login to merge, or discard this patch.
Test/Unit/Controller/Paypal/ReturnedTest.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -28,7 +28,6 @@
 block discarded – undo
28 28
 
29 29
 use Payone\Core\Controller\Paypal\Returned as ClassToTest;
30 30
 use Magento\Framework\TestFramework\Unit\Helper\ObjectManager;
31
-use Magento\Sales\Model\Order as OrderCore;
32 31
 use Magento\Framework\Controller\Result\Redirect;
33 32
 use Magento\Checkout\Model\Session;
34 33
 use Payone\Core\Model\Paypal\ReturnHandler;
Please login to merge, or discard this patch.
Controller/Adminhtml/Config/Export/Index.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -87,7 +87,7 @@
 block discarded – undo
87 87
                 $sXml = $this->configExport->generateConfigExportXml();
88 88
                 if ($sXml !== false) {
89 89
                     $oResultRaw->setHeader("Content-Type", "text/xml; charset=\"utf8\"", true);
90
-                    $oResultRaw->setHeader("Content-Disposition", "attachment; filename=\"payone_config_export" . date('Y-m-d H-i-s') . "_" . md5($sXml) . ".xml\"", true);
90
+                    $oResultRaw->setHeader("Content-Disposition", "attachment; filename=\"payone_config_export".date('Y-m-d H-i-s')."_".md5($sXml).".xml\"", true);
91 91
                     $oResultRaw->setContents($sXml);
92 92
                 }
93 93
             } catch (\Exception $e) {
Please login to merge, or discard this patch.
Model/Api/Invoice.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -221,7 +221,7 @@
 block discarded – undo
221 221
     {
222 222
         if (fmod(floatval($dItemAmount), 1.0) > 0) { // input does not represent an integer
223 223
             $sErrorMessage = "Unable to use floating point values for item amounts! Parameter was: ";
224
-            throw new \InvalidArgumentException($sErrorMessage . strval($dItemAmount), 1);
224
+            throw new \InvalidArgumentException($sErrorMessage.strval($dItemAmount), 1);
225 225
         } else { // return the integer value
226 226
             return intval($dItemAmount);
227 227
         }
Please login to merge, or discard this patch.
Test/Unit/Model/Methods/BaseMethodTest.php 1 patch
Unused Use Statements   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -28,11 +28,10 @@
 block discarded – undo
28 28
 
29 29
 use Magento\Payment\Model\Method\AbstractMethod;
30 30
 use Magento\Sales\Model\Order;
31
-use Payone\Core\Model\Methods\Paydirekt as ClassToTest;
31
+use Payone\Core\Model\Methods\Paydirekt;
32 32
 use Magento\Framework\TestFramework\Unit\Helper\ObjectManager;
33 33
 use Payone\Core\Helper\Shop;
34 34
 use Magento\Framework\App\Config\ScopeConfigInterface;
35
-use Payone\Core\Model\Methods\Paydirekt;
36 35
 use Payone\Core\Model\PayoneConfig;
37 36
 use Magento\Payment\Model\Info;
38 37
 use Payone\Core\Model\Api\Request\Authorization;
Please login to merge, or discard this patch.
Model/Api/Payolution/PrivacyDeclaration.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -169,7 +169,7 @@
 block discarded – undo
169 169
      * Get fallback template
170 170
      *
171 171
      * @param  string $sCompany
172
-     * @return mixed
172
+     * @return string
173 173
      */
174 174
     protected function getFallbackText($sCompany)
175 175
     {
Please login to merge, or discard this patch.
Unused Use Statements   -2 removed lines patch added patch discarded remove patch
@@ -26,8 +26,6 @@
 block discarded – undo
26 26
 
27 27
 namespace Payone\Core\Model\Api\Payolution;
28 28
 
29
-use Locale;
30
-
31 29
 /**
32 30
  * Class for requesting the privacy declaration from payolution
33 31
  */
Please login to merge, or discard this patch.