doxygen: Document problem with slashes and fix a few
authorTimo Tijhof <krinklemail@gmail.com>
Sat, 30 Aug 2014 16:51:42 +0000 (18:51 +0200)
committerKrinkle <krinklemail@gmail.com>
Tue, 23 Sep 2014 19:32:13 +0000 (19:32 +0000)
Change-Id: I39f8f394e7421fca71e7d26818f7118e85de7e4f

includes/resourceloader/ResourceLoader.php
maintenance/mwdoc-filter.php

index 4f1414b..57deb00 100644 (file)
@@ -35,22 +35,32 @@ class ResourceLoader {
        /** @var bool */
        protected static $debugMode = null;
 
-       /** @var array Module name/ResourceLoaderModule object pairs */
+       /**
+        * Module name/ResourceLoaderModule object pairs
+        * @var array
+        */
        protected $modules = array();
 
-       /** @var array Associative array mapping module name to info associative array */
+       /**
+        * Associative array mapping module name to info associative array
+        * @var array
+        */
        protected $moduleInfos = array();
 
        /** @var Config $config */
        private $config;
 
        /**
-        * @var array Associative array mapping framework ids to a list of names of test suite modules
-        *      like array( 'qunit' => array( 'mediawiki.tests.qunit.suites', 'ext.foo.tests', .. ), .. )
+        * Associative array mapping framework ids to a list of names of test suite modules
+        * like array( 'qunit' => array( 'mediawiki.tests.qunit.suites', 'ext.foo.tests', .. ), .. )
+        * @var array
         */
        protected $testModuleNames = array();
 
-       /** @var array E.g. array( 'source-id' => 'http://.../load.php' ) */
+       /**
+        * E.g. array( 'source-id' => 'http://.../load.php' )
+        * @var array
+        */
        protected $sources = array();
 
        /** @var bool */
index 8650689..0cc7bde 100644 (file)
@@ -23,7 +23,17 @@ $regexp = '#'
        . '([^\s]+)'
        . '\s+'
        // Any text or line(s) between type hint and '/' closing the comment
-       // (includes the star of "*/")
+       // (includes the star of "*/"). Descriptions containing a slash
+       // are not supported. Those will have to to be rewritten to have their
+       // description *before* the @var:
+       // /**
+       //  * Description with / in it.
+       //  * @var array
+       //  */
+       // instead of:
+       // /**
+       //  * @var array Description with / in it.
+       //  */
        . '([^/]+)'
        . '/'
        . '\s+'