Drop logger and debugger translator (#36)

* Update Translator.phpt

* Delete LoggerTranslator.phpt

* Delete DebuggerTranslator.phpt

* Update TranslationExtension.php

* Create compatibility.php

* Delete DebuggerTranslator.php

* Delete LoggerTranslator.php

* Update Panel.php

* Update Translator.php

* Update phpstan.neon

* Update ruleset.xml

* Update Panel.php

* Update compatibility.php

* Update compatibility.php

phpstan.neon

@@ -8,44 +8,39 @@ parameters:
 	level: max
 	paths:
 		- src
+	excludes_analyse:
+		- src/compatibility.php
 	ignoreErrors:
-		# src\Loaders\Doctrine.php
 		-
 			count: 2
 			message: '#^Variable property access on T\.$#'
 			path: src/Loaders/Doctrine.php
 
-		# src\Tracy\Panel.php
 		-
 			count: 1
 			message: '#^Only numeric types are allowed in post-increment, int\|string given\.$#'
 			path: src/Tracy/Panel.php
 
-		# src\Translator.php
 		-
 			count: 1
 			message: '#^Method Contributte\\Translation\\Translator\:\:addResource\(\) has no return typehint specified\.$#'
 			path: src/Translator.php
 
-		# src\Translator.php
 		-
 			count: 1
 			message: '#^Method Contributte\\Translation\\Translator\:\:setLocale\(\) has no return typehint specified\.$#'
 			path: src/Translator.php
 
-		# src\Translator.php
 		-
 			count: 1
 			message: '#^Method Contributte\\Translation\\Translator\:\:setFallbackLocales\(\) has no return typehint specified\.$#'
 			path: src/Translator.php
 
-		# src\Translator.php
 		-
 			count: 1
 			message: '#^Method Contributte\\Translation\\Translator\:\:computeFallbackLocales\(\) has no return typehint specified\.$#'
 			path: src/Translator.php
 
-		# src\Translator.php
 		-
 			count: 1
 			message: '#^Method Contributte\\Translation\\Translator\:\:computeFallbackLocales\(\) has parameter \$locale with no typehint specified\.$#'

ruleset.xml

@@ -13,4 +13,5 @@
 			</property>
 		</properties>
 	</rule>
+	<exclude-pattern>./src/compatibility.php</exclude-pattern>
 </ruleset>

src/DI/TranslationExtension.php

@@ -112,15 +112,7 @@ public function loadConfiguration(): void
 			->setFactory($this->config->cache->factory, [$this->config->debug]);
 
 		// Translator
-		if ($this->config->debug && $this->config->debugger) {
-			$factory = Contributte\Translation\DebuggerTranslator::class;
-
-		} elseif ($this->config->logger) {
-			$factory = Contributte\Translation\LoggerTranslator::class;
-
-		} else {
-			$factory = Contributte\Translation\Translator::class;
-		}
+		$factory = Contributte\Translation\Translator::class;
 
 		$translator = $builder->addDefinition($this->prefix('translator'))
 			->setFactory($factory, ['defaultLocale' => $this->config->locales->default, 'cacheDir' => $this->config->cache->dir, 'debug' => $this->config->debug, 'cacheVary' => $this->config->cache->vary])

src/Tracy/Panel.php

@@ -13,7 +13,7 @@ class Panel implements Tracy\IBarPanel
 
 	use Nette\SmartObject;
 
-	/** @var Contributte\Translation\DebuggerTranslator */
+	/** @var Contributte\Translation\Translator */
 	private $translator;
 
 	/** @var (string|int)[][] */
@@ -37,7 +37,7 @@ class Panel implements Tracy\IBarPanel
 	/** @var int */
 	private $ignoredResourcesCount = 0;
 
-	public function __construct(Contributte\Translation\DebuggerTranslator $translator)
+	public function __construct(Contributte\Translation\Translator $translator)
 	{
 		$this->translator = $translator;
 		$translator->setTracyPanel($this);

src/Translator.php

@@ -4,6 +4,7 @@
 
 use Contributte;
 use Nette;
+use Psr;
 use Symfony;
 
 /**
@@ -18,6 +19,8 @@
  * @property-read string $formattedPrefix
  * @property-read string[] $availableLocales
  * @property      string|null $locale
+ * @property      Psr\Log\LoggerInterface|null $psrLogger
+ * @property      Contributte\Translation\Tracy\Panel|null $tracyPanel
  */
 class Translator extends Symfony\Component\Translation\Translator implements Nette\Localization\ITranslator
 {
@@ -51,6 +54,12 @@ class Translator extends Symfony\Component\Translation\Translator implements Net
 	/** @var bool[] @internal */
 	private $resourcesLocales = [];
 
+	/** @var Psr\Log\LoggerInterface|null */
+	private $psrLogger;
+
+	/** @var Contributte\Translation\Tracy\Panel|null */
+	private $tracyPanel;
+
 	/**
 	 * @param string[] $cacheVary
 	 */
@@ -231,6 +240,28 @@ public function setFallbackLocales(array $locales)
 		$this->fallbackResolver->setFallbackLocales($locales);
 	}
 
+	public function getPsrLogger(): ?Psr\Log\LoggerInterface
+	{
+		return $this->psrLogger;
+	}
+
+	public function setPsrLogger(?Psr\Log\LoggerInterface $psrLogger): self
+	{
+		$this->psrLogger = $psrLogger;
+		return $this;
+	}
+
+	public function getTracyPanel(): ?Tracy\Panel
+	{
+		return $this->tracyPanel;
+	}
+
+	public function setTracyPanel(?Tracy\Panel $tracyPanel): self
+	{
+		$this->tracyPanel = $tracyPanel;
+		return $this;
+	}
+
 	/**
 	 * @param mixed $message
 	 * @param mixed ...$parameters
@@ -291,6 +322,42 @@ public function translate($message, ...$parameters): string
 		return $this->trans($message, $params, $domain, $locale);
 	}
 
+	/**
+	 * @param string|null $id
+	 * @param mixed[] $parameters
+	 * @param string|null $domain
+	 * @param string|null $locale
+	 * @return string
+	 * @phpcsSuppress SlevomatCodingStandard.TypeHints.TypeHintDeclaration.MissingParameterTypeHint
+	 * @phpcsSuppress SlevomatCodingStandard.TypeHints.TypeHintDeclaration.MissingReturnTypeHint
+	 */
+	public function trans($id, array $parameters = [], $domain = null, $locale = null)
+	{
+		if ($domain === null) {
+			$domain = 'messages';
+		}
+
+		if ($id !== null) {
+			if ($this->psrLogger !== null) {
+				if (!$this->getCatalogue()->has($id, $domain)) {
+					$this->psrLogger->notice('Missing translation', [
+						'id' => $id,
+						'domain' => $domain,
+						'locale' => $locale ?? $this->getLocale(),
+					]);
+				}
+			}
+
+			if ($this->tracyPanel !== null) {
+				if (!$this->getCatalogue()->has($id, $domain)) {
+					$this->tracyPanel->addMissingTranslation($id, $domain);
+				}
+			}
+		}
+
+		return parent::trans($id, $parameters, $domain, $locale);
+	}
+
 	/**
 	 * {@inheritdoc}
 	 */

src/compatibility.php

@@ -0,0 +1,27 @@
+<?php declare(strict_types = 1);
+
+namespace Contributte\Translation;
+
+if (false) {
+
+	/** @deprecated use Contributte\Translation\LoggerTranslator */
+	class LoggerTranslator
+	{
+
+	}
+
+} elseif (!class_exists(LoggerTranslator::class)) {
+	class_alias(Translator::class, LoggerTranslator::class);
+}
+
+if (false) {
+
+	/** @deprecated use Contributte\Translation\DebuggerTranslator */
+	class DebuggerTranslator
+	{
+
+	}
+
+} elseif (!class_exists(DebuggerTranslator::class)) {
+	class_alias(Translator::class, DebuggerTranslator::class);
+}