MAGETWO-67715: Fixed coding standard violations in the Framework\Translate namespace #9354

Oleksii Korshenko · web-flow · commit 090a0623ecd9 · 2017-04-21T21:30:40.000-05:00
diff --git a/lib/internal/Magento/Framework/Translate/Adapter.php b/lib/internal/Magento/Framework/Translate/Adapter.php
@@ -4,8 +4,6 @@
  * See COPYING.txt for license details.
  */
 
-// @codingStandardsIgnoreFile
-
 /**
  * Magento translate adapter
  */
@@ -31,6 +29,7 @@ public function translate($messageId, $locale = null)
         }
     }
 
+    // @codingStandardsIgnoreStart
     /**
      * Translate message string.
      *
@@ -47,4 +46,5 @@ public function __()
         }
         return $string;
     }
+    // @codingStandardsIgnoreEnd
 }
diff --git a/lib/internal/Magento/Framework/Translate/AdapterInterface.php b/lib/internal/Magento/Framework/Translate/AdapterInterface.php
@@ -4,8 +4,6 @@
  * See COPYING.txt for license details.
  */
 
-// @codingStandardsIgnoreFile
-
 namespace Magento\Framework\Translate;
 
 /**
@@ -24,11 +22,13 @@ interface AdapterInterface
      */
     public function translate($messageId, $locale = null);
 
+    // @codingStandardsIgnoreStart
     /**
      * Translate string
      *
      * @return string
      * @SuppressWarnings(PHPMD.ShortMethodName)
      */
     public function __();
+    // @codingStandardsIgnoreEnd
 }
diff --git a/lib/internal/Magento/Framework/Translate/Inline.php b/lib/internal/Magento/Framework/Translate/Inline.php
@@ -6,8 +6,6 @@
  * See COPYING.txt for license details.
  */
 
-// @codingStandardsIgnoreFile
-
 namespace Magento\Framework\Translate;
 
 class Inline implements \Magento\Framework\Translate\InlineInterface
@@ -242,7 +240,11 @@ protected function stripInlineTranslations(&$body)
             }
         } else {
             if (is_string($body)) {
-                $body = preg_replace('#' . \Magento\Framework\Translate\Inline\ParserInterface::REGEXP_TOKEN . '#', '$1', $body);
+                $body = preg_replace(
+                    '#' . \Magento\Framework\Translate\Inline\ParserInterface::REGEXP_TOKEN . '#',
+                    '$1',
+                    $body
+                );
             }
         }
         return $this;

Original file line number	Diff line number	Diff line change
`@@ -4,8 +4,6 @@`
`4`	`4`	`* See COPYING.txt for license details.`
`5`	`5`	`*/`
`6`	`6`
`7`		`-// @codingStandardsIgnoreFile`
`8`		`-`
`9`	`7`	`/**`
`10`	`8`	`* Magento translate adapter`
`11`	`9`	`*/`
`@@ -31,6 +29,7 @@ public function translate($messageId, $locale = null)`
`31`	`29`	`}`
`32`	`30`	`}`
`33`	`31`
	`32`	`+ // @codingStandardsIgnoreStart`
`34`	`33`	`/**`
`35`	`34`	`* Translate message string.`
`36`	`35`	`*`
`@@ -47,4 +46,5 @@ public function __()`
`47`	`46`	`}`
`48`	`47`	`return $string;`
`49`	`48`	`}`
	`49`	`+ // @codingStandardsIgnoreEnd`
`50`	`50`	`}`
Original file line number	Diff line number	Diff line change
`@@ -6,8 +6,6 @@`
`6`	`6`	`* See COPYING.txt for license details.`
`7`	`7`	`*/`
`8`	`8`
`9`		`-// @codingStandardsIgnoreFile`
`10`		`-`
`11`	`9`	`namespace Magento\Framework\Translate;`
`12`	`10`
`13`	`11`	`class Inline implements \Magento\Framework\Translate\InlineInterface`
`@@ -242,7 +240,11 @@ protected function stripInlineTranslations(&$body)`
`242`	`240`	`}`
`243`	`241`	`} else {`
`244`	`242`	`if (is_string($body)) {`
`245`		`- $body = preg_replace('#' . \Magento\Framework\Translate\Inline\ParserInterface::REGEXP_TOKEN . '#', '$1', $body);`
	`243`	`+ $body = preg_replace(`
	`244`	`+ '#' . \Magento\Framework\Translate\Inline\ParserInterface::REGEXP_TOKEN . '#',`
	`245`	`+ '$1',`
	`246`	`+ $body`
	`247`	`+ );`
`246`	`248`	`}`
`247`	`249`	`}`
`248`	`250`	`return $this;`