Fixed (again) empty repository check (issue 13)

James Moger

2011-07-22 b1dba764c201f4708b82767b2d91edb6e189ce6f

 src/com/gitblit/wicket/pages/EditRepositoryPage.java

@@ -1,16 +1,31 @@
/*

 * Copyright 2011 gitblit.com.

 *

 * Licensed under the Apache License, Version 2.0 (the "License");

 * you may not use this file except in compliance with the License.

 * You may obtain a copy of the License at

 *

 *     http://www.apache.org/licenses/LICENSE-2.0

 *

 * Unless required by applicable law or agreed to in writing, software

 * distributed under the License is distributed on an "AS IS" BASIS,

 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.

 * See the License for the specific language governing permissions and

 * limitations under the License.

 */

package com.gitblit.wicket.pages;



import java.text.MessageFormat;

import java.util.ArrayList;

import java.util.Arrays;

import java.util.Collections;

import java.util.Date;

import java.util.Iterator;

import java.util.List;

import java.util.Map;



import org.apache.wicket.PageParameters;

import org.apache.wicket.extensions.markup.html.form.palette.Palette;

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

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

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

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

@@ -25,22 +40,23 @@
import com.gitblit.GitBlit;

import com.gitblit.GitBlitException;

import com.gitblit.Keys;

import com.gitblit.models.RepositoryModel;

import com.gitblit.models.UserModel;

import com.gitblit.utils.StringUtils;

import com.gitblit.wicket.BasePage;

import com.gitblit.wicket.GitBlitWebSession;

import com.gitblit.wicket.WicketUtils;

import com.gitblit.wicket.models.RepositoryModel;

import com.gitblit.wicket.models.UserModel;



public class EditRepositoryPage extends BasePage {



   private final boolean isCreate;



   private boolean isAdmin;



   public EditRepositoryPage() {

      // create constructor

      super();

      isCreate = true;

      setupPage(new RepositoryModel("", "", "", new Date()));

      setupPage(new RepositoryModel());

   }



   public EditRepositoryPage(PageParameters params) {

@@ -55,7 +71,7 @@
   protected void setupPage(final RepositoryModel repositoryModel) {

      // ensure this user can create or edit this repository

      checkPermissions(repositoryModel);

		


      List<String> repositoryUsers = new ArrayList<String>();

      if (isCreate) {

         super.setupPage("", getString("gb.newRepository"));

@@ -67,8 +83,12 @@
         }

      }



      final Palette<String> usersPalette = new Palette<String>("users", new ListModel<String>(repositoryUsers), new CollectionModel<String>(GitBlit.self().getAllUsernames()), new ChoiceRenderer<String>("", ""), 10, false);

      CompoundPropertyModel<RepositoryModel> model = new CompoundPropertyModel<RepositoryModel>(repositoryModel);

      final String oldName = repositoryModel.name;

      final Palette<String> usersPalette = new Palette<String>("users", new ListModel<String>(

            repositoryUsers), new CollectionModel<String>(GitBlit.self().getAllUsernames()),

            new ChoiceRenderer<String>("", ""), 10, false);

      CompoundPropertyModel<RepositoryModel> model = new CompoundPropertyModel<RepositoryModel>(

            repositoryModel);

      Form<RepositoryModel> form = new Form<RepositoryModel>("editForm", model) {



         private static final long serialVersionUID = 1L;

@@ -84,6 +104,22 @@


               // automatically convert backslashes to forward slashes

               repositoryModel.name = repositoryModel.name.replace('\\', '/');

               // Automatically replace // with /

               repositoryModel.name = repositoryModel.name.replace("//", "/");



               // prohibit folder paths

               if (repositoryModel.name.startsWith("/")) {

                  error("Leading root folder references (/) are prohibited.");

                  return;

               }

               if (repositoryModel.name.startsWith("../")) {

                  error("Relative folder references (../) are prohibited.");

                  return;

               }

               if (repositoryModel.name.contains("/../")) {

                  error("Relative folder references (../) are prohibited.");

                  return;

               }



               // confirm valid characters in repository name

               char[] validChars = { '/', '.', '_', '-' };

@@ -94,7 +130,8 @@
                        ok |= c == vc;

                     }

                     if (!ok) {

                        error(MessageFormat.format("Illegal character '{0}' in repository name!", c));

                        error(MessageFormat.format(

                              "Illegal character ''{0}'' in repository name!", c));

                        return;

                     }

                  }

@@ -107,7 +144,7 @@
               }



               // save the repository

               GitBlit.self().editRepositoryModel(repositoryModel, isCreate);

               GitBlit.self().updateRepositoryModel(oldName, repositoryModel, isCreate);



               // save the repository access list

               if (repositoryModel.accessRestriction.exceeds(AccessRestrictionType.NONE)) {

@@ -117,7 +154,8 @@
                     repositoryUsers.add(users.next());

                  }

                  // ensure the owner is added to the user list

                  if (!repositoryUsers.contains(repositoryModel.owner)) {

                  if (repositoryModel.owner != null

                        && !repositoryUsers.contains(repositoryModel.owner)) {

                     repositoryUsers.add(repositoryModel.owner);

                  }

                  GitBlit.self().setRepositoryUsers(repositoryModel, repositoryUsers);

@@ -132,19 +170,34 @@
      };



      // field names reflective match RepositoryModel fields

      form.add(new TextField<String>("name").setEnabled(isCreate));

      form.add(new TextField<String>("name").setEnabled(isCreate || isAdmin));

      form.add(new TextField<String>("description"));

      form.add(new DropDownChoice<String>("owner", GitBlit.self().getAllUsernames()).setEnabled(GitBlitWebSession.get().canAdmin()));

      form.add(new DropDownChoice<AccessRestrictionType>("accessRestriction", Arrays.asList(AccessRestrictionType.values()), new AccessRestrictionRenderer()));

      form.add(new DropDownChoice<String>("owner", GitBlit.self().getAllUsernames())

            .setEnabled(GitBlitWebSession.get().canAdmin()));

      form.add(new DropDownChoice<AccessRestrictionType>("accessRestriction", Arrays

            .asList(AccessRestrictionType.values()), new AccessRestrictionRenderer()));

      form.add(new CheckBox("isFrozen"));

      form.add(new CheckBox("useTickets"));

      form.add(new CheckBox("useDocs"));

      form.add(new CheckBox("showRemoteBranches"));

      form.add(new CheckBox("showReadme"));

      form.add(usersPalette);



      form.add(new Button("save"));

      Button cancel = new Button("cancel") {

         private static final long serialVersionUID = 1L;



         @Override

         public void onSubmit() {

            setResponsePage(RepositoriesPage.class);

         }

      };

      cancel.setDefaultFormProcessing(false);

      form.add(cancel);



      add(form);

   }

	


   /**

    * Unfortunately must repeat part of AuthorizaitonStrategy here because that

    * mechanism does not take PageParameters into consideration, only page

@@ -153,8 +206,8 @@
    * Repository Owners should be able to edit their repository.

    */

   private void checkPermissions(RepositoryModel model) {

      boolean authenticateAdmin = GitBlit.self().settings().getBoolean(Keys.web.authenticateAdminPages, true);

      boolean allowAdmin = GitBlit.self().settings().getBoolean(Keys.web.allowAdministration, true);

      boolean authenticateAdmin = GitBlit.getBoolean(Keys.web.authenticateAdminPages, true);

      boolean allowAdmin = GitBlit.getBoolean(Keys.web.allowAdministration, true);



      GitBlitWebSession session = GitBlitWebSession.get();

      UserModel user = session.getUser();

@@ -167,21 +220,22 @@
            }

            if (isCreate) {

               // Create Repository

               if (!user.canAdmin()) {

               if (!user.canAdmin) {

                  // Only Administrators May Create

                  error("Only an administrator may create a repository", true);

               }

            } else {

               // Edit Repository

               if (user.canAdmin()) {

               if (user.canAdmin) {

                  // Admins can edit everything

                  isAdmin = true;

                  return;

               } else {

                  if (!model.owner.equalsIgnoreCase(user.getUsername())) {

                  if (!model.owner.equalsIgnoreCase(user.username)) {

                     // User is not an Admin nor Owner

                     error("Only an administrator or the owner may edit a repository", true);

                  }

               }					
               }

            }

         }

      } else {