Embrace @Inject for Managers, Servlets, and Filters

James Moger

2014-05-16 1b34b05f5b8d2d0fd51819b286e22d7ddd27cb27

 src/main/java/com/gitblit/GitBlit.java

@@ -20,9 +20,11 @@
import java.util.Collections;
import java.util.Comparator;
import java.util.HashSet;
import java.util.Iterator;
import java.util.List;
import java.util.Set;

import javax.inject.Inject;
import javax.inject.Singleton;
import javax.servlet.http.HttpServletRequest;

@@ -69,6 +71,7 @@

   private ITicketService ticketService;

   @Inject
   public GitBlit(
         IRuntimeManager runtimeManager,
         IPluginManager pluginManager,
@@ -115,6 +118,21 @@
   @Override
   public boolean isServingRepositories() {
      return servicesManager.isServingRepositories();
   }

   @Override
   public boolean isServingHTTP() {
      return servicesManager.isServingHTTP();
   }

   @Override
   public boolean isServingGIT() {
      return servicesManager.isServingGIT();
   }

   @Override
   public boolean isServingSSH() {
      return servicesManager.isServingSSH();
   }

   protected Object [] getModules() {
@@ -249,6 +267,24 @@
         }
      });

      // consider the user's transport preference
      RepositoryUrl preferredUrl = null;
      Transport preferredTransport = user.getPreferences().getTransport();
      if (preferredTransport != null) {
         Iterator<RepositoryUrl> itr = list.iterator();
         while (itr.hasNext()) {
            RepositoryUrl url = itr.next();
            if (url.transport.equals(preferredTransport)) {
               itr.remove();
               preferredUrl = url;
               break;
            }
         }
      }
      if (preferredUrl != null) {
         list.add(0, preferredUrl);
      }

      return list;
   }