From e5e32c4cb7454425a0116a1700aaf879a3d27e29 Mon Sep 17 00:00:00 2001 From: Jonathan Miller Date: Thu, 28 May 2026 17:40:24 -0500 Subject: [PATCH] fix(cli): fix joomla_release org, element derivation, isset bug - Change ORG from mokoconsulting-tech to MokoConsulting - Fix infourl from github.com to git.mokoconsulting.tech - Strip type prefix from name before deriving element - Use array_key_exists instead of isset for null-valued stability Authored-by: Moko Consulting Co-Authored-By: Claude Opus 4.6 (1M context) --- cli/joomla_release.php | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/cli/joomla_release.php b/cli/joomla_release.php index 5d75e2b..403da57 100644 --- a/cli/joomla_release.php +++ b/cli/joomla_release.php @@ -37,7 +37,7 @@ use MokoEnterprise\{ApiClient, AuditLogger, CliFramework, Config, PlatformAdapte class JoomlaRelease extends CliFramework { private const VERSION = '04.06.00'; - private const ORG = 'mokoconsulting-tech'; + private const ORG = 'MokoConsulting'; private const STABILITY_TAGS = [ 'development' => 'development', @@ -75,7 +75,7 @@ class JoomlaRelease extends CliFramework $stability = (string) $this->getArgument('--stability'); $dryRun = (bool) $this->getArgument('--dry-run'); - if (!isset(self::STABILITY_TAGS[$stability])) { + if (!array_key_exists($stability, self::STABILITY_TAGS)) { $this->log('ERROR', "Invalid stability: {$stability}. Use: " . implode(', ', array_keys(self::STABILITY_TAGS))); return 1; } @@ -213,7 +213,9 @@ class JoomlaRelease extends CliFramework // Templates don't have — derive from if ($element === '') { - $element = strtolower(str_replace(' ', '', $name)); + // Strip type prefix (e.g. "Template - ") before deriving element + $baseName = preg_replace('/^(Package|Plugin|Module|Component|Template|Library|File)\s*-\s*/i', '', $name); + $element = strtolower(str_replace([' ', '-'], '', $baseName)); } $tp = ''; @@ -435,7 +437,7 @@ class JoomlaRelease extends CliFramework $lines[] = ' '; $lines[] = " {$stability}"; $lines[] = ' '; - $lines[] = " https://github.com/" . self::ORG . ""; + $lines[] = " https://git.mokoconsulting.tech/" . self::ORG . ""; $lines[] = ' '; $lines[] = " {$zipUrl}"; $lines[] = " {$tarUrl}";