Refs #3646. "getRepositoryURL" is more descriptive than "getRepository".

diff --git a/engine/classes/ElggPluginManifest.php b/engine/classes/ElggPluginManifest.php
index 5eb248c2c8dd7022fb5ce858de668e2c5368d72e..5706d1e4274d8b7f300ff2cdbbeed59ec63cb4c3 100644 (file)
--- a/engine/classes/ElggPluginManifest.php
+++ b/engine/classes/ElggPluginManifest.php
@@ -264,7 +264,7 @@ class ElggPluginManifest {
        /**
         * Returns the license
         *
-        * @return sting
+        * @return string
         */
        public function getLicense() {
                // license vs licence.  Use license.
@@ -279,16 +279,16 @@ class ElggPluginManifest {
        /**
         * Returns the repository url
         *
-        * @return sting
+        * @return string
         */
-       public function getRepository() {
+       public function getRepositoryURL() {
                return $this->parser->getAttribute('repository');
        }
 
        /**
         * Returns the bug tracker page
         *
-        * @return sting
+        * @return string
         */
        public function getBugTracker() {
                return $this->parser->getAttribute('bugtracker');
@@ -297,7 +297,7 @@ class ElggPluginManifest {
        /**
         * Returns the donations page
         *
-        * @return sting
+        * @return string
         */
        public function getDonationsPage() {
                return $this->parser->getAttribute('donations');

diff --git a/engine/tests/api/plugins.php b/engine/tests/api/plugins.php
index 66297266c13138727f5ef37ebffd0b1f372bc14f..3b650f1b0199ae9ed5331f6b95cf9471932e1763 100644 (file)
--- a/engine/tests/api/plugins.php
+++ b/engine/tests/api/plugins.php
@@ -169,8 +169,8 @@ class ElggCorePluginsAPITest extends ElggCoreUnitTest {
        }
        
        public function testElggPluginManifestGetRepository() {
-               $this->assertEqual($this->manifest18->getRepository(), 'https://github.com/Elgg/Elgg');
-               $this->assertEqual($this->manifest17->getRepository(), '');
+               $this->assertEqual($this->manifest18->getRepositoryURL(), 'https://github.com/Elgg/Elgg');
+               $this->assertEqual($this->manifest17->getRepositoryURL(), '');
        }
        
                public function testElggPluginManifestGetBugtracker() {

diff --git a/views/default/object/plugin/full.php b/views/default/object/plugin/full.php
index a96acd4ec947dc08608c9f2d9e57f6170c8d6c71..76e1136f2f67c874b2785ebab68e1622ebd405ad 100644 (file)
--- a/views/default/object/plugin/full.php
+++ b/views/default/object/plugin/full.php
@@ -173,7 +173,7 @@ $website = elgg_view('output/url', array(
 ));
 
 $resources = array(
-       'repository' => $plugin->getManifest()->getRepository(),
+       'repository' => $plugin->getManifest()->getRepositoryURL(),
        'bugtracker' => $plugin->getManifest()->getBugTracker(),
        'donate' => $plugin->getManifest()->getDonationsPage(),
 );