Fixed sendemail script to list all commits in a push.

James Moger

2011-12-17 f4759059d4c363f5bd42550c9c35dd21f4d2bf41

 src/com/gitblit/GitBlit.java

@@ -69,6 +69,7 @@
import com.gitblit.models.ServerSettings;

import com.gitblit.models.ServerStatus;

import com.gitblit.models.SettingModel;

import com.gitblit.models.TeamModel;

import com.gitblit.models.UserModel;

import com.gitblit.utils.ByteFormat;

import com.gitblit.utils.FederationUtils;

@@ -511,6 +512,85 @@
   }



   /**

    * Returns the list of available teams that a user or repository may be

    * assigned to.

    * 
    * @return the list of teams

    */

   public List<String> getAllTeamnames() {

      List<String> teams = new ArrayList<String>(userService.getAllTeamNames());

      Collections.sort(teams);

      return teams;

   }



   /**

    * Returns the TeamModel object for the specified name.

    * 
    * @param teamname

    * @return a TeamModel object or null

    */

   public TeamModel getTeamModel(String teamname) {

      return userService.getTeamModel(teamname);

   }



   /**

    * Returns the list of all teams who are allowed to bypass the access

    * restriction placed on the specified repository.

    * 
    * @see IUserService.getTeamnamesForRepositoryRole(String)

    * @param repository

    * @return list of all teamnames that can bypass the access restriction

    */

   public List<String> getRepositoryTeams(RepositoryModel repository) {

      return userService.getTeamnamesForRepositoryRole(repository.name);

   }



   /**

    * Sets the list of all uses who are allowed to bypass the access

    * restriction placed on the specified repository.

    * 
    * @see IUserService.setTeamnamesForRepositoryRole(String, List<String>)

    * @param repository

    * @param teamnames

    * @return true if successful

    */

   public boolean setRepositoryTeams(RepositoryModel repository, List<String> repositoryTeams) {

      return userService.setTeamnamesForRepositoryRole(repository.name, repositoryTeams);

   }



   /**

    * Updates the TeamModel object for the specified name.

    * 
    * @param teamname

    * @param team

    * @param isCreate

    */

   public void updateTeamModel(String teamname, TeamModel team, boolean isCreate)

         throws GitBlitException {

      if (!teamname.equalsIgnoreCase(team.name)) {

         if (userService.getTeamModel(team.name) != null) {

            throw new GitBlitException(MessageFormat.format(

                  "Failed to rename ''{0}'' because ''{1}'' already exists.", teamname,

                  team.name));

         }

      }

      if (!userService.updateTeamModel(teamname, team)) {

         throw new GitBlitException(isCreate ? "Failed to add team!" : "Failed to update team!");

      }

   }



   /**

    * Delete the team object with the specified teamname

    * 
    * @see IUserService.deleteTeam(String)

    * @param teamname

    * @return true if successful

    */

   public boolean deleteTeam(String teamname) {

      return userService.deleteTeam(teamname);

   }



   /**

    * Clears all the cached data for the specified repository.

    * 

    * @param repositoryName

@@ -647,6 +727,10 @@
               "gitblit", null, "federationSets")));

         model.isFederated = getConfig(config, "isFederated", false);

         model.origin = config.getString("remote", "origin", "url");

         model.preReceiveScripts = new ArrayList<String>(Arrays.asList(config.getStringList(

               "gitblit", null, "preReceiveScript")));

         model.postReceiveScripts = new ArrayList<String>(Arrays.asList(config.getStringList(

               "gitblit", null, "postReceiveScript")));

      }

      r.close();

      return model;

@@ -866,6 +950,8 @@
      config.setString("gitblit", null, "federationStrategy",

            repository.federationStrategy.name());

      config.setBoolean("gitblit", null, "isFederated", repository.isFederated);

      config.setStringList("gitblit", null, "preReceiveScript", repository.preReceiveScripts);

      config.setStringList("gitblit", null, "postReceiveScript", repository.postReceiveScripts);

      try {

         config.save();

      } catch (IOException e) {

@@ -1115,6 +1201,7 @@
      case PULL_REPOSITORIES:

         return token.equals(all) || token.equals(unr) || token.equals(jur);

      case PULL_USERS:

      case PULL_TEAMS:

         return token.equals(all) || token.equals(unr);

      case PULL_SETTINGS:

         return token.equals(all);

@@ -1348,6 +1435,37 @@
   }



   /**

    * Notify users by email of something.

    * 
    * @param subject

    * @param message

    * @param toAddresses

    */

   public void notifyUsers(String subject, String message, ArrayList<String> toAddresses) {

      this.notifyUsers(subject, message, toAddresses.toArray(new String[0]));

   }



   /**

    * Notify users by email of something.

    * 
    * @param subject

    * @param message

    * @param toAddresses

    */

   public void notifyUsers(String subject, String message, String... toAddresses) {

      try {

         Message mail = mailExecutor.createMessage(toAddresses);

         if (mail != null) {

            mail.setSubject(subject);

            mail.setText(message);

            mailExecutor.queue(mail);

         }

      } catch (MessagingException e) {

         logger.error("Messaging error", e);

      }

   }



   /**

    * Returns the descriptions/comments of the Gitblit config settings.

    * 

    * @return SettingsModel

@@ -1473,7 +1591,7 @@
                     configService.updateUserModel(userModel);

                  }

               }

					


               // issue suggestion about switching to users.conf

               logger.warn("Please consider using \"users.conf\" instead of the deprecated \"users.properties\" file");

            } else if (realmFile.getName().toLowerCase().endsWith(".conf")) {