Updating DB withouth actually calling update #3019

7 Answer(s)

• 

  0

  ismcagdas created 7 years ago

  Support Team

  Hi,

  Actually there is no need to call update method in a unit of work.

  Save Cancel
• 

  0

  bilalhaidar created 7 years ago

  Thanks. But existing code of ABP Framework does it in more than places, so I got confused.

  Save Cancel
• 

  0

  ismcagdas created 7 years ago

  Support Team

  Hi,

  Can you share an example, I might be wrong about it.

  Thanks.

  Save Cancel
• 

  0

  bilalhaidar created 7 years ago

  Here you go:

  [AbpAuthorize(AppPermissions.Pages_Administration_Users_Edit)]
          protected virtual async Task UpdateUserAsync(CreateOrUpdateUserInput input)
          {
              Debug.Assert(input.User.Id != null, "input.User.Id should be set.");
  
              var user = await UserManager.FindByIdAsync(input.User.Id.Value);
  
              //Update user properties
              input.User.MapTo(user); //Passwords is not mapped (see mapping configuration)
  
              if (input.SetRandomPassword)
              {
                  input.User.Password = User.CreateRandomPassword();
              }
  
              if (!input.User.Password.IsNullOrEmpty())
              {
                  CheckErrors(await UserManager.ChangePasswordAsync(user, input.User.Password));
              }
  
              CheckErrors(await UserManager.UpdateAsync(user));
  
              //Update roles
              CheckErrors(await UserManager.SetRoles(user, input.AssignedRoleNames));
  
              if (input.SendActivationEmail)
              {
                  user.SetNewEmailConfirmationCode();
                  await _userEmailer.SendEmailActivationLinkAsync(user, input.User.Password);
              }
          }
  

  Save Cancel
• 

  0

  ismcagdas created 7 years ago

  Support Team

  Hi,

  In this case, it is used to check if there are errors on updating user. And the update method belongs to UserManager class which is not a repository.

  Thanks.

  Save Cancel
• 

  0

  bilalhaidar created 7 years ago

  I understand, so if there is no need to check for errors, you simply don't cal l any update method?

  Thanks

  Save Cancel
• 

  0

  alirizaadiyahsi created 7 years ago

  Yes, you are right.

  Save Cancel