app-inventor-gallery / aig

App Inventor Community Gallery
10 stars 8 forks source link

Users should not be allowed to change their uname to "guest" #116

Open fgmart-zz opened 12 years ago

fgmart-zz commented 12 years ago

Related to #115.

derrell commented 12 years ago

There are probably a few other special names that should be excluded. Off the top of my head, I can think of "admin" and "administrator" and maybe "superuser". There are probably a few others.

derrell commented 12 years ago

@mijoda to you. This should be simple. There is already a function that validates display names, so you just need to add a bit more sophistication to it. I suggest creating a static member which is an array of invalid display names, and ensuring that none of those (case-insensitive) is selected. (qx.lang.Array.contains(haystackArray, needleString) will be your friend.)

cpuwhiz11 commented 11 years ago

Fix for this in https://github.com/karuto/aig/pull/8.