Merged #173 "Fix MigrateTickets failure for view-restricted repositories"

James Moger

2014-09-08 0601663aec4ffd24e24abc97cf360529de5c716c

Merged #173 "Fix MigrateTickets failure for view-restricted repositories"

 src/main/java/com/gitblit/MigrateTickets.java

@@ -170,7 +170,7 @@
        long start = System.nanoTime();
        long totalTickets = 0;
        long totalChanges = 0;
        for (RepositoryModel repository : repositoryManager.getRepositoryModels(null)) {
        for (RepositoryModel repository : repositoryManager.getRepositoryModels()) {
            Set<Long> ids = inputService.getIds(repository);
            if (ids == null || ids.isEmpty()) {
                // nothing to migrate

 src/main/java/com/gitblit/manager/GitblitManager.java

@@ -79,11 +79,11 @@
import com.gitblit.transport.ssh.IPublicKeyManager;
import com.gitblit.transport.ssh.SshKey;
import com.gitblit.utils.ArrayUtils;
import com.gitblit.utils.XssFilter;
import com.gitblit.utils.HttpUtils;
import com.gitblit.utils.JsonUtils;
import com.gitblit.utils.ObjectCache;
import com.gitblit.utils.StringUtils;
import com.gitblit.utils.XssFilter;
import com.google.gson.Gson;
import com.google.gson.JsonIOException;
import com.google.gson.JsonSyntaxException;
@@ -982,6 +982,11 @@
    }

    @Override
    public List<RepositoryModel> getRepositoryModels() {
        return repositoryManager.getRepositoryModels();
    }

    @Override
    public List<RepositoryModel> getRepositoryModels(UserModel user) {
        return repositoryManager.getRepositoryModels(user);
    }

 src/main/java/com/gitblit/manager/IRepositoryManager.java

@@ -195,6 +195,14 @@
    Repository getRepository(String repositoryName, boolean logError);

    /**
     * Returns the list of all repository models.
     *
     * @return list of all repository models
      * @since 1.6.1
     */
    List<RepositoryModel> getRepositoryModels();

    /**
     * Returns the list of repository models that are accessible to the user.
     *
     * @param user

 src/main/java/com/gitblit/manager/RepositoryManager.java

@@ -625,6 +625,27 @@
    }

    /**
     * Returns the list of all repository models.
     *
     * @return list of all repository models
     */
    @Override
    public List<RepositoryModel> getRepositoryModels() {
        long methodStart = System.currentTimeMillis();
        List<String> list = getRepositoryList();
        List<RepositoryModel> repositories = new ArrayList<RepositoryModel>();
        for (String repo : list) {
            RepositoryModel model = getRepositoryModel(repo);
            if (model != null) {
                repositories.add(model);
            }
        }
        long duration = System.currentTimeMillis() - methodStart;
        logger.info(MessageFormat.format("{0} repository models loaded in {1} msecs", duration));
        return repositories;
    }

    /**
     * Returns the list of repository models that are accessible to the user.
     *
     * @param user