Overhaul EditRepositoryPage for layout and usability

James Moger

2014-06-03 85b5d72949ead641ba697543324ff5d236e23fd1

 src/main/java/com/gitblit/wicket/panels/AccessPolicyPanel.java

@@ -21,10 +21,13 @@


import org.apache.wicket.ajax.form.AjaxFormChoiceComponentUpdatingBehavior;

import org.apache.wicket.markup.html.basic.Label;

import org.apache.wicket.markup.html.form.CheckBox;

import org.apache.wicket.markup.html.form.Radio;

import org.apache.wicket.markup.html.form.RadioGroup;

import org.apache.wicket.markup.html.list.ListItem;

import org.apache.wicket.markup.html.list.ListView;

import org.apache.wicket.markup.html.panel.Fragment;

import org.apache.wicket.model.IModel;

import org.apache.wicket.model.Model;



import com.gitblit.Constants.AccessRestrictionType;

@@ -48,6 +51,8 @@
   private final AjaxFormChoiceComponentUpdatingBehavior callback;



   private RadioGroup<AccessPolicy> policiesGroup;



   private IModel<Boolean> allowForks;



   public AccessPolicyPanel(String wicketId, RepositoryModel repository) {

      this(wicketId, repository, null);

@@ -142,13 +147,28 @@
      }

      add(policiesGroup);



      allowForks = Model.of(true);

      add(newCheckbox("allowForks",

            getString("gb.allowForks"),

            getString("gb.allowForksDescription"),

            allowForks).setEnabled(app().settings().getBoolean(Keys.web.allowForking, true)));



      setOutputMarkupId(true);

   }



   private Fragment newCheckbox(String wicketId, String title, String description, IModel<Boolean> model) {

      Fragment fragment = new Fragment(wicketId, "checkboxOption", this);

      fragment.add(new Label("name", title));

      fragment.add(new Label("description", description));

      fragment.add(new CheckBox("checkbox", model));

      return fragment;

   }



   public void updateModel(RepositoryModel repository) {

      AccessPolicy policy = policiesGroup.getModelObject();

      repository.authorizationControl = policy.control;

      repository.accessRestriction = policy.type;

      repository.allowForks = allowForks.getObject();

   }



   @Override