diff --git a/src/Squidex/Areas/Api/Controllers/Apps/AppPatternsController.cs b/src/Squidex/Areas/Api/Controllers/Apps/AppPatternsController.cs index 74f9fc136..6ea790fab 100644 --- a/src/Squidex/Areas/Api/Controllers/Apps/AppPatternsController.cs +++ b/src/Squidex/Areas/Api/Controllers/Apps/AppPatternsController.cs @@ -97,7 +97,7 @@ namespace Squidex.Areas.Api.Controllers.Apps [ProducesResponseType(typeof(PatternsDto), 200)] [ApiPermission(Permissions.AppPatternsUpdate)] [ApiCosts(1)] - public async Task UpdatePattern(string app, Guid id, [FromBody] UpdatePatternDto request) + public async Task PutPattern(string app, Guid id, [FromBody] UpdatePatternDto request) { var command = request.ToUpdateCommand(id); diff --git a/src/Squidex/Areas/Api/Controllers/Apps/AppRolesController.cs b/src/Squidex/Areas/Api/Controllers/Apps/AppRolesController.cs index ac427a567..325c4c3e8 100644 --- a/src/Squidex/Areas/Api/Controllers/Apps/AppRolesController.cs +++ b/src/Squidex/Areas/Api/Controllers/Apps/AppRolesController.cs @@ -122,7 +122,7 @@ namespace Squidex.Areas.Api.Controllers.Apps [ProducesResponseType(typeof(RolesDto), 200)] [ApiPermission(Permissions.AppRolesUpdate)] [ApiCosts(1)] - public async Task UpdateRole(string app, string name, [FromBody] UpdateRoleDto request) + public async Task PutRole(string app, string name, [FromBody] UpdateRoleDto request) { var command = request.ToCommand(name); diff --git a/src/Squidex/Areas/Api/Controllers/Apps/Models/PatternDto.cs b/src/Squidex/Areas/Api/Controllers/Apps/Models/PatternDto.cs index 1377aee3e..678bca542 100644 --- a/src/Squidex/Areas/Api/Controllers/Apps/Models/PatternDto.cs +++ b/src/Squidex/Areas/Api/Controllers/Apps/Models/PatternDto.cs @@ -51,7 +51,7 @@ namespace Squidex.Areas.Api.Controllers.Apps.Models if (controller.HasPermission(Permissions.AppPatternsUpdate, app)) { - AddPutLink("update", controller.Url(x => nameof(x.UpdatePattern), values)); + AddPutLink("update", controller.Url(x => nameof(x.PutPattern), values)); } if (controller.HasPermission(Permissions.AppPatternsDelete, app)) diff --git a/src/Squidex/Areas/Api/Controllers/Apps/Models/RoleDto.cs b/src/Squidex/Areas/Api/Controllers/Apps/Models/RoleDto.cs index f703c3cf5..ff0ddf201 100644 --- a/src/Squidex/Areas/Api/Controllers/Apps/Models/RoleDto.cs +++ b/src/Squidex/Areas/Api/Controllers/Apps/Models/RoleDto.cs @@ -68,7 +68,7 @@ namespace Squidex.Areas.Api.Controllers.Apps.Models { if (controller.HasPermission(AllPermissions.AppRolesUpdate, app) && NumClients == 0 && NumContributors == 0) { - AddPutLink("update", controller.Url(x => nameof(x.UpdateRole), values)); + AddPutLink("update", controller.Url(x => nameof(x.PutRole), values)); } if (controller.HasPermission(AllPermissions.AppRolesDelete, app)) diff --git a/src/Squidex/Areas/Api/Controllers/Backups/RestoreController.cs b/src/Squidex/Areas/Api/Controllers/Backups/RestoreController.cs index 8a5a5c4d1..f648c7050 100644 --- a/src/Squidex/Areas/Api/Controllers/Backups/RestoreController.cs +++ b/src/Squidex/Areas/Api/Controllers/Backups/RestoreController.cs @@ -42,7 +42,7 @@ namespace Squidex.Areas.Api.Controllers.Backups [Route("apps/restore/")] [ProducesResponseType(typeof(RestoreJobDto), 200)] [ApiPermission(Permissions.AdminRestore)] - public async Task GetJob() + public async Task GetRestoreJob() { var restoreGrain = grainFactory.GetGrain(SingleGrain.Id); @@ -68,7 +68,7 @@ namespace Squidex.Areas.Api.Controllers.Backups [HttpPost] [Route("apps/restore/")] [ApiPermission(Permissions.AdminRestore)] - public async Task PostRestore([FromBody] RestoreRequestDto request) + public async Task PostRestoreJob([FromBody] RestoreRequestDto request) { var restoreGrain = grainFactory.GetGrain(SingleGrain.Id); diff --git a/src/Squidex/Areas/Api/Controllers/Ping/PingController.cs b/src/Squidex/Areas/Api/Controllers/Ping/PingController.cs index 388026dc4..fd6c294b6 100644 --- a/src/Squidex/Areas/Api/Controllers/Ping/PingController.cs +++ b/src/Squidex/Areas/Api/Controllers/Ping/PingController.cs @@ -35,7 +35,7 @@ namespace Squidex.Areas.Api.Controllers.Ping [HttpGet] [ProducesResponseType(typeof(ExposedValues), 200)] [Route("info/")] - public IActionResult Info() + public IActionResult GetInfo() { return Ok(exposedValues); } diff --git a/src/Squidex/Areas/Api/Controllers/Users/Models/ResourcesDto.cs b/src/Squidex/Areas/Api/Controllers/Users/Models/ResourcesDto.cs index 0b5fa32b2..c8eb9e3c3 100644 --- a/src/Squidex/Areas/Api/Controllers/Users/Models/ResourcesDto.cs +++ b/src/Squidex/Areas/Api/Controllers/Users/Models/ResourcesDto.cs @@ -29,7 +29,7 @@ namespace Squidex.Areas.Api.Controllers.Users.Models if (controller.HasPermission(Permissions.AdminRestore)) { - result.AddGetLink("admin/restore", controller.Url(x => nameof(x.GetJob))); + result.AddGetLink("admin/restore", controller.Url(x => nameof(x.GetRestoreJob))); } if (controller.HasPermission(Permissions.AdminUsersRead)) diff --git a/src/Squidex/Areas/IdentityServer/Controllers/Profile/ProfileController.cs b/src/Squidex/Areas/IdentityServer/Controllers/Profile/ProfileController.cs index dbb87c022..21808f9fe 100644 --- a/src/Squidex/Areas/IdentityServer/Controllers/Profile/ProfileController.cs +++ b/src/Squidex/Areas/IdentityServer/Controllers/Profile/ProfileController.cs @@ -132,19 +132,21 @@ namespace Squidex.Areas.IdentityServer.Controllers.Profile return IdentityResult.Failed(new IdentityError { Description = "Please upload a single file." }); } - var thumbnailStream = new MemoryStream(); - try + using (var thumbnailStream = new MemoryStream()) { - await assetThumbnailGenerator.CreateThumbnailAsync(file[0].OpenReadStream(), thumbnailStream, 128, 128, "Crop"); + try + { + await assetThumbnailGenerator.CreateThumbnailAsync(file[0].OpenReadStream(), thumbnailStream, 128, 128, "Crop"); - thumbnailStream.Position = 0; - } - catch - { - return IdentityResult.Failed(new IdentityError { Description = "Picture is not a valid image." }); - } + thumbnailStream.Position = 0; + } + catch + { + return IdentityResult.Failed(new IdentityError { Description = "Picture is not a valid image." }); + } - await userPictureStore.UploadAsync(user.Id, thumbnailStream); + await userPictureStore.UploadAsync(user.Id, thumbnailStream); + } return await userManager.UpdateSafeAsync(user.Identity, new UserValues { PictureUrl = SquidexClaimTypes.PictureUrlStore }); } diff --git a/src/Squidex/app/features/administration/pages/users/user-page.component.ts b/src/Squidex/app/features/administration/pages/users/user-page.component.ts index d2c2c1583..c930eb1ee 100644 --- a/src/Squidex/app/features/administration/pages/users/user-page.component.ts +++ b/src/Squidex/app/features/administration/pages/users/user-page.component.ts @@ -63,8 +63,8 @@ export class UserPageComponent extends ResourceOwner implements OnInit { if (value) { if (this.user) { this.usersState.update(this.user, value) - .subscribe(() => { - this.userForm.submitCompleted(); + .subscribe(user => { + this.userForm.submitCompleted({ newValue: user }); }, error => { this.userForm.submitFailed(error); });