[TASK] Use PHP_SAPI, PHP_VERSION and M_PI constant instead of functon…

…s calls Resolves: #85293 Releases: master Change-Id: Ifabbb20c1ef4ad482df676fe9999dcbdf18eb704 Reviewed-on: https://review.typo3.org/57249 Tested-by: TYPO3com <no-reply@typo3.com> Reviewed-by: Oliver Klee <typo3-coding@oliverklee.de> Reviewed-by: Rudy Gnodde <rgn@windinternet.nl> Reviewed-by: Jörg Bösche <typo3@joergboesche.de> Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de> Tested-by: Stefan Neufeind <typo3.neufeind@speedpartner.de> Reviewed-by: Frank Naegler <frank.naegler@typo3.org> Tested-by: Frank Naegler <frank.naegler@typo3.org>
TYPO3 · Jul 18, 2018 · ffede9f · ffede9f
1 parent bc191e1
commit ffede9f
Show file tree

Hide file tree

Showing 8 changed files with 11 additions and 11 deletions.
diff --git a/typo3/install/index.php b/typo3/install/index.php
@@ -1,7 +1,7 @@
 <?php
 
 // Exit early if php requirement is not satisfied.
-if (version_compare(PHP_VERSION, '7.2.0', '<')) {
+if (PHP_VERSION_ID < 70200) {
     die('This version of TYPO3 CMS requires PHP 7.2 or above');
 }
 

diff --git a/typo3/sysext/backend/Resources/Private/Php/backend.php b/typo3/sysext/backend/Resources/Private/Php/backend.php
@@ -13,7 +13,7 @@
  */
 
 // Exit early if php requirement is not satisfied.
-if (version_compare(PHP_VERSION, '7.2.0', '<')) {
+if (PHP_VERSION_ID < 70200) {
     die('This version of TYPO3 CMS requires PHP 7.2 or above');
 }
 

diff --git a/typo3/sysext/core/Classes/Console/CommandApplication.php b/typo3/sysext/core/Classes/Console/CommandApplication.php
@@ -57,7 +57,7 @@ public function run(callable $execute = null)
      */
     protected function checkEnvironmentOrDie()
     {
-        if (php_sapi_name() !== 'cli') {
+        if (PHP_SAPI !== 'cli') {
             die('Not called from a command line interface (e.g. a shell or scheduler).' . LF);
         }
     }

diff --git a/typo3/sysext/core/Classes/Imaging/GraphicalFunctions.php b/typo3/sysext/core/Classes/Imaging/GraphicalFunctions.php
@@ -676,7 +676,7 @@ public function makeText(&$im, $conf, $workArea)
      */
     public function txtPosition($conf, $workArea, $BB)
     {
-        $angle = (int)$conf['angle'] / 180 * pi();
+        $angle = (int)$conf['angle'] / 180 * M_PI;
         $conf['angle'] = 0;
         $straightBB = $this->calcBBox($conf);
         // offset, align, valign, workarea
@@ -1341,11 +1341,11 @@ public function circleOffset($distance, $iterations)
         $res = [];
         if ($distance && $iterations) {
             for ($a = 0; $a < $iterations; $a++) {
-                $yOff = round(sin(2 * pi() / $iterations * ($a + 1)) * 100 * $distance);
+                $yOff = round(sin(2 * M_PI / $iterations * ($a + 1)) * 100 * $distance);
                 if ($yOff) {
                     $yOff = (int)(ceil(abs($yOff / 100)) * ($yOff / abs($yOff)));
                 }
-                $xOff = round(cos(2 * pi() / $iterations * ($a + 1)) * 100 * $distance);
+                $xOff = round(cos(2 * M_PI / $iterations * ($a + 1)) * 100 * $distance);
                 if ($xOff) {
                     $xOff = (int)(ceil(abs($xOff / 100)) * ($xOff / abs($xOff)));
                 }

diff --git a/typo3/sysext/fluid/Tests/Unit/ViewHelpers/Format/NumberViewHelperTest.php b/typo3/sysext/fluid/Tests/Unit/ViewHelpers/Format/NumberViewHelperTest.php
@@ -34,7 +34,7 @@ protected function setUp()
         $this->injectDependenciesIntoViewHelper($this->viewHelper);
         $this->viewHelper->setRenderChildrenClosure(
             function () {
-                return pi();
+                return M_PI;
             }
         );
     }
@@ -88,7 +88,7 @@ public function formatNumberWithDecimals()
     public function formatNumberWithThousandsSeparator()
     {
         $this->viewHelper->setRenderChildrenClosure(function () {
-            return pi() * 1000;
+            return M_PI * 1000;
         });
         $this->setArgumentsUnderTest(
             $this->viewHelper,

diff --git a/typo3/sysext/frontend/Resources/Private/Php/frontend.php b/typo3/sysext/frontend/Resources/Private/Php/frontend.php
@@ -13,7 +13,7 @@
  */
 
 // Exit early if php requirement is not satisfied.
-if (version_compare(PHP_VERSION, '7.2.0', '<')) {
+if (PHP_VERSION_ID < 70200) {
     die('This version of TYPO3 CMS requires PHP 7.2 or above');
 }
 

diff --git a/typo3/sysext/install/Classes/SystemEnvironment/Check.php b/typo3/sysext/install/Classes/SystemEnvironment/Check.php
@@ -250,7 +250,7 @@ protected function checkMemorySettings()
     protected function checkPhpVersion()
     {
         $minimumPhpVersion = '7.2.0';
-        $currentPhpVersion = phpversion();
+        $currentPhpVersion = PHP_VERSION;
         if (version_compare($currentPhpVersion, $minimumPhpVersion) < 0) {
             $this->messageQueue->enqueue(new FlashMessage(
                 'Your PHP version ' . $currentPhpVersion . ' is too old. TYPO3 CMS does not run'

diff --git a/typo3/sysext/install/Resources/Private/Php/install.php b/typo3/sysext/install/Resources/Private/Php/install.php
@@ -94,7 +94,7 @@
  */
 
 // Exit early if php requirement is not satisfied.
-if (version_compare(PHP_VERSION, '7.2.0', '<')) {
+if (PHP_VERSION_ID < 70200) {
     die('This version of TYPO3 CMS requires PHP 7.2 or above');
 }