Clean up VLang by using "value" on language select

r2/r2/controllers/validator/validator.py

@@ -223,11 +223,8 @@ class nop(Validator):
 
 class VLang(Validator):
     def run(self, lang):
-        if lang:
-            lang = str(lang.split('[')[1].strip(']'))
-            if lang in g.all_languages:
-                return lang
-        #else
+        if lang in g.all_languages:
+            return lang
         return g.lang
 
 class VRequired(Validator):

r2/r2/templates/utils.html

@@ -201,7 +201,7 @@ ${unsafe(txt)}
   </option>
   %endif
   %for x in langs:
-  <option ${x == default_lang  and "selected='selected'" or ""}>
+  <option ${x == default_lang  and "selected='selected'" or ""} value="${x}">
     ${g.lang_name[x]} [${x}]
   </option>
   %endfor