Changeset 3764005 for langforge/trunk/admin/class-strings-screen.php

Timestamp:

10/25/2026 06:29:16 AM (22 hours ago)

Author:

avakodeforge

Message:

Release 1.3.0 (trunk + tag; see readme changelog)

File:

• langforge/trunk/admin/class-strings-screen.php (modified) (1 diff)

langforge/trunk/admin/class-strings-screen.php

@@ -291 +291 @@
     }
 
+    /**
+ * The glossary check of a save of translated strings (BUG-LANG-GLOSS-02).
+ *
+ * Each saved text of a language other than the source is checked against the source text of its string
+ * (LANGFORGE_Glossary::validate_translation); the terms of a language pair are read once. A string whose source
+ * text was not posted (the source cell of an email text is read-only on the screen) takes it from the table. It
+ * only reports: the texts are saved before this runs.
+ *
+ * @param array<int,array{name:string,domain:string,lang:string,value:string,source:?string}> $saved
+ * @param string $source_lang The language the strings are written in.
+ * @return array<int,array{name:string,domain:string,lang:string,warnings:string}>
+ */
+    public static function glossary_report(array $saved, $source_lang) {
+        if (!$saved || !class_exists('LANGFORGE_Glossary')) {
+            return [];
+        }
+        $glossary = LANGFORGE_Glossary::instance();
+        $terms    = [];
+        $sources  = [];
+        $report   = [];
+        foreach ($saved as $row) {
+            $lang = $row['lang'];
+            if (!isset($terms[$lang])) {
+                $terms[$lang] = (array) $glossary->get_terms($source_lang, $lang);
+            }
+            if (!$terms[$lang]) {
+                continue; // no term for this pair: nothing to follow
+            }
+            $source = $row['source'];
+            if ($source === null || $source === '') {
+                $domain = $row['domain'];
+                if (!isset($sources[$domain])) {
+                    $sources[$domain] = [];
+                    foreach ((array) LANGFORGE_String_Translation::instance()->get_strings($domain, $source_lang) as $source_row) {
+                        $sources[$domain][$source_row->string_name] = (string) $source_row->string_value;
+                    }
+                }
+                $source = $sources[$domain][$row['name']] ?? '';
+            }
+            if ($source === '') {
+                continue;
+            }
+            $warnings = $glossary->validate_translation($source, $row['value'], $source_lang, $lang, $terms[$lang]);
+            if ($warnings) {
+                $report[] = ['name' => $row['name'], 'domain' => $row['domain'], 'lang' => $lang, 'warnings' => $warnings];
+            }
+        }
+        return $report;
+    }
+
+    /** The notice after a save: which saved translations do not follow the glossary, and which terms. Nothing when none. */
+    public static function render_glossary_report(array $report) {
+        if (!$report) {
+            return;
+        }
+        echo '<div class="notice notice-warning"><p><strong>' . esc_html__('Glossary: saved, but these translations do not follow the glossary.', 'langforge') . '</strong></p><ul style="list-style:disc;margin-left:18px;">';
+        foreach ($report as $item) {
+            echo '<li><strong>' . esc_html($item['name']) . '</strong> (' . esc_html(strtoupper($item['lang'])) . '): ' . esc_html(implode(' ', $item['warnings'])) . '</li>';
+        }
+        echo '</ul></div>';
+    }
+
     /** The note under a name that other domains use too. */
     public static function render_other_domains_note(array $domains) {