Fixes #4133 updated type in plugin manifest test

diff --git a/engine/classes/ElggPluginManifest.php b/engine/classes/ElggPluginManifest.php
index eacc16455875db7f1aebec4621d8080d7c4b12a5..7592eb6676c7191ff016c1e51544f5cab67e919b 100644 (file)
--- a/engine/classes/ElggPluginManifest.php
+++ b/engine/classes/ElggPluginManifest.php
@@ -553,7 +553,7 @@ class ElggPluginManifest {
        }
 
        /**
-        * Returns the admin interface to use.
+        * Should this plugin be activated when Elgg is installed
         *
         *  @return bool
         */

diff --git a/engine/tests/api/plugins.php b/engine/tests/api/plugins.php
index a0faaff0ebe7dbf450c6af089252fe88c370b112..ac38f34eadf78ea4e7d137a68c0782bc128b64d7 100644 (file)
--- a/engine/tests/api/plugins.php
+++ b/engine/tests/api/plugins.php
@@ -105,7 +105,8 @@ class ElggCorePluginsAPITest extends ElggCoreUnitTest {
                                array('type' => 'plugin', 'name' => 'facebook_connect', 'version' => 1.0),
                        ),
 
-                       'activate_on_install' => true
+                       // string because we are reading from a file
+                       'activate_on_install' => 'true',
                );
 
                $this->assertEqual($this->manifest18->getManifest(), $manifest_array);