Merge pull request #542 from Smartling/WP-910

add debug (WP-910)

Author: vsolovei-smartling

composer.json

@@ -1,7 +1,7 @@
 {
   "name": "smartling/wordpress-connector",
   "license": "GPL-2.0-or-later",
-  "version": "3.9.9",
+  "version": "3.9.10",
   "description": "",
   "type": "wordpress-plugin",
   "repositories": [

inc/Smartling/Bootstrap.php

@@ -437,7 +437,7 @@ private function initializeBuildInContentTypes(ContainerBuilder $di): void
          * Post types and taxonomies are registered on 'init' hook, but this code is executed on 'plugins_loaded' hook,
          * so we need to postpone dynamic handlers execution
          */
-        add_action($action, static function () use ($di) {
+        add_action($action, static function () use ($action, $di) {
             // registering taxonomies first.
             $dynTermDefinitions = apply_filters(ExportedAPI::FILTER_SMARTLING_REGISTER_CUSTOM_TAXONOMY, []);
             if (!is_iterable($dynTermDefinitions)) {
@@ -452,6 +452,9 @@ private function initializeBuildInContentTypes(ContainerBuilder $di): void
             if (!is_iterable($externalDefinitions)) {
                 self::$loggerInstance->critical('External definitions not iterable after filter ' . ExportedAPI::FILTER_SMARTLING_REGISTER_CUSTOM_POST_TYPE . '. This is most likely due to an error outside of the plugins code.');
             }
+            $registeredTypes = array_column(array_column($externalDefinitions, 'type'), 'identifier');
+            self::$loggerInstance->debug('Registered custom post types: ' .
+                implode(', ', $registeredTypes) . ", action=$action, count=" . count($registeredTypes));
             foreach ($externalDefinitions as $externalDefinition) {
                 CustomPostType::registerCustomType($di, $externalDefinition);
             }

inc/Smartling/Jobs/DownloadTranslationJob.php

@@ -32,7 +32,7 @@ public function getJobHookName(): string
         return self::JOB_HOOK_NAME;
     }
 
-    public function run(): void
+    public function run(string $source): void
     {
         $this->getLogger()->debug('Started Translation Download Job.');
 

inc/Smartling/Jobs/JobAbstract.php

@@ -191,7 +191,7 @@ private function tryRunJob(string $source = ''): void
     {
         $this->placeLockFlag(source: $source);
         try {
-            $this->run();
+            $this->run($source);
         } finally {
             try {
                 $this->dropLockFlag();

inc/Smartling/Jobs/JobInterface.php

@@ -10,5 +10,5 @@ public function uninstall(): void;
 
     public function getJobHookName(): string;
 
-    public function run(): void;
+    public function run(string $source): void;
 }

inc/Smartling/Jobs/LastModifiedCheckJob.php

@@ -35,7 +35,7 @@ public function getJobHookName(): string
         return self::JOB_HOOK_NAME;
     }
 
-    public function run(): void
+    public function run(string $source): void
     {
         $this->getLogger()->info('Started Last-Modified Check Job.');
 

inc/Smartling/Jobs/SubmissionCollectorJob.php

@@ -55,7 +55,7 @@ private function fixEmptyFileUriSubmissions(int $iterationLimit = 100): void
         } while (0 < count($submissions));
     }
 
-    public function run(): void
+    public function run(string $source): void
     {
         $this->getLogger()->info('Started Submission Collector Job.');
         $this->fixEmptyFileUriSubmissions();

inc/Smartling/Jobs/UploadJob.php

@@ -34,15 +34,20 @@ public function getJobHookName(): string
         return self::JOB_HOOK_NAME;
     }
 
-    public function run(): void
+    public function run(string $source): void
     {
-        $this->getLogger()->debug('Started UploadJob.');
+        $message = 'UploadJob';
+        if ($source !== '') {
+            $message .= ", source=\"$source\"";
+        }
+        $message .= '.';
+        $this->getLogger()->debug("Started $message");
 
         $this->processUploadQueue();
 
         $this->processCloning();
 
-        $this->getLogger()->debug('Finished UploadJob.');
+        $this->getLogger()->debug("Finished $message");
     }
 
     private function processUploadQueue(): void

readme.txt

@@ -4,7 +4,7 @@ Tags: translation, localization, localisation, translate, multilingual, smartlin
 Requires at least: 5.5
 Tested up to: 6.4.1
 Requires PHP: 8.0
-Stable tag: 3.9.9
+Stable tag: 3.9.10
 License: GPLv2 or later
 
 Translate content in WordPress quickly and seamlessly with Smartling, the industry-leading Translation Management System.
@@ -62,6 +62,9 @@ Additional information on the Smartling Connector for WordPress can be found [he
 3. Track translation status within WordPress from the Submissions Board. View overall progress of submitted translation requests as well as resend updated content.
 
 == Changelog ==
+= 3.9.10 =
+* Added logging of registered post types
+
 = 3.9.9 =
 * Fixed submissions not being uploaded when related assets list has multiple entries that point to the same content
 

smartling-connector.php

@@ -11,7 +11,7 @@
  * Plugin Name:       Smartling Connector
  * Plugin URI:        https://www.smartling.com/products/automate/integrations/wordpress/
  * Description:       Integrate your WordPress site with Smartling to upload your content and download translations.
- * Version:           3.9.9
+ * Version:           3.9.10
  * Author:            Smartling
  * Author URI:        https://www.smartling.com
  * License:           GPL-2.0+