Skip to content

Commit 9f73b67

Browse files
author
Martin Spielmann
committed
support special characters in teams administration. addresses #1135
1 parent acf16a9 commit 9f73b67

File tree

1 file changed

+16
-3
lines changed

1 file changed

+16
-3
lines changed

src/main/java/com/gitblit/wicket/pages/EditUserPage.java

Lines changed: 16 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@
2727
import org.apache.wicket.markup.html.form.Button;
2828
import org.apache.wicket.markup.html.form.CheckBox;
2929
import org.apache.wicket.markup.html.form.Form;
30+
import org.apache.wicket.markup.html.form.IChoiceRenderer;
3031
import org.apache.wicket.markup.html.form.TextField;
3132
import org.apache.wicket.model.CompoundPropertyModel;
3233
import org.apache.wicket.model.Model;
@@ -43,7 +44,6 @@
4344
import com.gitblit.utils.StringUtils;
4445
import com.gitblit.wicket.NonTrimmedPasswordTextField;
4546
import com.gitblit.wicket.RequiresAdminRole;
46-
import com.gitblit.wicket.StringChoiceRenderer;
4747
import com.gitblit.wicket.WicketUtils;
4848
import com.gitblit.wicket.panels.RegistrantPermissionsPanel;
4949

@@ -106,8 +106,21 @@ protected void setupPage(final UserModel userModel) {
106106
final List<RegistrantAccessPermission> permissions = app().repositories().getUserAccessPermissions(userModel);
107107

108108
final Palette<String> teams = new Palette<String>("teams", new ListModel<String>(
109-
new ArrayList<String>(userTeams)), new CollectionModel<String>(app().users()
110-
.getAllTeamNames()), new StringChoiceRenderer(), 10, false);
109+
new ArrayList<String>(userTeams)), new CollectionModel<String>(app().users().getAllTeamNames()),
110+
//use index based IChoiceRenderer to add support for special characters in team names (e.g. R&D)
111+
new IChoiceRenderer<String>() {
112+
private static final long serialVersionUID = 1L;
113+
114+
@Override
115+
public Object getDisplayValue(String object) {
116+
return object;
117+
}
118+
119+
@Override
120+
public String getIdValue(String object, int index) {
121+
return String.valueOf(index);
122+
}
123+
}, 10, false);
111124
Form<UserModel> form = new Form<UserModel>("editForm", model) {
112125

113126
private static final long serialVersionUID = 1L;

0 commit comments

Comments
 (0)