chore: sync issue-branch.yml from Template-Generic [skip ci]

.mokogitea/workflows/issue-branch.yml

@@ -5,7 +5,7 @@
 # FILE INFORMATION
 # DEFGROUP: Gitea.Workflow
 # INGROUP: mokocli.Automation
-# VERSION: 01.07.03
+# VERSION: 01.00.00
 # BRIEF: Auto-create feature branch when an issue is opened
 
 name: "Universal: Issue Branch"

CHANGELOG.md

@@ -1,12 +1,17 @@
 # Changelog
 
+## [Unreleased]
+
+## [01.07.00] --- 2026-06-29
+
+
 All notable changes to MokoSuiteOpenGraph will be documented in this file.
 
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
 
-<!-- VERSION: 01.07.03 -->
+<!-- VERSION: 01.07.00 -->
 
-## [Unreleased]
+## [01.07.00] --- 2026-06-29
 
 ### Added
 - OG coverage **dashboard** as the default admin view — SVG donut gauge, coverage by content type, and a list of articles missing OG tags with a batch-generate shortcut (#94)

CODE_OF_CONDUCT.md

@@ -14,7 +14,7 @@
 	DEFGROUP: Template-Joomla
 	INGROUP: Template-Joomla.Documentation
 	REPO: https://github.com/mokoconsulting-tech/Template-Joomla/
-  VERSION: 01.07.03
+  VERSION: 01.07.00
   PATH: ./CODE_OF_CONDUCT.md
   BRIEF: Community expectations and enforcement guidelines
   NOTE: Adapted with attribution from the Contributor Covenant v2.1

GOVERNANCE.md

@@ -19,7 +19,7 @@
  DEFGROUP: mokoconsulting-tech.Template-Joomla
  INGROUP: MokoStandards.Governance
  REPO: https://github.com/mokoconsulting-tech/Template-Joomla
- VERSION: 01.07.03
+ VERSION: 01.07.00
  PATH: /GOVERNANCE.md
  BRIEF: Project governance rules, roles, and decision process for Template-Joomla
 -->

README.md

@@ -1,6 +1,6 @@
 # MokoSuiteOpenGraph
 
-<!-- VERSION: 01.07.03 -->
+<!-- VERSION: 01.07.00 -->
 
 Open Graph, Twitter Card, and social sharing meta tag management for Joomla 6 and higher.
 

SECURITY.md

@@ -23,7 +23,7 @@ DEFGROUP: Template-Joomla
 INGROUP: Template-Joomla.Documentation
 REPO: https://git.mokoconsulting.tech/MokoConsulting/Template-Joomla
 PATH: /SECURITY.md
-VERSION: 01.07.03
+VERSION: 01.07.00
 BRIEF: Security vulnerability reporting and handling policy
 -->
 

source/packages/com_mokoog/mokoog.xml

@@ -8,7 +8,7 @@
  -->
 <extension type="component" method="upgrade">
     <name>com_mokoog</name>
-    <version>01.07.03</version>
+    <version>01.07.00</version>
     <creationDate>2026-05-23</creationDate>
     <author>Moko Consulting</author>
     <authorEmail>hello@mokoconsulting.tech</authorEmail>

source/packages/com_mokoog/sql/updates/mysql/01.07.00.sql

@@ -0,0 +1 @@
+/* 01.07.00 — no schema changes */

source/packages/com_mokoog/sql/updates/mysql/01.07.01.sql

@@ -1 +0,0 @@
-/* 01.07.01 — no schema changes */

source/packages/com_mokoog/sql/updates/mysql/01.07.02.sql

@@ -1 +0,0 @@
-/* 01.07.02 — no schema changes */

source/packages/com_mokoog/sql/updates/mysql/01.07.03.sql

@@ -1 +0,0 @@
-/* 01.07.03 — no schema changes */

source/packages/com_mokoog/src/Controller/BatchController.php

@@ -73,9 +73,7 @@ class BatchController extends BaseController
         }
 
         $app    = Factory::getApplication();
-        $input  = $app->getInput();
-        $limit  = min($input->getInt('limit', 50), 200);
-        $lastId = max(0, $input->getInt('lastid', 0));
+        $limit  = min($app->getInput()->getInt('limit', 50), 200);
 
         $db    = Factory::getContainer()->get(\Joomla\Database\DatabaseInterface::class);
         $query = $db->getQuery(true)
@@ -90,25 +88,18 @@ class BatchController extends BaseController
             )
             ->where($db->quoteName('c.state') . ' = 1')
             ->where($db->quoteName('t.id') . ' IS NULL')
-            ->where($db->quoteName('c.id') . ' > ' . $lastId)
             ->order($db->quoteName('c.id') . ' ASC');
 
-        // Cursor-based pagination by id: each chunk fetches the next articles whose
-        // id is greater than the previous chunk's highest id. A row that fails to
-        // insert is passed over on the next chunk (its id is already behind the
-        // cursor) instead of being re-fetched forever, so the batch always reaches
-        // the end. The client stops when a chunk examines 0 rows.
+        // Always offset=0: processed articles now have #__mokoog_tags rows
+        // and are excluded by the LEFT JOIN ... IS NULL filter automatically.
         $db->setQuery($query, 0, $limit);
         $articles = $db->loadObjectList();
 
         $created = 0;
         $skipped = 0;
-        $lastProcessedId = $lastId;
         $now     = Factory::getDate()->toSql();
 
         foreach ($articles as $article) {
-            $lastProcessedId = (int) $article->id;
-
             $ogTitle       = $article->title;
             $ogDescription = $this->extractDescription($article);
             $ogImage       = $this->extractImage($article);
@@ -141,9 +132,6 @@ class BatchController extends BaseController
 
         echo new JsonResponse([
             'created'   => $created,
-            'skipped'  => $skipped,
-            'examined' => \count($articles),
-            'last_id'  => $lastProcessedId,
         ]);
 
         $app->close();

source/packages/com_mokoog/tmpl/tags/default.php

@@ -234,31 +234,27 @@ document.addEventListener('DOMContentLoaded', function() {
                     return;
                 }
                 status.textContent = total + ' <?php echo Text::_('COM_MOKOOG_BATCH_FOUND', true); ?>';
-                processChunk(0, 0, total, chunkSize, token, bar, status);
+                processChunk(0, total, chunkSize, token, bar, status);
             })
             .catch(function(err) {
                 status.textContent = '<?php echo Text::_('COM_MOKOOG_BATCH_ERROR', true); ?> ' + err.message;
             });
     }
 
-    function processChunk(lastId, processed, total, chunkSize, token, bar, status) {
-        // Cursor-based: pass the highest id seen so far. Failed rows fall behind
-        // the cursor and are not re-fetched, so the loop always terminates.
-        fetch('index.php?option=com_mokoog&task=batch.process&format=json&limit=' + chunkSize + '&lastid=' + lastId + '&' + token + '=1')
+    function processChunk(processed, total, chunkSize, token, bar, status) {
+        // Always offset=0: processed items are excluded by the IS NULL filter
+        fetch('index.php?option=com_mokoog&task=batch.process&format=json&limit=' + chunkSize + '&' + token + '=1')
             .then(function(r) { return r.json(); })
             .then(function(resp) {
-                var examined = resp.data.examined || 0;
-                processed += examined;
-                var pct = total > 0 ? Math.min(100, Math.round((processed / total) * 100)) : 100;
+                processed += resp.data.created;
+                var pct = Math.min(100, Math.round((processed / total) * 100));
                 bar.style.width = pct + '%';
                 bar.textContent = pct + '%';
                 status.textContent = processed + ' / ' + total + ' <?php echo Text::_('COM_MOKOOG_BATCH_PROCESSED', true); ?>';
 
-                if (examined > 0) {
-                    processChunk(resp.data.last_id, processed, total, chunkSize, token, bar, status);
+                if (resp.data.created > 0 && processed < total) {
+                    processChunk(processed, total, chunkSize, token, bar, status);
                 } else {
-                    bar.style.width = '100%';
-                    bar.textContent = '100%';
                     bar.classList.remove('progress-bar-animated');
                     bar.classList.add('bg-success');
                     status.textContent = '<?php echo Text::_('COM_MOKOOG_BATCH_COMPLETE', true); ?> ' + processed + ' articles.';

source/packages/plg_content_mokoog/mokoog.xml

@@ -8,7 +8,7 @@
  -->
 <extension type="plugin" group="content" method="upgrade">
     <name>Content - MokoSuiteOpenGraph</name>
-    <version>01.07.03</version>
+    <version>01.07.00</version>
     <creationDate>2026-05-23</creationDate>
     <author>Moko Consulting</author>
     <authorEmail>hello@mokoconsulting.tech</authorEmail>

source/packages/plg_system_mokoog/mokoog.xml

@@ -8,7 +8,7 @@
  -->
 <extension type="plugin" group="system" method="upgrade">
     <name>System - MokoSuiteOpenGraph</name>
-    <version>01.07.03</version>
+    <version>01.07.00</version>
     <creationDate>2026-05-23</creationDate>
     <author>Moko Consulting</author>
     <authorEmail>hello@mokoconsulting.tech</authorEmail>

source/packages/plg_system_mokoog/src/Extension/MokoOG.php

@@ -28,11 +28,6 @@ final class MokoOG extends CMSPlugin implements SubscriberInterface
      */
     protected $autoloadLanguage = true;
 
-    /**
-     * Minimum seconds between full sitemap regenerations (save-time throttle).
-     */
-    private const SITEMAP_MIN_INTERVAL = 60;
-
     /**
      * Returns the events this plugin subscribes to.
      *
@@ -850,15 +845,6 @@ final class MokoOG extends CMSPlugin implements SubscriberInterface
             return;
         }
 
-        // Throttle: rebuilding the whole sitemap on every save does not scale
-        // (bulk edits/imports). Regenerate at most once per interval — the
-        // sitemap is eventually consistent within that window.
-        $path = JPATH_ROOT . '/sitemap.xml';
-
-        if (is_file($path) && (time() - filemtime($path)) < self::SITEMAP_MIN_INTERVAL) {
-            return;
-        }
-
         $changefreq = $this->params->get('sitemap_changefreq', 'weekly');
         $xml = SitemapBuilder::generate($changefreq);
 

source/packages/plg_system_mokoog/src/Helper/ImageHelper.php

@@ -57,8 +57,96 @@ class ImageHelper
         int $targetHeight = self::TARGET_HEIGHT,
         int $quality = self::JPEG_QUALITY
     ): string {
-        // Thin wrapper over the shared implementation (no subdirectory).
-        return self::resizeToSize($imagePath, $targetWidth, $targetHeight, '', $quality);
+        // Resolve absolute path
+        $absPath = JPATH_ROOT . '/' . ltrim($imagePath, '/');
+
+        if (!is_file($absPath)) {
+            return $imagePath;
+        }
+
+        $imageInfo = getimagesize($absPath);
+
+        if (!$imageInfo) {
+            Log::add('MokoOG ImageHelper: Cannot read image dimensions: ' . basename($absPath), Log::WARNING, 'mokoog');
+
+            return $imagePath;
+        }
+
+        [$origWidth, $origHeight, $type] = $imageInfo;
+
+        // Skip if already at or below target size
+        if ($origWidth <= $targetWidth && $origHeight <= $targetHeight) {
+            return $imagePath;
+        }
+
+        // Ensure output directory exists
+        $outputDir = JPATH_ROOT . '/' . self::OUTPUT_DIR;
+
+        if (!is_dir($outputDir) && !Folder::create($outputDir)) {
+            Log::add('MokoOG ImageHelper: Cannot create output directory: ' . self::OUTPUT_DIR, Log::WARNING, 'mokoog');
+
+            return $imagePath;
+        }
+
+        // Generate output filename based on source hash + dimensions
+        $hash       = md5($imagePath . $targetWidth . $targetHeight);
+        $outputName = $hash . '.jpg';
+        $outputPath = $outputDir . '/' . $outputName;
+        $outputRel  = self::OUTPUT_DIR . '/' . $outputName;
+
+        // Skip if already generated
+        if (is_file($outputPath) && filemtime($outputPath) >= filemtime($absPath)) {
+            return $outputRel;
+        }
+
+        // Load source image
+        $source = self::loadImage($absPath, $type);
+
+        if (!$source) {
+            return $imagePath;
+        }
+
+        // Calculate crop dimensions (center crop to target aspect ratio)
+        $targetRatio = $targetWidth / $targetHeight;
+        $sourceRatio = $origWidth / $origHeight;
+
+        if ($sourceRatio > $targetRatio) {
+            // Source is wider — crop sides
+            $cropHeight = $origHeight;
+            $cropWidth  = (int) round($origHeight * $targetRatio);
+            $cropX      = (int) round(($origWidth - $cropWidth) / 2);
+            $cropY      = 0;
+        } else {
+            // Source is taller — crop top/bottom
+            $cropWidth  = $origWidth;
+            $cropHeight = (int) round($origWidth / $targetRatio);
+            $cropX      = 0;
+            $cropY      = (int) round(($origHeight - $cropHeight) / 2);
+        }
+
+        // Create output canvas and resample
+        $output = imagecreatetruecolor($targetWidth, $targetHeight);
+
+        imagecopyresampled(
+            $output,
+            $source,
+            0,
+            0,
+            $cropX,
+            $cropY,
+            $targetWidth,
+            $targetHeight,
+            $cropWidth,
+            $cropHeight
+        );
+
+        // Save as JPEG
+        imagejpeg($output, $outputPath, $quality);
+
+        imagedestroy($source);
+        imagedestroy($output);
+
+        return $outputRel;
     }
 
     /**
@@ -94,17 +182,11 @@ class ImageHelper
      * @param   int     $width      Target width
      * @param   int     $height     Target height
      * @param   string  $subdir     Subdirectory name for output (e.g. platform name)
-     * @param   int     $quality    JPEG quality 1-100
      *
      * @return  string  Path to the output image (relative to JPATH_ROOT)
      */
-    private static function resizeToSize(
-        string $imagePath,
-        int $width,
-        int $height,
-        string $subdir = '',
-        int $quality = self::JPEG_QUALITY
-    ): string {
+    private static function resizeToSize(string $imagePath, int $width, int $height, string $subdir = ''): string
+    {
         // Resolve absolute path
         $absPath = JPATH_ROOT . '/' . ltrim($imagePath, '/');
 
@@ -190,7 +272,7 @@ class ImageHelper
         );
 
         // Save as JPEG
-        imagejpeg($output, $outputPath, $quality);
+        imagejpeg($output, $outputPath, self::JPEG_QUALITY);
 
         imagedestroy($source);
         imagedestroy($output);
@@ -251,6 +333,43 @@ class ImageHelper
         }
     }
 
+    /**
+     * Check if an image meets minimum OG size requirements.
+     *
+     * @param   string  $imagePath  Image path relative to JPATH_ROOT
+     *
+     * @return  array{valid: bool, width: int, height: int, message: string}
+     */
+    public static function validate(string $imagePath): array
+    {
+        $absPath = JPATH_ROOT . '/' . ltrim($imagePath, '/');
+
+        if (!is_file($absPath)) {
+            return ['valid' => false, 'width' => 0, 'height' => 0, 'message' => 'File not found'];
+        }
+
+        $imageInfo = getimagesize($absPath);
+
+        if (!$imageInfo) {
+            return ['valid' => false, 'width' => 0, 'height' => 0, 'message' => 'Not a valid image'];
+        }
+
+        [$width, $height] = $imageInfo;
+
+        // Facebook minimum: 200x200, recommended: 1200x630
+        // WhatsApp minimum: 300x200
+        if ($width < 200 || $height < 200) {
+            return [
+                'valid'   => false,
+                'width'   => $width,
+                'height'  => $height,
+                'message' => "Image too small ({$width}x{$height}). Minimum: 200x200px.",
+            ];
+        }
+
+        return ['valid' => true, 'width' => $width, 'height' => $height, 'message' => 'OK'];
+    }
+
     /**
      * Load an image resource from a file.
      *

source/packages/plg_system_mokoog/src/Helper/SitemapBuilder.php

@@ -81,7 +81,7 @@ class SitemapBuilder
                 continue;
             }
 
-            $url     = self::articleUrl($article, $root);
+            $url     = $root . '/index.php?option=com_content&view=article&id=' . $article->id;
             $lastmod = $article->modified && $article->modified !== '0000-00-00 00:00:00'
                 ? date('Y-m-d', strtotime($article->modified)) : '';
 
@@ -102,45 +102,6 @@ class SitemapBuilder
         return $xml;
     }
 
-    /**
-     * Build the SEF/canonical site URL for an article, with a safe fallback.
-     *
-     * Routes through the site router so the sitemap matches the canonical URLs
-     * the plugin emits. If routing fails (or SEF is off), falls back to the
-     * non-SEF index.php URL — never an empty or broken URL.
-     *
-     * @param   object  $article  Row with id, alias, catid, language
-     * @param   string  $root     Site root without trailing slash
-     *
-     * @return  string  Absolute URL
-     */
-    private static function articleUrl(object $article, string $root): string
-    {
-        $fallback = $root . '/index.php?option=com_content&view=article&id=' . (int) $article->id;
-
-        $internal = 'index.php?option=com_content&view=article&id=' . (int) $article->id
-            . (!empty($article->alias) ? ':' . $article->alias : '')
-            . (!empty($article->catid) ? '&catid=' . (int) $article->catid : '');
-
-        try {
-            $routed = \Joomla\CMS\Router\Route::link(
-                'site',
-                $internal,
-                false,
-                \Joomla\CMS\Router\Route::TLS_IGNORE,
-                true
-            );
-
-            if (\is_string($routed) && $routed !== '') {
-                return $routed;
-            }
-        } catch (\Throwable $e) {
-            // Fall back to the non-SEF URL below.
-        }
-
-        return $fallback;
-    }
-
     /**
      * Write sitemap XML to the site root.
      *

source/packages/plg_webservices_mokoog/mokoog.xml

@@ -8,7 +8,7 @@
  -->
 <extension type="plugin" group="webservices" method="upgrade">
     <name>Web Services - MokoSuiteOpenGraph</name>
-    <version>01.07.03</version>
+    <version>01.07.00</version>
     <creationDate>2026-05-23</creationDate>
     <author>Moko Consulting</author>
     <authorEmail>hello@mokoconsulting.tech</authorEmail>

source/pkg_mokoog.xml

@@ -8,7 +8,7 @@
 <extension type="package" method="upgrade">
     <name>Package - MokoSuiteOpenGraph</name>
     <packagename>mokoog</packagename>
-    <version>01.07.03</version>
+    <version>01.07.00</version>
     <creationDate>2026-05-23</creationDate>
     <author>Moko Consulting</author>
     <authorEmail>hello@mokoconsulting.tech</authorEmail>