[Mediawiki-l] New Extension: AllTimeZones
nischayn22 at gmail.com
Wed Apr 4 17:47:34 UTC 2012
Thanks everyone for your feedback
On Wed, Apr 4, 2012 at 10:58 PM, Jeroen De Dauw <jeroendedauw at gmail.com>wrote:
> I had a quick look at this extension and have some suggestions:
> * Escape ALL variables in HTML you are building. You can use
> htmlspecialchars, the methods of MediaWikis HTML class, and in this case
> probably want to just construct the whole thing with XmlSelect.
> * You are not fully following the style guidelines. The spacing is off and
> you are using underscores in var names instead of lowerCamelCase. See
> * Don't use the 'description' in wgExtensionCredits, use 'descriptionmsg'
> instead, and properly internationalize it.
> Next to that it looks great, although I'm not completely sure the i18n is
> all in order when using the native PHP time functions :)
Yes, you are right. I have only spend about 5-7 hours in making that
extension and was too excited to mail the list.
It needs a lot of cleanup and i18n and some better UI also. I will also
change the var names.
Besides that I also look forward to make a general extension to handle
conversion of units for length,weight etc
B.tech 3rd year
Department of Information Technology
More information about the MediaWiki-l