Add clone transport user preference

James Moger

2014-06-05 1b04d7730f56a08c935974504b2d849d01870c02

 src/main/java/com/gitblit/ConfigUserService.java

@@ -36,6 +36,7 @@


import com.gitblit.Constants.AccessPermission;

import com.gitblit.Constants.AccountType;

import com.gitblit.Constants.Transport;

import com.gitblit.manager.IRuntimeManager;

import com.gitblit.models.TeamModel;

import com.gitblit.models.UserModel;

@@ -96,7 +97,13 @@


   private static final String LOCALE = "locale";



   private static final String EMAILONMYTICKETCHANGES = "emailMeOnMyTicketChanges";



   private static final String TRANSPORT = "transport";



   private static final String ACCOUNTTYPE = "accountType";



   private static final String DISABLED = "disabled";



   private final File realmFile;



@@ -701,9 +708,19 @@
         if (!StringUtils.isEmpty(model.countryCode)) {

            config.setString(USER, model.username, COUNTRYCODE, model.countryCode);

         }

         if (model.disabled) {

            config.setBoolean(USER, model.username, DISABLED, true);

         }

         if (model.getPreferences() != null) {

            if (!StringUtils.isEmpty(model.getPreferences().locale)) {

               config.setString(USER, model.username, LOCALE, model.getPreferences().locale);

            if (model.getPreferences().getLocale() != null) {

               String val = model.getPreferences().getLocale().getLanguage() + "_" + model.getPreferences().getLocale().getCountry();

               config.setString(USER, model.username, LOCALE, val);

            }



            config.setBoolean(USER, model.username, EMAILONMYTICKETCHANGES, model.getPreferences().isEmailMeOnMyTicketChanges());



            if (model.getPreferences().getTransport() != null) {

               config.setString(USER, model.username, TRANSPORT, model.getPreferences().getTransport().name());

            }

         }



@@ -868,17 +885,22 @@
               if (Constants.EXTERNAL_ACCOUNT.equals(user.password) && user.accountType.isLocal()) {

                  user.accountType = AccountType.EXTERNAL;

               }

               user.disabled = config.getBoolean(USER, username, DISABLED, false);

               user.organizationalUnit = config.getString(USER, username, ORGANIZATIONALUNIT);

               user.organization = config.getString(USER, username, ORGANIZATION);

               user.locality = config.getString(USER, username, LOCALITY);

               user.stateProvince = config.getString(USER, username, STATEPROVINCE);

               user.countryCode = config.getString(USER, username, COUNTRYCODE);

               user.cookie = config.getString(USER, username, COOKIE);

               user.getPreferences().locale = config.getString(USER, username, LOCALE);

               if (StringUtils.isEmpty(user.cookie) && !StringUtils.isEmpty(user.password)) {

                  user.cookie = StringUtils.getSHA1(user.username + user.password);

               }



               // preferences

               user.getPreferences().setLocale(config.getString(USER, username, LOCALE));

               user.getPreferences().setEmailMeOnMyTicketChanges(config.getBoolean(USER, username, EMAILONMYTICKETCHANGES, true));

               user.getPreferences().setTransport(Transport.fromString(config.getString(USER, username, TRANSPORT)));



               // user roles

               Set<String> roles = new HashSet<String>(Arrays.asList(config.getStringList(

                     USER, username, ROLE)));