Teammates: Use enum instead of string for gender field in StudentProfileAttributes and related classes

Created on 26 Jun 2017  路  8Comments  路  Source: TEAMMATES/teammates

Replace 'gender' field with corresponding enum for StudentProfileAttributes and related classes.
Currently gender field is String that only accepts "male", "female" or "other" values, so It's better to use enum over String value in that case.

a-CodeQuality help wanted p.Low

All 8 comments

@whipermr5 I will do this :)

@shivakrishna9 Are you taking this issue ?? else would you let me do it ..

<%@ tag description="studentProfile - Edit profile div" %>
<%@ taglib uri="http://java.sun.com/jsp/jstl/core" prefix="c" %>
<%@ tag import="teammates.common.util.Const" %>
<%@ attribute name="profile" type="teammates.ui.template.StudentProfileEditBox" required="true" %>
<%@ attribute name="sessionToken" required="true" %>




where is the declaration of var in studentProfileDiv.java? I hope I am checking the correct java file. In this issue, it is said that var is of String datatype but I am not able to figure out where is var declared String?

teammates.ui.controller.StudentProfileEditSaveAction#extractProfileData
55 line

i dont know weather have i done correctly,,, please help

@rajujha373 You can submit a PR for us to take a look :)

@LimeTheCoder @whipermr5, Is this issue still open? If so I will be glad to work on this issue.

@gaurav122197 someone else is currently working on this in #8208, I suggest you pick another issue :).

Was this page helpful?
0 / 5 - 0 ratings