Compare commits
2 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| ee80111336 | |||
| 29f83d06ee |
@@ -5,7 +5,7 @@
|
|||||||
# FILE INFORMATION
|
# FILE INFORMATION
|
||||||
# DEFGROUP: Gitea.Workflow
|
# DEFGROUP: Gitea.Workflow
|
||||||
# INGROUP: mokocli.Automation
|
# INGROUP: mokocli.Automation
|
||||||
# VERSION: 01.07.03
|
# VERSION: 01.07.06
|
||||||
# BRIEF: Auto-create feature branch when an issue is opened
|
# BRIEF: Auto-create feature branch when an issue is opened
|
||||||
|
|
||||||
name: "Universal: Issue Branch"
|
name: "Universal: Issue Branch"
|
||||||
|
|||||||
+4
-1
@@ -4,10 +4,13 @@ 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/).
|
The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
|
||||||
|
|
||||||
<!-- VERSION: 01.07.03 -->
|
<!-- VERSION: 01.07.06 -->
|
||||||
|
|
||||||
## [Unreleased]
|
## [Unreleased]
|
||||||
|
|
||||||
|
### Fixed
|
||||||
|
- **Honest install success** — the installer no longer claims success or shows the license message when the install partially failed. The package postflight verifies every child extension declared in the manifest registered in `#__extensions`; the `com_mokoog` component script verifies its declared tables exist before echoing the success message. Both checks fail open, so a transient DB/IO glitch never fakes a failure.
|
||||||
|
|
||||||
### Added
|
### 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)
|
- 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)
|
||||||
- Single OG tag **create/edit screen** in the admin (the tag manager was previously read-only) (#98)
|
- Single OG tag **create/edit screen** in the admin (the tag manager was previously read-only) (#98)
|
||||||
|
|||||||
+1
-1
@@ -14,7 +14,7 @@
|
|||||||
DEFGROUP: Template-Joomla
|
DEFGROUP: Template-Joomla
|
||||||
INGROUP: Template-Joomla.Documentation
|
INGROUP: Template-Joomla.Documentation
|
||||||
REPO: https://github.com/mokoconsulting-tech/Template-Joomla/
|
REPO: https://github.com/mokoconsulting-tech/Template-Joomla/
|
||||||
VERSION: 01.07.03
|
VERSION: 01.07.06
|
||||||
PATH: ./CODE_OF_CONDUCT.md
|
PATH: ./CODE_OF_CONDUCT.md
|
||||||
BRIEF: Community expectations and enforcement guidelines
|
BRIEF: Community expectations and enforcement guidelines
|
||||||
NOTE: Adapted with attribution from the Contributor Covenant v2.1
|
NOTE: Adapted with attribution from the Contributor Covenant v2.1
|
||||||
|
|||||||
+1
-1
@@ -19,7 +19,7 @@
|
|||||||
DEFGROUP: mokoconsulting-tech.Template-Joomla
|
DEFGROUP: mokoconsulting-tech.Template-Joomla
|
||||||
INGROUP: MokoStandards.Governance
|
INGROUP: MokoStandards.Governance
|
||||||
REPO: https://github.com/mokoconsulting-tech/Template-Joomla
|
REPO: https://github.com/mokoconsulting-tech/Template-Joomla
|
||||||
VERSION: 01.07.03
|
VERSION: 01.07.06
|
||||||
PATH: /GOVERNANCE.md
|
PATH: /GOVERNANCE.md
|
||||||
BRIEF: Project governance rules, roles, and decision process for Template-Joomla
|
BRIEF: Project governance rules, roles, and decision process for Template-Joomla
|
||||||
-->
|
-->
|
||||||
|
|||||||
@@ -1,6 +1,6 @@
|
|||||||
# MokoSuiteOpenGraph
|
# MokoSuiteOpenGraph
|
||||||
|
|
||||||
<!-- VERSION: 01.07.03 -->
|
<!-- VERSION: 01.07.06 -->
|
||||||
|
|
||||||
Open Graph, Twitter Card, and social sharing meta tag management for Joomla 6 and higher.
|
Open Graph, Twitter Card, and social sharing meta tag management for Joomla 6 and higher.
|
||||||
|
|
||||||
|
|||||||
+1
-1
@@ -23,7 +23,7 @@ DEFGROUP: Template-Joomla
|
|||||||
INGROUP: Template-Joomla.Documentation
|
INGROUP: Template-Joomla.Documentation
|
||||||
REPO: https://git.mokoconsulting.tech/MokoConsulting/Template-Joomla
|
REPO: https://git.mokoconsulting.tech/MokoConsulting/Template-Joomla
|
||||||
PATH: /SECURITY.md
|
PATH: /SECURITY.md
|
||||||
VERSION: 01.07.03
|
VERSION: 01.07.06
|
||||||
BRIEF: Security vulnerability reporting and handling policy
|
BRIEF: Security vulnerability reporting and handling policy
|
||||||
-->
|
-->
|
||||||
|
|
||||||
|
|||||||
@@ -8,7 +8,7 @@
|
|||||||
-->
|
-->
|
||||||
<extension type="component" method="upgrade">
|
<extension type="component" method="upgrade">
|
||||||
<name>com_mokoog</name>
|
<name>com_mokoog</name>
|
||||||
<version>01.07.03</version>
|
<version>01.07.06</version>
|
||||||
<creationDate>2026-05-23</creationDate>
|
<creationDate>2026-05-23</creationDate>
|
||||||
<author>Moko Consulting</author>
|
<author>Moko Consulting</author>
|
||||||
<authorEmail>hello@mokoconsulting.tech</authorEmail>
|
<authorEmail>hello@mokoconsulting.tech</authorEmail>
|
||||||
|
|||||||
@@ -10,7 +10,9 @@
|
|||||||
|
|
||||||
defined('_JEXEC') or die;
|
defined('_JEXEC') or die;
|
||||||
|
|
||||||
|
use Joomla\CMS\Factory;
|
||||||
use Joomla\CMS\Installer\InstallerAdapter;
|
use Joomla\CMS\Installer\InstallerAdapter;
|
||||||
|
use Joomla\Database\DatabaseInterface;
|
||||||
|
|
||||||
class Com_MokoOGInstallerScript
|
class Com_MokoOGInstallerScript
|
||||||
{
|
{
|
||||||
@@ -23,6 +25,20 @@ class Com_MokoOGInstallerScript
|
|||||||
*/
|
*/
|
||||||
public function install(InstallerAdapter $parent): void
|
public function install(InstallerAdapter $parent): void
|
||||||
{
|
{
|
||||||
|
// Be honest about success: don't claim a clean install if the component's
|
||||||
|
// own tables never landed. Fails open (see missingTables) so a query/IO
|
||||||
|
// glitch never fakes a failure.
|
||||||
|
$missing = $this->missingTables();
|
||||||
|
|
||||||
|
if (!empty($missing))
|
||||||
|
{
|
||||||
|
echo '<p>MokoSuiteOpenGraph component did not install correctly. '
|
||||||
|
. 'Missing database table(s): '
|
||||||
|
. htmlspecialchars(implode(', ', $missing), ENT_QUOTES) . '.</p>';
|
||||||
|
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
echo '<p>MokoSuiteOpenGraph component installed successfully.</p>';
|
echo '<p>MokoSuiteOpenGraph component installed successfully.</p>';
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -35,6 +51,69 @@ class Com_MokoOGInstallerScript
|
|||||||
*/
|
*/
|
||||||
public function update(InstallerAdapter $parent): void
|
public function update(InstallerAdapter $parent): void
|
||||||
{
|
{
|
||||||
|
$missing = $this->missingTables();
|
||||||
|
|
||||||
|
if (!empty($missing))
|
||||||
|
{
|
||||||
|
echo '<p>MokoSuiteOpenGraph component did not update correctly. '
|
||||||
|
. 'Missing database table(s): '
|
||||||
|
. htmlspecialchars(implode(', ', $missing), ENT_QUOTES) . '.</p>';
|
||||||
|
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
echo '<p>MokoSuiteOpenGraph component updated successfully.</p>';
|
echo '<p>MokoSuiteOpenGraph component updated successfully.</p>';
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Verify the component's declared tables actually exist. Reads the shipped
|
||||||
|
* install SQL, extracts every CREATE TABLE #__name, resolves the live prefix,
|
||||||
|
* and checks each against the database's table list (case-insensitive).
|
||||||
|
*
|
||||||
|
* FAILS OPEN: any error (unreadable SQL, query failure) returns [] so a
|
||||||
|
* transient glitch never turns a good install into a false failure.
|
||||||
|
*
|
||||||
|
* @return string[] Prefixed names of any missing tables.
|
||||||
|
*/
|
||||||
|
private function missingTables(): array
|
||||||
|
{
|
||||||
|
try
|
||||||
|
{
|
||||||
|
$sqlFile = JPATH_ADMINISTRATOR . '/components/com_mokoog/sql/install.mysql.sql';
|
||||||
|
|
||||||
|
if (!is_file($sqlFile))
|
||||||
|
{
|
||||||
|
return [];
|
||||||
|
}
|
||||||
|
|
||||||
|
$sql = (string) file_get_contents($sqlFile);
|
||||||
|
|
||||||
|
if ($sql === '' || !preg_match_all('/CREATE\s+TABLE\s+(?:IF\s+NOT\s+EXISTS\s+)?[`"\']?(#__[A-Za-z0-9_]+)/i', $sql, $m))
|
||||||
|
{
|
||||||
|
return [];
|
||||||
|
}
|
||||||
|
|
||||||
|
$db = Factory::getContainer()->get(DatabaseInterface::class);
|
||||||
|
$prefix = $db->getPrefix();
|
||||||
|
$existing = array_map('strtolower', (array) $db->getTableList());
|
||||||
|
$missing = [];
|
||||||
|
|
||||||
|
foreach (array_unique($m[1]) as $table)
|
||||||
|
{
|
||||||
|
$real = str_replace('#__', $prefix, $table);
|
||||||
|
|
||||||
|
if (!in_array(strtolower($real), $existing, true))
|
||||||
|
{
|
||||||
|
$missing[] = $real;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
return $missing;
|
||||||
|
}
|
||||||
|
catch (\Throwable $e)
|
||||||
|
{
|
||||||
|
// Fail open — never fake a failure on a glitch.
|
||||||
|
return [];
|
||||||
|
}
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -0,0 +1 @@
|
|||||||
|
/* 01.07.06 — no schema changes */
|
||||||
@@ -8,7 +8,7 @@
|
|||||||
-->
|
-->
|
||||||
<extension type="plugin" group="content" method="upgrade">
|
<extension type="plugin" group="content" method="upgrade">
|
||||||
<name>Content - MokoSuiteOpenGraph</name>
|
<name>Content - MokoSuiteOpenGraph</name>
|
||||||
<version>01.07.03</version>
|
<version>01.07.06</version>
|
||||||
<creationDate>2026-05-23</creationDate>
|
<creationDate>2026-05-23</creationDate>
|
||||||
<author>Moko Consulting</author>
|
<author>Moko Consulting</author>
|
||||||
<authorEmail>hello@mokoconsulting.tech</authorEmail>
|
<authorEmail>hello@mokoconsulting.tech</authorEmail>
|
||||||
|
|||||||
@@ -8,7 +8,7 @@
|
|||||||
-->
|
-->
|
||||||
<extension type="plugin" group="system" method="upgrade">
|
<extension type="plugin" group="system" method="upgrade">
|
||||||
<name>System - MokoSuiteOpenGraph</name>
|
<name>System - MokoSuiteOpenGraph</name>
|
||||||
<version>01.07.03</version>
|
<version>01.07.06</version>
|
||||||
<creationDate>2026-05-23</creationDate>
|
<creationDate>2026-05-23</creationDate>
|
||||||
<author>Moko Consulting</author>
|
<author>Moko Consulting</author>
|
||||||
<authorEmail>hello@mokoconsulting.tech</authorEmail>
|
<authorEmail>hello@mokoconsulting.tech</authorEmail>
|
||||||
|
|||||||
@@ -8,7 +8,7 @@
|
|||||||
-->
|
-->
|
||||||
<extension type="plugin" group="webservices" method="upgrade">
|
<extension type="plugin" group="webservices" method="upgrade">
|
||||||
<name>Web Services - MokoSuiteOpenGraph</name>
|
<name>Web Services - MokoSuiteOpenGraph</name>
|
||||||
<version>01.07.03</version>
|
<version>01.07.06</version>
|
||||||
<creationDate>2026-05-23</creationDate>
|
<creationDate>2026-05-23</creationDate>
|
||||||
<author>Moko Consulting</author>
|
<author>Moko Consulting</author>
|
||||||
<authorEmail>hello@mokoconsulting.tech</authorEmail>
|
<authorEmail>hello@mokoconsulting.tech</authorEmail>
|
||||||
|
|||||||
@@ -8,7 +8,7 @@
|
|||||||
<extension type="package" method="upgrade">
|
<extension type="package" method="upgrade">
|
||||||
<name>Package - MokoSuiteOpenGraph</name>
|
<name>Package - MokoSuiteOpenGraph</name>
|
||||||
<packagename>mokoog</packagename>
|
<packagename>mokoog</packagename>
|
||||||
<version>01.07.03</version>
|
<version>01.07.06</version>
|
||||||
<creationDate>2026-05-23</creationDate>
|
<creationDate>2026-05-23</creationDate>
|
||||||
<author>Moko Consulting</author>
|
<author>Moko Consulting</author>
|
||||||
<authorEmail>hello@mokoconsulting.tech</authorEmail>
|
<authorEmail>hello@mokoconsulting.tech</authorEmail>
|
||||||
|
|||||||
+96
-1
@@ -51,8 +51,8 @@ class Pkg_MokoOGInstallerScript
|
|||||||
public function postflight(string $type, InstallerAdapter $parent): void
|
public function postflight(string $type, InstallerAdapter $parent): void
|
||||||
{
|
{
|
||||||
$this->restoreDownloadKey();
|
$this->restoreDownloadKey();
|
||||||
$this->warnMissingLicenseKey();
|
|
||||||
|
|
||||||
|
// Enable plugins after first install (unconditional module/plugin setup)
|
||||||
if ($type === 'install')
|
if ($type === 'install')
|
||||||
{
|
{
|
||||||
$db = Factory::getContainer()->get(\Joomla\Database\DatabaseInterface::class);
|
$db = Factory::getContainer()->get(\Joomla\Database\DatabaseInterface::class);
|
||||||
@@ -69,6 +69,101 @@ class Pkg_MokoOGInstallerScript
|
|||||||
)->execute();
|
)->execute();
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// Be honest about success. Joomla's package installer only LOGS a failed child
|
||||||
|
// sub-install but still runs this postflight, so don't show the license /
|
||||||
|
// next-steps message if a bundled extension is actually missing. Fails open
|
||||||
|
// (see missingChildExtensions) so a query/IO glitch never fakes a failure.
|
||||||
|
$missing = $this->missingChildExtensions($parent);
|
||||||
|
|
||||||
|
if (!empty($missing))
|
||||||
|
{
|
||||||
|
Factory::getApplication()->enqueueMessage(
|
||||||
|
'<h4>MokoSuiteOpenGraph did not install correctly.</h4>'
|
||||||
|
. '<p>The following bundled extensions are missing: '
|
||||||
|
. htmlspecialchars(implode(', ', $missing), ENT_QUOTES) . '</p>'
|
||||||
|
. '<p>Please uninstall MokoSuiteOpenGraph and reinstall the full package.</p>',
|
||||||
|
'error'
|
||||||
|
);
|
||||||
|
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
$this->warnMissingLicenseKey();
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Verify every child extension declared in the package manifest actually landed
|
||||||
|
* in #__extensions. Returns readable labels of any that are missing.
|
||||||
|
*
|
||||||
|
* element = <file> "id", EXCEPT plugins strip a leading plg_<group>_; plugins also
|
||||||
|
* match folder = <file> "group". FAILS OPEN — any error returns [] so a transient
|
||||||
|
* glitch never turns a good install into a false failure.
|
||||||
|
*/
|
||||||
|
private function missingChildExtensions($parent): array
|
||||||
|
{
|
||||||
|
try
|
||||||
|
{
|
||||||
|
$manifest = $parent->getParent()->getManifest();
|
||||||
|
|
||||||
|
if (!$manifest || !isset($manifest->files) || !isset($manifest->files->file))
|
||||||
|
{
|
||||||
|
return [];
|
||||||
|
}
|
||||||
|
|
||||||
|
$db = Factory::getContainer()->get(\Joomla\Database\DatabaseInterface::class);
|
||||||
|
$missing = [];
|
||||||
|
|
||||||
|
foreach ($manifest->files->file as $file)
|
||||||
|
{
|
||||||
|
$attrs = $file->attributes();
|
||||||
|
$id = isset($attrs['id']) ? (string) $attrs['id'] : '';
|
||||||
|
$exType = isset($attrs['type']) ? (string) $attrs['type'] : '';
|
||||||
|
|
||||||
|
if ($id === '' || $exType === '')
|
||||||
|
{
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
|
$group = isset($attrs['group']) ? (string) $attrs['group'] : '';
|
||||||
|
$element = $id;
|
||||||
|
|
||||||
|
// Plugin element in #__extensions is the id minus any leading plg_<group>_.
|
||||||
|
if ($exType === 'plugin' && $group !== '')
|
||||||
|
{
|
||||||
|
$prefix = 'plg_' . $group . '_';
|
||||||
|
|
||||||
|
if (strpos($element, $prefix) === 0)
|
||||||
|
{
|
||||||
|
$element = substr($element, \strlen($prefix));
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
$query = $db->getQuery(true)
|
||||||
|
->select('COUNT(*)')
|
||||||
|
->from($db->quoteName('#__extensions'))
|
||||||
|
->where($db->quoteName('element') . ' = ' . $db->quote($element))
|
||||||
|
->where($db->quoteName('type') . ' = ' . $db->quote($exType));
|
||||||
|
|
||||||
|
if ($exType === 'plugin' && $group !== '')
|
||||||
|
{
|
||||||
|
$query->where($db->quoteName('folder') . ' = ' . $db->quote($group));
|
||||||
|
}
|
||||||
|
|
||||||
|
if ((int) $db->setQuery($query)->loadResult() === 0)
|
||||||
|
{
|
||||||
|
$label = trim((string) $file);
|
||||||
|
$missing[] = $label !== '' ? preg_replace('/\.zip$/i', '', $label) : $id;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
return $missing;
|
||||||
|
}
|
||||||
|
catch (\Throwable $e)
|
||||||
|
{
|
||||||
|
// Fail open — never fake a failure on a glitch.
|
||||||
|
return [];
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user