Fixed regression in repository url panel layout

James Moger

2013-05-15 f6b9cd98683f19489b10ea6153d2585cb306e88e

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

@@ -35,18 +35,14 @@
import org.apache.wicket.MarkupContainer;

import org.apache.wicket.PageParameters;

import org.apache.wicket.RedirectToUrlException;

import org.apache.wicket.RequestCycle;

import org.apache.wicket.RestartResponseException;

import org.apache.wicket.markup.html.CSSPackageResource;

import org.apache.wicket.markup.html.WebPage;

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

import org.apache.wicket.markup.html.link.BookmarkablePageLink;

import org.apache.wicket.markup.html.link.ExternalLink;

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

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

import org.apache.wicket.protocol.http.RequestUtils;

import org.apache.wicket.protocol.http.WebRequest;

import org.apache.wicket.protocol.http.WebResponse;

import org.apache.wicket.protocol.http.servlet.ServletWebRequest;

import org.slf4j.Logger;

import org.slf4j.LoggerFactory;

@@ -59,7 +55,6 @@
import com.gitblit.GitBlit;

import com.gitblit.Keys;

import com.gitblit.models.ProjectModel;

import com.gitblit.models.RepositoryModel;

import com.gitblit.models.TeamModel;

import com.gitblit.models.UserModel;

import com.gitblit.utils.StringUtils;

@@ -68,7 +63,7 @@
import com.gitblit.wicket.WicketUtils;

import com.gitblit.wicket.panels.LinkPanel;



public abstract class BasePage extends WebPage {

public abstract class BasePage extends SessionPage {



   private final Logger logger;

   

@@ -78,14 +73,12 @@
      super();

      logger = LoggerFactory.getLogger(getClass());

      customizeHeader();

      login();

   }



   public BasePage(PageParameters params) {

      super(params);

      logger = LoggerFactory.getLogger(getClass());

      customizeHeader();

      login();

   }

   

   private void customizeHeader() {

@@ -133,39 +126,11 @@
      super.onAfterRender();

   }   



   private void login() {

      GitBlitWebSession session = GitBlitWebSession.get();

      if (session.isLoggedIn() && !session.isSessionInvalidated()) {

         // already have a session, refresh usermodel to pick up

         // any changes to permissions or roles (issue-186)

         UserModel user = GitBlit.self().getUserModel(session.getUser().username);

         session.setUser(user);

         return;

      }

		
      // try to authenticate by servlet request

      HttpServletRequest httpRequest = ((WebRequest) getRequestCycle().getRequest()).getHttpServletRequest();

      UserModel user = GitBlit.self().authenticate(httpRequest);



      // Login the user

      if (user != null) {

         // issue 62: fix session fixation vulnerability

         session.replaceSession();

         session.setUser(user);



         // Set Cookie

         WebResponse response = (WebResponse) getRequestCycle().getResponse();

         GitBlit.self().setCookie(response, user);

			
         session.continueRequest();

      }

   }



   protected void setupPage(String repositoryName, String pageName) {

      if (repositoryName != null && repositoryName.trim().length() > 0) {

         add(new Label("title", getServerName() + " - " + repositoryName));

         add(new Label("title", repositoryName + " - " + Keys.web.siteName));

      } else {

         add(new Label("title", getServerName()));

         add(new Label("title", Keys.web.siteName));

      }



      ExternalLink rootLink = new ExternalLink("rootLink", urlFor(RepositoriesPage.class, null).toString());

@@ -286,21 +251,6 @@
      ServletWebRequest servletWebRequest = (ServletWebRequest) getRequest();

      HttpServletRequest req = servletWebRequest.getHttpServletRequest();

      return req.getServerName();

   }

	
   public static String getRepositoryUrl(RepositoryModel repository) {

      StringBuilder sb = new StringBuilder();

      sb.append(WicketUtils.getGitblitURL(RequestCycle.get().getRequest()));

      sb.append(Constants.GIT_PATH);

      sb.append(repository.name);

		
      // inject username into repository url if authentication is required

      if (repository.accessRestriction.exceeds(AccessRestrictionType.NONE)

            && GitBlitWebSession.get().isLoggedIn()) {

         String username = GitBlitWebSession.get().getUsername();

         sb.insert(sb.indexOf("://") + 3, username + "@");

      }

      return sb.toString();

   }

   

   protected List<ProjectModel> getProjectModels() {