[Phpmyadmin-devel] jshint errors

adam adamgsoc2013 at gmail.com
Tue Apr 16 17:05:16 CEST 2013


HI,
I also send a simple changes to fix jshint errors;
1 already defined
2 missing semicolon
https://github.com/phpmyadmin/phpmyadmin/pull/260
>> 
>> Dne Tue, 16 Apr 2013 06:33:52 -0400
>> Marc Delisle <marc at infomarc.info> napsal(a):
>> 
>>> Le 2013-04-16 03:06, Michal Čihař a écrit :
>>>> I think PMA_messages.strSave is shorter and looks nicer than
>>>> PMA_messages['strSave'], though I don't have strong opinion on that.
>>>> Also with messages (the most frequent of this warning in our case) we
>>>> can be pretty sure we don't use reserved words, so the dot syntax is
>>>> always safe here.
>>> I also think the dot notation is better, and here is another reason. 
>>> When seeing the string in quotes, I always wonder whether the string 
>>> 'strSave' will be shown to the user.
>> 
>> I've made pull request for this change:
>> 
>> https://github.com/phpmyadmin/phpmyadmin/pull/257
>> 
>> Please review and eventually merge that.
>
>Looks great, merged.


Adam


More information about the Developers mailing list