@@ -203,6 +203,13 @@ class PageMetadataController extends PluginController { * @param the current page */ public static function Callback_page_page_updated($page) { + // If there is no metadata submited at all, just delete (possible) existing ones for this page. + // There could be invisible metadata (from other plug-ins). Thanks to: Andy + if (!isset($_POST[self::PLUGIN_ID])) { + PageMetadata::DeleteAllByPage($page); + return; + } + // Get all posted metadata foreach ($_POST[self::PLUGIN_ID] as $metadata) { $value = trim($metadata["value"]); PageMetadataController.php @@ -2,7 +2,7 @@ <frog-plugins> <frog-plugin> <id>page_metadata</id> - <version>1.0.0</version> + <version>1.0.1</version> <status>final</status> </frog-plugin> </frog-plugins> \ No newline at end of file frog-plugins.xml @@ -7,7 +7,7 @@ Plugin::setInfos(array( 'id' => $plugin_id, 'title' => __('Page Metadata'), 'description' => __('Allows to add more metadata to a page.'), - 'version' => '1.0.0', + 'version' => '1.0.1', 'license' => 'MIT', 'author' => 'THE M', 'website' => 'http://github.com/them/frog_page_metadata/', index.php @@ -39,7 +39,7 @@ class PageMetadata extends Record { return array(); } - return Record::findAllFrom(__CLASS__, 'page_id = ? ORDER BY keyword', array($page_id)); + return self::findAllFrom(__CLASS__, 'page_id = ? ORDER BY keyword', array($page_id)); } /** @@ -71,7 +71,7 @@ class PageMetadata extends Record { return null; } - return Record::findOneFrom(__CLASS__, 'page_id = ? AND keyword = ?', array($page_id, $keyword)); + return self::findOneFrom(__CLASS__, 'page_id = ? AND keyword = ?', array($page_id, $keyword)); } /** @@ -84,7 +84,7 @@ class PageMetadata extends Record { return; } - Record::deleteWhere(__CLASS__, 'page_id = ?', array($page_id)); + self::deleteWhere(__CLASS__, 'page_id = ?', array($page_id)); } /** @@ -97,7 +97,7 @@ class PageMetadata extends Record { return; } - Record::deleteWhere(__CLASS__, 'keyword = ?', array($keyword)); + self::deleteWhere(__CLASS__, 'keyword = ?', array($keyword)); } } ?> \ No newline at end of file models/PageMetadata.php 4aebe10acdf350a3f66277e3e7d7e7737b52ab75 The M only.the.em@googlemail.com http://github.com/them/frog_page_metadata/commit/d66a1accf048e0c8c31a60db1b33f05793059692 d66a1accf048e0c8c31a60db1b33f05793059692 2009-07-22T09:09:40-07:00 2009-07-22T09:09:40-07:00 Version: 1.0.1 - [fix] No processing (and warning due to lousy programming (too much Perl hacking)) if there is no metadata for a page - Use inherited method instead of delegation 5acde3aba684f3c03cdac52b4ec8a69cac36f756 The M only.the.em@googlemail.com