Changeset 3764005 for langforge/trunk/includes/class-cli.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/includes/class-cli.php (modified) (7 diffs)

langforge/trunk/includes/class-cli.php

@@ -16 +16 @@
 // phpcs:ignore WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedClassFound -- LANGFORGE_ is the established plugin class prefix
 class LANGFORGE_CLI extends WP_CLI_Command {
+
+    /**
+ * The language code as the site stores it, or an error that names the languages the site has.
+ *
+ * Same gate as the REST routes and the metabox (LANGFORGE_Post_Translation::require_language_available):
+ * the code, cleaned, must be one of the active languages. A command that skipped it left a post in a language
+ * the site does not have, or made a draft in it.
+ *
+ * @param mixed $language Language code as typed.
+ * @return string The cleaned code.
+ */
+    private static function require_site_language($language) {
+        $language = sanitize_text_field((string) $language);
+        if (!LANGFORGE_Post_Translation::instance()->is_language_available($language)) {
+            $active = array_keys((array) LANGFORGE_Core::instance()->get_active_languages());
+            WP_CLI::error(sprintf(
+                "Language '%s' is not active on this site. Active languages: %s.",
+                $language,
+                $active ? implode(', ', $active) : '(none)'
+            ));
+        }
+        return $language;
+    }
 
     /**
@@ -102 +125 @@
         }
 
+        // The REST route and the metabox refuse a language the site does not have; the command asks the same.
+        $language = self::require_site_language($language);
+
         $existing = LANGFORGE_Post_Translation::instance()->get_translation($post_id, $language);
         if ($existing) {
@@ -158 +184 @@
         }
 
+        $language = self::require_site_language($language);
+
         LANGFORGE_Post_Translation::instance()->set_post_language((int) $post_id, $language);
         WP_CLI::success("Set language '$language' for post #$post_id.");
@@ -176 +204 @@
     public function link($args, $assoc_args) {
         list($id1, $id2) = $args;
-        LANGFORGE_Post_Translation::instance()->link_translations((int) $id1, (int) $id2);
+        $id1 = (int) $id1;
+        $id2 = (int) $id2;
+
+        // A row for a post that is not there is an orphan in the translations table: both posts are looked up
+        // first, as translate and set-language do, and nothing is written for a missing one.
+        foreach ([$id1, $id2] as $id) {
+            if ($id <= 0 || !get_post($id)) {
+                WP_CLI::error("Post $id not found.");
+            }
+        }
+
+        LANGFORGE_Post_Translation::instance()->link_translations($id1, $id2);
         WP_CLI::success("Linked posts #$id1 and #$id2 as translations.");
     }
@@ -194 +233 @@
     public function unlink($args, $assoc_args) {
         list($post_id, $language) = $args;
-        LANGFORGE_Post_Translation::instance()->unlink_translation((int) $post_id, $language);
+        $post_id = (int) $post_id;
+
+        // As link does: a post that is not there is an error, not a "Success" for nothing.
+        if ($post_id <= 0 || !get_post($post_id)) {
+            WP_CLI::error("Post $post_id not found.");
+        }
+
+        LANGFORGE_Post_Translation::instance()->unlink_translation($post_id, $language);
         WP_CLI::success("Unlinked $language translation from post #$post_id.");
     }
@@ -221 +267 @@
  * --lang=<language>
  * : Target language code (letters, digits, "-" and "_" only, for example
- * de or pt-BR).
+ * de or pt-BR). It has to be an active language of the site.
  *
  * [--format=<format>]
@@ -258 +304 @@
             ));
         }
+
+        // A language the site does not have gets the same refusal as set-language: the code is an active language
+        // of the site (the one gate of the REST route and the metabox), and the error names the languages it has.
+        $target = self::require_site_language($target);
 
         if (!class_exists('LANGFORGE_Import_Export')) {