[Phpmyadmin-devel] AJAXify phpMyAdmin Interface - GSOC 2011 - Add a new user in db-> privileges

Thilanka Kaushalya lgtkaushalya at gmail.com
Sun Jun 5 04:29:37 CEST 2011


Hi Marc,


Works fine now. Are you done with this feature? Before I put it in
> production, please polish your code in js/server_privileges.js:
>
> - remove tab characters (we use indents of four spaces)
>
> I'm using 4 space indentation . I checked the code but I cannot find where
 have added tab characters.

- use spaces around operators
>  "url.length-23" should be "url.length - 23"
>
>
Okay I fixed that.


> - watch proper code indentations
>
> - use more spaces around keywords
>  "if(" should be "if ("
>  "}else{" should be "} else {"
>
>
Okay I fixed that.

I fixed the above issues and pushed to the repo. Please check that.

Regards,
Thilanka.


-- 
L G Thilanka Kaushalya
Computer Science & Engineering,
University of Moratuwa,
Sri Lanka

Connect with me at,
      Gmail        : lgtkaushalya at gmail.com
      Twitter       : @thilanka_k
      Facebook  : Thilanka Kaushalya
      Yahoo       : lgtkaushalya at yahoo.com
      Blog          : http://coders-view.blogspot.com/
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.phpmyadmin.net/pipermail/developers/attachments/20110605/807136b0/attachment.html>


More information about the Developers mailing list