From d980ffe7837f3f9779d43fd46f290fa1c4cafd7a Mon Sep 17 00:00:00 2001 From: Lennard Fonteijn Date: Wed, 23 Aug 2017 23:14:49 +0200 Subject: [PATCH 01/47] Added possibilty to ignore collisions from an IContentFinder --- src/Umbraco.Web/Routing/PublishedContentRequest.cs | 5 ++++- src/Umbraco.Web/Routing/UrlProviderExtensions.cs | 2 +- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/src/Umbraco.Web/Routing/PublishedContentRequest.cs b/src/Umbraco.Web/Routing/PublishedContentRequest.cs index 9c21958603..4a4968e4e3 100644 --- a/src/Umbraco.Web/Routing/PublishedContentRequest.cs +++ b/src/Umbraco.Web/Routing/PublishedContentRequest.cs @@ -614,6 +614,9 @@ namespace Umbraco.Web.Routing set { _headers = value; } } - + /// + /// Gets of sets a value indicating whether the Umbraco Backoffice should ignore a collision for this request. + /// + public bool IgnorePublishedContentCollisions { get; set; } } } \ No newline at end of file diff --git a/src/Umbraco.Web/Routing/UrlProviderExtensions.cs b/src/Umbraco.Web/Routing/UrlProviderExtensions.cs index d464b8962e..accd7018fe 100644 --- a/src/Umbraco.Web/Routing/UrlProviderExtensions.cs +++ b/src/Umbraco.Web/Routing/UrlProviderExtensions.cs @@ -78,7 +78,7 @@ namespace Umbraco.Web.Routing { urls.Add(ui.Text("content", "routeError", "(error)", umbracoContext.Security.CurrentUser)); } - else if (pcr.PublishedContent.Id != content.Id) + else if (pcr.IgnorePublishedContentCollisions == false && pcr.PublishedContent.Id != content.Id) { var o = pcr.PublishedContent; string s; From bf1a90020c8c66d594da1b737713be671b834e66 Mon Sep 17 00:00:00 2001 From: John Churchley Date: Wed, 20 Dec 2017 16:35:24 +0000 Subject: [PATCH 02/47] Allow Members tree to work independently of Member Types tree by giving it permission to access the MemberType.GetAllTypes Method --- src/Umbraco.Web/Editors/MemberTypeController.cs | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/src/Umbraco.Web/Editors/MemberTypeController.cs b/src/Umbraco.Web/Editors/MemberTypeController.cs index 1d8766f253..7e9475bbd7 100644 --- a/src/Umbraco.Web/Editors/MemberTypeController.cs +++ b/src/Umbraco.Web/Editors/MemberTypeController.cs @@ -24,7 +24,7 @@ namespace Umbraco.Web.Editors /// An API controller used for dealing with content types /// [PluginController("UmbracoApi")] - [UmbracoTreeAuthorize(Constants.Trees.MemberTypes)] + [UmbracoTreeAuthorize(new string[] { Constants.Trees.MemberTypes, Constants.Trees.Members})] public class MemberTypeController : ContentTypeControllerBase { /// @@ -48,6 +48,7 @@ namespace Umbraco.Web.Editors private readonly MembershipProvider _provider; + [UmbracoTreeAuthorize(Constants.Trees.MemberTypes)] public MemberTypeDisplay GetById(int id) { var ct = Services.MemberTypeService.Get(id); @@ -67,6 +68,7 @@ namespace Umbraco.Web.Editors /// [HttpDelete] [HttpPost] + [UmbracoTreeAuthorize(Constants.Trees.MemberTypes)] public HttpResponseMessage DeleteById(int id) { var foundType = Services.MemberTypeService.Get(id); @@ -93,6 +95,8 @@ namespace Umbraco.Web.Editors /// be looked up via the db, they need to be passed in. /// /// + + [UmbracoTreeAuthorize(Constants.Trees.MemberTypes)] public HttpResponseMessage GetAvailableCompositeMemberTypes(int contentTypeId, [FromUri]string[] filterContentTypes, [FromUri]string[] filterPropertyTypes) @@ -106,6 +110,7 @@ namespace Umbraco.Web.Editors return Request.CreateResponse(result); } + [UmbracoTreeAuthorize(Constants.Trees.MemberTypes)] public MemberTypeDisplay GetEmpty() { var ct = new MemberType(-1); @@ -129,6 +134,7 @@ namespace Umbraco.Web.Editors return Enumerable.Empty(); } + [UmbracoTreeAuthorize(Constants.Trees.MemberTypes)] public MemberTypeDisplay PostSave(MemberTypeSave contentTypeSave) { var savedCt = PerformPostSave( From c040cfefcdf413a096d643171fba0b4c0be4cb88 Mon Sep 17 00:00:00 2001 From: Rebecca Date: Thu, 22 Feb 2018 13:02:34 +1000 Subject: [PATCH 03/47] Failed Logon Attempts resets to 0 after maximum number of attempts --- .../Security/BackOfficeUserManager.cs | 62 ++++++++++++++----- .../src/views/users/user.controller.js | 2 + .../src/views/users/views/user/details.html | 2 +- 3 files changed, 49 insertions(+), 17 deletions(-) diff --git a/src/Umbraco.Core/Security/BackOfficeUserManager.cs b/src/Umbraco.Core/Security/BackOfficeUserManager.cs index 4969fa5f2f..38861f300c 100644 --- a/src/Umbraco.Core/Security/BackOfficeUserManager.cs +++ b/src/Umbraco.Core/Security/BackOfficeUserManager.cs @@ -57,7 +57,7 @@ namespace Umbraco.Core.Security [Obsolete("Use the overload specifying all dependencies instead")] public static BackOfficeUserManager Create( IdentityFactoryOptions options, - IUserService userService, + IUserService userService, IExternalLoginService externalLoginService, MembershipProviderBase membershipProvider) { @@ -94,7 +94,7 @@ namespace Umbraco.Core.Security manager.InitUserManager(manager, membershipProvider, contentSectionConfig, options); return manager; } - + [EditorBrowsable(EditorBrowsableState.Never)] [Obsolete("Use the overload specifying all dependencies instead")] public static BackOfficeUserManager Create( @@ -129,7 +129,7 @@ namespace Umbraco.Core.Security [Obsolete("Use the overload specifying all dependencies instead")] protected void InitUserManager( BackOfficeUserManager manager, - MembershipProviderBase membershipProvider, + MembershipProviderBase membershipProvider, IdentityFactoryOptions options) { InitUserManager(manager, membershipProvider, UmbracoConfig.For.UmbracoSettings().Content, options); @@ -216,8 +216,8 @@ namespace Umbraco.Core.Security /// /// protected void InitUserManager( - BackOfficeUserManager manager, - MembershipProviderBase membershipProvider, + BackOfficeUserManager manager, + MembershipProviderBase membershipProvider, IDataProtectionProvider dataProtectionProvider, IContentSection contentSectionConfig) { @@ -233,7 +233,7 @@ namespace Umbraco.Core.Security //use a custom hasher based on our membership provider manager.PasswordHasher = GetDefaultPasswordHasher(membershipProvider); - + if (dataProtectionProvider != null) { manager.UserTokenProvider = new DataProtectorTokenProvider(dataProtectionProvider.Create("ASP.NET Identity")); @@ -373,7 +373,7 @@ namespace Umbraco.Core.Security } #region Overrides for password logic - + /// /// Logic used to validate a username and password /// @@ -469,7 +469,7 @@ namespace Umbraco.Core.Security await UpdateSecurityStampInternal(user); return IdentityResult.Success; - + } /// @@ -507,15 +507,22 @@ namespace Umbraco.Core.Security #endregion - public override Task SetLockoutEndDateAsync(int userId, DateTimeOffset lockoutEnd) + public override async Task SetLockoutEndDateAsync(int userId, DateTimeOffset lockoutEnd) { - var result = base.SetLockoutEndDateAsync(userId, lockoutEnd); + var result = await base.SetLockoutEndDateAsync(userId, lockoutEnd); // The way we unlock is by setting the lockoutEnd date to the current datetime - if (result.Result.Succeeded && lockoutEnd >= DateTimeOffset.UtcNow) + if (result.Succeeded && lockoutEnd >= DateTimeOffset.UtcNow) + { RaiseAccountLockedEvent(userId); + } else + { RaiseAccountUnlockedEvent(userId); + //Resets the login attempt fails back to 0 when unlock is clicked + await ResetAccessFailedCountAsync(userId); + + } return result; } @@ -537,16 +544,39 @@ namespace Umbraco.Core.Security RaiseResetAccessFailedCountEvent(userId); return await UpdateAsync(user); } - - - public override Task AccessFailedAsync(int userId) + + /// + /// Overides the microsoft ASP.NET user managment method + /// + /// + /// + /// returns a Async Task + /// + /// + /// Doesnt set fail attempts back to 0 + /// + public override async Task AccessFailedAsync(int userId) { - var result = base.AccessFailedAsync(userId); + var lockoutStore = (IUserLockoutStore)Store; + var user = await FindByIdAsync(userId); + if (user == null) + throw new InvalidOperationException("No user found by user id " + userId); + + var count = await lockoutStore.IncrementAccessFailedCountAsync(user); + + if (count >= MaxFailedAccessAttemptsBeforeLockout) + { + await lockoutStore.SetLockoutEndDateAsync(user, DateTimeOffset.UtcNow.Add(DefaultAccountLockoutTimeSpan)); + //NOTE: in normal aspnet identity this would do set the number of failed attempts back to 0 + //here we are persisting the value for the back office + } + + var result = await UpdateAsync(user); //Slightly confusing: this will return a Success if we successfully update the AccessFailed count - if (result.Result.Succeeded) + if (result.Succeeded) RaiseLoginFailedEvent(userId); return result; diff --git a/src/Umbraco.Web.UI.Client/src/views/users/user.controller.js b/src/Umbraco.Web.UI.Client/src/views/users/user.controller.js index 0f3519c7ba..3a59a50c2c 100644 --- a/src/Umbraco.Web.UI.Client/src/views/users/user.controller.js +++ b/src/Umbraco.Web.UI.Client/src/views/users/user.controller.js @@ -341,8 +341,10 @@ vm.unlockUserButtonState = "busy"; usersResource.unlockUsers([vm.user.id]).then(function (data) { vm.user.userState = 0; + vm.user.failedPasswordAttempts = 0; setUserDisplayState(); vm.unlockUserButtonState = "success"; + formHelper.showNotifications(data); }, function (error) { vm.unlockUserButtonState = "error"; diff --git a/src/Umbraco.Web.UI.Client/src/views/users/views/user/details.html b/src/Umbraco.Web.UI.Client/src/views/users/views/user/details.html index 67f204aa2f..83c28f9f99 100644 --- a/src/Umbraco.Web.UI.Client/src/views/users/views/user/details.html +++ b/src/Umbraco.Web.UI.Client/src/views/users/views/user/details.html @@ -358,4 +358,4 @@ - \ No newline at end of file + From 863679b3864889ca31b2316b949e28eab0554b07 Mon Sep 17 00:00:00 2001 From: Samuel Sperling Date: Thu, 22 Feb 2018 14:06:26 +1100 Subject: [PATCH 04/47] U4-10849 fixed Umbraco user field "User Last updated" and "last locked date" not updating after password change / account locking.. --- .../Models/Identity/BackOfficeIdentityUser.cs | 15 +++++++++++++-- .../Models/Identity/IdentityModelMappings.cs | 3 ++- src/Umbraco.Core/Models/Identity/IdentityUser.cs | 8 +++++++- .../Security/BackOfficeUserManager.cs | 1 + src/Umbraco.Core/Security/BackOfficeUserStore.cs | 11 +++++++++-- 5 files changed, 32 insertions(+), 6 deletions(-) diff --git a/src/Umbraco.Core/Models/Identity/BackOfficeIdentityUser.cs b/src/Umbraco.Core/Models/Identity/BackOfficeIdentityUser.cs index 1a8cdc446f..3151492106 100644 --- a/src/Umbraco.Core/Models/Identity/BackOfficeIdentityUser.cs +++ b/src/Umbraco.Core/Models/Identity/BackOfficeIdentityUser.cs @@ -120,13 +120,22 @@ namespace Umbraco.Core.Models.Identity set { _tracker.SetPropertyValueAndDetectChanges(value, ref _userName, Ps.Value.UserNameSelector); } } + /// + /// LastPasswordChangeDateUtc so we can track changes to it + /// + public override DateTime? LastPasswordChangeDateUtc + { + get { return _lastPasswordChangeDateUtc; } + set { _tracker.SetPropertyValueAndDetectChanges(value, ref _lastPasswordChangeDateUtc, Ps.Value.LastPasswordChangeDateUtcSelector); } + } + /// /// Override LastLoginDateUtc so we can track changes to it /// public override DateTime? LastLoginDateUtc { get { return _lastLoginDateUtc; } - set { _tracker.SetPropertyValueAndDetectChanges(value, ref _lastLoginDateUtc, Ps.Value.LastLoginDateUtcSelector); } + set { _tracker.SetPropertyValueAndDetectChanges(value, ref _lastLoginDateUtc, Ps.Value.LastLoginDateUtcSelector); } } /// @@ -392,6 +401,7 @@ namespace Umbraco.Core.Models.Identity public readonly PropertyInfo EmailSelector = ExpressionHelper.GetPropertyInfo(x => x.Email); public readonly PropertyInfo UserNameSelector = ExpressionHelper.GetPropertyInfo(x => x.UserName); public readonly PropertyInfo LastLoginDateUtcSelector = ExpressionHelper.GetPropertyInfo(x => x.LastLoginDateUtc); + public readonly PropertyInfo LastPasswordChangeDateUtcSelector = ExpressionHelper.GetPropertyInfo(x => x.LastPasswordChangeDateUtc); public readonly PropertyInfo EmailConfirmedSelector = ExpressionHelper.GetPropertyInfo(x => x.EmailConfirmed); public readonly PropertyInfo NameSelector = ExpressionHelper.GetPropertyInfo(x => x.Name); public readonly PropertyInfo AccessFailedCountSelector = ExpressionHelper.GetPropertyInfo(x => x.AccessFailedCount); @@ -419,6 +429,7 @@ namespace Umbraco.Core.Models.Identity private int _id; private bool _hasIdentity = false; private DateTime? _lastLoginDateUtc; + private DateTime? _lastPasswordChangeDateUtc; private bool _emailConfirmed; private string _name; private int _accessFailedCount; @@ -449,4 +460,4 @@ namespace Umbraco.Core.Models.Identity } -} \ No newline at end of file +} diff --git a/src/Umbraco.Core/Models/Identity/IdentityModelMappings.cs b/src/Umbraco.Core/Models/Identity/IdentityModelMappings.cs index 954705d544..5ef270c789 100644 --- a/src/Umbraco.Core/Models/Identity/IdentityModelMappings.cs +++ b/src/Umbraco.Core/Models/Identity/IdentityModelMappings.cs @@ -19,6 +19,7 @@ namespace Umbraco.Core.Models.Identity }) .ConstructUsing(user => new BackOfficeIdentityUser(user.Id, user.Groups)) .ForMember(user => user.LastLoginDateUtc, expression => expression.MapFrom(user => user.LastLoginDate.ToUniversalTime())) + .ForMember(user => user.LastPasswordChangeDateUtc, expression => expression.MapFrom(user => user.LastPasswordChangeDate.ToUniversalTime())) .ForMember(user => user.Email, expression => expression.MapFrom(user => user.Email)) .ForMember(user => user.EmailConfirmed, expression => expression.MapFrom(user => user.EmailConfirmedDate.HasValue)) .ForMember(user => user.Id, expression => expression.MapFrom(user => user.Id)) @@ -47,4 +48,4 @@ namespace Umbraco.Core.Models.Identity return storedPass.StartsWith(Constants.Security.EmptyPasswordPrefix) ? null : storedPass; } } -} \ No newline at end of file +} diff --git a/src/Umbraco.Core/Models/Identity/IdentityUser.cs b/src/Umbraco.Core/Models/Identity/IdentityUser.cs index c867dcf622..81c7c56d13 100644 --- a/src/Umbraco.Core/Models/Identity/IdentityUser.cs +++ b/src/Umbraco.Core/Models/Identity/IdentityUser.cs @@ -83,6 +83,12 @@ namespace Umbraco.Core.Models.Identity /// public virtual DateTime? LockoutEndDateUtc { get; set; } + /// + /// DateTime in UTC when the password was last changed. + /// + /// + public virtual DateTime? LastPasswordChangeDateUtc { get; set; } + /// /// Is lockout enabled for this user /// @@ -127,4 +133,4 @@ namespace Umbraco.Core.Models.Identity } -} \ No newline at end of file +} diff --git a/src/Umbraco.Core/Security/BackOfficeUserManager.cs b/src/Umbraco.Core/Security/BackOfficeUserManager.cs index 4969fa5f2f..fff97f145c 100644 --- a/src/Umbraco.Core/Security/BackOfficeUserManager.cs +++ b/src/Umbraco.Core/Security/BackOfficeUserManager.cs @@ -457,6 +457,7 @@ namespace Umbraco.Core.Security /// protected override async Task UpdatePassword(IUserPasswordStore passwordStore, T user, string newPassword) { + user.LastPasswordChangeDateUtc = DateTime.UtcNow; var userAwarePasswordHasher = PasswordHasher as IUserAwarePasswordHasher; if (userAwarePasswordHasher == null) return await base.UpdatePassword(passwordStore, user, newPassword); diff --git a/src/Umbraco.Core/Security/BackOfficeUserStore.cs b/src/Umbraco.Core/Security/BackOfficeUserStore.cs index 9db8347040..e0b91ce175 100644 --- a/src/Umbraco.Core/Security/BackOfficeUserStore.cs +++ b/src/Umbraco.Core/Security/BackOfficeUserStore.cs @@ -622,7 +622,7 @@ namespace Umbraco.Core.Security private bool UpdateMemberProperties(IUser user, BackOfficeIdentityUser identityUser) { var anythingChanged = false; - + //don't assign anything if nothing has changed as this will trigger the track changes of the model if (identityUser.IsPropertyDirty("LastLoginDateUtc") @@ -632,6 +632,13 @@ namespace Umbraco.Core.Security anythingChanged = true; user.LastLoginDate = identityUser.LastLoginDateUtc.Value.ToLocalTime(); } + if (identityUser.IsPropertyDirty("LastPasswordChangeDateUtc") + || (user.LastPasswordChangeDate != default(DateTime) && identityUser.LastPasswordChangeDateUtc.HasValue == false) + || identityUser.LastPasswordChangeDateUtc.HasValue && user.LastPasswordChangeDate.ToUniversalTime() != identityUser.LastPasswordChangeDateUtc.Value) + { + anythingChanged = true; + user.LastPasswordChangeDate = identityUser.LastPasswordChangeDateUtc.Value.ToLocalTime(); + } if (identityUser.IsPropertyDirty("EmailConfirmed") || (user.EmailConfirmedDate.HasValue && user.EmailConfirmedDate.Value != default(DateTime) && identityUser.EmailConfirmed == false) || ((user.EmailConfirmedDate.HasValue == false || user.EmailConfirmedDate.Value == default(DateTime)) && identityUser.EmailConfirmed)) @@ -765,4 +772,4 @@ namespace Umbraco.Core.Security return Task.FromResult(false); } } -} \ No newline at end of file +} From b4926416333ab5c830836230b6059dcf745685ce Mon Sep 17 00:00:00 2001 From: mvanhelmont Date: Sun, 29 Apr 2018 08:48:49 +0200 Subject: [PATCH 05/47] U4-11275 This is a simple fix for U4-11275 so that packages also can set routePath to something like a overview page. Like the new user section when you don't want to display children. Is a copy of /:section/:tree/:method/:id --- src/Umbraco.Web.UI.Client/src/routes.js | 35 +++++++++++++++++++------ 1 file changed, 27 insertions(+), 8 deletions(-) diff --git a/src/Umbraco.Web.UI.Client/src/routes.js b/src/Umbraco.Web.UI.Client/src/routes.js index 3fc4d3f78e..314b3b0c53 100644 --- a/src/Umbraco.Web.UI.Client/src/routes.js +++ b/src/Umbraco.Web.UI.Client/src/routes.js @@ -31,11 +31,11 @@ app.config(function ($routeProvider) { userService.getCurrentUser({ broadcastEvent: broadcast }).then(function (user) { //is auth, check if we allow or reject if (isRequired) { - + //This checks the current section and will force a redirect to 'content' as the default if ($route.current.params.section.toLowerCase() === "default" || $route.current.params.section.toLowerCase() === "umbraco" || $route.current.params.section === "") { $route.current.params.section = "content"; - } + } // U4-5430, Benjamin Howarth // We need to change the current route params if the user only has access to a single section @@ -141,13 +141,32 @@ app.config(function ($routeProvider) { resolve: canRoute(true) }) .when('/:section/:tree/:method', { - templateUrl: function (rp) { + //This allows us to dynamically change the template for this route since you cannot inject services into the templateUrl method. + template: "
", + //This controller will execute for this route, then we replace the template dynamnically based on the current tree. + controller: function ($scope, $route, $routeParams, treeService) { - //if there is no method registered for this then show the dashboard - if (!rp.method) - return "views/common/dashboard.html"; - - return ('views/' + rp.tree + '/' + rp.method + '.html'); + if (!$routeParams.tree || !$routeParams.method) { + $scope.templateUrl = "views/common/dashboard.html"; + } + + // Here we need to figure out if this route is for a package tree and if so then we need + // to change it's convention view path to: + // /App_Plugins/{mypackage}/backoffice/{treetype}/{method}.html + + // otherwise if it is a core tree we use the core paths: + // views/{treetype}/{method}.html + + var packageTreeFolder = treeService.getTreePackageFolder($routeParams.tree); + + if (packageTreeFolder) { + $scope.templateUrl = (Umbraco.Sys.ServerVariables.umbracoSettings.appPluginsPath + + "/" + packageTreeFolder + + "/backoffice/" + $routeParams.tree + "/" + $routeParams.method + ".html"); + } + else { + $scope.templateUrl = ('views/' + $routeParams.tree + '/' + $routeParams.method + '.html'); + } }, resolve: canRoute(true) }) From 8be303cb05d6100f5c47d5b128dfd7009132428d Mon Sep 17 00:00:00 2001 From: Nathan Woulfe Date: Sun, 17 Jun 2018 14:33:32 +1000 Subject: [PATCH 06/47] Adds key for 'anchor' to en variants. Needs translations. --- src/Umbraco.Web.UI/umbraco/config/lang/en.xml | 1 + src/Umbraco.Web.UI/umbraco/config/lang/en_us.xml | 1 + 2 files changed, 2 insertions(+) diff --git a/src/Umbraco.Web.UI/umbraco/config/lang/en.xml b/src/Umbraco.Web.UI/umbraco/config/lang/en.xml index 93d8e8b0f6..22596ff9dd 100644 --- a/src/Umbraco.Web.UI/umbraco/config/lang/en.xml +++ b/src/Umbraco.Web.UI/umbraco/config/lang/en.xml @@ -527,6 +527,7 @@ Add Alias All + Anchor Are you sure? Back Border diff --git a/src/Umbraco.Web.UI/umbraco/config/lang/en_us.xml b/src/Umbraco.Web.UI/umbraco/config/lang/en_us.xml index 15256505a9..23d7f3698b 100644 --- a/src/Umbraco.Web.UI/umbraco/config/lang/en_us.xml +++ b/src/Umbraco.Web.UI/umbraco/config/lang/en_us.xml @@ -526,6 +526,7 @@ Add Alias All + Anchor Are you sure? Back Border From f3375c01d8881181371978812792f4318fe0c16e Mon Sep 17 00:00:00 2001 From: Nathan Woulfe Date: Sun, 17 Jun 2018 14:38:37 +1000 Subject: [PATCH 07/47] remove unused dialog service, add logic for managing anchor value in link object, add function for parsing an array of anchors from a string --- .../src/common/services/tinymce.service.js | 46 ++++++++++++++++--- 1 file changed, 40 insertions(+), 6 deletions(-) diff --git a/src/Umbraco.Web.UI.Client/src/common/services/tinymce.service.js b/src/Umbraco.Web.UI.Client/src/common/services/tinymce.service.js index 86de5b586d..d862789539 100644 --- a/src/Umbraco.Web.UI.Client/src/common/services/tinymce.service.js +++ b/src/Umbraco.Web.UI.Client/src/common/services/tinymce.service.js @@ -6,7 +6,7 @@ * @description * A service containing all logic for all of the Umbraco TinyMCE plugins */ -function tinyMceService(dialogService, $log, imageHelper, $http, $timeout, macroResource, macroService, $routeParams, umbRequestHelper, angularHelper, userService) { +function tinyMceService($log, imageHelper, $http, $timeout, macroResource, macroService, $routeParams, umbRequestHelper, angularHelper, userService) { return { /** @@ -645,6 +645,7 @@ function tinyMceService(dialogService, $log, imageHelper, $http, $timeout, macro data.href = anchorElm ? dom.getAttrib(anchorElm, 'href') : ''; data.target = anchorElm ? dom.getAttrib(anchorElm, 'target') : ''; data.rel = anchorElm ? dom.getAttrib(anchorElm, 'rel') : ''; + data.anchor = anchorElm ? dom.getAttrib(anchorElm, 'anchor') : ''; if (selectedElm.nodeName === "IMG") { data.text = initialText = " "; @@ -677,8 +678,6 @@ function tinyMceService(dialogService, $log, imageHelper, $http, $timeout, macro }; } - var injector = angular.element(document.getElementById("umbracoMainPageBody")).injector(); - var dialogService = injector.get("dialogService"); var currentTarget = null; //if we already have a link selected, we want to pass that data over to the dialog @@ -690,10 +689,20 @@ function tinyMceService(dialogService, $log, imageHelper, $http, $timeout, macro target: anchor.attr("target") }; + // split the URL to check for an anchor or querystring, then add that value to the currentTarget object + var urlParts = currentTarget.url.split(/(\?|#)/); + if (urlParts.length === 3) { + currentTarget.anchor = urlParts[2]; + } + //locallink detection, we do this here, to avoid poluting the dialogservice //so the dialog service can just expect to get a node-like structure - if (currentTarget.url.indexOf("localLink:") > 0) { - var linkId = currentTarget.url.substring(currentTarget.url.indexOf(":") + 1, currentTarget.url.length - 1); + if (currentTarget.url.indexOf("localLink:") > 0) { + // if the current link has an anchor, it needs to be considered when getting the udi/id + // if an anchor exists, reduce the substring max by its length plus two to offset the removed prefix and trailing curly brace + var linkId = currentTarget.url.substring(currentTarget.url.indexOf(":") + 1, + currentTarget.url.length - (currentTarget.anchor ? currentTarget.anchor.length + 2 : 1)); + //we need to check if this is an INT or a UDI var parsedIntId = parseInt(linkId, 10); if (isNaN(parsedIntId)) { @@ -741,6 +750,20 @@ function tinyMceService(dialogService, $log, imageHelper, $http, $timeout, macro }); }, + + getAnchorNames: function(propertyString) { + var anchorPattern = //gi; + var matches = propertyString.match(anchorPattern); + var anchors = []; + + if (matches) { + anchors = matches.map(function(v) { + return v.substring(v.indexOf('"') + 1, v.lastIndexOf('\\')); + }); + } + + return anchors; + }, insertLinkInEditor: function(editor, target, anchorElm) { @@ -749,13 +772,18 @@ function tinyMceService(dialogService, $log, imageHelper, $http, $timeout, macro var hasUdi = target.udi ? true : false; var id = hasUdi ? target.udi : (target.id ? target.id : null); + // if an anchor exists, check that it is appropriately prefixed + if (target.anchor && target.anchor[0] !== '?' && target.anchor[0] !== '#') { + target.anchor = (target.anchor.indexOf('=') === -1 ? '#' : '?') + target.anchor; + } + //Create a json obj used to create the attributes for the tag function createElemAttributes() { var a = { href: href, title: target.name, target: target.target ? target.target : null, - rel: target.rel ? target.rel : null + rel: target.rel ? target.rel : null }; if (hasUdi) { a["data-udi"] = target.udi; @@ -763,6 +791,12 @@ function tinyMceService(dialogService, $log, imageHelper, $http, $timeout, macro else if (target.id) { a["data-id"] = target.id; } + + if (target.anchor) { + a["data-anchor"] = target.anchor; + a.href = a.href + target.anchor; + } + return a; } From 1cbb456dcd61485af8705b5cc78715ce6f2ad965 Mon Sep 17 00:00:00 2001 From: Nathan Woulfe Date: Sun, 17 Jun 2018 14:39:45 +1000 Subject: [PATCH 08/47] update rte controllers to add anchors array to overlay model - anchors are parsed from the current editor properties --- .../src/views/propertyeditors/grid/editors/rte.controller.js | 4 +++- .../src/views/propertyeditors/rte/rte.controller.js | 5 +++-- 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/src/Umbraco.Web.UI.Client/src/views/propertyeditors/grid/editors/rte.controller.js b/src/Umbraco.Web.UI.Client/src/views/propertyeditors/grid/editors/rte.controller.js index 99c3fd80ff..397438d5a0 100644 --- a/src/Umbraco.Web.UI.Client/src/views/propertyeditors/grid/editors/rte.controller.js +++ b/src/Umbraco.Web.UI.Client/src/views/propertyeditors/grid/editors/rte.controller.js @@ -1,7 +1,7 @@ (function() { "use strict"; - function GridRichTextEditorController($scope, tinyMceService, macroService) { + function GridRichTextEditorController($scope, tinyMceService, macroService, editorState) { var vm = this; @@ -11,9 +11,11 @@ vm.openEmbed = openEmbed; function openLinkPicker(editor, currentTarget, anchorElement) { + vm.linkPickerOverlay = { view: "linkpicker", currentTarget: currentTarget, + anchors: tinyMceService.getAnchorNames(JSON.stringify(editorState.current.properties)), show: true, submit: function(model) { tinyMceService.insertLinkInEditor(editor, model.target, anchorElement); diff --git a/src/Umbraco.Web.UI.Client/src/views/propertyeditors/rte/rte.controller.js b/src/Umbraco.Web.UI.Client/src/views/propertyeditors/rte/rte.controller.js index 17f0f3b698..c5a739bec2 100644 --- a/src/Umbraco.Web.UI.Client/src/views/propertyeditors/rte/rte.controller.js +++ b/src/Umbraco.Web.UI.Client/src/views/propertyeditors/rte/rte.controller.js @@ -1,6 +1,6 @@ angular.module("umbraco") .controller("Umbraco.PropertyEditors.RTEController", - function ($rootScope, $scope, $q, $locale, dialogService, $log, imageHelper, assetsService, $timeout, tinyMceService, angularHelper, stylesheetResource, macroService) { + function ($rootScope, $scope, $q, $locale, dialogService, $log, imageHelper, assetsService, $timeout, tinyMceService, angularHelper, stylesheetResource, macroService, editorState) { $scope.isLoading = true; @@ -273,11 +273,12 @@ angular.module("umbraco") syncContent(editor); }); - + tinyMceService.createLinkPicker(editor, $scope, function(currentTarget, anchorElement) { $scope.linkPickerOverlay = { view: "linkpicker", currentTarget: currentTarget, + anchors: tinyMceService.getAnchorNames(JSON.stringify(editorState.current.properties)), show: true, submit: function(model) { tinyMceService.insertLinkInEditor(editor, model.target, anchorElement); From 3fecd55c49f5078212ccf3e18f0c5c3315c2e023 Mon Sep 17 00:00:00 2001 From: Nathan Woulfe Date: Sun, 17 Jun 2018 14:41:41 +1000 Subject: [PATCH 09/47] adds check for anchors when overlay opens, and fetches anchors when target updates. --- .../common/dialogs/linkpicker.controller.js | 13 +++++++---- .../linkpicker/linkpicker.controller.js | 23 +++++++++++-------- 2 files changed, 23 insertions(+), 13 deletions(-) diff --git a/src/Umbraco.Web.UI.Client/src/views/common/dialogs/linkpicker.controller.js b/src/Umbraco.Web.UI.Client/src/views/common/dialogs/linkpicker.controller.js index e76db90f47..ba6835a6b1 100644 --- a/src/Umbraco.Web.UI.Client/src/views/common/dialogs/linkpicker.controller.js +++ b/src/Umbraco.Web.UI.Client/src/views/common/dialogs/linkpicker.controller.js @@ -1,6 +1,6 @@ //used for the media picker dialog angular.module("umbraco").controller("Umbraco.Dialogs.LinkPickerController", - function ($scope, eventsService, dialogService, entityResource, contentResource, mediaHelper, userService, localizationService) { + function ($scope, eventsService, dialogService, entityResource, contentResource, mediaHelper, userService, localizationService, tinyMceService) { var dialogOptions = $scope.dialogOptions; var searchText = "Search..."; @@ -39,6 +39,10 @@ angular.module("umbraco").controller("Umbraco.Dialogs.LinkPickerController", }); } } + + if (dialogOptions.anchors) { + $scope.anchorValues = dialogOptions.anchors; + } function nodeSelectHandler(ev, args) { args.event.preventDefault(); @@ -69,9 +73,10 @@ angular.module("umbraco").controller("Umbraco.Dialogs.LinkPickerController", $scope.target.url = "/"; } else { - contentResource.getNiceUrl(args.node.id).then(function (url) { - $scope.target.url = url; - }); + contentResource.getById(args.node.id).then(function (resp) { + $scope.anchorValues = tinyMceService.getAnchorNames(JSON.stringify(resp.properties)); + $scope.model.target.url = resp.urls[0]; + }); } if (!angular.isUndefined($scope.target.isMedia)) { diff --git a/src/Umbraco.Web.UI.Client/src/views/common/overlays/linkpicker/linkpicker.controller.js b/src/Umbraco.Web.UI.Client/src/views/common/overlays/linkpicker/linkpicker.controller.js index 91c74311b3..5d6b0916f8 100644 --- a/src/Umbraco.Web.UI.Client/src/views/common/overlays/linkpicker/linkpicker.controller.js +++ b/src/Umbraco.Web.UI.Client/src/views/common/overlays/linkpicker/linkpicker.controller.js @@ -1,8 +1,10 @@ //used for the media picker dialog angular.module("umbraco").controller("Umbraco.Overlays.LinkPickerController", - function ($scope, eventsService, dialogService, entityResource, contentResource, mediaHelper, userService, localizationService) { + function ($scope, eventsService, dialogService, entityResource, contentResource, mediaHelper, userService, localizationService, tinyMceService) { var dialogOptions = $scope.model; + var anchorPattern = //gi; + var searchText = "Search..."; localizationService.localize("general_search").then(function (value) { searchText = value + "..."; @@ -22,11 +24,10 @@ angular.module("umbraco").controller("Umbraco.Overlays.LinkPickerController", selectedSearchResults: [] }; - $scope.showTarget = $scope.model.hideTarget !== true; + $scope.showTarget = $scope.model.hideTarget !== true; if (dialogOptions.currentTarget) { $scope.model.target = dialogOptions.currentTarget; - //if we have a node ID, we fetch the current node to build the form data if ($scope.model.target.id || $scope.model.target.udi) { @@ -47,9 +48,12 @@ angular.module("umbraco").controller("Umbraco.Overlays.LinkPickerController", }); } } - - function nodeSelectHandler(ev, args) { - + + if (dialogOptions.anchors) { + $scope.anchorValues = dialogOptions.anchors; + } + + function nodeSelectHandler(ev, args) { if(args && args.event) { args.event.preventDefault(); args.event.stopPropagation(); @@ -71,9 +75,10 @@ angular.module("umbraco").controller("Umbraco.Overlays.LinkPickerController", if (args.node.id < 0) { $scope.model.target.url = "/"; } - else { - contentResource.getNiceUrl(args.node.id).then(function (url) { - $scope.model.target.url = url; + else { + contentResource.getById(args.node.id).then(function (resp) { + $scope.anchorValues = tinyMceService.getAnchorNames(JSON.stringify(resp.properties)); + $scope.model.target.url = resp.urls[0]; }); } From 9cfc997ba209780ac3e36f5b104cf33f8ff80ac2 Mon Sep 17 00:00:00 2001 From: Nathan Woulfe Date: Sun, 17 Jun 2018 14:42:16 +1000 Subject: [PATCH 10/47] update overlay/dialog to add input for anchor value - value can be a string or selection from the datalist. Is the dialog in use anymore? --- .../src/views/common/dialogs/linkpicker.html | 14 +++++++++++++- .../common/overlays/linkpicker/linkpicker.html | 15 ++++++++++++++- 2 files changed, 27 insertions(+), 2 deletions(-) diff --git a/src/Umbraco.Web.UI.Client/src/views/common/dialogs/linkpicker.html b/src/Umbraco.Web.UI.Client/src/views/common/dialogs/linkpicker.html index 442357edcc..0cc1c52595 100644 --- a/src/Umbraco.Web.UI.Client/src/views/common/dialogs/linkpicker.html +++ b/src/Umbraco.Web.UI.Client/src/views/common/dialogs/linkpicker.html @@ -2,13 +2,25 @@
- + + + + + diff --git a/src/Umbraco.Web.UI.Client/src/views/common/overlays/linkpicker/linkpicker.html b/src/Umbraco.Web.UI.Client/src/views/common/overlays/linkpicker/linkpicker.html index e1b13206df..93d13d275a 100644 --- a/src/Umbraco.Web.UI.Client/src/views/common/overlays/linkpicker/linkpicker.html +++ b/src/Umbraco.Web.UI.Client/src/views/common/overlays/linkpicker/linkpicker.html @@ -5,11 +5,24 @@ localize="placeholder" placeholder="@general_url" class="umb-editor umb-textstring" + style="float:left; min-width:0; width:65%" ng-model="model.target.url" ng-disabled="model.target.id" focus-when="{{true}} "/> + + + + + + - + Date: Sun, 17 Jun 2018 14:47:02 +1000 Subject: [PATCH 11/47] adds ngdoc --- .../src/common/services/tinymce.service.js | 15 +++++++++++++-- 1 file changed, 13 insertions(+), 2 deletions(-) diff --git a/src/Umbraco.Web.UI.Client/src/common/services/tinymce.service.js b/src/Umbraco.Web.UI.Client/src/common/services/tinymce.service.js index d862789539..d60c264e7a 100644 --- a/src/Umbraco.Web.UI.Client/src/common/services/tinymce.service.js +++ b/src/Umbraco.Web.UI.Client/src/common/services/tinymce.service.js @@ -751,9 +751,20 @@ function tinyMceService($log, imageHelper, $http, $timeout, macroResource, macro }, - getAnchorNames: function(propertyString) { + /** + * @ngdoc method + * @name umbraco.services.tinyMceService#getAnchorNames + * @methodOf umbraco.services.tinyMceService + * + * @description + * From the given string, generates a string array where each item is the id attribute value from a named anchor + * 'some string with a named anchor' returns ['anchor'] + * + * @param {string} input the string to parse + */ + getAnchorNames: function(input) { var anchorPattern = //gi; - var matches = propertyString.match(anchorPattern); + var matches = input.match(anchorPattern); var anchors = []; if (matches) { From d327299d76f5e3ea93bf9621f3e6f545d20e4c39 Mon Sep 17 00:00:00 2001 From: Nathan Woulfe Date: Sun, 17 Jun 2018 18:11:01 +1000 Subject: [PATCH 12/47] data.anchor was not used, no need to keep it --- src/Umbraco.Web.UI.Client/src/common/services/tinymce.service.js | 1 - 1 file changed, 1 deletion(-) diff --git a/src/Umbraco.Web.UI.Client/src/common/services/tinymce.service.js b/src/Umbraco.Web.UI.Client/src/common/services/tinymce.service.js index d60c264e7a..6593a6ebd5 100644 --- a/src/Umbraco.Web.UI.Client/src/common/services/tinymce.service.js +++ b/src/Umbraco.Web.UI.Client/src/common/services/tinymce.service.js @@ -645,7 +645,6 @@ function tinyMceService($log, imageHelper, $http, $timeout, macroResource, macro data.href = anchorElm ? dom.getAttrib(anchorElm, 'href') : ''; data.target = anchorElm ? dom.getAttrib(anchorElm, 'target') : ''; data.rel = anchorElm ? dom.getAttrib(anchorElm, 'rel') : ''; - data.anchor = anchorElm ? dom.getAttrib(anchorElm, 'anchor') : ''; if (selectedElm.nodeName === "IMG") { data.text = initialText = " "; From ef580558b18afa7649b2eda5c79392a199d616d1 Mon Sep 17 00:00:00 2001 From: Nathan Woulfe Date: Sun, 17 Jun 2018 19:18:16 +1000 Subject: [PATCH 13/47] ensure anchor attribute is removed if anchor value is empty --- .../src/common/services/tinymce.service.js | 17 ++++++++++------- 1 file changed, 10 insertions(+), 7 deletions(-) diff --git a/src/Umbraco.Web.UI.Client/src/common/services/tinymce.service.js b/src/Umbraco.Web.UI.Client/src/common/services/tinymce.service.js index 6593a6ebd5..bc1dd987fd 100644 --- a/src/Umbraco.Web.UI.Client/src/common/services/tinymce.service.js +++ b/src/Umbraco.Web.UI.Client/src/common/services/tinymce.service.js @@ -686,12 +686,12 @@ function tinyMceService($log, imageHelper, $http, $timeout, macroResource, macro name: anchor.attr("title"), url: anchor.attr("href"), target: anchor.attr("target") - }; - - // split the URL to check for an anchor or querystring, then add that value to the currentTarget object - var urlParts = currentTarget.url.split(/(\?|#)/); - if (urlParts.length === 3) { - currentTarget.anchor = urlParts[2]; + }; + + // drop the lead char from the anchor text, if it has a value + var anchorVal = anchor.data("anchor"); + if (anchorVal) { + currentTarget.anchor = anchorVal.substring(1); } //locallink detection, we do this here, to avoid poluting the dialogservice @@ -795,6 +795,7 @@ function tinyMceService($log, imageHelper, $http, $timeout, macroResource, macro target: target.target ? target.target : null, rel: target.rel ? target.rel : null }; + if (hasUdi) { a["data-udi"] = target.udi; } @@ -805,8 +806,10 @@ function tinyMceService($log, imageHelper, $http, $timeout, macroResource, macro if (target.anchor) { a["data-anchor"] = target.anchor; a.href = a.href + target.anchor; + } else { + a["data-anchor"] = null; } - + return a; } From 9f90c99350a1a97d76c021ff4a1595e17c1d18af Mon Sep 17 00:00:00 2001 From: Nathan Woulfe Date: Sun, 17 Jun 2018 19:46:07 +1000 Subject: [PATCH 14/47] fetch anchors for existing links --- .../views/common/dialogs/linkpicker.controller.js | 6 ++++-- .../overlays/linkpicker/linkpicker.controller.js | 12 ++++++------ 2 files changed, 10 insertions(+), 8 deletions(-) diff --git a/src/Umbraco.Web.UI.Client/src/views/common/dialogs/linkpicker.controller.js b/src/Umbraco.Web.UI.Client/src/views/common/dialogs/linkpicker.controller.js index ba6835a6b1..043d5af1c5 100644 --- a/src/Umbraco.Web.UI.Client/src/views/common/dialogs/linkpicker.controller.js +++ b/src/Umbraco.Web.UI.Client/src/views/common/dialogs/linkpicker.controller.js @@ -34,8 +34,10 @@ angular.module("umbraco").controller("Umbraco.Dialogs.LinkPickerController", }); } - contentResource.getNiceUrl(id).then(function (url) { - $scope.target.url = url; + // if a link exists, get the properties to build the anchor name list + contentResource.getById(id).then(function (resp) { + $scope.anchorValues = tinyMceService.getAnchorNames(JSON.stringify(resp.properties)); + $scope.model.target.url = resp.urls[0]; }); } } diff --git a/src/Umbraco.Web.UI.Client/src/views/common/overlays/linkpicker/linkpicker.controller.js b/src/Umbraco.Web.UI.Client/src/views/common/overlays/linkpicker/linkpicker.controller.js index 5d6b0916f8..a5fa5f5e3e 100644 --- a/src/Umbraco.Web.UI.Client/src/views/common/overlays/linkpicker/linkpicker.controller.js +++ b/src/Umbraco.Web.UI.Client/src/views/common/overlays/linkpicker/linkpicker.controller.js @@ -42,14 +42,14 @@ angular.module("umbraco").controller("Umbraco.Overlays.LinkPickerController", $scope.dialogTreeEventHandler.syncTree({ path: $scope.model.target.path, tree: "content" }); }); } - - contentResource.getNiceUrl(id).then(function (url) { - $scope.model.target.url = url; + + // if a link exists, get the properties to build the anchor name list + contentResource.getById(id).then(function (resp) { + $scope.anchorValues = tinyMceService.getAnchorNames(JSON.stringify(resp.properties)); + $scope.model.target.url = resp.urls[0]; }); } - } - - if (dialogOptions.anchors) { + } else if (dialogOptions.anchors) { $scope.anchorValues = dialogOptions.anchors; } From bb28d3f4a9456fc13bf9cd2932b80e9fa3e73f4e Mon Sep 17 00:00:00 2001 From: Nathan Woulfe Date: Sun, 17 Jun 2018 20:06:39 +1000 Subject: [PATCH 15/47] extend contentcontroller to allow udi or guid params for getbyid, which also requires extending the attribute for the different param types --- src/Umbraco.Web/Editors/ContentController.cs | 71 ++++++++++++++----- ...EnsureUserPermissionForContentAttribute.cs | 20 +++++- 2 files changed, 73 insertions(+), 18 deletions(-) diff --git a/src/Umbraco.Web/Editors/ContentController.cs b/src/Umbraco.Web/Editors/ContentController.cs index cb5c9ebf71..899c6a0ab3 100644 --- a/src/Umbraco.Web/Editors/ContentController.cs +++ b/src/Umbraco.Web/Editors/ContentController.cs @@ -67,7 +67,9 @@ namespace Umbraco.Web.Editors public void Initialize(HttpControllerSettings controllerSettings, HttpControllerDescriptor controllerDescriptor) { controllerSettings.Services.Replace(typeof(IHttpActionSelector), new ParameterSwapControllerActionSelector( - new ParameterSwapControllerActionSelector.ParameterSwapInfo("GetNiceUrl", "id", typeof(int), typeof(Guid), typeof(Udi)))); + new ParameterSwapControllerActionSelector.ParameterSwapInfo("GetNiceUrl", "id", typeof(int), typeof(Guid), typeof(Udi)), + new ParameterSwapControllerActionSelector.ParameterSwapInfo("GetById", "id", typeof(int), typeof(Guid), typeof(Udi)) + )); } } @@ -100,7 +102,7 @@ namespace Umbraco.Web.Editors var content = Services.ContentService.GetById(saveModel.ContentId); if (content == null) throw new HttpResponseException(Request.CreateResponse(HttpStatusCode.NotFound)); - + //current permissions explicitly assigned to this content item var contentPermissions = Services.ContentService.GetPermissionsForEntity(content) .ToDictionary(x => x.UserGroupId, x => x); @@ -136,10 +138,10 @@ namespace Umbraco.Web.Editors //if they are different we need to update, otherwise there's nothing to update else if (contentPermissions.ContainsKey(userGroup.Id) == false || contentPermissions[userGroup.Id].AssignedPermissions.UnsortedSequenceEqual(groupPermissionCodes) == false) { - + Services.UserService.ReplaceUserGroupPermissions(userGroup.Id, groupPermissionCodes.Select(x => x[0]), content.Id); - } - } + } + } } return GetDetailedPermissions(content, allUserGroups); @@ -158,7 +160,7 @@ namespace Umbraco.Web.Editors { if (contentId <= 0) throw new HttpResponseException(Request.CreateResponse(HttpStatusCode.NotFound)); var content = Services.ContentService.GetById(contentId); - if (content == null) throw new HttpResponseException(Request.CreateResponse(HttpStatusCode.NotFound)); + if (content == null) throw new HttpResponseException(Request.CreateResponse(HttpStatusCode.NotFound)); //TODO: Should non-admins be able to see detailed permissions? @@ -195,9 +197,9 @@ namespace Umbraco.Web.Editors permission.Checked = false; permission.Checked = assignedGroupPermission.AssignedPermissions.Contains(permission.PermissionCode, StringComparer.InvariantCulture); } - + } - + return defaultPermissionsByGroup; } @@ -246,10 +248,10 @@ namespace Umbraco.Web.Editors //set a custom path since the tree that renders this has the content type id as the parent content.Path = string.Format("-1,{0},{1}", persistedContent.ContentTypeId, content.Id); - content.AllowedActions = new[] {"A"}; + content.AllowedActions = new[] { "A" }; content.IsBlueprint = true; - var excludeProps = new[] {"_umb_urls", "_umb_releasedate", "_umb_expiredate", "_umb_template"}; + var excludeProps = new[] { "_umb_urls", "_umb_releasedate", "_umb_expiredate", "_umb_template" }; var propsTab = content.Tabs.Last(); propsTab.Properties = propsTab.Properties .Where(p => excludeProps.Contains(p.Alias) == false); @@ -274,6 +276,43 @@ namespace Umbraco.Web.Editors return content; } + /// + /// Gets the content json for the content id + /// + /// + /// + [OutgoingEditorModelEvent] + [EnsureUserPermissionForContent("id")] + public ContentItemDisplay GetById(Guid id) + { + var foundContent = GetObjectFromRequest(() => Services.ContentService.GetById(id)); + if (foundContent == null) + { + HandleContentNotFound(id); + } + + var content = AutoMapperExtensions.MapWithUmbracoContext(foundContent, UmbracoContext); + return content; + } + + /// + /// Gets the content json for the content id + /// + /// + /// + [OutgoingEditorModelEvent] + [EnsureUserPermissionForContent("id")] + public ContentItemDisplay GetById(Udi id) + { + var guidUdi = id as GuidUdi; + if (guidUdi != null) + { + return GetById(guidUdi.Guid); + } + + throw new HttpResponseException(HttpStatusCode.NotFound); + } + [EnsureUserPermissionForContent("id")] public ContentItemDisplay GetWithTreeDefinition(int id) { @@ -373,7 +412,7 @@ namespace Umbraco.Web.Editors { return GetNiceUrl(guidUdi.Guid); } - throw new HttpResponseException(HttpStatusCode.NotFound); + throw new HttpResponseException(HttpStatusCode.NotFound); } /// @@ -459,7 +498,7 @@ namespace Umbraco.Web.Editors { var permissions = Services.UserService .GetPermissions(Security.CurrentUser, nodeIds); - + var permissionsDictionary = new Dictionary(); foreach (var nodeId in nodeIds) { @@ -513,7 +552,7 @@ namespace Umbraco.Web.Editors var notificationModel = new SimpleNotificationModel(); notificationModel.AddSuccessNotification( Services.TextService.Localize("blueprints/createdBlueprintHeading"), - Services.TextService.Localize("blueprints/createdBlueprintMessage", new[]{ content.Name}) + Services.TextService.Localize("blueprints/createdBlueprintMessage", new[] { content.Name }) ); return notificationModel; @@ -563,7 +602,7 @@ namespace Umbraco.Web.Editors [ModelBinder(typeof(ContentItemBinder))] ContentItemSave contentItem) { - return PostSaveInternal(contentItem, + return PostSaveInternal(contentItem, content => Services.ContentService.WithResult().Save(contentItem.PersistedContent, Security.CurrentUser.Id)); } @@ -1083,7 +1122,7 @@ namespace Umbraco.Web.Editors { throw new HttpResponseException(HttpStatusCode.NotFound); } - + var hasPathAccess = (nodeId == Constants.System.Root) ? user.HasContentRootAccess(entityService) : (nodeId == Constants.System.RecycleBinContent) @@ -1108,7 +1147,7 @@ namespace Umbraco.Web.Editors var allowed = true; foreach (var p in permissionsToCheck) { - if (permission == null + if (permission == null || permission.GetAllPermissions().Contains(p.ToString(CultureInfo.InvariantCulture)) == false) { allowed = false; diff --git a/src/Umbraco.Web/WebApi/Filters/EnsureUserPermissionForContentAttribute.cs b/src/Umbraco.Web/WebApi/Filters/EnsureUserPermissionForContentAttribute.cs index 129bac1c4d..1a5fc4d07c 100644 --- a/src/Umbraco.Web/WebApi/Filters/EnsureUserPermissionForContentAttribute.cs +++ b/src/Umbraco.Web/WebApi/Filters/EnsureUserPermissionForContentAttribute.cs @@ -79,7 +79,23 @@ namespace Umbraco.Web.WebApi.Filters if (parts.Length == 1) { - nodeId = (int)actionContext.ActionArguments[parts[0]]; + var argument = actionContext.ActionArguments[parts[0]].ToString(); + // if the argument is an int, it will parse and can be assigned to nodeId + // if might be a udi, so check that next + // otherwise treat it as a guid - unlikely we ever get here + if (int.TryParse(argument, out int parsedId)) + { + nodeId = parsedId; + } + else if (Udi.TryParse(argument, true, out Udi udi)) + { + nodeId = ApplicationContext.Current.Services.EntityService.GetIdForUdi(udi).Result; + } + else + { + Guid.TryParse(argument, out Guid key); + nodeId = ApplicationContext.Current.Services.EntityService.GetIdForKey(key, UmbracoObjectTypes.Document).Result; + } } else { @@ -118,4 +134,4 @@ namespace Umbraco.Web.WebApi.Filters } -} \ No newline at end of file +} From ecaa853799c9275683897fe82b2fb823b6b5f089 Mon Sep 17 00:00:00 2001 From: Nathan Woulfe Date: Sun, 17 Jun 2018 20:07:14 +1000 Subject: [PATCH 16/47] tidy up get/set for anchor when opening overlay --- .../src/common/services/tinymce.service.js | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/Umbraco.Web.UI.Client/src/common/services/tinymce.service.js b/src/Umbraco.Web.UI.Client/src/common/services/tinymce.service.js index bc1dd987fd..860a25b09d 100644 --- a/src/Umbraco.Web.UI.Client/src/common/services/tinymce.service.js +++ b/src/Umbraco.Web.UI.Client/src/common/services/tinymce.service.js @@ -689,7 +689,7 @@ function tinyMceService($log, imageHelper, $http, $timeout, macroResource, macro }; // drop the lead char from the anchor text, if it has a value - var anchorVal = anchor.data("anchor"); + var anchorVal = anchor[0].dataset.anchor; if (anchorVal) { currentTarget.anchor = anchorVal.substring(1); } @@ -699,8 +699,7 @@ function tinyMceService($log, imageHelper, $http, $timeout, macroResource, macro if (currentTarget.url.indexOf("localLink:") > 0) { // if the current link has an anchor, it needs to be considered when getting the udi/id // if an anchor exists, reduce the substring max by its length plus two to offset the removed prefix and trailing curly brace - var linkId = currentTarget.url.substring(currentTarget.url.indexOf(":") + 1, - currentTarget.url.length - (currentTarget.anchor ? currentTarget.anchor.length + 2 : 1)); + var linkId = currentTarget.url.substring(currentTarget.url.indexOf(":") + 1, currentTarget.url.lastIndexOf("}")); //we need to check if this is an INT or a UDI var parsedIntId = parseInt(linkId, 10); From 411ff80b3e2926e73a65dcb9cf8a766a4e869efc Mon Sep 17 00:00:00 2001 From: Nathan Woulfe Date: Thu, 21 Jun 2018 07:15:54 +1000 Subject: [PATCH 17/47] removes inline styles --- .../src/less/property-editors.less | 14 ++++++++++++++ .../common/overlays/linkpicker/linkpicker.html | 6 ++---- 2 files changed, 16 insertions(+), 4 deletions(-) diff --git a/src/Umbraco.Web.UI.Client/src/less/property-editors.less b/src/Umbraco.Web.UI.Client/src/less/property-editors.less index 2d317fa4a0..77fd17ea38 100644 --- a/src/Umbraco.Web.UI.Client/src/less/property-editors.less +++ b/src/Umbraco.Web.UI.Client/src/less/property-editors.less @@ -3,10 +3,24 @@ // -------------------------------------------------- .umb-editor { min-width:66.6%; + + &-pull { + float:left; + width:66.6%; + } + + &-push { + float:right; + } } .umb-editor-tiny { width: 60px; + + &.umb-editor-push { + width:30%; + min-width:0; + } } .umb-editor-small { diff --git a/src/Umbraco.Web.UI.Client/src/views/common/overlays/linkpicker/linkpicker.html b/src/Umbraco.Web.UI.Client/src/views/common/overlays/linkpicker/linkpicker.html index 93d13d275a..2ad9177e8c 100644 --- a/src/Umbraco.Web.UI.Client/src/views/common/overlays/linkpicker/linkpicker.html +++ b/src/Umbraco.Web.UI.Client/src/views/common/overlays/linkpicker/linkpicker.html @@ -4,8 +4,7 @@ @@ -14,8 +13,7 @@ list="anchors" localize="placeholder" placeholder="@general_anchor" - class="umb-editor umb-textstring" - style="float:right; min-width:0; width:30%" + class="umb-editor umb-editor-push umb-editor-tiny umb-textstring" ng-model="model.target.anchor" /> From 576ff6d2b91078647eb7dc3f4a0315fe58a99d51 Mon Sep 17 00:00:00 2001 From: Nathan Woulfe Date: Mon, 25 Jun 2018 16:09:28 +1000 Subject: [PATCH 18/47] add trackby in repeater, filter out dupes --- .../propertyeditors/tags/tags.controller.js | 46 ++++++++++--------- .../src/views/propertyeditors/tags/tags.html | 2 +- 2 files changed, 25 insertions(+), 23 deletions(-) diff --git a/src/Umbraco.Web.UI.Client/src/views/propertyeditors/tags/tags.controller.js b/src/Umbraco.Web.UI.Client/src/views/propertyeditors/tags/tags.controller.js index 06365d0483..745fe48d0f 100644 --- a/src/Umbraco.Web.UI.Client/src/views/propertyeditors/tags/tags.controller.js +++ b/src/Umbraco.Web.UI.Client/src/views/propertyeditors/tags/tags.controller.js @@ -5,15 +5,12 @@ angular.module("umbraco") var $typeahead; $scope.isLoading = true; - $scope.tagToAdd = ""; - - assetsService.loadJs("lib/typeahead.js/typeahead.bundle.min.js", $scope).then(function () { - - $scope.isLoading = false; - - //load current value - - if ($scope.model.value) { + $scope.tagToAdd = ""; + + function setModelValue(val) { + + $scope.model.value = val || $scope.model.value; + if ($scope.model.value) { if (!$scope.model.config.storageType || $scope.model.config.storageType !== "Json") { //it is csv if (!$scope.model.value) { @@ -21,7 +18,14 @@ angular.module("umbraco") } else { if($scope.model.value.length > 0) { - $scope.model.value = $scope.model.value.split(","); + // split the csv string, and remove any duplicate values + var tempArray = $scope.model.value.split(',').map(function(v) { + return v.trim(); + }); + + $scope.model.value = tempArray.filter(function(v, i, self) { + return self.indexOf(v) === i; + }); } } } @@ -29,6 +33,14 @@ angular.module("umbraco") else { $scope.model.value = []; } + } + + assetsService.loadJs("lib/typeahead.js/typeahead.bundle.min.js", $scope).then(function () { + + $scope.isLoading = false; + + //load current value + setModelValue(); // Method required by the valPropertyValidator directive (returns true if the property editor has at least one tag selected) $scope.validateMandatory = function () { @@ -85,17 +97,7 @@ angular.module("umbraco") //vice versa $scope.model.onValueChanged = function (newVal, oldVal) { //update the display val again if it has changed from the server - $scope.model.value = newVal; - - if (!$scope.model.config.storageType || $scope.model.config.storageType !== "Json") { - //it is csv - if (!$scope.model.value) { - $scope.model.value = []; - } - else { - $scope.model.value = $scope.model.value.split(","); - } - } + setModelValue(newVal); }; //configure the tags data source @@ -190,4 +192,4 @@ angular.module("umbraco") }); } -); \ No newline at end of file +); diff --git a/src/Umbraco.Web.UI.Client/src/views/propertyeditors/tags/tags.html b/src/Umbraco.Web.UI.Client/src/views/propertyeditors/tags/tags.html index e9fbde9027..7e4de164de 100644 --- a/src/Umbraco.Web.UI.Client/src/views/propertyeditors/tags/tags.html +++ b/src/Umbraco.Web.UI.Client/src/views/propertyeditors/tags/tags.html @@ -8,7 +8,7 @@ - + From 9b8f4c1a9a9242bb87b741d1379ed5e0acdb95ee Mon Sep 17 00:00:00 2001 From: Nathan Woulfe Date: Fri, 29 Jun 2018 20:27:28 +1000 Subject: [PATCH 19/47] check for existing file content when creating/saving new template --- src/Umbraco.Core/Services/FileService.cs | 40 ++++++++++++++++++- src/Umbraco.Core/Services/IFileService.cs | 9 ++++- src/Umbraco.Web/Editors/TemplateController.cs | 11 +++++ 3 files changed, 58 insertions(+), 2 deletions(-) diff --git a/src/Umbraco.Core/Services/FileService.cs b/src/Umbraco.Core/Services/FileService.cs index 916c036a69..4428175fa9 100644 --- a/src/Umbraco.Core/Services/FileService.cs +++ b/src/Umbraco.Core/Services/FileService.cs @@ -291,6 +291,14 @@ namespace Umbraco.Core.Services {"ContentTypeAlias", contentTypeAlias}, }; + // check that the template hasn't been created on disk before creating the content type + // if it exists, set the new template content to the existing file content + string content = GetViewContent(contentTypeAlias); + if (content.IsNullOrWhiteSpace() == false) + { + template.Content = content; + } + using (var uow = UowProvider.GetUnitOfWork()) { var saveEventArgs = new SaveEventArgs(template, true, evtMsgs, additionalData); @@ -1045,6 +1053,19 @@ namespace Umbraco.Core.Services : Attempt.Fail(); } + internal Attempt TryGetViewPath(string fileName) + { + if (fileName.EndsWith(".cshtml") == false) + { + fileName += ".cshtml"; + } + + string viewPath = IOHelper.MapPath(SystemDirectories.MvcViews + "/" + fileName); + return System.IO.File.Exists(viewPath) + ? Attempt.Succeed(viewPath) + : Attempt.Fail(); + } + private IPartialViewRepository GetPartialViewRepository(PartialViewType partialViewType, IUnitOfWork uow) { switch (partialViewType) @@ -1076,6 +1097,23 @@ namespace Umbraco.Core.Services return GetPartialViewMacroSnippetContent(snippetName, PartialViewType.PartialViewMacro); } + public string GetViewContent(string filename) + { + if (filename.IsNullOrWhiteSpace()) + throw new ArgumentNullException(nameof(filename)); + + Attempt viewAttempt = TryGetViewPath(filename); + if (viewAttempt.Success == false) + { + return string.Empty; + } + + using (var view = new StreamReader(System.IO.File.OpenRead(viewAttempt.Result))) + { + return view.ReadToEnd().Trim(); + } + } + private string GetPartialViewMacroSnippetContent(string snippetName, PartialViewType partialViewType) { if (snippetName.IsNullOrWhiteSpace()) @@ -1265,4 +1303,4 @@ namespace Umbraco.Core.Services #endregion } -} \ No newline at end of file +} diff --git a/src/Umbraco.Core/Services/IFileService.cs b/src/Umbraco.Core/Services/IFileService.cs index 74de0861d8..0031562697 100644 --- a/src/Umbraco.Core/Services/IFileService.cs +++ b/src/Umbraco.Core/Services/IFileService.cs @@ -427,5 +427,12 @@ namespace Umbraco.Core.Services /// The filesystem path to the partial view. /// The size of the partial view. long GetPartialViewFileSize(string filepath); + + /// + /// Gets the content of a view. + /// + /// The name of the view. + /// + string GetViewContent(string filename); } -} \ No newline at end of file +} diff --git a/src/Umbraco.Web/Editors/TemplateController.cs b/src/Umbraco.Web/Editors/TemplateController.cs index 78855c95c2..3df6cb2aba 100644 --- a/src/Umbraco.Web/Editors/TemplateController.cs +++ b/src/Umbraco.Web/Editors/TemplateController.cs @@ -1,9 +1,12 @@ using System.Collections.Generic; +using System.IO; using System.Linq; using System.Net; using System.Net.Http; +using System.Web; using System.Web.Http; using AutoMapper; +using Umbraco.Core; using Umbraco.Core.IO; using Umbraco.Core.Models; using Umbraco.Core.Services; @@ -178,6 +181,14 @@ namespace Umbraco.Web.Editors else { //create + + // file might already be on disk, if so grab the content to avoid overwriting + string content = Services.FileService.GetViewContent(display.Alias); + if (string.IsNullOrEmpty(content) == false) + { + display.Content = content; + } + ITemplate master = null; if (string.IsNullOrEmpty(display.MasterTemplateAlias) == false) { From e6842a10206721e9be7bd35e6474d4b34683212f Mon Sep 17 00:00:00 2001 From: Nathan Woulfe Date: Fri, 29 Jun 2018 20:34:04 +1000 Subject: [PATCH 20/47] clean up usings --- src/Umbraco.Web/Editors/TemplateController.cs | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/src/Umbraco.Web/Editors/TemplateController.cs b/src/Umbraco.Web/Editors/TemplateController.cs index 3df6cb2aba..132e804a95 100644 --- a/src/Umbraco.Web/Editors/TemplateController.cs +++ b/src/Umbraco.Web/Editors/TemplateController.cs @@ -1,18 +1,13 @@ -using System.Collections.Generic; -using System.IO; +using AutoMapper; +using System.Collections.Generic; using System.Linq; using System.Net; using System.Net.Http; -using System.Web; using System.Web.Http; -using AutoMapper; -using Umbraco.Core; using Umbraco.Core.IO; using Umbraco.Core.Models; -using Umbraco.Core.Services; using Umbraco.Web.Models.ContentEditing; using Umbraco.Web.Mvc; -using Umbraco.Web.WebApi; using Umbraco.Web.WebApi.Filters; using Constants = Umbraco.Core.Constants; From e34926acd62f0590f4929dfdcd5e2ccedfeed514 Mon Sep 17 00:00:00 2001 From: Nathan Woulfe Date: Fri, 29 Jun 2018 20:27:28 +1000 Subject: [PATCH 21/47] check for existing file content when creating/saving new template --- src/Umbraco.Core/Services/FileService.cs | 40 ++++++++++++++++++- src/Umbraco.Core/Services/IFileService.cs | 9 ++++- src/Umbraco.Web/Editors/TemplateController.cs | 11 +++++ 3 files changed, 58 insertions(+), 2 deletions(-) diff --git a/src/Umbraco.Core/Services/FileService.cs b/src/Umbraco.Core/Services/FileService.cs index 916c036a69..4428175fa9 100644 --- a/src/Umbraco.Core/Services/FileService.cs +++ b/src/Umbraco.Core/Services/FileService.cs @@ -291,6 +291,14 @@ namespace Umbraco.Core.Services {"ContentTypeAlias", contentTypeAlias}, }; + // check that the template hasn't been created on disk before creating the content type + // if it exists, set the new template content to the existing file content + string content = GetViewContent(contentTypeAlias); + if (content.IsNullOrWhiteSpace() == false) + { + template.Content = content; + } + using (var uow = UowProvider.GetUnitOfWork()) { var saveEventArgs = new SaveEventArgs(template, true, evtMsgs, additionalData); @@ -1045,6 +1053,19 @@ namespace Umbraco.Core.Services : Attempt.Fail(); } + internal Attempt TryGetViewPath(string fileName) + { + if (fileName.EndsWith(".cshtml") == false) + { + fileName += ".cshtml"; + } + + string viewPath = IOHelper.MapPath(SystemDirectories.MvcViews + "/" + fileName); + return System.IO.File.Exists(viewPath) + ? Attempt.Succeed(viewPath) + : Attempt.Fail(); + } + private IPartialViewRepository GetPartialViewRepository(PartialViewType partialViewType, IUnitOfWork uow) { switch (partialViewType) @@ -1076,6 +1097,23 @@ namespace Umbraco.Core.Services return GetPartialViewMacroSnippetContent(snippetName, PartialViewType.PartialViewMacro); } + public string GetViewContent(string filename) + { + if (filename.IsNullOrWhiteSpace()) + throw new ArgumentNullException(nameof(filename)); + + Attempt viewAttempt = TryGetViewPath(filename); + if (viewAttempt.Success == false) + { + return string.Empty; + } + + using (var view = new StreamReader(System.IO.File.OpenRead(viewAttempt.Result))) + { + return view.ReadToEnd().Trim(); + } + } + private string GetPartialViewMacroSnippetContent(string snippetName, PartialViewType partialViewType) { if (snippetName.IsNullOrWhiteSpace()) @@ -1265,4 +1303,4 @@ namespace Umbraco.Core.Services #endregion } -} \ No newline at end of file +} diff --git a/src/Umbraco.Core/Services/IFileService.cs b/src/Umbraco.Core/Services/IFileService.cs index 74de0861d8..0031562697 100644 --- a/src/Umbraco.Core/Services/IFileService.cs +++ b/src/Umbraco.Core/Services/IFileService.cs @@ -427,5 +427,12 @@ namespace Umbraco.Core.Services /// The filesystem path to the partial view. /// The size of the partial view. long GetPartialViewFileSize(string filepath); + + /// + /// Gets the content of a view. + /// + /// The name of the view. + /// + string GetViewContent(string filename); } -} \ No newline at end of file +} diff --git a/src/Umbraco.Web/Editors/TemplateController.cs b/src/Umbraco.Web/Editors/TemplateController.cs index 78855c95c2..3df6cb2aba 100644 --- a/src/Umbraco.Web/Editors/TemplateController.cs +++ b/src/Umbraco.Web/Editors/TemplateController.cs @@ -1,9 +1,12 @@ using System.Collections.Generic; +using System.IO; using System.Linq; using System.Net; using System.Net.Http; +using System.Web; using System.Web.Http; using AutoMapper; +using Umbraco.Core; using Umbraco.Core.IO; using Umbraco.Core.Models; using Umbraco.Core.Services; @@ -178,6 +181,14 @@ namespace Umbraco.Web.Editors else { //create + + // file might already be on disk, if so grab the content to avoid overwriting + string content = Services.FileService.GetViewContent(display.Alias); + if (string.IsNullOrEmpty(content) == false) + { + display.Content = content; + } + ITemplate master = null; if (string.IsNullOrEmpty(display.MasterTemplateAlias) == false) { From 416cea796e7cd5493c41fcbd929be356ebda4a3b Mon Sep 17 00:00:00 2001 From: Nathan Woulfe Date: Fri, 29 Jun 2018 20:34:04 +1000 Subject: [PATCH 22/47] clean up usings --- src/Umbraco.Web/Editors/TemplateController.cs | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/src/Umbraco.Web/Editors/TemplateController.cs b/src/Umbraco.Web/Editors/TemplateController.cs index 3df6cb2aba..132e804a95 100644 --- a/src/Umbraco.Web/Editors/TemplateController.cs +++ b/src/Umbraco.Web/Editors/TemplateController.cs @@ -1,18 +1,13 @@ -using System.Collections.Generic; -using System.IO; +using AutoMapper; +using System.Collections.Generic; using System.Linq; using System.Net; using System.Net.Http; -using System.Web; using System.Web.Http; -using AutoMapper; -using Umbraco.Core; using Umbraco.Core.IO; using Umbraco.Core.Models; -using Umbraco.Core.Services; using Umbraco.Web.Models.ContentEditing; using Umbraco.Web.Mvc; -using Umbraco.Web.WebApi; using Umbraco.Web.WebApi.Filters; using Constants = Umbraco.Core.Constants; From d3c8b46e0090b7f4e5872c44416b1b617c7d2646 Mon Sep 17 00:00:00 2001 From: Nathan Woulfe Date: Fri, 6 Jul 2018 09:52:43 +1000 Subject: [PATCH 23/47] replace node.published with node.hasPublishedVersion --- .../src/views/components/content/umb-content-node-info.html | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/Umbraco.Web.UI.Client/src/views/components/content/umb-content-node-info.html b/src/Umbraco.Web.UI.Client/src/views/components/content/umb-content-node-info.html index 04a2c710bb..7e2ac85017 100644 --- a/src/Umbraco.Web.UI.Client/src/views/components/content/umb-content-node-info.html +++ b/src/Umbraco.Web.UI.Client/src/views/components/content/umb-content-node-info.html @@ -7,11 +7,11 @@
  • - + {{url}} -
    +
    {{url}}
    From 9d5b089227b611119ff64e5569eac0378d2b190a Mon Sep 17 00:00:00 2001 From: Michael Latouche Date: Mon, 9 Jul 2018 13:05:15 +0200 Subject: [PATCH 24/47] Update fr.xml --- src/Umbraco.Web.UI/umbraco/config/lang/fr.xml | 785 +++++++++++++++--- 1 file changed, 662 insertions(+), 123 deletions(-) diff --git a/src/Umbraco.Web.UI/umbraco/config/lang/fr.xml b/src/Umbraco.Web.UI/umbraco/config/lang/fr.xml index d721a8fc8b..adff0526ca 100644 --- a/src/Umbraco.Web.UI/umbraco/config/lang/fr.xml +++ b/src/Umbraco.Web.UI/umbraco/config/lang/fr.xml @@ -5,18 +5,21 @@ http://our.umbraco.org/documentation/Extending-Umbraco/Language-Files - Gérer les noms d'hôtes + Culture et noms d'hôte Informations d'audit Parcourir Changer le type de document Copier Créer + Exporter + Créer un groupe Créer un package Supprimer Désactiver Vider la corbeille - Exporter un type - Importer un type + Activer + Exporter le type de document + Importer un type de document Importer un package Editer dans Canvas Déconnexion @@ -27,6 +30,7 @@ Dépublier Rafraîchir Republier le site tout entier + Renommer Récupérer Spécifiez les permissions pour la page %0% Choisissez où déplacer @@ -35,10 +39,40 @@ Version antérieure Envoyer pour publication Envoyer pour traduction + Spécifier le groupe Trier Traduire Mettre à jour + Spécifier les permissions + Débloquer + Créer un modèle de contenu + + Contenu + Administration + Structure + Autre + + + Permettre d'attribuer la culture et des noms d'hôte + Permettre d'accéder au journal d'historique d'un noeud + Permettre d'accéder à un noeud + Permettre de modifier le type de document d'un noeud + Permettre de copier un noeud + Permettre de créer des noeuds + Permettre de supprimer des noeuds + Permettre de déplacemer un noeud + Permettre de définir et modifier l'accès public à un noeud + Permettre de publier un noeud + Permettre de modifier les permissions pour un noeud + Permettre de revenir à une situation antérieure + Permettre d'envoyer un noeud pour approbation avant publication + Permettre d'envoyer un noeud à la traduction + Permettre de modifier l'ordonnancement des noeuds + Permettre de traduire un noeud + Permettre de sauvegarder un noeud + Permettre la création d'un Modèle de Contenu + Permission refusée. Ajouter un nouveau domaine @@ -53,13 +87,17 @@ Domaine '%0%' déjà assigné Domaine '%0%' mis à jour Editer les domaines actuels - +
    Les domaines contenant un chemin d'un niveau sont autorisés, ex : "example.com/en". Pour autant, cela - devrait être évité. Utilisez plutôt la gestion des noms d'hôte.]]>     + devrait être évité. Utilisez plutôt la gestion de la culture et des noms d'hôte.]]> + Hériter Culture - ou hériter de la culture des noeuds parents. S'appliquera aussi
    - au noeud courant, à moins qu'un domaine ci-dessous soit aussi d'application.]]>     + + ou hériter de la culture des noeuds parents. S'appliquera aussi
    + au noeud courant, à moins qu'un domaine ci-dessous soit aussi d'application.]]> +     Domaines @@ -90,7 +128,8 @@ Retourner à la liste Sauver Sauver et publier - Sauver et envoyer pour approbation + Sauver et planifier + Sauver et envoyer pour approbation Sauver la mise en page de la liste Prévisualiser La prévisualisation est désactivée car aucun modèle n'a été assigné. @@ -98,6 +137,7 @@ Afficher les styles Insérer un tableau Générer les modèles + Sauver et générer les modèles Défaire Refaire @@ -141,30 +181,38 @@ Dernière publication Il n'y a aucun élément à afficher Il n'y a aucun élément à afficher dans cette liste. + Aucun contenu n'a encore été ajouté + Aucun membre n'a encore été ajouté Type de Média Lien vers des média(s) Groupe de membres Rôle Type de membre + Aucune modification n'a été faite Aucune date choisie Titre de la page + Ce média n'a pas de lien Propriétés Ce document est publié mais n'est pas visible car son parent '%0%' n'est pas publié Oups : ce document est publié mais n'est pas présent dans le cache (erreur interne Umbraco) Oups: impossible d'obtenir cet url (erreur interne - voir fichier log) Oups: ce document est publié mais son url entrerait en collision avec le contenu %0% Publier + Publié + Publié(changements en cours) Statut de publication Publié le Dépublié le Supprimer la date - Ordre de tri mis à jour + Défininir la date + Ordre de tri mis à jour Pour trier les noeuds, faites-les simplement glisser à l'aide de la souris ou cliquez sur les entêtes de colonne. Vous pouvez séléctionner plusieurs noeuds en gardant la touche "shift" ou "ctrl" enfoncée pendant votre séléction. Statistiques Titre (optionnel) Texte alternatif (optionnel) Type Dépublier + Dépublié Dernière édition Date/heure à laquelle ce document a été édité Supprimer le(s) fichier(s) @@ -175,17 +223,33 @@ Cible Ceci se traduit par l'heure suivante sur le serveur : Qu'est-ce que cela signifie?]]> + Etes-vous certain(e) de vouloir supprimer cet éléménent? + La propriété %0% utilise l'éditeur %1% qui n'est pas supporté par Nested Content. Ajouter un autre champ texte Enlever ce champ texte + Racine du contenu + Cette valeur est masquée. Si vous avez besoin de pouvoir accéder à cette valeur, veuillez prendre contact avec l'administrateur du site web. + Cette valeur est masquée. + + Créer un nouveau Modèle de Contenu à partir de '%0%' + Vide + Sélectionner un Modèle de Contenu + Modèle de Contenu créé + Un modèle de Contenu a été créé à partir de '%0%' + Un autre Modèle de Contenu existe déjà avec le même nom + Un Modèle de Contenu est du contenu pré-défini qu'un éditeur peut sélectionner et utiliser comme base pour la création de nouveau contenu + Cliquez pour télécharger Faites glisser vos fichier ici... Lien vers le média ou cliquez ici pour choisir un fichier Les seuls types de fichiers autorisés sont + Ce fichier ne peut pas ête chargé, il n'est pas un type de fichier autorisé. La taille maximum de fichier est - + Racine du média + Créer un nouveau membre Tous les membres @@ -193,6 +257,7 @@ Où voulez-vous créer le nouveau %0% Créer un élément sous + Sélectionnez le type de document pour lequel vous voulez créer un modèle de contenu Choisissez un type et un titre "Types de documents".]]> "Types de médias".]]> @@ -280,7 +345,10 @@ Cette macro ne contient aucune propriété éditable Coller Editer les permissions pour - Les éléments dans la corbeille sont en cours de suppression. Ne fermez pas cette fenêtre avant que cette opération soit terminée. + Définir les permissions pour + Définir les permissions pour %0% pour le groupe d'utilisateurs %1% + Sélectionnez les groupes d'utilisateurs pour lesquels vous souhaitez définir les permissions + Les éléments dans la corbeille sont en cours de suppression. Ne fermez pas cette fenêtre avant que cette opération soit terminée. La corbeille est maintenant vide Les éléments supprimés de la corbeille seront supprimés définitivement regexlib.com rencontre actuellement des problèmes sur lesquels nous n'avons aucun contrôle. Nous sommes sincèrement désolés pour le désagrément.]]> @@ -292,10 +360,14 @@ Le cache du site va être mis à jour. Tous les contenus publiés seront mis à jour. Et tous les contenus dépubliés resteront invisibles. Nombre de colonnes Nombre de lignes - Définir un placeholder ID. En mettant un ID sur votre placeholder, vous pouvez injecter du contenu à cet endroit depuis les modèles enfants, - en faisant référence à cet ID au sein d'un élément <asp:content />.]]> - Séléctionnez un placeholder id dans la liste ci-dessous. Vous pouvez seulement - choisir un ID se trouvant dans le parent du modèle actuel.]]> + + Définir un placeholder ID. En mettant un ID sur votre placeholder, vous pouvez injecter du contenu à cet endroit depuis les modèles enfants, + en faisant référence à cet ID au sein d'un élément <asp:content />.]]> + + + Séléctionnez un placeholder id dans la liste ci-dessous. Vous pouvez seulement + choisir un ID se trouvant dans le parent du modèle actuel.]]> + Cliquez sur l'image pour la voir en taille réelle Sélectionner un élément Voir l'élément de cache @@ -307,14 +379,19 @@ Ouvre le document lié dans une nouvelle fenêtre ou un nouvel onglet Lier à un media Lier à un fichier + Sélectionner le noeud de base du contenu Sélectionner le media Sélectionner l'icône Sélectionner l'élément Sélectionner le lien Sélectionner la macro Sélectionner le contenu + Sélectionner le noeud de base des media Sélectionner le membre Sélectionner le groupe de membres + Sélectionner le noeud + Sélectionner les sections + Sélectionner les utilisateurs Aucune icone n'a été trouvée Il n'y a pas de paramètres pour cette macro Il n'y a pas de macro disponible à insérer @@ -329,9 +406,11 @@ Selectionner un snippet - + %0%' ci-dessous.
    Vous pouvez ajouter d'autres langues depuis le menu ci-dessous "Langues". - ]]>     + ]]> + Nom de Culture Modifiez la clé de l'élément de dictionaire. @@ -346,13 +425,16 @@ Confirmation de votre mot de passe Nommer %0%... Entrez un nom... + Entrez un email... + Entrez un nom d'utilisateur... Libellé... Entrez une description... Rechercher... Filtrer... Ajouter des tags (appuyer sur enter entre chaque tag)... Entrez votre email - Votre nom d'utilisateur est généralement votre adresse email + Entrez un message... + Votre nom d'utilisateur est généralement votre adresse email Autoriser à la racine @@ -372,6 +454,11 @@ Créer une liste personnalisée Supprimer la liste personnalisée + + Renommé + Entrez un nouveau nom de répertoire ici + %0% a été renommé en %1% + Ajouter une valeur de base Type de donnée en base de donées @@ -432,6 +519,7 @@ Il y a une erreur de configuration du type de données utilisé pour cette propriété, veuillez vérifier le type de données. + Options A propos Action Actions @@ -472,18 +560,25 @@ Erreur Trouver Premier + Général + Groupes Hauteur Aide + Cacher + Historique Icône Importer + Info Marge intérieure Insérer Installer Non valide Justifier + Libellé Langue Dernier Mise en page + Liens En cours de chargement Bloqué Connexion @@ -491,6 +586,7 @@ Déconnexion Macro Obligatoire + Message Déplacer Plus Nom @@ -498,9 +594,12 @@ Suivant Non de + Off OK Ouvrir + On ou + Trier par Mot de passe Chemin Placeholder ID @@ -509,13 +608,16 @@ Propriétés Email de réception des données de formulaire Corbeille + Votre corbeille est vide Restant Enlever Renommer Renouveller Requis + Retrouver Réessayer Permissions + Publication Programmée Rechercher Désolé, nous ne pouvons pas trouver ce que vous recherchez Aucun élément n'a été ajouté @@ -524,6 +626,7 @@ Afficher la page à l'envoi Taille Trier + Statut Envoyer Type Rechercher... @@ -550,6 +653,7 @@ Sauvegarde... actuel Intégrer + Retrouver sélectionné @@ -559,7 +663,20 @@ Jaune Orange Bleu + Bleu-gris + Gris + Brun + Bleu Clair + Cyan + Vert Clair + Limon + Ambre + Orange Foncé Rouge + Rose + Mauve + Mauve Foncé + Indigo @@ -590,7 +707,6 @@ Général Editeur - Couleur de fond Gras @@ -598,7 +714,6 @@ Police Texte - Page @@ -607,28 +722,38 @@ Impossible de sauvegarder le fichier web.config. Veuillez modifier la "connection string" manuellement. Votre base de données a été détectée et est identifiée comme étant Configuration de la base de données - + installer pour installer la base de données Umbraco %0% - ]]> + ]]> + Suivant pour poursuivre.]]> - Base de données non trouvée ! Veuillez vérifier les informations de la "connection string" dans le fichier web.config.

    + + Base de données non trouvée ! Veuillez vérifier les informations de la "connection string" dans le fichier web.config.

    Pour poursuivre, veuillez éditer le fichier "web.config" (avec Visual Studio ou votre éditeur de texte favori), scroller jusqu'en bas, ajouter le "connection string" pour votre base de données dans la ligne avec la clé "umbracoDbDSN" et sauvegarder le fichier.

    Cliquez sur le bouton Réessayer lorsque cela est fait.
    - Plus d'informations sur l'édition du fichier web.config ici.

    ]]>     - + Plus d'informations sur l'édition du fichier web.config ici.

    ]]> +     + + Veuillez contacter votre fournisseur de services internet si nécessaire. - Si vous installez Umbraco sur un ordinateur ou un serveur local, vous aurez peut-être besoin de consulter votre administrateur système.]]> - + + + Appuyez sur le bouton Upgrader pour mettre à jour votre base de données vers Umbraco %0%

    N'ayez pas d'inquiétude : aucun contenu ne sera supprimé et tout continuera à fonctionner parfaitement par après !

    - ]]>     - Appuyez sur Suivant pour - poursuivre. ]]> + ]]> +     + + Appuyez sur Suivant pour + poursuivre. ]]> + Suivant pour poursuivre la configuration]]> Le mot de passe par défaut doit être modifié !]]> L'utilisateur par défaut a été désactivé ou n'a pas accès à Umbraco!

    Aucune autre action n'est requise. Cliquez sur Suivant pour poursuivre.]]> @@ -640,41 +765,56 @@ Fichiers et dossiers concernés Plus d'informations sur la configuration des permissions Vous devez donner à ASP.NET les droits de modification sur les fichiers/dossiers suivants - Vos configurations de permissions sont presque parfaites !

    - Vous pouvez faire fonctionner Umbraco sans problèmes, mais vous ne serez pas en mesure d'installer des packages, ce qui est hautement recommandé pour tirer pleinement profit d'Umbraco.]]>     + + Vos configurations de permissions sont presque parfaites !

    + Vous pouvez faire fonctionner Umbraco sans problèmes, mais vous ne serez pas en mesure d'installer des packages, ce qui est hautement recommandé pour tirer pleinement profit d'Umbraco.]]> +     Comment résoudre Cliquez ici pour lire la version texte tutoriel vidéo sur la définition des permissions des répertoires pour Umbraco, ou lisez la version texte.]]> - Vos configurations de permissions pourraient poser problème ! + + Vos configurations de permissions pourraient poser problème !

    - Vous pouvez faire fonctionner Umbraco sans problèmes, mais vous ne serez pas en mesure d'installer des packages, ce qui est hautement recommandé pour tirer pleinement profit d'Umbraco.]]>     - Vos configurations de permissions ne sont pas prêtes pour Umbraco ! + Vous pouvez faire fonctionner Umbraco sans problèmes, mais vous ne serez pas en mesure d'installer des packages, ce qui est hautement recommandé pour tirer pleinement profit d'Umbraco.]]> + + + Vos configurations de permissions ne sont pas prêtes pour Umbraco !

    - Pour faire fonctionner Umbraco, vous aurez besoin de mettre à jour les permissions sur les fichiers/dossiers.]]>     - Vos configurations de permissions sont parfaites !

    - Vous êtes prêt(e) à faire fonctionner Umbraco et à installer des packages !]]>     + Pour faire fonctionner Umbraco, vous aurez besoin de mettre à jour les permissions sur les fichiers/dossiers.]]> +     + + Vos configurations de permissions sont parfaites !

    + Vous êtes prêt(e) à faire fonctionner Umbraco et à installer des packages !]]> +     Résoudre un problème sur un dossier Suivez ce lien pour plus d'informations sur les problèmes avec ASP.NET et la création de dossiers Définir les permissions de dossier - + + ]]> + Je veux démarrer "from scratch" - + Apprenez comment) Vous pouvez toujours choisir d'installer Runway plus tard. Pour cela, allez dans la section "Développeur" et sélectionnez "Packages". - ]]> + ]]> + Vous venez de mettre en place une plateforme Umbraco toute nette. Que voulez-vous faire ensuite ? Runway est installé - + Voici la liste des modules recommandés, cochez ceux que vous souhaitez installer, ou regardez la liste complète des modules - ]]> + ]]> + Recommandé uniquement pour les utilisateurs expérimentés Je veux commencer avec un site simple - + "Runway" est un site simple qui fournit des types de documents et des modèles de base. L'installateur peut mettre en place Runway automatiquement pour vous, mais vous pouvez facilement l'éditer, l'enrichir, ou le supprimer par la suite. Il n'est pas nécessaire, et vous pouvez parfaitement vous en passer pour utiliser Umbraco. Cela étant dit, @@ -685,7 +825,8 @@ Inclus avec Runway : Home page, Getting Started page, Installing Modules page.
    Modules optionnels : Top Navigation, Sitemap, Contact, Gallery. - ]]>     + ]]> + Qu'est-ce que Runway Etape 1/5 : Accepter la licence Etape 2/5 : Configuration de la base de données @@ -693,24 +834,36 @@ Etape 4/5 : Sécurité Umbraco Etape 5/5 : Umbraco est prêt Merci d'avoir choisi Umbraco - Parcourir votre nouveau site -Vous avez installé Runway, alors pourquoi ne pas jeter un oeil au look de votre nouveau site ?]]> - Aide et informations complémentaires -Obtenez de l'aide de notre "award winning" communauté, parcourez la documentation ou regardez quelques vidéos gratuites sur la manière de construire un site simple, d'utiliser les packages ainsi qu'un guide rapide sur la terminologie Umbraco]]> + + Parcourir votre nouveau site +Vous avez installé Runway, alors pourquoi ne pas jeter un oeil au look de votre nouveau site ?]]> + + + Aide et informations complémentaires +Obtenez de l'aide de notre "award winning" communauté, parcourez la documentation ou regardez quelques vidéos gratuites sur la manière de construire un site simple, d'utiliser les packages ainsi qu'un guide rapide sur la terminologie Umbraco]]> + Umbraco %0% est installé et prêt à être utilisé - fichier /web.config et mettre à jour le paramètre AppSetting umbracoConfigurationStatus situé en bas à la valeur '%0%'.]]> - démarrer instantanément en cliquant sur le bouton "Lancer Umbraco" ci-dessous.
    -Si vous débutez avec Umbraco, vous pouvez trouver une foule de ressources dans nos pages "Getting Started".]]>     - Lancer Umbraco -Pour gérer votre site, ouvrez simplement le backoffice Umbraco et commencez à ajouter du contenu, à mettre à jour les modèles d'affichage et feuilles de styles ou à ajouter de nouvelles fonctionnalités]]> + + fichier /web.config et mettre à jour le paramètre AppSetting umbracoConfigurationStatus situé en bas à la valeur '%0%'.]]> + + + démarrer instantanément en cliquant sur le bouton "Lancer Umbraco" ci-dessous.
    +Si vous débutez avec Umbraco, vous pouvez trouver une foule de ressources dans nos pages "Getting Started".]]> +     + + Lancer Umbraco +Pour gérer votre site, ouvrez simplement le backoffice Umbraco et commencez à ajouter du contenu, à mettre à jour les modèles d'affichage et feuilles de styles ou à ajouter de nouvelles fonctionnalités]]> + La connexion à la base de données a échoué. Umbraco Version 3 Umbraco Version 4 Regarder - Umbraco %0%, qu'il s'agisse d'une nouvelle installation ou d'une mise à jour à partir de la version 3.0 + + Umbraco %0%, qu'il s'agisse d'une nouvelle installation ou d'une mise à jour à partir de la version 3.0

    - Appuyez sur "suivant" pour commencer l'assistant.]]>     + Appuyez sur "suivant" pour commencer l'assistant.]]> +     Code de la Culture @@ -721,13 +874,13 @@ Pour gérer votre site, ouvrez simplement le backoffice Umbraco et commencez à Renouvellez votre session maintenant pour sauvegarder votre travail - Joyeux dimanche - Joyeux lundi - Joyeux mardi - Joyeux mercredi + Joyeux dimanche détonnant + Joyeux lundi lumineux + Joyeux mardi magique + Joyeux mercredi merveilleux Joyeux jeudi Joyeux vendredi - Joyeux samedi + Joyeux chamedi Connectez-vous ci-dessous Identifiez-vous avec La session a expiré @@ -741,7 +894,87 @@ Pour gérer votre site, ouvrez simplement le backoffice Umbraco et commencez à Le lien sur lequel vous avez cliqué est non valide ou a expiré. Umbraco: Ré-initialiser le mot de passe - Votre nom d'utilisateur pour vous connecter au back-office Umbraco est : %0%.

    Cliquez ici pour ré-initialiser votre mot de passe, ou recopiez cet URL dans votre navigateur :

    %1%

    ]]> + + + + + + + + + + + +
    + + + + + +
    + +
    + +
    +
    + + + + + + +
    +
    +
    + + + + +
    + + + + +
    +

    + Une réinitialisation de votre mot de passe a été demandée +

    +

    + Votre nom d'utilisateur pour vous connecter au back-office Umbraco est : %0% +

    +

    + + + + + + +
    + + Cliquez sur ce lien pour réinitialiser votre mot de passe + +
    +

    +

    Si vous ne pouvez pas cliquer sur le lien, recopiez cet URL dans votre navigateur :

    + + + + +
    + + %1% + +
    +

    +
    +
    +


    +
    +
    + + + ]]>     @@ -765,7 +998,8 @@ Pour gérer votre site, ouvrez simplement le backoffice Umbraco et commencez à Editez vos notifications pour %0% - + - Hello %0%

    - -

    Ceci est un email automatique pour vous informer que la tâche '%1%' - a été executée sur la page '%2%' - par l'utilisateur '%3%' -

    - -

    -

    Résumé de la mise à jour :

    - - %6% + ]]> + + + + + + + + + +
    + + + +
    + + + + + +
    + +
    + +
    +
    + + + + + +
    +
    +
    + + + + +
    + + + + +
    +

    + Salut %0%, +

    +

    + Ceci est un email automatique pour vous informer que la tâche '%1%' a été executée sur la page '%2%' par l'utilisateur '%3%' +

    + + + + + + +
    + +
    + MODIFIER
    +
    +

    +

    Résumé de la mise à jour :

    + + %6% +
    +

    +

    + Bonne journée !

    + Avec les salutations du Robot Umbraco +

    +
    +
    +


    +
    +
    -

    - - - -

    Bonne journée !

    - Avec les salutations du Robot Umbraco -

    ]]>     + + + ]]> + La notification [%0%] à propos de %1% a été executée sur %2% Notifications - + et localisez le package. Les packages Umbraco ont généralement une extension ".umb" ou ".zip". - ]]> + ]]> + Déposez pour uploader ou cliquez ici pour choisir les fichiers Uploader un package @@ -852,8 +1142,10 @@ Pour gérer votre site, ouvrez simplement le backoffice Umbraco et commencez à Meta data du package Nom du package Le package ne contient aucun élément -
    - Vous pouvez supprimer tranquillement ce package de votre installation en cliquant sur "Désinstaller le package" ci-dessous.]]>     + +
    + Vous pouvez supprimer tranquillement ce package de votre installation en cliquant sur "Désinstaller le package" ci-dessous.]]> +     Aucune mise à jour disponible Options du package Package readme @@ -862,9 +1154,11 @@ Pour gérer votre site, ouvrez simplement le backoffice Umbraco et commencez à Le package a été désinstallé Le package a été désinstallé avec succès Désinstaller le package - + + Remarque : tous les documents, media etc. dépendant des éléments que vous supprimez vont cesser de fonctionner, ce qui peut provoquer une instabilité du système, - désinstallez donc avec prudence. En cas de doute, contactez l'auteur du package.]]> + désinstallez donc avec prudence. En cas de doute, contactez l'auteur du package.]]> + Télécharger la mise à jour depuis le repository Mettre à jour le package Instructions de mise à jour @@ -913,32 +1207,51 @@ Pour gérer votre site, ouvrez simplement le backoffice Umbraco et commencez à %0% n'a pas pu être publié car cet élément est programmé pour être publié bientôt. ]]> - + - + + + - + + + - + + + + ]]> + Inclure les pages enfant non publiées Publication en cours - veuillez patienter... %0% pages sur %1% ont été publiées... %0% a été publié %0% et ses pages enfants ont été publiées Publier %0% et toutes ses pages enfant - Publier pour publier %0% et la rendre ainsi accessible publiquement.

    + + Publier pour publier %0% et la rendre ainsi accessible publiquement.

    Vous pouvez publier cette page et toutes ses sous-pages en cochant Inclure les pages enfant non pubiées ci-dessous. - ]]>     + ]]> +     Vous n'avez configuré aucune couleur approuvée - + + Vous avez choisi un élément de contenu actuellement supprimé ou dans la corbeille + Vous avez choisi des éléments de contenu actuellement supprimés ou dans la corbeille + + + Vous avez choisi un élément media actuellement supprimé ou dans la corbeille + Vous avez choisi des éléments media actuellement supprimés ou dans la corbeille + Elément supprimé + + introduire un lien externe choisir une page interne Légende @@ -1021,6 +1334,7 @@ Pour gérer votre site, ouvrez simplement le backoffice Umbraco et commencez à Validation Les erreurs de validation doivent être corrigées avant de pouvoir sauvegarder l'élément Echec + Sauvegardé Permissions utilisateur insuffisantes, l'opération n'a pas pu être complétée Annulation L'opération a été annulée par une extension tierce @@ -1054,6 +1368,7 @@ Pour gérer votre site, ouvrez simplement le backoffice Umbraco et commencez à Erreur lors de la sauvegarde de l'utilisateur (consultez les logs) Utilisateur sauvegardé Type d'utilisateur sauvegardé + Groupe d'utilisateurs sauvegardé Fichier non sauvegardé Le fichier n'a pas pu être sauvegardé. Vérifiez les permissions de fichier. Fichier sauvegardé @@ -1079,11 +1394,31 @@ Pour gérer votre site, ouvrez simplement le backoffice Umbraco et commencez à Vue partielle sauvegardée sans erreurs ! Vue partielle non sauvegardée Une erreur est survenue lors de la sauvegarde du fichier. + Permissions sauvegardées pour Vue script sauvegardée Vue script sauvegardée sans erreur ! Vue script non sauvegardée Une erreur est survenue lors de la sauvegarde du fichier. Une erreur est survenue lors de la sauvegarde du fichier. + %0% groupes d'utilisateurs supprimés + %0% a été supprimé + %0% utilisateurs activés + Une erreur est survenue lors de l'activation des utilisateurs + %0% utilisateurs désactivés + Une erreur est survenue lors de la désactivation des utilisateurs + %0% est à présent activé + Une erreur est survenue lors de l'activation de l'utilisateur + %0% est à présent désactivé + Une erreur est survenue lors de la désactivation de l'utilisateur + Les groupes d'utilisateurs ont été définis + %0% groupes d'utilisateurs supprimés + %0% a été supprimé + %0% utilisateurs débloqués + Une erreur est survenue lors du débloquage des utilisateurs + %0% est à présent débloqué + Une erreur est survenue lors du débloquage de l'utilisateur + Le membre a été exporté vers le fichier + Une erreur est survenue lors de l'export du membre Utilise la syntaxe CSS. Ex : h1, .redHeader, .blueTex @@ -1201,7 +1536,7 @@ Pour gérer votre site, ouvrez simplement le backoffice Umbraco et commencez à - Rich Text Editor + Rich Text Editor Image Macro Embed @@ -1244,6 +1579,8 @@ Pour gérer votre site, ouvrez simplement le backoffice Umbraco et commencez à Autoriser tous les éditeurs Autoriser toutes les configurations de rangées + Eléments maximum + Laisser vide ou mettre à 0 pour un nombre illimté Configurer comme défaut Choisir en plus Choisir le défaut @@ -1308,11 +1645,27 @@ Pour gérer votre site, ouvrez simplement le backoffice Umbraco et commencez à utilisant cet éditeur seront mis à jour avec la nouvelle configuration Le membre peut éditer - Afficher dans le profil du membre - + Autoriser la modification de la valeur de cette propriété par le membre à partir de sa page de profil + Est une donnée sensible + Cacher cette propriété aux éditeurs de contenu qui n'ont pas accès à la visualisation des données sensibles + Afficher dans le profil du membre + Permettre d'afficher la valeur de cette propriété sur la page de profil du membre + + l'onglet n'a pas d'ordre de tri + + Où cette composition est-elle utilisée? + Cette composition est actuellement utilisée dans la composition des types de contenu suivants : - + + Fabrication des modèles + ceci peut prendre un peu de temps, ne vous inquiétez pas + Modèles générés + Les modèles n'ont pas pu être générés + La génération des modèles a échoué, voyez les exceptions dans les U log + + + Ajouter un champ de rechange Champ de rechange Ajouter une valeur par défaut @@ -1353,10 +1706,12 @@ Pour gérer votre site, ouvrez simplement le backoffice Umbraco et commencez à Tâches qui vous sont assignées - vous sont assignées. Pour voir un aperçu détaillé incluant les commentaires, cliquez sur "Détails" ou juste sur le nom de la page. + + vous sont assignées. Pour voir un aperçu détaillé incluant les commentaires, cliquez sur "Détails" ou juste sur le nom de la page. Vous pouvez aussi télécharger la page au format XML en cliquant sur le lien "Télécharger XML".
    Pour clôturer une tâche de traduction, allez sur Détails, puis cliquez sur le bouton "Terminer la tâche". - ]]>     + ]]> +     Terminer la tâche Détails Télécharger toutes les tâches de traductions au format XML @@ -1364,7 +1719,8 @@ Pour gérer votre site, ouvrez simplement le backoffice Umbraco et commencez à Télécharger la DTD XML Champs Inclure les pages enfants - + + ]]> + [%0%] tâches de traductions pour %1% Aucun utilisateur traducteur trouvé. Veuillez créer un utilisateur traducteur avant d'envoyer du contenu pour traduction Tâches que vous avez créées - que vous avez créées. Pour voir un aperçu détaillé incluant les commentaires, + + que vous avez créées. Pour voir un aperçu détaillé incluant les commentaires, cliquez sur "Détails" ou juste sur le nom de la page. Vous pouvez aussi télécharger la page au format XML en cliquant sur le lien "Télécharger XML". Pour clôturer une tâche de traduction, allez sur Détails, puis cliquez sur le bouton "Terminer tâche". - ]]> + ]]> + La page '%0%' a été envoyée pour traduction Veuillez choisir la langue dans laquelle le contenu doit être traduit Envoyer la page '%0%' pour traduction @@ -1401,6 +1760,9 @@ Pour gérer votre site, ouvrez simplement le backoffice Umbraco et commencez à Uploader le fichier de traduction XML + Contenu + Types de contenu + Media Navigateur de cache Corbeille Packages créés @@ -1421,6 +1783,8 @@ Pour gérer votre site, ouvrez simplement le backoffice Umbraco et commencez à Types de relations Packages Packages + Vues Partielles + Vues Partielles pour les Fichiers Macro Fichiers Python Installer depuis le repository Installer Runway @@ -1431,8 +1795,7 @@ Pour gérer votre site, ouvrez simplement le backoffice Umbraco et commencez à Modèles Fichiers XSLT Analytique - Vues partielles - Fichiers Macro pour les vues partielles + Utilisateurs Nouvelle mise à jour disponible @@ -1441,23 +1804,45 @@ Pour gérer votre site, ouvrez simplement le backoffice Umbraco et commencez à Erreur lors de la recherche de mises à jour. Veuillez vérifier le stack trace pour obtenir plus d'informations sur l'erreur. + Accès + Sur base des groupes et noeuds de départ, l'utilisateur a accès aux noeuds suivants + Donner accès Administrateur Champ catégorie + Utilisateur créé Changer le mot de passe + Changer la photo Nouveau mot de passe + n'a pas été bloqué + Le mot de passe n'a pas été modifié Confirmez votre nouveau mot de passe Vous pouvez changer votre mot de passe d'accès au Back Office Umbraco en remplissant le formulaire ci-dessous puis en cliquant sur le bouton "Changer le mot de passe" Canal de contenu + Créer un autre utilisateur + Créer de nouveaux utilisateurs pour leur donner accès à Umbraco. Lors de la création d'un nouvel utilisateur, un mot de passe est généré que vous pouvez partager avec l'utilisateur. Champ description Désactiver l'utilisateur Type de document Editeur Champ extrait + Tentatives de connexion échouées + Voir le profil de l'utilisateur + Ajouter des groupes pour donner les accès et permissions + Inviter un autre utilisateur + Inviter de nouveaux utilisateurs pour leur donner accès à Umbraco. Un email d'invitation sera envoyé à chaque utilisateur avec des informations concernant la connexion à Umbraco. Langue + Spécifiez la langue dans laquelle vous souhaitez voir les menus et dialogues + Date du dernier bloquage + Dernière connexion + Dernière modification du mot de passe Identifiant Noeud de départ dans la librarie de média + Limiter la librairie média à un noeud de départ spécifique + Noeuds de départ dans la librairie de média + Limiter la librairie média à des noeuds de départ spécifique Sections Désactiver l'accès Umbraco + ne s'est pas encore connecté Ancien mot de passe Mot de passe Réinitialiser le mot de passe @@ -1472,17 +1857,150 @@ Pour gérer votre site, ouvrez simplement le backoffice Umbraco et commencez à Remplacer les permissions sur les noeuds enfants Vous êtes en train de modifiez les permissions pour les pages : Choisissez les pages dont les permissions doivent être modifiées + Supprimer la photo + Permissions par défaut + Permissions granulaires + Définir les permissions sur des noeuds spécifiques + Profil Rechercher tous les enfants + Ajouter les sections auxquelles les utilisateur peuvent accéder + Sélectionner les groupes d'utilisateurs + Aucun noeud de départ sélectionné + Aucun noeud de départ sélectionné + Actif + Tous + Désactivé + Bloqué + Invité Noeud de départ du contenu + Limiter l'arborescence de contenu à un noeud de départ spécifique + Noeuds de départ du contenu + Limiter l'arborescence de contenu à des noeuds de départ spécifiques + Nom (A-Z) + Nom (Z-A) + Plus récent + Plus ancien + Dernière connexion + Dernière mise à jour de l'utilisateur + a été créé + Le nouvel utilisateur a été créé avec succès. Utilisez le mot de passe ci-dessous pour la connexion à Umbraco. + Gestion des utilisateurs Nom d'utilisateur - Permissions utilisateur + Permissions de l'utilisateur + Permissions du groupe d'utilisateurs + Groupe d'utilisateurs + Groupes d'utilisateurs + a été invité + Une invitation a été envoyée au nouvel utilisateur avec les détails concernant la oonnexion à Umbraco. + Bien le bonjour et bienvenue dans Umbraco! Vous serez prêt.e dans moins d'1 minute, vous devez encore simplement configurer votre mot de passe et ajouter une photo pour votre avatar. + Chargez une photo afin que les autres utilisateurs puissent vous reconnaître facilement. Rédacteur Traducteur Modifier Votre profil Votre historique récent La session expire dans + Inviter un utilisateur + Créer un utilisateur + Envoyer l'invitation + Retour aux utilisateurs + Umbraco: Invitation + + + + + + + + + + + + +
    + + + + + +
    + +
    + +
    +
    + + + + + + +
    +
    +
    + + + + +
    + + + + +
    +

    + Salut %0%, +

    +

    + Vous avez été invité.e par %1% à accéder au Umbraco Back Office. +

    +

    + Message de %1%: +
    + %2% +

    + + + + + + +
    + + + + + + +
    + + Cliquez sur ce lien pour accepter l'invitation + +
    +
    +

    Si vous ne pouvez pas cliquer sur le lien, copiez cet URL dans votre navigateur :

    + + + + +
    + + %3% + +
    +

    +
    +
    +


    +
    +
    + + ]]> +     + Inviter + Validation Valider comme email @@ -1543,6 +2061,8 @@ Pour gérer votre site, ouvrez simplement le backoffice Umbraco et commencez à Le certificat de votre site a été marqué comme valide. Erreur de validation du certificat : '%0%' + Le certificat SSL de votre site web a expiré. + Le certificat SSL de votre site web va expirer dans %0% jours. Erreur en essayant de contacter l'URL %0% - '%1%' Vous êtes actuellement %0% à voir le site via le schéma HTTPS. La valeur appSetting 'umbracoUseSSL' est fixée à 'false' dans votre fichier web.config. Une fois que vous donnerez accès à ce site en utilisant le schéma HTTPS, cette valeur devra être mise à 'true'. @@ -1588,6 +2108,21 @@ Pour gérer votre site, ouvrez simplement le backoffice Umbraco et commencez à Une configuration générant un header qui empêche l'intégration du site par d'autres sites via IFRAME a été ajoutée à votre fichier web.config. Impossible de modifier le fichier web.config. Erreur : %0% + X-Content-Type-Options utilisé pour la protection contre les vulnérabilités de MIME sniffing a été trouvé.]]> + X-Content-Type-Options utilisé pour la protection contre les vulnérabilités de MIME sniffing n'a pas été trouvé.]]> + Ajoute une valeur dans la section httpProtocol/customHeaders du fichier web.config pour protéger contre les vulnérabilités de MIME sniffing. + Une configuration a été ajoutée à votre fichier web.config pour créer un header protégeant contre les vulnérabilités de MIME sniffing. + + Strict-Transport-Security, aussi connu sous le nom de HSTS-header, a été trouvé.]]> + Strict-Transport-Security, aussi connu sous le nom de HSTS-header, n'a pas été trouvé.]]> + Ajoute l'en-tête 'Strict-Transport-Security' avec la valeur 'max-age=10886400; preload' à la section httpProtocol/customHeaders du fichier web.config. Utilisez cette correction uniquement si vos domaines vont fonctionner en https pour les 18 prochaines semaines (minimum). + L'en-tête HSTS a été ajouté à votre fichier web.config. + + X-XSS-Protection a été trouvé.]]> + X-XSS-Protection n'a pas été trouvé.]]> + Ajoute l'en-tête 'X-XSS-Protection' avec la valeur '1; mode=block' à la section httpProtocol/customHeaders du fichier web.config. + L'en-tête X-XSS-Protection a été ajouté à votre fichier web.config. + @@ -1601,6 +2136,8 @@ Pour gérer votre site, ouvrez simplement le backoffice Umbraco et commencez à %0%.]]> %0%.]]> +

    Les résultats de l'exécution du Umbraco Health Checks planifiée le %0% à %1% sont les suivants :

    %2%]]>     + Statut du Umbraco Health Check Désactiver URL tracker @@ -1622,5 +2159,7 @@ Pour gérer votre site, ouvrez simplement le backoffice Umbraco et commencez à Pas d'élément de dictionaire à choisir + + caractères restant + - From ca13782e2aeb2145f9b9f9bee1dd02a33b92de7c Mon Sep 17 00:00:00 2001 From: Michael Latouche Date: Tue, 10 Jul 2018 13:57:40 +0200 Subject: [PATCH 25/47] Update fr.xml --- src/Umbraco.Web.UI/umbraco/config/lang/fr.xml | 47 ++++++++++++++++--- 1 file changed, 40 insertions(+), 7 deletions(-) diff --git a/src/Umbraco.Web.UI/umbraco/config/lang/fr.xml b/src/Umbraco.Web.UI/umbraco/config/lang/fr.xml index adff0526ca..afc06213f3 100644 --- a/src/Umbraco.Web.UI/umbraco/config/lang/fr.xml +++ b/src/Umbraco.Web.UI/umbraco/config/lang/fr.xml @@ -46,6 +46,7 @@ Spécifier les permissions Débloquer Créer un modèle de contenu + Envoyer à nouveau l'invitation Contenu @@ -100,9 +101,6 @@ Domaines - - Aperçu pour - Vider la sélection Choisir @@ -141,6 +139,27 @@ Défaire Refaire + + Aperçu pour + Suppressions de contenu réalisées par utilisateur + Dépublications réalisées par utilisateur + Sauvegardes et publications réalisées par utilisateur + Sauvegardes de contenu réalisées par utilisateur + Déplacements de contenu réalisés par utilisateur + Copies de contenu réalisées par utilisateur + Récupérations de contenu réalisées par utilisateur + Envois de contenu pour publication réalisés par utilisateur + Envois de contenu pour traduction réalisés par utilisateur + Copie + Publication + Déplacement + Sauvegarde + Suppression + Dépublication + Récupération + Envoi pour publication + Envoi pour traduction + Pour changer le type de document du contenu séléctionné, faites d'abord un choix dans la liste des types valides à cet endroit. Puis modifiez si nécessaire la correspondance des propriétés du type actuel vers le nouveau, et cliquez sur Sauver. @@ -286,7 +305,8 @@ Invalider les changements Vous avez des changements en cours Etes-vous certain(e) de vouloir quitter cette page? - vous avez des changements en cours - + La dépublication va supprimer cette page ainsi que tous ses descendants du site. + Terminé @@ -418,6 +438,7 @@ La clé '%0%' existe déjà. ]]> + Aperçu du dictionaire Votre nom d'utilisateur @@ -594,10 +615,10 @@ Suivant Non de - Off + Inactif OK Ouvrir - On + Actif ou Trier par Mot de passe @@ -888,6 +909,8 @@ Pour gérer votre site, ouvrez simplement le backoffice Umbraco et commencez à Mot de passe oublié? Un email contenant un lien pour ré-initialiser votre mot de passe sera envoyé à l'adresse spécifiée Un email contenant les instructions de ré-initialisation de votre mot de passe sera envoyée à l'adresse spécifiée si elle correspond à nos informations. + Montrer le mot de passe + Cacher le mot de passe Revenir au formulaire de connexion Veuillez fournir un nouveau mot de passe Votre mot de passe a été mis à jour @@ -1419,6 +1442,9 @@ Pour gérer votre site, ouvrez simplement le backoffice Umbraco et commencez à Une erreur est survenue lors du débloquage de l'utilisateur Le membre a été exporté vers le fichier Une erreur est survenue lors de l'export du membre + L'utilisateur %0% a été supprimé + Inviter l'utilisateur + L'invitation a été envoyée à nouveau à %0% Utilise la syntaxe CSS. Ex : h1, .redHeader, .blueTex @@ -1997,8 +2023,11 @@ Pour gérer votre site, ouvrez simplement le backoffice Umbraco et commencez à ]]> - + Inviter + Nouvel envoi de l'invitation en cours... + Supprimer l'Utilisateur + Etes-vous certain(e) de vouloir supprimer le compte de cet utilisateur? @@ -2059,6 +2088,10 @@ Pour gérer votre site, ouvrez simplement le backoffice Umbraco et commencez à Total XML : %0%, Total : %1% Total XML : %0%, Total publié : %1% + Base de données - Le schéma de la base de données est correct pour cette version de Umbraco + %0% problèmes ont été détectés avec le schéma de votre base de données (Voyez le fichier log pour les détails) + Des erreurs ont été détectées lors de la validation du schéma de la base de données par rapport à la version actuelle de Umbraco. + Le certificat de votre site a été marqué comme valide. Erreur de validation du certificat : '%0%' Le certificat SSL de votre site web a expiré. From f416fc6919c1ccc26ac9a600a8d9c371f3fbf806 Mon Sep 17 00:00:00 2001 From: Nathan Woulfe Date: Wed, 11 Jul 2018 15:25:39 +1000 Subject: [PATCH 26/47] clear preview cookie when clicking the info-box link --- .../Configuration/UmbracoSettings/ContentElement.cs | 4 ++-- .../UmbracoSettings/umbracoSettings.config | 2 +- .../components/content/umb-content-node-info.html | 2 +- .../config/umbracoSettings.Release.config | 12 ++++++------ src/Umbraco.Web.UI/config/umbracoSettings.config | 2 +- 5 files changed, 11 insertions(+), 11 deletions(-) diff --git a/src/Umbraco.Core/Configuration/UmbracoSettings/ContentElement.cs b/src/Umbraco.Core/Configuration/UmbracoSettings/ContentElement.cs index 64c9755f7d..17523ab3a1 100644 --- a/src/Umbraco.Core/Configuration/UmbracoSettings/ContentElement.cs +++ b/src/Umbraco.Core/Configuration/UmbracoSettings/ContentElement.cs @@ -112,7 +112,7 @@ namespace Umbraco.Core.Configuration.UmbracoSettings { get { - return GetOptionalTextElement("PreviewBadge", @"In Preview Mode - click to end"); + return GetOptionalTextElement("PreviewBadge", @"In Preview Mode - click to end"); } } @@ -360,4 +360,4 @@ namespace Umbraco.Core.Configuration.UmbracoSettings } } -} \ No newline at end of file +} diff --git a/src/Umbraco.Tests/Configurations/UmbracoSettings/umbracoSettings.config b/src/Umbraco.Tests/Configurations/UmbracoSettings/umbracoSettings.config index 5f5d47e8ba..80a10446c4 100644 --- a/src/Umbraco.Tests/Configurations/UmbracoSettings/umbracoSettings.config +++ b/src/Umbraco.Tests/Configurations/UmbracoSettings/umbracoSettings.config @@ -83,7 +83,7 @@ true - In Preview Mode - click to end]]> + In Preview Mode - click to end]]> diff --git a/src/Umbraco.Web.UI.Client/src/views/components/content/umb-content-node-info.html b/src/Umbraco.Web.UI.Client/src/views/components/content/umb-content-node-info.html index 7e2ac85017..ec24ba2674 100644 --- a/src/Umbraco.Web.UI.Client/src/views/components/content/umb-content-node-info.html +++ b/src/Umbraco.Web.UI.Client/src/views/components/content/umb-content-node-info.html @@ -7,7 +7,7 @@
    • - + {{url}} diff --git a/src/Umbraco.Web.UI/config/umbracoSettings.Release.config b/src/Umbraco.Web.UI/config/umbracoSettings.Release.config index 3f06c398f5..8fc637a1d8 100644 --- a/src/Umbraco.Web.UI/config/umbracoSettings.Release.config +++ b/src/Umbraco.Web.UI/config/umbracoSettings.Release.config @@ -1,11 +1,11 @@ - @@ -40,7 +40,7 @@ - In Preview Mode - click to end]]> + In Preview Mode - click to end]]> - In Preview Mode - click to end + In Preview Mode - click to end ]]> From 914a8fdc2bff67a4bc8feb95733d7eaffe0105b1 Mon Sep 17 00:00:00 2001 From: Nathan Woulfe Date: Wed, 11 Jul 2018 16:27:53 +1000 Subject: [PATCH 27/47] revert link change, config test now passes --- .../Configurations/UmbracoSettings/ContentElementTests.cs | 2 +- .../src/views/components/content/umb-content-node-info.html | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/Umbraco.Tests/Configurations/UmbracoSettings/ContentElementTests.cs b/src/Umbraco.Tests/Configurations/UmbracoSettings/ContentElementTests.cs index 0287159cd9..ae5e6eb118 100644 --- a/src/Umbraco.Tests/Configurations/UmbracoSettings/ContentElementTests.cs +++ b/src/Umbraco.Tests/Configurations/UmbracoSettings/ContentElementTests.cs @@ -156,7 +156,7 @@ namespace Umbraco.Tests.Configurations.UmbracoSettings [Test] public void PreviewBadge() { - Assert.IsTrue(SettingsSection.Content.PreviewBadge == @"In Preview Mode - click to end"); + Assert.IsTrue(SettingsSection.Content.PreviewBadge == @"In Preview Mode - click to end"); } [Test] public void UmbracoLibraryCacheDuration() diff --git a/src/Umbraco.Web.UI.Client/src/views/components/content/umb-content-node-info.html b/src/Umbraco.Web.UI.Client/src/views/components/content/umb-content-node-info.html index ec24ba2674..7e2ac85017 100644 --- a/src/Umbraco.Web.UI.Client/src/views/components/content/umb-content-node-info.html +++ b/src/Umbraco.Web.UI.Client/src/views/components/content/umb-content-node-info.html @@ -7,7 +7,7 @@
      • - + {{url}} From 583b25a85d7b80082437750c453c81fe0d8bb230 Mon Sep 17 00:00:00 2001 From: Mundairson Date: Wed, 11 Jul 2018 17:24:36 +0100 Subject: [PATCH 28/47] Correct Umbraco.Web.UmbracoHelper and tests to be coding standards compliant. --- .../FrontEnd/UmbracoHelperTests.cs | 144 +++++++++--------- src/Umbraco.Web/UmbracoHelper.cs | 3 - 2 files changed, 68 insertions(+), 79 deletions(-) diff --git a/src/Umbraco.Tests/FrontEnd/UmbracoHelperTests.cs b/src/Umbraco.Tests/FrontEnd/UmbracoHelperTests.cs index 8d1bc23880..01351a62fc 100644 --- a/src/Umbraco.Tests/FrontEnd/UmbracoHelperTests.cs +++ b/src/Umbraco.Tests/FrontEnd/UmbracoHelperTests.cs @@ -12,59 +12,51 @@ namespace Umbraco.Tests.FrontEnd public class UmbracoHelperTests : BaseUmbracoApplicationTest { - [Test] - public void Truncate_Simple() - { - var text = "Hello world, this is some text with a link"; + private const string SampleWithAnchorElement = "Hello world, this is some text with a link"; + private const string SampleWithBoldAndAnchorElements = "Hello world, this is some text with a link"; + [Test] + public static void Truncate_Simple() + { var helper = new UmbracoHelper(); - var result = helper.Truncate(text, 25).ToString(); - - Assert.AreEqual("Hello world, this is some…", result); - } - - /// - /// If a truncated string ends with a space, we should trim the space before appending the ellipsis. - /// - [Test] - public void Truncate_Simple_With_Trimming() - { - var text = "Hello world, this is some text with a link"; - - var helper = new UmbracoHelper(); - - var result = helper.Truncate(text, 26).ToString(); + var result = helper.Truncate(SampleWithAnchorElement, 25).ToString(); Assert.AreEqual("Hello world, this is some…", result); } [Test] - public void Truncate_Inside_Word() + public static void When_Truncating_A_String_Ends_With_A_Space_We_Should_Trim_The_Space_Before_Appending_The_Ellipsis() { - var text = "Hello world, this is some text with a link"; - var helper = new UmbracoHelper(); - var result = helper.Truncate(text, 24).ToString(); + var result = helper.Truncate(SampleWithAnchorElement, 26).ToString(); + + Assert.AreEqual("Hello world, this is some…", result); + } + + [Test] + public static void Truncate_Inside_Word() + { + var helper = new UmbracoHelper(); + + var result = helper.Truncate(SampleWithAnchorElement, 24).ToString(); Assert.AreEqual("Hello world, this is som…", result); } [Test] - public void Truncate_With_Tag() + public static void Truncate_With_Tag() { - var text = "Hello world, this is some text with a link"; - var helper = new UmbracoHelper(); - var result = helper.Truncate(text, 35).ToString(); + var result = helper.Truncate(SampleWithAnchorElement, 35).ToString(); Assert.AreEqual("Hello world, this is some text with…", result); } [Test] - public void Create_Encrypted_RouteString_From_Anonymous_Object() + public static void Create_Encrypted_RouteString_From_Anonymous_Object() { var additionalRouteValues = new { @@ -73,15 +65,22 @@ namespace Umbraco.Tests.FrontEnd Key3 = "Value3", keY4 = "valuE4" }; - var encryptedRouteString = UmbracoHelper.CreateEncryptedRouteString("FormController", "FormAction", "", additionalRouteValues); + + var encryptedRouteString = UmbracoHelper.CreateEncryptedRouteString( + "FormController", + "FormAction", + "", + additionalRouteValues + ); + var result = encryptedRouteString.DecryptWithMachineKey(); - var expectedResult = "c=FormController&a=FormAction&ar=&key1=value1&key2=value2&Key3=Value3&keY4=valuE4"; + const string expectedResult = "c=FormController&a=FormAction&ar=&key1=value1&key2=value2&Key3=Value3&keY4=valuE4"; Assert.AreEqual(expectedResult, result); } [Test] - public void Create_Encrypted_RouteString_From_Dictionary() + public static void Create_Encrypted_RouteString_From_Dictionary() { var additionalRouteValues = new Dictionary() { @@ -90,77 +89,74 @@ namespace Umbraco.Tests.FrontEnd {"Key3", "Value3"}, {"keY4", "valuE4"} }; - var encryptedRouteString = UmbracoHelper.CreateEncryptedRouteString("FormController", "FormAction", "", additionalRouteValues); + + var encryptedRouteString = UmbracoHelper.CreateEncryptedRouteString( + "FormController", + "FormAction", + "", + additionalRouteValues + ); + var result = encryptedRouteString.DecryptWithMachineKey(); - var expectedResult = "c=FormController&a=FormAction&ar=&key1=value1&key2=value2&Key3=Value3&keY4=valuE4"; + const string expectedResult = "c=FormController&a=FormAction&ar=&key1=value1&key2=value2&Key3=Value3&keY4=valuE4"; Assert.AreEqual(expectedResult, result); } [Test] - public void Truncate_By_Words() + public static void Truncate_By_Words() { - var text = "Hello world, this is some text with a link"; - var helper = new UmbracoHelper(); - var result = helper.TruncateByWords(text, 4).ToString(); + var result = helper.TruncateByWords(SampleWithAnchorElement, 4).ToString(); Assert.AreEqual("Hello world, this is…", result); } [Test] - public void Truncate_By_Words_With_Tag() + public static void Truncate_By_Words_With_Tag() { - var text = "Hello world, this is some text with a link"; - var helper = new UmbracoHelper(); - var result = helper.TruncateByWords(text, 4).ToString(); + var result = helper.TruncateByWords(SampleWithBoldAndAnchorElements, 4).ToString(); Assert.AreEqual("Hello world, this is…", result); } [Test] - public void Truncate_By_Words_Mid_Tag() + public static void Truncate_By_Words_Mid_Tag() { - var text = "Hello world, this is some text with a link"; - var helper = new UmbracoHelper(); - var result = helper.TruncateByWords(text, 7).ToString(); + var result = helper.TruncateByWords(SampleWithAnchorElement, 7).ToString(); Assert.AreEqual("Hello world, this is some text with…", result); } [Test] - public void Strip_All_Html() + public static void Strip_All_Html() { - var text = "Hello world, this is some text with a link"; - var helper = new UmbracoHelper(); - var result = helper.StripHtml(text, null).ToString(); + var result = helper.StripHtml(SampleWithBoldAndAnchorElements, null).ToString(); Assert.AreEqual("Hello world, this is some text with a link", result); } [Test] - public void Strip_Specific_Html() + public static void Strip_Specific_Html() { - var text = "Hello world, this is some text with a link"; - string[] tags = { "b" }; var helper = new UmbracoHelper(); - var result = helper.StripHtml(text, tags).ToString(); + var result = helper.StripHtml(SampleWithBoldAndAnchorElements, tags).ToString(); - Assert.AreEqual("Hello world, this is some text with a link", result); + Assert.AreEqual(SampleWithAnchorElement, result); } [Test] - public void Strip_Invalid_Html() + public static void Strip_Invalid_Html() { var text = "Hello world, is some text with a link"; @@ -173,7 +169,7 @@ namespace Umbraco.Tests.FrontEnd // ------- Int32 conversion tests [Test] - public static void Converting_boxed_34_to_an_int_returns_34() + public static void Converting_Boxed_34_To_An_Int_Returns_34() { // Arrange const int sample = 34; @@ -190,7 +186,7 @@ namespace Umbraco.Tests.FrontEnd } [Test] - public static void Converting_string_54_to_an_int_returns_54() + public static void Converting_String_54_To_An_Int_Returns_54() { // Arrange const string sample = "54"; @@ -207,7 +203,7 @@ namespace Umbraco.Tests.FrontEnd } [Test] - public static void Converting_hello_to_an_int_returns_false() + public static void Converting_Hello_To_An_Int_Returns_False() { // Arrange const string sample = "Hello"; @@ -224,7 +220,7 @@ namespace Umbraco.Tests.FrontEnd } [Test] - public static void Converting_unsupported_object_to_an_int_returns_false() + public static void Converting_Unsupported_Object_To_An_Int_Returns_False() { // Arrange var clearlyWillNotConvertToInt = new StringBuilder(0); @@ -242,7 +238,7 @@ namespace Umbraco.Tests.FrontEnd // ------- GUID conversion tests [Test] - public static void Converting_boxed_guid_to_a_guid_returns_original_guid_value() + public static void Converting_Boxed_Guid_To_A_Guid_Returns_Original_Guid_Value() { // Arrange Guid sample = Guid.NewGuid(); @@ -259,7 +255,7 @@ namespace Umbraco.Tests.FrontEnd } [Test] - public static void Converting_string_guid_to_a_guid_returns_original_guid_value() + public static void Converting_String_Guid_To_A_Guid_Returns_Original_Guid_Value() { // Arrange Guid sample = Guid.NewGuid(); @@ -276,7 +272,7 @@ namespace Umbraco.Tests.FrontEnd } [Test] - public static void Converting_hello_to_a_guid_returns_false() + public static void Converting_Hello_To_A_Guid_Returns_False() { // Arrange const string sample = "Hello"; @@ -293,7 +289,7 @@ namespace Umbraco.Tests.FrontEnd } [Test] - public static void Converting_unsupported_object_to_a_guid_returns_false() + public static void Converting_Unsupported_Object_To_A_Guid_Returns_False() { // Arrange var clearlyWillNotConvertToGuid = new StringBuilder(0); @@ -311,11 +307,10 @@ namespace Umbraco.Tests.FrontEnd // ------- UDI Conversion Tests /// - /// This requires PluginManager.Current to be initialised before - /// running. + /// This requires PluginManager.Current to be initialised before running. /// [Test] - public static void Converting_boxed_udi_to_a_udi_returns_original_udi_value() + public static void Converting_Boxed_Udi_To_A_Udi_Returns_Original_Udi_Value() { // Arrange Udi.ResetUdiTypes(); @@ -333,11 +328,10 @@ namespace Umbraco.Tests.FrontEnd } /// - /// This requires PluginManager.Current to be initialised before - /// running. + /// This requires PluginManager.Current to be initialised before running. /// [Test] - public static void Converting_string_udi_to_a_udi_returns_original_udi_value() + public static void Converting_String_Udi_To_A_Udi_Returns_Original_Udi_Value() { // Arrange Udi.ResetUdiTypes(); @@ -355,11 +349,10 @@ namespace Umbraco.Tests.FrontEnd } /// - /// This requires PluginManager.Current to be initialised before - /// running. + /// This requires PluginManager.Current to be initialised before running. /// [Test] - public static void Converting_hello_to_a_udi_returns_false() + public static void Converting_Hello_To_A_Udi_Returns_False() { // Arrange Udi.ResetUdiTypes(); @@ -377,11 +370,10 @@ namespace Umbraco.Tests.FrontEnd } /// - /// This requires PluginManager.Current to be initialised before - /// running. + /// This requires PluginManager.Current to be initialised before running. /// [Test] - public static void Converting_unsupported_object_to_a_udi_returns_false() + public static void Converting_Unsupported_Object_To_A_Udi_Returns_False() { // Arrange Udi.ResetUdiTypes(); diff --git a/src/Umbraco.Web/UmbracoHelper.cs b/src/Umbraco.Web/UmbracoHelper.cs index 98a5c6d932..b06b786104 100644 --- a/src/Umbraco.Web/UmbracoHelper.cs +++ b/src/Umbraco.Web/UmbracoHelper.cs @@ -910,7 +910,6 @@ namespace Umbraco.Web return ContentQuery.ContentAtRoot(); } - /// Had to change to internal for testing. internal static bool ConvertIdObjectToInt(object id, out int intId) { switch (id) @@ -928,7 +927,6 @@ namespace Umbraco.Web } } - /// Had to change to internal for testing. internal static bool ConvertIdObjectToGuid(object id, out Guid guidId) { switch (id) @@ -946,7 +944,6 @@ namespace Umbraco.Web } } - /// Had to change to internal for testing. internal static bool ConvertIdObjectToUdi(object id, out Udi guidId) { switch (id) From f2fafbc63174941c42b523d62cb5548c80302c61 Mon Sep 17 00:00:00 2001 From: Nathan Woulfe Date: Thu, 12 Jul 2018 11:37:04 +1000 Subject: [PATCH 29/47] handle manual/external links, add label to input, update placeholder --- .../src/common/services/tinymce.service.js | 1562 +++++++++-------- .../src/less/property-editors.less | 21 + .../common/dialogs/linkpicker.controller.js | 251 +-- .../src/views/common/dialogs/linkpicker.html | 26 +- .../linkpicker/linkpicker.controller.js | 190 +- .../overlays/linkpicker/linkpicker.html | 124 +- src/Umbraco.Web.UI/umbraco/config/lang/en.xml | 3 +- .../umbraco/config/lang/en_us.xml | 3 +- 8 files changed, 1091 insertions(+), 1089 deletions(-) diff --git a/src/Umbraco.Web.UI.Client/src/common/services/tinymce.service.js b/src/Umbraco.Web.UI.Client/src/common/services/tinymce.service.js index 860a25b09d..baa7b87a6a 100644 --- a/src/Umbraco.Web.UI.Client/src/common/services/tinymce.service.js +++ b/src/Umbraco.Web.UI.Client/src/common/services/tinymce.service.js @@ -7,856 +7,864 @@ * A service containing all logic for all of the Umbraco TinyMCE plugins */ function tinyMceService($log, imageHelper, $http, $timeout, macroResource, macroService, $routeParams, umbRequestHelper, angularHelper, userService) { - return { - - /** - * @ngdoc method - * @name umbraco.services.tinyMceService#configuration - * @methodOf umbraco.services.tinyMceService - * - * @description - * Returns a collection of plugins available to the tinyMCE editor - * - */ - configuration: function () { - return umbRequestHelper.resourcePromise( - $http.get( - umbRequestHelper.getApiUrl( - "rteApiBaseUrl", - "GetConfiguration"), { cache: true }), - 'Failed to retrieve tinymce configuration'); - }, - - /** - * @ngdoc method - * @name umbraco.services.tinyMceService#defaultPrevalues - * @methodOf umbraco.services.tinyMceService - * - * @description - * Returns a default configration to fallback on in case none is provided - * - */ - defaultPrevalues: function () { - var cfg = {}; - cfg.toolbar = ["code", "bold", "italic", "styleselect","alignleft", "aligncenter", "alignright", "bullist","numlist", "outdent", "indent", "link", "image", "umbmediapicker", "umbembeddialog", "umbmacro"]; - cfg.stylesheets = []; - cfg.dimensions = { height: 500 }; - cfg.maxImageSize = 500; - return cfg; - }, - - /** - * @ngdoc method - * @name umbraco.services.tinyMceService#createInsertEmbeddedMedia - * @methodOf umbraco.services.tinyMceService - * - * @description - * Creates the umbrco insert embedded media tinymce plugin - * - * @param {Object} editor the TinyMCE editor instance - * @param {Object} $scope the current controller scope - */ - createInsertEmbeddedMedia: function (editor, scope, callback) { - editor.addButton('umbembeddialog', { - icon: 'custom icon-tv', - tooltip: 'Embed', - onclick: function () { - if (callback) { - callback(); - } - } - }); - }, - - insertEmbeddedMediaInEditor: function(editor, preview) { - editor.insertContent(preview); - }, - - /** - * @ngdoc method - * @name umbraco.services.tinyMceService#createMediaPicker - * @methodOf umbraco.services.tinyMceService - * - * @description - * Creates the umbrco insert media tinymce plugin - * - * @param {Object} editor the TinyMCE editor instance - * @param {Object} $scope the current controller scope - */ - createMediaPicker: function (editor, scope, callback) { - editor.addButton('umbmediapicker', { - icon: 'custom icon-picture', - tooltip: 'Media Picker', - stateSelector: 'img', - onclick: function () { - - var selectedElm = editor.selection.getNode(), - currentTarget; - - - if(selectedElm.nodeName === 'IMG'){ - var img = $(selectedElm); - - var hasUdi = img.attr("data-udi") ? true : false; - - currentTarget = { - altText: img.attr("alt"), - url: img.attr("src") - }; - - if (hasUdi) { - currentTarget["udi"] = img.attr("data-udi"); - } - else { - currentTarget["id"] = img.attr("rel"); - } - } - - userService.getCurrentUser().then(function(userData) { - if(callback) { - callback(currentTarget, userData); - } - }); - - } - }); - }, - - insertMediaInEditor: function(editor, img) { - if(img) { - - var hasUdi = img.udi ? true : false; - - var data = { - alt: img.altText || "", - src: (img.url) ? img.url : "nothing.jpg", - id: '__mcenew' - }; - - if (hasUdi) { - data["data-udi"] = img.udi; - } - else { - //Considering these fixed because UDI will now be used and thus - // we have no need for rel http://issues.umbraco.org/issue/U4-6228, http://issues.umbraco.org/issue/U4-6595 - data["rel"] = img.id; - data["data-id"] = img.id; - } - - editor.insertContent(editor.dom.createHTML('img', data)); - - $timeout(function () { - var imgElm = editor.dom.get('__mcenew'); - var size = editor.dom.getSize(imgElm); - - if (editor.settings.maxImageSize && editor.settings.maxImageSize !== 0) { - var newSize = imageHelper.scaleToMaxSize(editor.settings.maxImageSize, size.w, size.h); - - var s = "width: " + newSize.width + "px; height:" + newSize.height + "px;"; - editor.dom.setAttrib(imgElm, 'style', s); - editor.dom.setAttrib(imgElm, 'id', null); - - if (img.url) { - var src = img.url + "?width=" + newSize.width + "&height=" + newSize.height; - editor.dom.setAttrib(imgElm, 'data-mce-src', src); - } - } - }, 500); - } - }, - - /** - * @ngdoc method - * @name umbraco.services.tinyMceService#createUmbracoMacro - * @methodOf umbraco.services.tinyMceService - * - * @description - * Creates the insert umbrco macro tinymce plugin - * - * @param {Object} editor the TinyMCE editor instance - * @param {Object} $scope the current controller scope - */ - createInsertMacro: function (editor, $scope, callback) { - - var createInsertMacroScope = this; - - /** Adds custom rules for the macro plugin and custom serialization */ - editor.on('preInit', function (args) { - //this is requires so that we tell the serializer that a 'div' is actually allowed in the root, otherwise the cleanup will strip it out - editor.serializer.addRules('div'); - - /** This checks if the div is a macro container, if so, checks if its wrapped in a p tag and then unwraps it (removes p tag) */ - editor.serializer.addNodeFilter('div', function (nodes, name) { - for (var i = 0; i < nodes.length; i++) { - if (nodes[i].attr("class") === "umb-macro-holder" && nodes[i].parent && nodes[i].parent.name.toUpperCase() === "P") { - nodes[i].parent.unwrap(); - } - } - }); - - }); - - /** - * Because the macro gets wrapped in a P tag because of the way 'enter' works, this - * method will return the macro element if not wrapped in a p, or the p if the macro - * element is the only one inside of it even if we are deep inside an element inside the macro - */ - function getRealMacroElem(element) { - var e = $(element).closest(".umb-macro-holder"); - if (e.length > 0) { - if (e.get(0).parentNode.nodeName === "P") { - //now check if we're the only element - if (element.parentNode.childNodes.length === 1) { - return e.get(0).parentNode; - } - } - return e.get(0); - } - return null; - } - - /** Adds the button instance */ - editor.addButton('umbmacro', { - icon: 'custom icon-settings-alt', - tooltip: 'Insert macro', - onPostRender: function () { - - var ctrl = this; - var isOnMacroElement = false; - - /** - if the selection comes from a different element that is not the macro's - we need to check if the selection includes part of the macro, if so we'll force the selection - to clear to the next element since if people can select part of the macro markup they can then modify it. - */ - function handleSelectionChange() { - - if (!editor.selection.isCollapsed()) { - var endSelection = tinymce.activeEditor.selection.getEnd(); - var startSelection = tinymce.activeEditor.selection.getStart(); - //don't proceed if it's an entire element selected - if (endSelection !== startSelection) { - - //if the end selection is a macro then move the cursor - //NOTE: we don't have to handle when the selection comes from a previous parent because - // that is automatically taken care of with the normal onNodeChanged logic since the - // evt.element will be the macro once it becomes part of the selection. - var $testForMacro = $(endSelection).closest(".umb-macro-holder"); - if ($testForMacro.length > 0) { - - //it came from before so move after, if there is no after then select ourselves - var next = $testForMacro.next(); - if (next.length > 0) { - editor.selection.setCursorLocation($testForMacro.next().get(0)); - } - else { - selectMacroElement($testForMacro.get(0)); - } - - } - } - } - } - - /** helper method to select the macro element */ - function selectMacroElement(macroElement) { - - // move selection to top element to ensure we can't edit this - editor.selection.select(macroElement); - - // check if the current selection *is* the element (ie bug) - var currentSelection = editor.selection.getStart(); - if (tinymce.isIE) { - if (!editor.dom.hasClass(currentSelection, 'umb-macro-holder')) { - while (!editor.dom.hasClass(currentSelection, 'umb-macro-holder') && currentSelection.parentNode) { - currentSelection = currentSelection.parentNode; - } - editor.selection.select(currentSelection); - } - } - } - - /** - * Add a node change handler, test if we're editing a macro and select the whole thing, then set our isOnMacroElement flag. - * If we change the selection inside this method, then we end up in an infinite loop, so we have to remove ourselves - * from the event listener before changing selection, however, it seems that putting a break point in this method - * will always cause an 'infinite' loop as the caret keeps changing. - */ - function onNodeChanged(evt) { - - //set our macro button active when on a node of class umb-macro-holder - var $macroElement = $(evt.element).closest(".umb-macro-holder"); - - handleSelectionChange(); - - //set the button active - ctrl.active($macroElement.length !== 0); - - if ($macroElement.length > 0) { - var macroElement = $macroElement.get(0); - - //remove the event listener before re-selecting - editor.off('NodeChange', onNodeChanged); - - selectMacroElement(macroElement); - - //set the flag - isOnMacroElement = true; - - //re-add the event listener - editor.on('NodeChange', onNodeChanged); - } - else { - isOnMacroElement = false; - } - - } - - /** when the contents load we need to find any macros declared and load in their content */ - editor.on("LoadContent", function (o) { - - //get all macro divs and load their content - $(editor.dom.select(".umb-macro-holder.mceNonEditable")).each(function() { - createInsertMacroScope.loadMacroContent($(this), null, $scope); - }); - - }); - - /** This prevents any other commands from executing when the current element is the macro so the content cannot be edited */ - editor.on('BeforeExecCommand', function (o) { - if (isOnMacroElement) { - if (o.preventDefault) { - o.preventDefault(); - } - if (o.stopImmediatePropagation) { - o.stopImmediatePropagation(); - } - return; - } - }); - - /** This double checks and ensures you can't paste content into the rendered macro */ - editor.on("Paste", function (o) { - if (isOnMacroElement) { - if (o.preventDefault) { - o.preventDefault(); - } - if (o.stopImmediatePropagation) { - o.stopImmediatePropagation(); - } - return; - } - }); - - //set onNodeChanged event listener - editor.on('NodeChange', onNodeChanged); - - /** - * Listen for the keydown in the editor, we'll check if we are currently on a macro element, if so - * we'll check if the key down is a supported key which requires an action, otherwise we ignore the request - * so the macro cannot be edited. - */ - editor.on('KeyDown', function (e) { - if (isOnMacroElement) { - var macroElement = editor.selection.getNode(); - - //get the 'real' element (either p or the real one) - macroElement = getRealMacroElem(macroElement); - - //prevent editing - e.preventDefault(); - e.stopPropagation(); - - var moveSibling = function (element, isNext) { - var $e = $(element); - var $sibling = isNext ? $e.next() : $e.prev(); - if ($sibling.length > 0) { - editor.selection.select($sibling.get(0)); - editor.selection.collapse(true); - } - else { - //if we're moving previous and there is no sibling, then lets recurse and just select the next one - if (!isNext) { - moveSibling(element, true); - return; - } - - //if there is no sibling we'll generate a new p at the end and select it - editor.setContent(editor.getContent() + "

         

        "); - editor.selection.select($(editor.dom.getRoot()).children().last().get(0)); - editor.selection.collapse(true); - - } - }; - - //supported keys to move to the next or prev element (13-enter, 27-esc, 38-up, 40-down, 39-right, 37-left) - //supported keys to remove the macro (8-backspace, 46-delete) - //TODO: Should we make the enter key insert a line break before or leave it as moving to the next element? - if ($.inArray(e.keyCode, [13, 40, 39]) !== -1) { - //move to next element - moveSibling(macroElement, true); - } - else if ($.inArray(e.keyCode, [27, 38, 37]) !== -1) { - //move to prev element - moveSibling(macroElement, false); - } - else if ($.inArray(e.keyCode, [8, 46]) !== -1) { - //delete macro element - - //move first, then delete - moveSibling(macroElement, false); - editor.dom.remove(macroElement); - } - return ; - } - }); - - }, - - /** The insert macro button click event handler */ - onclick: function () { - - var dialogData = { - //flag for use in rte so we only show macros flagged for the editor - richTextEditor: true - }; - - //when we click we could have a macro already selected and in that case we'll want to edit the current parameters - //so we'll need to extract them and submit them to the dialog. - var macroElement = editor.selection.getNode(); - macroElement = getRealMacroElem(macroElement); - if (macroElement) { - //we have a macro selected so we'll need to parse it's alias and parameters - var contents = $(macroElement).contents(); - var comment = _.find(contents, function(item) { - return item.nodeType === 8; - }); - if (!comment) { - throw "Cannot parse the current macro, the syntax in the editor is invalid"; - } - var syntax = comment.textContent.trim(); - var parsed = macroService.parseMacroSyntax(syntax); - dialogData = { - macroData: parsed - }; - } - - if(callback) { - callback(dialogData); - } - - } - }); - }, - - insertMacroInEditor: function(editor, macroObject, $scope) { - - //put the macro syntax in comments, we will parse this out on the server side to be used - //for persisting. - var macroSyntaxComment = ""; - //create an id class for this element so we can re-select it after inserting - var uniqueId = "umb-macro-" + editor.dom.uniqueId(); - var macroDiv = editor.dom.create('div', - { - 'class': 'umb-macro-holder ' + macroObject.macroAlias + ' mceNonEditable ' + uniqueId - }, - macroSyntaxComment + 'Macro alias: ' + macroObject.macroAlias + ''); - - editor.selection.setNode(macroDiv); - - var $macroDiv = $(editor.dom.select("div.umb-macro-holder." + uniqueId)); - - //async load the macro content - this.loadMacroContent($macroDiv, macroObject, $scope); - - }, - - /** loads in the macro content async from the server */ - loadMacroContent: function($macroDiv, macroData, $scope) { - - //if we don't have the macroData, then we'll need to parse it from the macro div - if (!macroData) { - var contents = $macroDiv.contents(); - var comment = _.find(contents, function (item) { - return item.nodeType === 8; - }); - if (!comment) { - throw "Cannot parse the current macro, the syntax in the editor is invalid"; - } - var syntax = comment.textContent.trim(); - var parsed = macroService.parseMacroSyntax(syntax); - macroData = parsed; - } - - var $ins = $macroDiv.find("ins"); - - //show the throbber - $macroDiv.addClass("loading"); - - var contentId = $routeParams.id; - - //need to wrap in safe apply since this might be occuring outside of angular - angularHelper.safeApply($scope, function() { - macroResource.getMacroResultAsHtmlForEditor(macroData.macroAlias, contentId, macroData.macroParamsDictionary) - .then(function (htmlResult) { - - $macroDiv.removeClass("loading"); - htmlResult = htmlResult.trim(); - if (htmlResult !== "") { - $ins.html(htmlResult); - } - }); - }); - - }, - - createLinkPicker: function(editor, $scope, onClick) { - - function createLinkList(callback) { - return function() { - var linkList = editor.settings.link_list; - - if (typeof(linkList) === "string") { - tinymce.util.XHR.send({ - url: linkList, - success: function(text) { - callback(tinymce.util.JSON.parse(text)); - } - }); - } else { - callback(linkList); - } - }; - } - - function showDialog(linkList) { - var data = {}, selection = editor.selection, dom = editor.dom, selectedElm, anchorElm, initialText; - var win, linkListCtrl, relListCtrl, targetListCtrl; - - function linkListChangeHandler(e) { - var textCtrl = win.find('#text'); - - if (!textCtrl.value() || (e.lastControl && textCtrl.value() === e.lastControl.text())) { - textCtrl.value(e.control.text()); - } - - win.find('#href').value(e.control.value()); - } - - function buildLinkList() { - var linkListItems = [{ - text: 'None', - value: '' + return { + + /** + * @ngdoc method + * @name umbraco.services.tinyMceService#configuration + * @methodOf umbraco.services.tinyMceService + * + * @description + * Returns a collection of plugins available to the tinyMCE editor + * + */ + configuration: function () { + return umbRequestHelper.resourcePromise( + $http.get( + umbRequestHelper.getApiUrl( + "rteApiBaseUrl", + "GetConfiguration"), { + cache: true + }), + 'Failed to retrieve tinymce configuration'); + }, + + /** + * @ngdoc method + * @name umbraco.services.tinyMceService#defaultPrevalues + * @methodOf umbraco.services.tinyMceService + * + * @description + * Returns a default configration to fallback on in case none is provided + * + */ + defaultPrevalues: function () { + var cfg = {}; + cfg.toolbar = ["code", "bold", "italic", "styleselect", "alignleft", "aligncenter", "alignright", "bullist", "numlist", "outdent", "indent", "link", "image", "umbmediapicker", "umbembeddialog", "umbmacro"]; + cfg.stylesheets = []; + cfg.dimensions = { + height: 500 + }; + cfg.maxImageSize = 500; + return cfg; + }, + + /** + * @ngdoc method + * @name umbraco.services.tinyMceService#createInsertEmbeddedMedia + * @methodOf umbraco.services.tinyMceService + * + * @description + * Creates the umbrco insert embedded media tinymce plugin + * + * @param {Object} editor the TinyMCE editor instance + * @param {Object} $scope the current controller scope + */ + createInsertEmbeddedMedia: function (editor, scope, callback) { + editor.addButton('umbembeddialog', { + icon: 'custom icon-tv', + tooltip: 'Embed', + onclick: function () { + if (callback) { + callback(); + } + } + }); + }, + + insertEmbeddedMediaInEditor: function (editor, preview) { + editor.insertContent(preview); + }, + + /** + * @ngdoc method + * @name umbraco.services.tinyMceService#createMediaPicker + * @methodOf umbraco.services.tinyMceService + * + * @description + * Creates the umbrco insert media tinymce plugin + * + * @param {Object} editor the TinyMCE editor instance + * @param {Object} $scope the current controller scope + */ + createMediaPicker: function (editor, scope, callback) { + editor.addButton('umbmediapicker', { + icon: 'custom icon-picture', + tooltip: 'Media Picker', + stateSelector: 'img', + onclick: function () { + + var selectedElm = editor.selection.getNode(), + currentTarget; + + + if (selectedElm.nodeName === 'IMG') { + var img = $(selectedElm); + + var hasUdi = img.attr("data-udi") ? true : false; + + currentTarget = { + altText: img.attr("alt"), + url: img.attr("src") + }; + + if (hasUdi) { + currentTarget["udi"] = img.attr("data-udi"); + } else { + currentTarget["id"] = img.attr("rel"); + } + } + + userService.getCurrentUser().then(function (userData) { + if (callback) { + callback(currentTarget, userData); + } + }); + + } + }); + }, + + insertMediaInEditor: function (editor, img) { + if (img) { + + var hasUdi = img.udi ? true : false; + + var data = { + alt: img.altText || "", + src: (img.url) ? img.url : "nothing.jpg", + id: '__mcenew' + }; + + if (hasUdi) { + data["data-udi"] = img.udi; + } else { + //Considering these fixed because UDI will now be used and thus + // we have no need for rel http://issues.umbraco.org/issue/U4-6228, http://issues.umbraco.org/issue/U4-6595 + data["rel"] = img.id; + data["data-id"] = img.id; + } + + editor.insertContent(editor.dom.createHTML('img', data)); + + $timeout(function () { + var imgElm = editor.dom.get('__mcenew'); + var size = editor.dom.getSize(imgElm); + + if (editor.settings.maxImageSize && editor.settings.maxImageSize !== 0) { + var newSize = imageHelper.scaleToMaxSize(editor.settings.maxImageSize, size.w, size.h); + + var s = "width: " + newSize.width + "px; height:" + newSize.height + "px;"; + editor.dom.setAttrib(imgElm, 'style', s); + editor.dom.setAttrib(imgElm, 'id', null); + + if (img.url) { + var src = img.url + "?width=" + newSize.width + "&height=" + newSize.height; + editor.dom.setAttrib(imgElm, 'data-mce-src', src); + } + } + }, 500); + } + }, + + /** + * @ngdoc method + * @name umbraco.services.tinyMceService#createUmbracoMacro + * @methodOf umbraco.services.tinyMceService + * + * @description + * Creates the insert umbrco macro tinymce plugin + * + * @param {Object} editor the TinyMCE editor instance + * @param {Object} $scope the current controller scope + */ + createInsertMacro: function (editor, $scope, callback) { + + var createInsertMacroScope = this; + + /** Adds custom rules for the macro plugin and custom serialization */ + editor.on('preInit', function (args) { + //this is requires so that we tell the serializer that a 'div' is actually allowed in the root, otherwise the cleanup will strip it out + editor.serializer.addRules('div'); + + /** This checks if the div is a macro container, if so, checks if its wrapped in a p tag and then unwraps it (removes p tag) */ + editor.serializer.addNodeFilter('div', function (nodes, name) { + for (var i = 0; i < nodes.length; i++) { + if (nodes[i].attr("class") === "umb-macro-holder" && nodes[i].parent && nodes[i].parent.name.toUpperCase() === "P") { + nodes[i].parent.unwrap(); + } + } + }); + + }); + + /** + * Because the macro gets wrapped in a P tag because of the way 'enter' works, this + * method will return the macro element if not wrapped in a p, or the p if the macro + * element is the only one inside of it even if we are deep inside an element inside the macro + */ + function getRealMacroElem(element) { + var e = $(element).closest(".umb-macro-holder"); + if (e.length > 0) { + if (e.get(0).parentNode.nodeName === "P") { + //now check if we're the only element + if (element.parentNode.childNodes.length === 1) { + return e.get(0).parentNode; + } + } + return e.get(0); + } + return null; + } + + /** Adds the button instance */ + editor.addButton('umbmacro', { + icon: 'custom icon-settings-alt', + tooltip: 'Insert macro', + onPostRender: function () { + + var ctrl = this; + var isOnMacroElement = false; + + /** + if the selection comes from a different element that is not the macro's + we need to check if the selection includes part of the macro, if so we'll force the selection + to clear to the next element since if people can select part of the macro markup they can then modify it. + */ + function handleSelectionChange() { + + if (!editor.selection.isCollapsed()) { + var endSelection = tinymce.activeEditor.selection.getEnd(); + var startSelection = tinymce.activeEditor.selection.getStart(); + //don't proceed if it's an entire element selected + if (endSelection !== startSelection) { + + //if the end selection is a macro then move the cursor + //NOTE: we don't have to handle when the selection comes from a previous parent because + // that is automatically taken care of with the normal onNodeChanged logic since the + // evt.element will be the macro once it becomes part of the selection. + var $testForMacro = $(endSelection).closest(".umb-macro-holder"); + if ($testForMacro.length > 0) { + + //it came from before so move after, if there is no after then select ourselves + var next = $testForMacro.next(); + if (next.length > 0) { + editor.selection.setCursorLocation($testForMacro.next().get(0)); + } else { + selectMacroElement($testForMacro.get(0)); + } + + } + } + } + } + + /** helper method to select the macro element */ + function selectMacroElement(macroElement) { + + // move selection to top element to ensure we can't edit this + editor.selection.select(macroElement); + + // check if the current selection *is* the element (ie bug) + var currentSelection = editor.selection.getStart(); + if (tinymce.isIE) { + if (!editor.dom.hasClass(currentSelection, 'umb-macro-holder')) { + while (!editor.dom.hasClass(currentSelection, 'umb-macro-holder') && currentSelection.parentNode) { + currentSelection = currentSelection.parentNode; + } + editor.selection.select(currentSelection); + } + } + } + + /** + * Add a node change handler, test if we're editing a macro and select the whole thing, then set our isOnMacroElement flag. + * If we change the selection inside this method, then we end up in an infinite loop, so we have to remove ourselves + * from the event listener before changing selection, however, it seems that putting a break point in this method + * will always cause an 'infinite' loop as the caret keeps changing. + */ + function onNodeChanged(evt) { + + //set our macro button active when on a node of class umb-macro-holder + var $macroElement = $(evt.element).closest(".umb-macro-holder"); + + handleSelectionChange(); + + //set the button active + ctrl.active($macroElement.length !== 0); + + if ($macroElement.length > 0) { + var macroElement = $macroElement.get(0); + + //remove the event listener before re-selecting + editor.off('NodeChange', onNodeChanged); + + selectMacroElement(macroElement); + + //set the flag + isOnMacroElement = true; + + //re-add the event listener + editor.on('NodeChange', onNodeChanged); + } else { + isOnMacroElement = false; + } + + } + + /** when the contents load we need to find any macros declared and load in their content */ + editor.on("LoadContent", function (o) { + + //get all macro divs and load their content + $(editor.dom.select(".umb-macro-holder.mceNonEditable")).each(function () { + createInsertMacroScope.loadMacroContent($(this), null, $scope); + }); + + }); + + /** This prevents any other commands from executing when the current element is the macro so the content cannot be edited */ + editor.on('BeforeExecCommand', function (o) { + if (isOnMacroElement) { + if (o.preventDefault) { + o.preventDefault(); + } + if (o.stopImmediatePropagation) { + o.stopImmediatePropagation(); + } + return; + } + }); + + /** This double checks and ensures you can't paste content into the rendered macro */ + editor.on("Paste", function (o) { + if (isOnMacroElement) { + if (o.preventDefault) { + o.preventDefault(); + } + if (o.stopImmediatePropagation) { + o.stopImmediatePropagation(); + } + return; + } + }); + + //set onNodeChanged event listener + editor.on('NodeChange', onNodeChanged); + + /** + * Listen for the keydown in the editor, we'll check if we are currently on a macro element, if so + * we'll check if the key down is a supported key which requires an action, otherwise we ignore the request + * so the macro cannot be edited. + */ + editor.on('KeyDown', function (e) { + if (isOnMacroElement) { + var macroElement = editor.selection.getNode(); + + //get the 'real' element (either p or the real one) + macroElement = getRealMacroElem(macroElement); + + //prevent editing + e.preventDefault(); + e.stopPropagation(); + + var moveSibling = function (element, isNext) { + var $e = $(element); + var $sibling = isNext ? $e.next() : $e.prev(); + if ($sibling.length > 0) { + editor.selection.select($sibling.get(0)); + editor.selection.collapse(true); + } else { + //if we're moving previous and there is no sibling, then lets recurse and just select the next one + if (!isNext) { + moveSibling(element, true); + return; + } + + //if there is no sibling we'll generate a new p at the end and select it + editor.setContent(editor.getContent() + "

         

        "); + editor.selection.select($(editor.dom.getRoot()).children().last().get(0)); + editor.selection.collapse(true); + + } + }; + + //supported keys to move to the next or prev element (13-enter, 27-esc, 38-up, 40-down, 39-right, 37-left) + //supported keys to remove the macro (8-backspace, 46-delete) + //TODO: Should we make the enter key insert a line break before or leave it as moving to the next element? + if ($.inArray(e.keyCode, [13, 40, 39]) !== -1) { + //move to next element + moveSibling(macroElement, true); + } else if ($.inArray(e.keyCode, [27, 38, 37]) !== -1) { + //move to prev element + moveSibling(macroElement, false); + } else if ($.inArray(e.keyCode, [8, 46]) !== -1) { + //delete macro element + + //move first, then delete + moveSibling(macroElement, false); + editor.dom.remove(macroElement); + } + return; + } + }); + + }, + + /** The insert macro button click event handler */ + onclick: function () { + + var dialogData = { + //flag for use in rte so we only show macros flagged for the editor + richTextEditor: true + }; + + //when we click we could have a macro already selected and in that case we'll want to edit the current parameters + //so we'll need to extract them and submit them to the dialog. + var macroElement = editor.selection.getNode(); + macroElement = getRealMacroElem(macroElement); + if (macroElement) { + //we have a macro selected so we'll need to parse it's alias and parameters + var contents = $(macroElement).contents(); + var comment = _.find(contents, function (item) { + return item.nodeType === 8; + }); + if (!comment) { + throw "Cannot parse the current macro, the syntax in the editor is invalid"; + } + var syntax = comment.textContent.trim(); + var parsed = macroService.parseMacroSyntax(syntax); + dialogData = { + macroData: parsed + }; + } + + if (callback) { + callback(dialogData); + } + + } + }); + }, + + insertMacroInEditor: function (editor, macroObject, $scope) { + + //put the macro syntax in comments, we will parse this out on the server side to be used + //for persisting. + var macroSyntaxComment = ""; + //create an id class for this element so we can re-select it after inserting + var uniqueId = "umb-macro-" + editor.dom.uniqueId(); + var macroDiv = editor.dom.create('div', { + 'class': 'umb-macro-holder ' + macroObject.macroAlias + ' mceNonEditable ' + uniqueId + }, + macroSyntaxComment + 'Macro alias: ' + macroObject.macroAlias + ''); + + editor.selection.setNode(macroDiv); + + var $macroDiv = $(editor.dom.select("div.umb-macro-holder." + uniqueId)); + + //async load the macro content + this.loadMacroContent($macroDiv, macroObject, $scope); + + }, + + /** loads in the macro content async from the server */ + loadMacroContent: function ($macroDiv, macroData, $scope) { + + //if we don't have the macroData, then we'll need to parse it from the macro div + if (!macroData) { + var contents = $macroDiv.contents(); + var comment = _.find(contents, function (item) { + return item.nodeType === 8; + }); + if (!comment) { + throw "Cannot parse the current macro, the syntax in the editor is invalid"; + } + var syntax = comment.textContent.trim(); + var parsed = macroService.parseMacroSyntax(syntax); + macroData = parsed; + } + + var $ins = $macroDiv.find("ins"); + + //show the throbber + $macroDiv.addClass("loading"); + + var contentId = $routeParams.id; + + //need to wrap in safe apply since this might be occuring outside of angular + angularHelper.safeApply($scope, function () { + macroResource.getMacroResultAsHtmlForEditor(macroData.macroAlias, contentId, macroData.macroParamsDictionary) + .then(function (htmlResult) { + + $macroDiv.removeClass("loading"); + htmlResult = htmlResult.trim(); + if (htmlResult !== "") { + $ins.html(htmlResult); + } + }); + }); + + }, + + createLinkPicker: function (editor, $scope, onClick) { + + function createLinkList(callback) { + return function () { + var linkList = editor.settings.link_list; + + if (typeof (linkList) === "string") { + tinymce.util.XHR.send({ + url: linkList, + success: function (text) { + callback(tinymce.util.JSON.parse(text)); + } + }); + } else { + callback(linkList); + } + }; + } + + function showDialog(linkList) { + var data = {}, + selection = editor.selection, + dom = editor.dom, + selectedElm, anchorElm, initialText; + var win, linkListCtrl, relListCtrl, targetListCtrl; + + function linkListChangeHandler(e) { + var textCtrl = win.find('#text'); + + if (!textCtrl.value() || (e.lastControl && textCtrl.value() === e.lastControl.text())) { + textCtrl.value(e.control.text()); + } + + win.find('#href').value(e.control.value()); + } + + function buildLinkList() { + var linkListItems = [{ + text: 'None', + value: '' }]; - tinymce.each(linkList, function(link) { - linkListItems.push({ - text: link.text || link.title, - value: link.value || link.url, - menu: link.menu - }); - }); + tinymce.each(linkList, function (link) { + linkListItems.push({ + text: link.text || link.title, + value: link.value || link.url, + menu: link.menu + }); + }); - return linkListItems; - } + return linkListItems; + } - function buildRelList(relValue) { - var relListItems = [{ - text: 'None', - value: '' + function buildRelList(relValue) { + var relListItems = [{ + text: 'None', + value: '' }]; - tinymce.each(editor.settings.rel_list, function(rel) { - relListItems.push({ - text: rel.text || rel.title, - value: rel.value, - selected: relValue === rel.value - }); - }); + tinymce.each(editor.settings.rel_list, function (rel) { + relListItems.push({ + text: rel.text || rel.title, + value: rel.value, + selected: relValue === rel.value + }); + }); - return relListItems; - } + return relListItems; + } - function buildTargetList(targetValue) { - var targetListItems = [{ - text: 'None', - value: '' + function buildTargetList(targetValue) { + var targetListItems = [{ + text: 'None', + value: '' }]; - if (!editor.settings.target_list) { - targetListItems.push({ - text: 'New window', - value: '_blank' - }); - } + if (!editor.settings.target_list) { + targetListItems.push({ + text: 'New window', + value: '_blank' + }); + } - tinymce.each(editor.settings.target_list, function(target) { - targetListItems.push({ - text: target.text || target.title, - value: target.value, - selected: targetValue === target.value - }); - }); + tinymce.each(editor.settings.target_list, function (target) { + targetListItems.push({ + text: target.text || target.title, + value: target.value, + selected: targetValue === target.value + }); + }); - return targetListItems; - } + return targetListItems; + } - function buildAnchorListControl(url) { - var anchorList = []; + function buildAnchorListControl(url) { + var anchorList = []; - tinymce.each(editor.dom.select('a:not([href])'), function(anchor) { - var id = anchor.name || anchor.id; + tinymce.each(editor.dom.select('a:not([href])'), function (anchor) { + var id = anchor.name || anchor.id; - if (id) { - anchorList.push({ - text: id, - value: '#' + id, - selected: url.indexOf('#' + id) !== -1 - }); - } - }); + if (id) { + anchorList.push({ + text: id, + value: '#' + id, + selected: url.indexOf('#' + id) !== -1 + }); + } + }); - if (anchorList.length) { - anchorList.unshift({ - text: 'None', - value: '' - }); + if (anchorList.length) { + anchorList.unshift({ + text: 'None', + value: '' + }); - return { - name: 'anchor', - type: 'listbox', - label: 'Anchors', - values: anchorList, - onselect: linkListChangeHandler - }; - } - } + return { + name: 'anchor', + type: 'listbox', + label: 'Anchors', + values: anchorList, + onselect: linkListChangeHandler + }; + } + } - function updateText() { - if (!initialText && data.text.length === 0) { - this.parent().parent().find('#text')[0].value(this.value()); - } - } + function updateText() { + if (!initialText && data.text.length === 0) { + this.parent().parent().find('#text')[0].value(this.value()); + } + } - selectedElm = selection.getNode(); - anchorElm = dom.getParent(selectedElm, 'a[href]'); + selectedElm = selection.getNode(); + anchorElm = dom.getParent(selectedElm, 'a[href]'); - data.text = initialText = anchorElm ? (anchorElm.innerText || anchorElm.textContent) : selection.getContent({format: 'text'}); - data.href = anchorElm ? dom.getAttrib(anchorElm, 'href') : ''; - data.target = anchorElm ? dom.getAttrib(anchorElm, 'target') : ''; - data.rel = anchorElm ? dom.getAttrib(anchorElm, 'rel') : ''; + data.text = initialText = anchorElm ? (anchorElm.innerText || anchorElm.textContent) : selection.getContent({ + format: 'text' + }); + data.href = anchorElm ? dom.getAttrib(anchorElm, 'href') : ''; + data.target = anchorElm ? dom.getAttrib(anchorElm, 'target') : ''; + data.rel = anchorElm ? dom.getAttrib(anchorElm, 'rel') : ''; - if (selectedElm.nodeName === "IMG") { - data.text = initialText = " "; - } + if (selectedElm.nodeName === "IMG") { + data.text = initialText = " "; + } - if (linkList) { - linkListCtrl = { - type: 'listbox', - label: 'Link list', - values: buildLinkList(), - onselect: linkListChangeHandler - }; - } + if (linkList) { + linkListCtrl = { + type: 'listbox', + label: 'Link list', + values: buildLinkList(), + onselect: linkListChangeHandler + }; + } - if (editor.settings.target_list !== false) { - targetListCtrl = { - name: 'target', - type: 'listbox', - label: 'Target', - values: buildTargetList(data.target) - }; - } + if (editor.settings.target_list !== false) { + targetListCtrl = { + name: 'target', + type: 'listbox', + label: 'Target', + values: buildTargetList(data.target) + }; + } - if (editor.settings.rel_list) { - relListCtrl = { - name: 'rel', - type: 'listbox', - label: 'Rel', - values: buildRelList(data.rel) - }; - } + if (editor.settings.rel_list) { + relListCtrl = { + name: 'rel', + type: 'listbox', + label: 'Rel', + values: buildRelList(data.rel) + }; + } - var currentTarget = null; + var currentTarget = null; + + //if we already have a link selected, we want to pass that data over to the dialog + if (anchorElm) { + var anchor = $(anchorElm); + currentTarget = { + name: anchor.attr("title"), + url: anchor.attr("href"), + target: anchor.attr("target") + }; - //if we already have a link selected, we want to pass that data over to the dialog - if(anchorElm){ - var anchor = $(anchorElm); - currentTarget = { - name: anchor.attr("title"), - url: anchor.attr("href"), - target: anchor.attr("target") - }; - // drop the lead char from the anchor text, if it has a value var anchorVal = anchor[0].dataset.anchor; if (anchorVal) { currentTarget.anchor = anchorVal.substring(1); } - - //locallink detection, we do this here, to avoid poluting the dialogservice - //so the dialog service can just expect to get a node-like structure - if (currentTarget.url.indexOf("localLink:") > 0) { + + //locallink detection, we do this here, to avoid poluting the dialogservice + //so the dialog service can just expect to get a node-like structure + if (currentTarget.url.indexOf("localLink:") > 0) { // if the current link has an anchor, it needs to be considered when getting the udi/id // if an anchor exists, reduce the substring max by its length plus two to offset the removed prefix and trailing curly brace var linkId = currentTarget.url.substring(currentTarget.url.indexOf(":") + 1, currentTarget.url.lastIndexOf("}")); - //we need to check if this is an INT or a UDI - var parsedIntId = parseInt(linkId, 10); - if (isNaN(parsedIntId)) { - //it's a UDI - currentTarget.udi = linkId; - } - else { - currentTarget.id = linkId; - } - } - } + //we need to check if this is an INT or a UDI + var parsedIntId = parseInt(linkId, 10); + if (isNaN(parsedIntId)) { + //it's a UDI + currentTarget.udi = linkId; + } else { + currentTarget.id = linkId; + } + } + } - if(onClick) { - onClick(currentTarget, anchorElm); - } + if (onClick) { + onClick(currentTarget, anchorElm); + } - } + } - editor.addButton('link', { - icon: 'link', - tooltip: 'Insert/edit link', - shortcut: 'Ctrl+K', - onclick: createLinkList(showDialog), - stateSelector: 'a[href]' - }); + editor.addButton('link', { + icon: 'link', + tooltip: 'Insert/edit link', + shortcut: 'Ctrl+K', + onclick: createLinkList(showDialog), + stateSelector: 'a[href]' + }); - editor.addButton('unlink', { - icon: 'unlink', - tooltip: 'Remove link', - cmd: 'unlink', - stateSelector: 'a[href]' - }); + editor.addButton('unlink', { + icon: 'unlink', + tooltip: 'Remove link', + cmd: 'unlink', + stateSelector: 'a[href]' + }); - editor.addShortcut('Ctrl+K', '', createLinkList(showDialog)); - this.showDialog = showDialog; + editor.addShortcut('Ctrl+K', '', createLinkList(showDialog)); + this.showDialog = showDialog; - editor.addMenuItem('link', { - icon: 'link', - text: 'Insert link', - shortcut: 'Ctrl+K', - onclick: createLinkList(showDialog), - stateSelector: 'a[href]', - context: 'insert', - prependToContext: true - }); + editor.addMenuItem('link', { + icon: 'link', + text: 'Insert link', + shortcut: 'Ctrl+K', + onclick: createLinkList(showDialog), + stateSelector: 'a[href]', + context: 'insert', + prependToContext: true + }); + + }, - }, - /** - * @ngdoc method - * @name umbraco.services.tinyMceService#getAnchorNames - * @methodOf umbraco.services.tinyMceService - * - * @description - * From the given string, generates a string array where each item is the id attribute value from a named anchor - * 'some string with a named anchor' returns ['anchor'] - * - * @param {string} input the string to parse - */ - getAnchorNames: function(input) { - var anchorPattern = //gi; + * @ngdoc method + * @name umbraco.services.tinyMceService#getAnchorNames + * @methodOf umbraco.services.tinyMceService + * + * @description + * From the given string, generates a string array where each item is the id attribute value from a named anchor + * 'some string with a named anchor' returns ['anchor'] + * + * @param {string} input the string to parse + */ + getAnchorNames: function (input) { + var anchorPattern = //gi; var matches = input.match(anchorPattern); - var anchors = []; - + var anchors = []; + if (matches) { - anchors = matches.map(function(v) { + anchors = matches.map(function (v) { return v.substring(v.indexOf('"') + 1, v.lastIndexOf('\\')); }); } - + return anchors; }, - insertLinkInEditor: function(editor, target, anchorElm) { + insertLinkInEditor: function (editor, target, anchorElm) { - var href = target.url; - // We want to use the Udi. If it is set, we use it, else fallback to id, and finally to null - var hasUdi = target.udi ? true : false; - var id = hasUdi ? target.udi : (target.id ? target.id : null); + var href = target.url; + // We want to use the Udi. If it is set, we use it, else fallback to id, and finally to null + var hasUdi = target.udi ? true : false; + var id = hasUdi ? target.udi : (target.id ? target.id : null); // if an anchor exists, check that it is appropriately prefixed if (target.anchor && target.anchor[0] !== '?' && target.anchor[0] !== '#') { - target.anchor = (target.anchor.indexOf('=') === -1 ? '#' : '?') + target.anchor; + target.anchor = (target.anchor.indexOf('=') === -1 ? '#' : '?') + target.anchor; + } + + // the href might be an external url, so check the value for an anchor/qs + // href has the anchor re-appended later, hence the reset here to avoid duplicating the anchor + if (!target.anchor) { + var urlParts = href.split(/(#|\?)/); + if (urlParts.length === 3) { + href = urlParts[0]; + target.anchor = urlParts[1] + urlParts[2]; + } } - //Create a json obj used to create the attributes for the tag - function createElemAttributes() { - var a = { - href: href, - title: target.name, - target: target.target ? target.target : null, - rel: target.rel ? target.rel : null - }; - - if (hasUdi) { - a["data-udi"] = target.udi; - } - else if (target.id) { - a["data-id"] = target.id; - } - + //Create a json obj used to create the attributes for the tag + function createElemAttributes() { + var a = { + href: href, + title: target.name, + target: target.target ? target.target : null, + rel: target.rel ? target.rel : null + }; + + if (hasUdi) { + a["data-udi"] = target.udi; + } else if (target.id) { + a["data-id"] = target.id; + } + if (target.anchor) { a["data-anchor"] = target.anchor; a.href = a.href + target.anchor; } else { a["data-anchor"] = null; } - - return a; - } - function insertLink() { - if (anchorElm) { - editor.dom.setAttribs(anchorElm, createElemAttributes()); + return a; + } - editor.selection.select(anchorElm); - editor.execCommand('mceEndTyping'); - } - else { - editor.execCommand('mceInsertLink', false, createElemAttributes()); - } - } + function insertLink() { + if (anchorElm) { + editor.dom.setAttribs(anchorElm, createElemAttributes()); - if (!href) { - editor.execCommand('unlink'); - return; - } + editor.selection.select(anchorElm); + editor.execCommand('mceEndTyping'); + } else { + editor.execCommand('mceInsertLink', false, createElemAttributes()); + } + } - //if we have an id, it must be a locallink:id, aslong as the isMedia flag is not set - if(id && (angular.isUndefined(target.isMedia) || !target.isMedia)){ - - href = "/{localLink:" + id + "}"; + if (!href) { + editor.execCommand('unlink'); + return; + } - insertLink(); - return; - } + //if we have an id, it must be a locallink:id, aslong as the isMedia flag is not set + if (id && (angular.isUndefined(target.isMedia) || !target.isMedia)) { - // Is email and not //user@domain.com - if (href.indexOf('@') > 0 && href.indexOf('//') === -1 && href.indexOf('mailto:') === -1) { - href = 'mailto:' + href; - insertLink(); - return; - } + href = "/{localLink:" + id + "}"; - // Is www. prefixed - if (/^\s*www\./i.test(href)) { - href = 'http://' + href; - insertLink(); - return; - } + insertLink(); + return; + } - insertLink(); + // Is email and not //user@domain.com + if (href.indexOf('@') > 0 && href.indexOf('//') === -1 && href.indexOf('mailto:') === -1) { + href = 'mailto:' + href; + insertLink(); + return; + } - } + // Is www. prefixed + if (/^\s*www\./i.test(href)) { + href = 'http://' + href; + insertLink(); + return; + } - }; + insertLink(); + + } + + }; } angular.module('umbraco.services').factory('tinyMceService', tinyMceService); diff --git a/src/Umbraco.Web.UI.Client/src/less/property-editors.less b/src/Umbraco.Web.UI.Client/src/less/property-editors.less index 77fd17ea38..dd0e4d3e77 100644 --- a/src/Umbraco.Web.UI.Client/src/less/property-editors.less +++ b/src/Umbraco.Web.UI.Client/src/less/property-editors.less @@ -42,6 +42,27 @@ width: 99%; } +// displays property inline with preceeding +.umb-property { + &--pull { + float:left; + width:60%; + } + + &--push { + float:right; + width:35%; + } + + &--pull, &--push { + .umb-editor { + min-width:0; + width:100%; + } + } +} + + // // Content picker // -------------------------------------------------- diff --git a/src/Umbraco.Web.UI.Client/src/views/common/dialogs/linkpicker.controller.js b/src/Umbraco.Web.UI.Client/src/views/common/dialogs/linkpicker.controller.js index 043d5af1c5..73f6003bb9 100644 --- a/src/Umbraco.Web.UI.Client/src/views/common/dialogs/linkpicker.controller.js +++ b/src/Umbraco.Web.UI.Client/src/views/common/dialogs/linkpicker.controller.js @@ -1,156 +1,163 @@ //used for the media picker dialog angular.module("umbraco").controller("Umbraco.Dialogs.LinkPickerController", function ($scope, eventsService, dialogService, entityResource, contentResource, mediaHelper, userService, localizationService, tinyMceService) { - var dialogOptions = $scope.dialogOptions; + var dialogOptions = $scope.dialogOptions; - var searchText = "Search..."; - localizationService.localize("general_search").then(function (value) { - searchText = value + "..."; - }); + var searchText = "Search..."; + localizationService.localize("general_search").then(function (value) { + searchText = value + "..."; + }); - $scope.dialogTreeEventHandler = $({}); - $scope.target = {}; - $scope.searchInfo = { - searchFromId: null, - searchFromName: null, - showSearch: false, - results: [], - selectedSearchResults: [] - } + $scope.dialogTreeEventHandler = $({}); + $scope.target = {}; + $scope.searchInfo = { + searchFromId: null, + searchFromName: null, + showSearch: false, + results: [], + selectedSearchResults: [] + } - if (dialogOptions.currentTarget) { - $scope.target = dialogOptions.currentTarget; + if (dialogOptions.currentTarget) { + $scope.target = dialogOptions.currentTarget; - //if we have a node ID, we fetch the current node to build the form data - if ($scope.target.id || $scope.target.udi) { + //if we have a node ID, we fetch the current node to build the form data + if ($scope.target.id || $scope.target.udi) { - var id = $scope.target.udi ? $scope.target.udi : $scope.target.id; + var id = $scope.target.udi ? $scope.target.udi : $scope.target.id; - if (!$scope.target.path) { - entityResource.getPath(id, "Document").then(function (path) { - $scope.target.path = path; - //now sync the tree to this path - $scope.dialogTreeEventHandler.syncTree({ path: $scope.target.path, tree: "content" }); - }); - } + if (!$scope.target.path) { + entityResource.getPath(id, "Document").then(function (path) { + $scope.target.path = path; + //now sync the tree to this path + $scope.dialogTreeEventHandler.syncTree({ + path: $scope.target.path, + tree: "content" + }); + }); + } // if a link exists, get the properties to build the anchor name list - contentResource.getById(id).then(function (resp) { + contentResource.getById(id).then(function (resp) { $scope.anchorValues = tinyMceService.getAnchorNames(JSON.stringify(resp.properties)); - $scope.model.target.url = resp.urls[0]; - }); - } - } - + $scope.model.target.url = resp.urls[0]; + }); + } else if ($scope.target.url.length) { + // a url but no id/udi indicates an external link - trim the url to remove the anchor/qs + $scope.target.url = $scope.model.url.substring(0, $scope.model.url.search(/(#|\?)/)); + } + } + if (dialogOptions.anchors) { $scope.anchorValues = dialogOptions.anchors; } - function nodeSelectHandler(ev, args) { - args.event.preventDefault(); - args.event.stopPropagation(); + function nodeSelectHandler(ev, args) { + args.event.preventDefault(); + args.event.stopPropagation(); - if (args.node.metaData.listViewNode) { - //check if list view 'search' node was selected + if (args.node.metaData.listViewNode) { + //check if list view 'search' node was selected - $scope.searchInfo.showSearch = true; - $scope.searchInfo.searchFromId = args.node.metaData.listViewNode.id; - $scope.searchInfo.searchFromName = args.node.metaData.listViewNode.name; - } - else { - eventsService.emit("dialogs.linkPicker.select", args); + $scope.searchInfo.showSearch = true; + $scope.searchInfo.searchFromId = args.node.metaData.listViewNode.id; + $scope.searchInfo.searchFromName = args.node.metaData.listViewNode.name; + } else { + eventsService.emit("dialogs.linkPicker.select", args); - if ($scope.currentNode) { - //un-select if there's a current one selected - $scope.currentNode.selected = false; - } + if ($scope.currentNode) { + //un-select if there's a current one selected + $scope.currentNode.selected = false; + } - $scope.currentNode = args.node; - $scope.currentNode.selected = true; - $scope.target.id = args.node.id; - $scope.target.udi = args.node.udi; - $scope.target.name = args.node.name; + $scope.currentNode = args.node; + $scope.currentNode.selected = true; + $scope.target.id = args.node.id; + $scope.target.udi = args.node.udi; + $scope.target.name = args.node.name; - if (args.node.id < 0) { - $scope.target.url = "/"; - } - else { - contentResource.getById(args.node.id).then(function (resp) { - $scope.anchorValues = tinyMceService.getAnchorNames(JSON.stringify(resp.properties)); + if (args.node.id < 0) { + $scope.target.url = "/"; + } else { + contentResource.getById(args.node.id).then(function (resp) { + $scope.anchorValues = tinyMceService.getAnchorNames(JSON.stringify(resp.properties)); $scope.model.target.url = resp.urls[0]; }); - } + } - if (!angular.isUndefined($scope.target.isMedia)) { - delete $scope.target.isMedia; - } - } - } + if (!angular.isUndefined($scope.target.isMedia)) { + delete $scope.target.isMedia; + } + } + } - function nodeExpandedHandler(ev, args) { - if (angular.isArray(args.children)) { + function nodeExpandedHandler(ev, args) { + if (angular.isArray(args.children)) { - //iterate children - _.each(args.children, function (child) { - //check if any of the items are list views, if so we need to add a custom - // child: A node to activate the search - if (child.metaData.isContainer) { - child.hasChildren = true; - child.children = [ - { - level: child.level + 1, - hasChildren: false, - name: searchText, - metaData: { - listViewNode: child, - }, - cssClass: "icon umb-tree-icon sprTree icon-search", - cssClasses: ["not-published"] + //iterate children + _.each(args.children, function (child) { + //check if any of the items are list views, if so we need to add a custom + // child: A node to activate the search + if (child.metaData.isContainer) { + child.hasChildren = true; + child.children = [ + { + level: child.level + 1, + hasChildren: false, + name: searchText, + metaData: { + listViewNode: child, + }, + cssClass: "icon umb-tree-icon sprTree icon-search", + cssClasses: ["not-published"] } ]; - } - }); - } - } + } + }); + } + } - $scope.switchToMediaPicker = function () { - userService.getCurrentUser().then(function (userData) { - dialogService.mediaPicker({ - startNodeId: userData.startMediaIds.length == 0 ? -1 : userData.startMediaIds[0], - callback: function(media) { - $scope.target.id = media.id; - $scope.target.isMedia = true; - $scope.target.name = media.name; - $scope.target.url = mediaHelper.resolveFile(media); - } - }); - }); - }; + $scope.switchToMediaPicker = function () { + userService.getCurrentUser().then(function (userData) { + dialogService.mediaPicker({ + startNodeId: userData.startMediaIds.length == 0 ? -1 : userData.startMediaIds[0], + callback: function (media) { + $scope.target.id = media.id; + $scope.target.isMedia = true; + $scope.target.name = media.name; + $scope.target.url = mediaHelper.resolveFile(media); + } + }); + }); + }; - $scope.hideSearch = function () { - $scope.searchInfo.showSearch = false; - $scope.searchInfo.searchFromId = null; - $scope.searchInfo.searchFromName = null; - $scope.searchInfo.results = []; - } + $scope.hideSearch = function () { + $scope.searchInfo.showSearch = false; + $scope.searchInfo.searchFromId = null; + $scope.searchInfo.searchFromName = null; + $scope.searchInfo.results = []; + } - // method to select a search result - $scope.selectResult = function (evt, result) { - result.selected = result.selected === true ? false : true; - nodeSelectHandler(evt, {event: evt, node: result}); - }; + // method to select a search result + $scope.selectResult = function (evt, result) { + result.selected = result.selected === true ? false : true; + nodeSelectHandler(evt, { + event: evt, + node: result + }); + }; - //callback when there are search results - $scope.onSearchResults = function (results) { - $scope.searchInfo.results = results; - $scope.searchInfo.showSearch = true; - }; + //callback when there are search results + $scope.onSearchResults = function (results) { + $scope.searchInfo.results = results; + $scope.searchInfo.showSearch = true; + }; - $scope.dialogTreeEventHandler.bind("treeNodeSelect", nodeSelectHandler); - $scope.dialogTreeEventHandler.bind("treeNodeExpanded", nodeExpandedHandler); + $scope.dialogTreeEventHandler.bind("treeNodeSelect", nodeSelectHandler); + $scope.dialogTreeEventHandler.bind("treeNodeExpanded", nodeExpandedHandler); - $scope.$on('$destroy', function () { - $scope.dialogTreeEventHandler.unbind("treeNodeSelect", nodeSelectHandler); - $scope.dialogTreeEventHandler.unbind("treeNodeExpanded", nodeExpandedHandler); - }); - }); \ No newline at end of file + $scope.$on('$destroy', function () { + $scope.dialogTreeEventHandler.unbind("treeNodeSelect", nodeSelectHandler); + $scope.dialogTreeEventHandler.unbind("treeNodeExpanded", nodeExpandedHandler); + }); + }); diff --git a/src/Umbraco.Web.UI.Client/src/views/common/dialogs/linkpicker.html b/src/Umbraco.Web.UI.Client/src/views/common/dialogs/linkpicker.html index 0cc1c52595..ab60f67b61 100644 --- a/src/Umbraco.Web.UI.Client/src/views/common/dialogs/linkpicker.html +++ b/src/Umbraco.Web.UI.Client/src/views/common/dialogs/linkpicker.html @@ -1,27 +1,17 @@
        - - - + + + - + + + + - + /gi; - - var searchText = "Search..."; - localizationService.localize("general_search").then(function (value) { - searchText = value + "..."; - }); - if(!$scope.model.title) { - $scope.model.title = localizationService.localize("defaultdialogs_selectLink"); + var searchText = "Search..."; + localizationService.localize("general_search").then(function (value) { + searchText = value + "..."; + }); + + if (!$scope.model.title) { + $scope.model.title = localizationService.localize("defaultdialogs_selectLink"); } - $scope.dialogTreeEventHandler = $({}); - $scope.model.target = {}; - $scope.searchInfo = { - searchFromId: null, - searchFromName: null, - showSearch: false, - results: [], - selectedSearchResults: [] - }; + $scope.dialogTreeEventHandler = $({}); + $scope.model.target = {}; + $scope.searchInfo = { + searchFromId: null, + searchFromName: null, + showSearch: false, + results: [], + selectedSearchResults: [] + }; $scope.showTarget = $scope.model.hideTarget !== true; - if (dialogOptions.currentTarget) { - $scope.model.target = dialogOptions.currentTarget; - //if we have a node ID, we fetch the current node to build the form data - if ($scope.model.target.id || $scope.model.target.udi) { + if (dialogOptions.currentTarget) { + $scope.model.target = dialogOptions.currentTarget; + //if we have a node ID, we fetch the current node to build the form data + if ($scope.model.target.id || $scope.model.target.udi) { - //will be either a udi or an int - var id = $scope.model.target.udi ? $scope.model.target.udi : $scope.model.target.id; + //will be either a udi or an int + var id = $scope.model.target.udi ? $scope.model.target.udi : $scope.model.target.id; + + if (!$scope.model.target.path) { + + entityResource.getPath(id, "Document").then(function (path) { + $scope.model.target.path = path; + //now sync the tree to this path + $scope.dialogTreeEventHandler.syncTree({ + path: $scope.model.target.path, + tree: "content" + }); + }); + } - if (!$scope.model.target.path) { - - entityResource.getPath(id, "Document").then(function (path) { - $scope.model.target.path = path; - //now sync the tree to this path - $scope.dialogTreeEventHandler.syncTree({ path: $scope.model.target.path, tree: "content" }); - }); - } - // if a link exists, get the properties to build the anchor name list - contentResource.getById(id).then(function (resp) { + contentResource.getById(id).then(function (resp) { $scope.anchorValues = tinyMceService.getAnchorNames(JSON.stringify(resp.properties)); - $scope.model.target.url = resp.urls[0]; - }); - } - } else if (dialogOptions.anchors) { + $scope.model.target.url = resp.urls[0]; + }); + } else if ($scope.model.target.url.length) { + // a url but no id/udi indicates an external link - trim the url to remove the anchor/qs + $scope.model.target.url = $scope.model.target.url.substring(0, $scope.model.target.url.search(/(#|\?)/)); + } + } else if (dialogOptions.anchors) { $scope.anchorValues = dialogOptions.anchors; } - + function nodeSelectHandler(ev, args) { - if(args && args.event) { + if (args && args.event) { args.event.preventDefault(); args.event.stopPropagation(); } @@ -74,10 +80,9 @@ angular.module("umbraco").controller("Umbraco.Overlays.LinkPickerController", if (args.node.id < 0) { $scope.model.target.url = "/"; - } - else { - contentResource.getById(args.node.id).then(function (resp) { - $scope.anchorValues = tinyMceService.getAnchorNames(JSON.stringify(resp.properties)); + } else { + contentResource.getById(args.node.id).then(function (resp) { + $scope.anchorValues = tinyMceService.getAnchorNames(JSON.stringify(resp.properties)); $scope.model.target.url = resp.urls[0]; }); } @@ -85,69 +90,76 @@ angular.module("umbraco").controller("Umbraco.Overlays.LinkPickerController", if (!angular.isUndefined($scope.model.target.isMedia)) { delete $scope.model.target.isMedia; } - } + } - function nodeExpandedHandler(ev, args) { + function nodeExpandedHandler(ev, args) { // open mini list view for list views if (args.node.metaData.isContainer) { openMiniListView(args.node); } - } + } - $scope.switchToMediaPicker = function () { - userService.getCurrentUser().then(function (userData) { - $scope.mediaPickerOverlay = { - view: "mediapicker", - startNodeId: userData.startMediaIds.length !== 1 ? -1 : userData.startMediaIds[0], - startNodeIsVirtual: userData.startMediaIds.length !== 1, - show: true, - submit: function(model) { - var media = model.selectedImages[0]; + $scope.switchToMediaPicker = function () { + userService.getCurrentUser().then(function (userData) { + $scope.mediaPickerOverlay = { + view: "mediapicker", + startNodeId: userData.startMediaIds.length !== 1 ? -1 : userData.startMediaIds[0], + startNodeIsVirtual: userData.startMediaIds.length !== 1, + show: true, + submit: function (model) { + var media = model.selectedImages[0]; - $scope.model.target.id = media.id; - $scope.model.target.udi = media.udi; - $scope.model.target.isMedia = true; - $scope.model.target.name = media.name; - $scope.model.target.url = mediaHelper.resolveFile(media); + $scope.model.target.id = media.id; + $scope.model.target.udi = media.udi; + $scope.model.target.isMedia = true; + $scope.model.target.name = media.name; + $scope.model.target.url = mediaHelper.resolveFile(media); + + debugger; - $scope.mediaPickerOverlay.show = false; - $scope.mediaPickerOverlay = null; - } - }; - }); - }; + $scope.mediaPickerOverlay.show = false; + $scope.mediaPickerOverlay = null; + } + }; + }); + }; - $scope.hideSearch = function () { - $scope.searchInfo.showSearch = false; - $scope.searchInfo.searchFromId = null; - $scope.searchInfo.searchFromName = null; - $scope.searchInfo.results = []; - } + $scope.hideSearch = function () { + $scope.searchInfo.showSearch = false; + $scope.searchInfo.searchFromId = null; + $scope.searchInfo.searchFromName = null; + $scope.searchInfo.results = []; + } - // method to select a search result - $scope.selectResult = function (evt, result) { - result.selected = result.selected === true ? false : true; - nodeSelectHandler(evt, {event: evt, node: result}); - }; + // method to select a search result + $scope.selectResult = function (evt, result) { + result.selected = result.selected === true ? false : true; + nodeSelectHandler(evt, { + event: evt, + node: result + }); + }; - //callback when there are search results - $scope.onSearchResults = function (results) { - $scope.searchInfo.results = results; - $scope.searchInfo.showSearch = true; - }; + //callback when there are search results + $scope.onSearchResults = function (results) { + $scope.searchInfo.results = results; + $scope.searchInfo.showSearch = true; + }; - $scope.dialogTreeEventHandler.bind("treeNodeSelect", nodeSelectHandler); - $scope.dialogTreeEventHandler.bind("treeNodeExpanded", nodeExpandedHandler); + $scope.dialogTreeEventHandler.bind("treeNodeSelect", nodeSelectHandler); + $scope.dialogTreeEventHandler.bind("treeNodeExpanded", nodeExpandedHandler); - $scope.$on('$destroy', function () { - $scope.dialogTreeEventHandler.unbind("treeNodeSelect", nodeSelectHandler); - $scope.dialogTreeEventHandler.unbind("treeNodeExpanded", nodeExpandedHandler); - }); + $scope.$on('$destroy', function () { + $scope.dialogTreeEventHandler.unbind("treeNodeSelect", nodeSelectHandler); + $scope.dialogTreeEventHandler.unbind("treeNodeExpanded", nodeExpandedHandler); + }); // Mini list view $scope.selectListViewNode = function (node) { node.selected = node.selected === true ? false : true; - nodeSelectHandler({}, { node: node }); + nodeSelectHandler({}, { + node: node + }); }; $scope.closeMiniListView = function () { diff --git a/src/Umbraco.Web.UI.Client/src/views/common/overlays/linkpicker/linkpicker.html b/src/Umbraco.Web.UI.Client/src/views/common/overlays/linkpicker/linkpicker.html index 2ad9177e8c..80d2b6cddd 100644 --- a/src/Umbraco.Web.UI.Client/src/views/common/overlays/linkpicker/linkpicker.html +++ b/src/Umbraco.Web.UI.Client/src/views/common/overlays/linkpicker/linkpicker.html @@ -1,100 +1,62 @@ diff --git a/src/Umbraco.Web.UI/umbraco/config/lang/en.xml b/src/Umbraco.Web.UI/umbraco/config/lang/en.xml index 22596ff9dd..640a4275db 100644 --- a/src/Umbraco.Web.UI/umbraco/config/lang/en.xml +++ b/src/Umbraco.Web.UI/umbraco/config/lang/en.xml @@ -318,6 +318,7 @@ Link title Link + Anchor / querystring Name Manage hostnames Close this window @@ -436,6 +437,7 @@ Enter your email... Enter a message... Your username is usually your email + #value or ?key=value Allow at root @@ -527,7 +529,6 @@ Add Alias All - Anchor Are you sure? Back Border diff --git a/src/Umbraco.Web.UI/umbraco/config/lang/en_us.xml b/src/Umbraco.Web.UI/umbraco/config/lang/en_us.xml index 23d7f3698b..a53db9a50a 100644 --- a/src/Umbraco.Web.UI/umbraco/config/lang/en_us.xml +++ b/src/Umbraco.Web.UI/umbraco/config/lang/en_us.xml @@ -318,6 +318,7 @@ Link title Link + Anchor / querystring Name Close this window Are you sure you want to delete @@ -435,6 +436,7 @@ Enter your email... Enter a message... Your username is usually your email + #value or ?key=value Allow at root @@ -526,7 +528,6 @@ Add Alias All - Anchor Are you sure? Back Border From f9761ad2f848c6ab421814b09fe3fb1916c043aa Mon Sep 17 00:00:00 2001 From: Nathan Woulfe Date: Thu, 12 Jul 2018 14:32:13 +1000 Subject: [PATCH 30/47] auto save on create to fetch existing view --- .../src/views/templates/edit.controller.js | 19 ++++++++++++++++++- 1 file changed, 18 insertions(+), 1 deletion(-) diff --git a/src/Umbraco.Web.UI.Client/src/views/templates/edit.controller.js b/src/Umbraco.Web.UI.Client/src/views/templates/edit.controller.js index fa30571708..1e2934028b 100644 --- a/src/Umbraco.Web.UI.Client/src/views/templates/edit.controller.js +++ b/src/Umbraco.Web.UI.Client/src/views/templates/edit.controller.js @@ -27,7 +27,7 @@ vm.page.keyboardShortcutsOverview.push(templateHelper.getTemplateEditorShortcuts()); - vm.save = function () { + vm.save = function (suppressNotification) { vm.page.saveButtonState = "busy"; vm.template.content = vm.editor.getValue(); @@ -44,11 +44,13 @@ rebindCallback: function (orignal, saved) {} }).then(function (saved) { + if (!suppressNotification) { localizationService.localizeMany(["speechBubbles_templateSavedHeader", "speechBubbles_templateSavedText"]).then(function(data){ var header = data[0]; var message = data[1]; notificationsService.success(header, message); }); + } vm.page.saveButtonState = "success"; @@ -134,6 +136,21 @@ vm.page.loading = false; vm.template = template; + // if this is a new template, bind to the blur event on the name + if ($routeParams.create) { + $timeout(function() { + var nameField = angular.element(document.querySelector('[data-element="editor-name-field"]')); + if (nameField) { + nameField.bind('blur', function(event) { + if (event.target.value) { + vm.save(true); + } + }); + } + }); + } + + //sync state editorState.set(vm.template); navigationService.syncTree({ tree: "templates", path: vm.template.path, forceReload: true }).then(function (syncArgs) { From 2a2cea0b7d957d678bc4997d22b44c1680d78fee Mon Sep 17 00:00:00 2001 From: Michael Latouche Date: Thu, 12 Jul 2018 13:00:33 +0200 Subject: [PATCH 31/47] Update fr.xml --- src/Umbraco.Web.UI/umbraco/config/lang/fr.xml | 30 +++++++++---------- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/src/Umbraco.Web.UI/umbraco/config/lang/fr.xml b/src/Umbraco.Web.UI/umbraco/config/lang/fr.xml index afc06213f3..37c22b1a44 100644 --- a/src/Umbraco.Web.UI/umbraco/config/lang/fr.xml +++ b/src/Umbraco.Web.UI/umbraco/config/lang/fr.xml @@ -62,7 +62,7 @@ Permettre de copier un noeud Permettre de créer des noeuds Permettre de supprimer des noeuds - Permettre de déplacemer un noeud + Permettre de déplacer un noeud Permettre de définir et modifier l'accès public à un noeud Permettre de publier un noeud Permettre de modifier les permissions pour un noeud @@ -218,7 +218,7 @@ Oups: ce document est publié mais son url entrerait en collision avec le contenu %0% Publier Publié - Publié(changements en cours) + Publié (changements en cours) Statut de publication Publié le Dépublié le @@ -265,7 +265,7 @@ Lien vers le média ou cliquez ici pour choisir un fichier Les seuls types de fichiers autorisés sont - Ce fichier ne peut pas ête chargé, il n'est pas un type de fichier autorisé. + Ce fichier ne peut pas ête chargé, il n'est pas d'un type de fichier autorisé. La taille maximum de fichier est Racine du média @@ -276,7 +276,7 @@ Où voulez-vous créer le nouveau %0% Créer un élément sous - Sélectionnez le type de document pour lequel vous voulez créer un modèle de contenu + Sélectionnez le type de document pour lequel vous souhaitez créer un modèle de contenu Choisissez un type et un titre "Types de documents".]]> "Types de médias".]]> @@ -305,7 +305,7 @@ Invalider les changements Vous avez des changements en cours Etes-vous certain(e) de vouloir quitter cette page? - vous avez des changements en cours - La dépublication va supprimer cette page ainsi que tous ses descendants du site. + La dépublication va supprimer du site cette page ainsi que tous ses descendants. Terminé @@ -385,7 +385,7 @@ en faisant référence à cet ID au sein d'un élément <asp:content />.]]> - Séléctionnez un placeholder id dans la liste ci-dessous. Vous pouvez seulement + Séléctionnez un placeholder ID dans la liste ci-dessous. Vous pouvez seulement choisir un ID se trouvant dans le parent du modèle actuel.]]> Cliquez sur l'image pour la voir en taille réelle @@ -861,7 +861,7 @@ Vous avez installé Runway, alors pourquoi ne pas jeter un oeil au look de votre Aide et informations complémentaires -Obtenez de l'aide de notre "award winning" communauté, parcourez la documentation ou regardez quelques vidéos gratuites sur la manière de construire un site simple, d'utiliser les packages ainsi qu'un guide rapide sur la terminologie Umbraco]]> +Obtenez de l'aide de notre communauté "award winning", parcourez la documentation ou regardez quelques vidéos gratuites sur la manière de construire un site simple, d'utiliser les packages ainsi qu'un guide rapide sur la terminologie Umbraco]]> Umbraco %0% est installé et prêt à être utilisé @@ -1077,7 +1077,7 @@ Pour gérer votre site, ouvrez simplement le backoffice Umbraco et commencez à Salut %0%,

        - Ceci est un email automatique pour vous informer que la tâche '%1%' a été executée sur la page '%2%' par l'utilisateur '%3%' + Ceci est un email automatique pour vous informer que la tâche '%1%' a été exécutée sur la page '%2%' par l'utilisateur '%3%'

        @@ -1831,7 +1831,7 @@ Pour gérer votre site, ouvrez simplement le backoffice Umbraco et commencez à Accès - Sur base des groupes et noeuds de départ, l'utilisateur a accès aux noeuds suivants + Sur base des groupes et des noeuds de départ, l'utilisateur a accès aux noeuds suivantsDonner accèsAdministrateurChamp catégorie @@ -1845,7 +1845,7 @@ Pour gérer votre site, ouvrez simplement le backoffice Umbraco et commencez à Vous pouvez changer votre mot de passe d'accès au Back Office Umbraco en remplissant le formulaire ci-dessous puis en cliquant sur le bouton "Changer le mot de passe"Canal de contenuCréer un autre utilisateur - Créer de nouveaux utilisateurs pour leur donner accès à Umbraco. Lors de la création d'un nouvel utilisateur, un mot de passe est généré que vous pouvez partager avec l'utilisateur. + Créer de nouveaux utilisateurs pour leur donner accès à Umbraco. Lors de la création d'un nouvel utilisateur, un mot de passe est généré que vous pouvez partager avec ce dernier.Champ descriptionDésactiver l'utilisateurType de document @@ -1889,7 +1889,7 @@ Pour gérer votre site, ouvrez simplement le backoffice Umbraco et commencez à Définir les permissions sur des noeuds spécifiquesProfilRechercher tous les enfants - Ajouter les sections auxquelles les utilisateur peuvent accéder + Ajouter les sections auxquelles les utilisateurs peuvent accéderSélectionner les groupes d'utilisateursAucun noeud de départ sélectionnéAucun noeud de départ sélectionné @@ -1917,7 +1917,7 @@ Pour gérer votre site, ouvrez simplement le backoffice Umbraco et commencez à Groupe d'utilisateursGroupes d'utilisateursa été invité - Une invitation a été envoyée au nouvel utilisateur avec les détails concernant la oonnexion à Umbraco. + Une invitation a été envoyée au nouvel utilisateur avec les détails concernant la connexion à Umbraco.Bien le bonjour et bienvenue dans Umbraco! Vous serez prêt.e dans moins d'1 minute, vous devez encore simplement configurer votre mot de passe et ajouter une photo pour votre avatar.Chargez une photo afin que les autres utilisateurs puissent vous reconnaître facilement.Rédacteur @@ -2144,17 +2144,17 @@ Pour gérer votre site, ouvrez simplement le backoffice Umbraco et commencez à X-Content-Type-Options utilisé pour la protection contre les vulnérabilités de MIME sniffing a été trouvé.]]>X-Content-Type-Options utilisé pour la protection contre les vulnérabilités de MIME sniffing n'a pas été trouvé.]]>Ajoute une valeur dans la section httpProtocol/customHeaders du fichier web.config pour protéger contre les vulnérabilités de MIME sniffing. - Une configuration a été ajoutée à votre fichier web.config pour créer un header protégeant contre les vulnérabilités de MIME sniffing. + Une configuration a été ajoutée dans votre fichier web.config pour créer un header protégeant contre les vulnérabilités de MIME sniffing.Strict-Transport-Security, aussi connu sous le nom de HSTS-header, a été trouvé.]]>Strict-Transport-Security, aussi connu sous le nom de HSTS-header, n'a pas été trouvé.]]>Ajoute l'en-tête 'Strict-Transport-Security' avec la valeur 'max-age=10886400; preload' à la section httpProtocol/customHeaders du fichier web.config. Utilisez cette correction uniquement si vos domaines vont fonctionner en https pour les 18 prochaines semaines (minimum). - L'en-tête HSTS a été ajouté à votre fichier web.config. + L'en-tête HSTS a été ajouté dans votre fichier web.config.X-XSS-Protection a été trouvé.]]>X-XSS-Protection n'a pas été trouvé.]]>Ajoute l'en-tête 'X-XSS-Protection' avec la valeur '1; mode=block' à la section httpProtocol/customHeaders du fichier web.config. - L'en-tête X-XSS-Protection a été ajouté à votre fichier web.config. + L'en-tête X-XSS-Protection a été ajouté dans votre fichier web.config. + + + + \ No newline at end of file From a03173da4dc1a964e0ba8771c166ba0aadf1e3d1 Mon Sep 17 00:00:00 2001 From: ssougnez Date: Sat, 14 Jul 2018 00:11:04 +0200 Subject: [PATCH 35/47] Update the code to use the GetFieldName method instead of recreating it --- .../Persistence/PetaPocoSqlExtensions.cs | 36 ++++++++----------- 1 file changed, 14 insertions(+), 22 deletions(-) diff --git a/src/Umbraco.Core/Persistence/PetaPocoSqlExtensions.cs b/src/Umbraco.Core/Persistence/PetaPocoSqlExtensions.cs index 7b383f85d1..1af0f4d4fa 100644 --- a/src/Umbraco.Core/Persistence/PetaPocoSqlExtensions.cs +++ b/src/Umbraco.Core/Persistence/PetaPocoSqlExtensions.cs @@ -24,7 +24,7 @@ namespace Umbraco.Core.Persistence [Obsolete("Use the overload specifying ISqlSyntaxProvider instead")] public static Sql Select(this Sql sql, params Expression>[] fields) { - return sql.Select(GetColumnsList(SqlSyntaxContext.SqlSyntaxProvider, fields)); + return sql.Select(GetFieldNames(SqlSyntaxContext.SqlSyntaxProvider, fields)); } /// @@ -37,7 +37,7 @@ namespace Umbraco.Core.Persistence /// public static Sql Select(this Sql sql, ISqlSyntaxProvider sqlSyntax, params Expression>[] fields) { - return sql.Select(GetColumnsList(sqlSyntax, fields)); + return sql.Select(GetFieldNames(sqlSyntax, fields)); } /// @@ -49,7 +49,7 @@ namespace Umbraco.Core.Persistence /// public static Sql AndSelect(this Sql sql, params Expression>[] fields) { - return sql.AndSelect(GetColumnsList(SqlSyntaxContext.SqlSyntaxProvider, fields)); + return sql.AndSelect(GetFieldNames(SqlSyntaxContext.SqlSyntaxProvider, fields)); } /// @@ -62,7 +62,7 @@ namespace Umbraco.Core.Persistence /// public static Sql AndSelect(this Sql sql, ISqlSyntaxProvider sqlSyntax, params Expression>[] fields) { - return sql.AndSelect(GetColumnsList(sqlSyntax, fields)); + return sql.AndSelect(GetFieldNames(sqlSyntax, fields)); } [Obsolete("Use the overload specifying ISqlSyntaxProvider instead")] @@ -105,6 +105,16 @@ namespace Umbraco.Core.Persistence return sqlSyntax.GetQuotedTableName(tableName) + "." + sqlSyntax.GetQuotedColumnName(fieldName); } + private static string[] GetFieldNames(ISqlSyntaxProvider sqlSyntax, params Expression>[] fields) + { + if (fields.Length == 0) + { + return new[] { string.Format("{0}.*", sqlSyntax.GetQuotedTableName(typeof(T).GetTableName())) }; + } + + return fields.Select(field => GetFieldName(field, sqlSyntax)).ToArray(); + } + [Obsolete("Use the overload specifying ISqlSyntaxProvider instead")] public static Sql WhereIn(this Sql sql, Expression> fieldSelector, IEnumerable values) { @@ -289,23 +299,5 @@ namespace Umbraco.Core.Persistence var attr = column.FirstAttribute(); return attr == null || string.IsNullOrWhiteSpace(attr.Name) ? column.Name : attr.Name; } - - private static string[] GetColumnsList(ISqlSyntaxProvider sqlSyntax, params Expression>[] fields) - { - string tableName = sqlSyntax.GetQuotedTableName(GetTableName(typeof(T))); - - if (fields.Length == 0) - { - return new[] { string.Format("{0}.*", tableName) }; - } - - return fields.Select(field => - { - var column = ExpressionHelper.FindProperty(field) as PropertyInfo; - var columnName = GetColumnName(column); - - return string.Format("{0}.{1}", tableName, sqlSyntax.GetQuotedColumnName(columnName)); - }).ToArray(); - } } } From 7a07649e2b3acd58876fbf1ca515079166814953 Mon Sep 17 00:00:00 2001 From: Morten Larsen Date: Sat, 14 Jul 2018 10:26:46 +0200 Subject: [PATCH 36/47] Update links in the noNodes splash screen --- src/Umbraco.Web.UI/config/splashes/noNodes.aspx | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/Umbraco.Web.UI/config/splashes/noNodes.aspx b/src/Umbraco.Web.UI/config/splashes/noNodes.aspx index d46ecd6113..bf68c3ac32 100644 --- a/src/Umbraco.Web.UI/config/splashes/noNodes.aspx +++ b/src/Umbraco.Web.UI/config/splashes/noNodes.aspx @@ -39,14 +39,14 @@

        Easy start with Umbraco.tv

        We have created a bunch of 'how-to' videos, to get you easily started with Umbraco. Learn how to build projects in just a couple of minutes. Easiest CMS in the world.

        - Umbraco.tv → + Umbraco.tv →

        Be a part of the community

        The Umbraco community is the best of its kind, be sure to visit, and if you have any questions, we're sure that you can get your answers from the community.

        - our.Umbraco → + our.Umbraco →
        From 9daae665e4afe731798a581e0772fc1ce35b1ebe Mon Sep 17 00:00:00 2001 From: Sebastiaan Janssen Date: Sat, 14 Jul 2018 17:08:42 +0200 Subject: [PATCH 37/47] Adds unit tests for IsAncestor(OrSelf) and IsDescendant(OrSelf) --- .../PublishedContent/PublishedContentTests.cs | 145 +++++++++++++++++- 1 file changed, 144 insertions(+), 1 deletion(-) diff --git a/src/Umbraco.Tests/PublishedContent/PublishedContentTests.cs b/src/Umbraco.Tests/PublishedContent/PublishedContentTests.cs index 9ffe42b7f3..8cd3f07238 100644 --- a/src/Umbraco.Tests/PublishedContent/PublishedContentTests.cs +++ b/src/Umbraco.Tests/PublishedContent/PublishedContentTests.cs @@ -94,7 +94,9 @@ namespace Umbraco.Tests.PublishedContent - + + + 1 @@ -590,6 +592,77 @@ namespace Umbraco.Tests.PublishedContent Assert.IsTrue(result.Select(x => ((dynamic)x).Id).ContainsAll(new dynamic[] { 1173, 1046 })); } + [Test] + public void IsAncestor() + { + // Structure: + // - Root : 1046 (no parent) + // -- Home: 1173 (parent 1046) + // -- Custom Doc: 1178 (parent 1173) + // --- Custom Doc2: 1179 (parent: 1178) + + var home = GetNode(1173); + var root = GetNode(1046); + var customDoc = GetNode(1178); + var customDoc2 = GetNode(1179); + + Assert.IsTrue(root.IsAncestor(customDoc2)); + Assert.IsTrue(root.IsAncestor(customDoc)); + Assert.IsTrue(root.IsAncestor(home)); + Assert.IsFalse(root.IsAncestor(root)); + + Assert.IsTrue(home.IsAncestor(customDoc2)); + Assert.IsTrue(home.IsAncestor(customDoc)); + Assert.IsFalse(home.IsAncestor(home)); + Assert.IsFalse(home.IsAncestor(root)); + + Assert.IsTrue(customDoc.IsAncestor(customDoc2)); + Assert.IsFalse(customDoc.IsAncestor(customDoc)); + Assert.IsFalse(customDoc.IsAncestor(home)); + Assert.IsFalse(customDoc.IsAncestor(root)); + + Assert.IsFalse(customDoc2.IsAncestor(customDoc2)); + Assert.IsFalse(customDoc2.IsAncestor(customDoc)); + Assert.IsFalse(customDoc2.IsAncestor(home)); + Assert.IsFalse(customDoc2.IsAncestor(root)); + } + + [Test] + public void IsAncestorOrSelf() + { + // Structure: + // - Root : 1046 (no parent) + // -- Home: 1173 (parent 1046) + // -- Custom Doc: 1178 (parent 1173) + // --- Custom Doc2: 1179 (parent: 1178) + + var home = GetNode(1173); + var root = GetNode(1046); + var customDoc = GetNode(1178); + var customDoc2 = GetNode(1179); + + Assert.IsTrue(root.IsAncestorOrSelf(customDoc2)); + Assert.IsTrue(root.IsAncestorOrSelf(customDoc)); + Assert.IsTrue(root.IsAncestorOrSelf(home)); + Assert.IsTrue(root.IsAncestorOrSelf(root)); + + Assert.IsTrue(home.IsAncestorOrSelf(customDoc2)); + Assert.IsTrue(home.IsAncestorOrSelf(customDoc)); + Assert.IsTrue(home.IsAncestorOrSelf(home)); + Assert.IsFalse(home.IsAncestorOrSelf(root)); + + Assert.IsTrue(customDoc.IsAncestorOrSelf(customDoc2)); + Assert.IsTrue(customDoc.IsAncestorOrSelf(customDoc)); + Assert.IsFalse(customDoc.IsAncestorOrSelf(home)); + Assert.IsFalse(customDoc.IsAncestorOrSelf(root)); + + Assert.IsTrue(customDoc2.IsAncestorOrSelf(customDoc2)); + Assert.IsFalse(customDoc2.IsAncestorOrSelf(customDoc)); + Assert.IsFalse(customDoc2.IsAncestorOrSelf(home)); + Assert.IsFalse(customDoc2.IsAncestorOrSelf(root)); + } + + [Test] public void Descendants_Or_Self() { @@ -616,6 +689,76 @@ namespace Umbraco.Tests.PublishedContent Assert.IsTrue(result.Select(x => ((dynamic)x).Id).ContainsAll(new dynamic[] { 1173, 1174, 1176, 1175, 4444 })); } + [Test] + public void IsDescendant() + { + // Structure: + // - Root : 1046 (no parent) + // -- Home: 1173 (parent 1046) + // -- Custom Doc: 1178 (parent 1173) + // --- Custom Doc2: 1179 (parent: 1178) + + var home = GetNode(1173); + var root = GetNode(1046); + var customDoc = GetNode(1178); + var customDoc2 = GetNode(1179); + + Assert.IsFalse(root.IsDescendant(root)); + Assert.IsFalse(root.IsDescendant(home)); + Assert.IsFalse(root.IsDescendant(customDoc)); + Assert.IsFalse(root.IsDescendant(customDoc2)); + + Assert.IsTrue(home.IsDescendant(root)); + Assert.IsFalse(home.IsDescendant(home)); + Assert.IsFalse(home.IsDescendant(customDoc)); + Assert.IsFalse(home.IsDescendant(customDoc2)); + + Assert.IsTrue(customDoc.IsDescendant(root)); + Assert.IsTrue(customDoc.IsDescendant(home)); + Assert.IsFalse(customDoc.IsDescendant(customDoc)); + Assert.IsFalse(customDoc.IsDescendant(customDoc2)); + + Assert.IsTrue(customDoc2.IsDescendant(root)); + Assert.IsTrue(customDoc2.IsDescendant(home)); + Assert.IsTrue(customDoc2.IsDescendant(customDoc)); + Assert.IsFalse(customDoc2.IsDescendant(customDoc2)); + } + + [Test] + public void IsDescendantOrSelf() + { + // Structure: + // - Root : 1046 (no parent) + // -- Home: 1173 (parent 1046) + // -- Custom Doc: 1178 (parent 1173) + // --- Custom Doc2: 1179 (parent: 1178) + + var home = GetNode(1173); + var root = GetNode(1046); + var customDoc = GetNode(1178); + var customDoc2 = GetNode(1179); + + Assert.IsTrue(root.IsDescendantOrSelf(root)); + Assert.IsFalse(root.IsDescendantOrSelf(home)); + Assert.IsFalse(root.IsDescendantOrSelf(customDoc)); + Assert.IsFalse(root.IsDescendantOrSelf(customDoc2)); + + Assert.IsTrue(home.IsDescendantOrSelf(root)); + Assert.IsTrue(home.IsDescendantOrSelf(home)); + Assert.IsFalse(home.IsDescendantOrSelf(customDoc)); + Assert.IsFalse(home.IsDescendantOrSelf(customDoc2)); + + Assert.IsTrue(customDoc.IsDescendantOrSelf(root)); + Assert.IsTrue(customDoc.IsDescendantOrSelf(home)); + Assert.IsTrue(customDoc.IsDescendantOrSelf(customDoc)); + Assert.IsFalse(customDoc.IsDescendantOrSelf(customDoc2)); + + Assert.IsTrue(customDoc2.IsDescendantOrSelf(root)); + Assert.IsTrue(customDoc2.IsDescendantOrSelf(home)); + Assert.IsTrue(customDoc2.IsDescendantOrSelf(customDoc)); + Assert.IsTrue(customDoc2.IsDescendantOrSelf(customDoc2)); + } + [Test] public void Up() { From 861b2c61a849b26f4d124c689ead7ad120089014 Mon Sep 17 00:00:00 2001 From: Sebastiaan Janssen Date: Sat, 14 Jul 2018 17:16:59 +0200 Subject: [PATCH 38/47] Adds a few more test cases --- .../PublishedContent/PublishedContentTests.cs | 36 +++++++++++++++++-- 1 file changed, 34 insertions(+), 2 deletions(-) diff --git a/src/Umbraco.Tests/PublishedContent/PublishedContentTests.cs b/src/Umbraco.Tests/PublishedContent/PublishedContentTests.cs index 8cd3f07238..261fc738fe 100644 --- a/src/Umbraco.Tests/PublishedContent/PublishedContentTests.cs +++ b/src/Umbraco.Tests/PublishedContent/PublishedContentTests.cs @@ -600,31 +600,39 @@ namespace Umbraco.Tests.PublishedContent // -- Home: 1173 (parent 1046) // -- Custom Doc: 1178 (parent 1173) // --- Custom Doc2: 1179 (parent: 1178) + // - Custom Doc3: 1172 (no parent) var home = GetNode(1173); var root = GetNode(1046); var customDoc = GetNode(1178); var customDoc2 = GetNode(1179); + var customDoc3 = GetNode(1172); + Assert.IsFalse(root.IsAncestor(customDoc3)); Assert.IsTrue(root.IsAncestor(customDoc2)); Assert.IsTrue(root.IsAncestor(customDoc)); Assert.IsTrue(root.IsAncestor(home)); Assert.IsFalse(root.IsAncestor(root)); + Assert.IsFalse(home.IsAncestor(customDoc3)); Assert.IsTrue(home.IsAncestor(customDoc2)); Assert.IsTrue(home.IsAncestor(customDoc)); Assert.IsFalse(home.IsAncestor(home)); Assert.IsFalse(home.IsAncestor(root)); + Assert.IsFalse(customDoc.IsAncestor(customDoc3)); Assert.IsTrue(customDoc.IsAncestor(customDoc2)); Assert.IsFalse(customDoc.IsAncestor(customDoc)); Assert.IsFalse(customDoc.IsAncestor(home)); Assert.IsFalse(customDoc.IsAncestor(root)); + Assert.IsFalse(customDoc2.IsAncestor(customDoc3)); Assert.IsFalse(customDoc2.IsAncestor(customDoc2)); Assert.IsFalse(customDoc2.IsAncestor(customDoc)); Assert.IsFalse(customDoc2.IsAncestor(home)); Assert.IsFalse(customDoc2.IsAncestor(root)); + + Assert.IsFalse(customDoc3.IsAncestor(customDoc3)); } [Test] @@ -635,31 +643,39 @@ namespace Umbraco.Tests.PublishedContent // -- Home: 1173 (parent 1046) // -- Custom Doc: 1178 (parent 1173) // --- Custom Doc2: 1179 (parent: 1178) + // - Custom Doc3: 1172 (no parent) var home = GetNode(1173); var root = GetNode(1046); var customDoc = GetNode(1178); var customDoc2 = GetNode(1179); + var customDoc3 = GetNode(1172); + Assert.IsFalse(root.IsAncestorOrSelf(customDoc3)); Assert.IsTrue(root.IsAncestorOrSelf(customDoc2)); Assert.IsTrue(root.IsAncestorOrSelf(customDoc)); Assert.IsTrue(root.IsAncestorOrSelf(home)); Assert.IsTrue(root.IsAncestorOrSelf(root)); + Assert.IsFalse(home.IsAncestorOrSelf(customDoc3)); Assert.IsTrue(home.IsAncestorOrSelf(customDoc2)); Assert.IsTrue(home.IsAncestorOrSelf(customDoc)); Assert.IsTrue(home.IsAncestorOrSelf(home)); Assert.IsFalse(home.IsAncestorOrSelf(root)); + Assert.IsFalse(customDoc.IsAncestorOrSelf(customDoc3)); Assert.IsTrue(customDoc.IsAncestorOrSelf(customDoc2)); Assert.IsTrue(customDoc.IsAncestorOrSelf(customDoc)); Assert.IsFalse(customDoc.IsAncestorOrSelf(home)); Assert.IsFalse(customDoc.IsAncestorOrSelf(root)); + Assert.IsFalse(customDoc2.IsAncestorOrSelf(customDoc3)); Assert.IsTrue(customDoc2.IsAncestorOrSelf(customDoc2)); Assert.IsFalse(customDoc2.IsAncestorOrSelf(customDoc)); Assert.IsFalse(customDoc2.IsAncestorOrSelf(home)); Assert.IsFalse(customDoc2.IsAncestorOrSelf(root)); + + Assert.IsTrue(customDoc3.IsAncestorOrSelf(customDoc3)); } @@ -697,31 +713,39 @@ namespace Umbraco.Tests.PublishedContent // -- Home: 1173 (parent 1046) // -- Custom Doc: 1178 (parent 1173) // --- Custom Doc2: 1179 (parent: 1178) - + // - Custom Doc3: 1172 (no parent) + var home = GetNode(1173); var root = GetNode(1046); var customDoc = GetNode(1178); var customDoc2 = GetNode(1179); + var customDoc3 = GetNode(1172); Assert.IsFalse(root.IsDescendant(root)); Assert.IsFalse(root.IsDescendant(home)); Assert.IsFalse(root.IsDescendant(customDoc)); Assert.IsFalse(root.IsDescendant(customDoc2)); + Assert.IsFalse(root.IsDescendant(customDoc3)); Assert.IsTrue(home.IsDescendant(root)); Assert.IsFalse(home.IsDescendant(home)); Assert.IsFalse(home.IsDescendant(customDoc)); Assert.IsFalse(home.IsDescendant(customDoc2)); + Assert.IsFalse(home.IsDescendant(customDoc3)); Assert.IsTrue(customDoc.IsDescendant(root)); Assert.IsTrue(customDoc.IsDescendant(home)); Assert.IsFalse(customDoc.IsDescendant(customDoc)); Assert.IsFalse(customDoc.IsDescendant(customDoc2)); + Assert.IsFalse(customDoc.IsDescendant(customDoc3)); Assert.IsTrue(customDoc2.IsDescendant(root)); Assert.IsTrue(customDoc2.IsDescendant(home)); Assert.IsTrue(customDoc2.IsDescendant(customDoc)); Assert.IsFalse(customDoc2.IsDescendant(customDoc2)); + Assert.IsFalse(customDoc2.IsDescendant(customDoc3)); + + Assert.IsFalse(customDoc3.IsDescendant(customDoc3)); } [Test] @@ -732,31 +756,39 @@ namespace Umbraco.Tests.PublishedContent // -- Home: 1173 (parent 1046) // -- Custom Doc: 1178 (parent 1173) // --- Custom Doc2: 1179 (parent: 1178) - + // - Custom Doc3: 1172 (no parent) + var home = GetNode(1173); var root = GetNode(1046); var customDoc = GetNode(1178); var customDoc2 = GetNode(1179); + var customDoc3 = GetNode(1172); Assert.IsTrue(root.IsDescendantOrSelf(root)); Assert.IsFalse(root.IsDescendantOrSelf(home)); Assert.IsFalse(root.IsDescendantOrSelf(customDoc)); Assert.IsFalse(root.IsDescendantOrSelf(customDoc2)); + Assert.IsFalse(root.IsDescendantOrSelf(customDoc3)); Assert.IsTrue(home.IsDescendantOrSelf(root)); Assert.IsTrue(home.IsDescendantOrSelf(home)); Assert.IsFalse(home.IsDescendantOrSelf(customDoc)); Assert.IsFalse(home.IsDescendantOrSelf(customDoc2)); + Assert.IsFalse(home.IsDescendantOrSelf(customDoc3)); Assert.IsTrue(customDoc.IsDescendantOrSelf(root)); Assert.IsTrue(customDoc.IsDescendantOrSelf(home)); Assert.IsTrue(customDoc.IsDescendantOrSelf(customDoc)); Assert.IsFalse(customDoc.IsDescendantOrSelf(customDoc2)); + Assert.IsFalse(customDoc.IsDescendantOrSelf(customDoc3)); Assert.IsTrue(customDoc2.IsDescendantOrSelf(root)); Assert.IsTrue(customDoc2.IsDescendantOrSelf(home)); Assert.IsTrue(customDoc2.IsDescendantOrSelf(customDoc)); Assert.IsTrue(customDoc2.IsDescendantOrSelf(customDoc2)); + Assert.IsFalse(customDoc2.IsDescendantOrSelf(customDoc3)); + + Assert.IsTrue(customDoc3.IsDescendantOrSelf(customDoc3)); } [Test] From ca993187b00fff908484806da196e86d04451c01 Mon Sep 17 00:00:00 2001 From: Sebastiaan Janssen Date: Sat, 14 Jul 2018 17:37:14 +0200 Subject: [PATCH 39/47] Fixes unit tests after adding an extra node in the cache --- src/Umbraco.Tests/PublishedContent/PublishedContentTests.cs | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/Umbraco.Tests/PublishedContent/PublishedContentTests.cs b/src/Umbraco.Tests/PublishedContent/PublishedContentTests.cs index 261fc738fe..abb22c2478 100644 --- a/src/Umbraco.Tests/PublishedContent/PublishedContentTests.cs +++ b/src/Umbraco.Tests/PublishedContent/PublishedContentTests.cs @@ -315,7 +315,7 @@ namespace Umbraco.Tests.PublishedContent { var doc = GetNode(1046); - var expected = new[] { 1046, 1173, 1174, 1177, 1178, 1176, 1175, 4444, 1172 }; + var expected = new[] { 1046, 1173, 1174, 1177, 1178, 1179, 1176, 1175, 4444, 1172 }; var exindex = 0; // must respect the XPath descendants-or-self axis! @@ -688,7 +688,7 @@ namespace Umbraco.Tests.PublishedContent Assert.IsNotNull(result); - Assert.AreEqual(8, result.Count()); + Assert.AreEqual(9, result.Count()); Assert.IsTrue(result.Select(x => ((dynamic)x).Id).ContainsAll(new dynamic[] { 1046, 1173, 1174, 1176, 1175 })); } @@ -701,7 +701,7 @@ namespace Umbraco.Tests.PublishedContent Assert.IsNotNull(result); - Assert.AreEqual(7, result.Count()); + Assert.AreEqual(8, result.Count()); Assert.IsTrue(result.Select(x => ((dynamic)x).Id).ContainsAll(new dynamic[] { 1173, 1174, 1176, 1175, 4444 })); } From 65d68aefadbe77f8d7d92355ad897eb76589cffd Mon Sep 17 00:00:00 2001 From: ssougnez Date: Sat, 14 Jul 2018 20:18:17 +0200 Subject: [PATCH 40/47] - Removed new obsolete method - Fix tests --- .../Persistence/PetaPocoSqlExtensions.cs | 27 +---- .../Persistence/Querying/PetaPocoSqlTests.cs | 107 +++++++++++------- src/Umbraco.Web.UI.Client/vwd.webinfo | 8 -- 3 files changed, 67 insertions(+), 75 deletions(-) delete mode 100644 src/Umbraco.Web.UI.Client/vwd.webinfo diff --git a/src/Umbraco.Core/Persistence/PetaPocoSqlExtensions.cs b/src/Umbraco.Core/Persistence/PetaPocoSqlExtensions.cs index 1af0f4d4fa..b9cb8801b9 100644 --- a/src/Umbraco.Core/Persistence/PetaPocoSqlExtensions.cs +++ b/src/Umbraco.Core/Persistence/PetaPocoSqlExtensions.cs @@ -14,19 +14,6 @@ namespace Umbraco.Core.Persistence ///         public static class PetaPocoSqlExtensions { - /// - /// Defines the column to select in the generated SQL query - /// - /// - /// Sql object - /// Columns to select - /// - [Obsolete("Use the overload specifying ISqlSyntaxProvider instead")] - public static Sql Select(this Sql sql, params Expression>[] fields) - { - return sql.Select(GetFieldNames(SqlSyntaxContext.SqlSyntaxProvider, fields)); - } - /// /// Defines the column to select in the generated SQL query /// @@ -39,19 +26,7 @@ namespace Umbraco.Core.Persistence { return sql.Select(GetFieldNames(sqlSyntax, fields)); } - - /// - /// Adds another set of field to select. This method must be used with "Select" when fecthing fields from different tables. - /// - /// - /// Sql object - /// Additional columns to select - /// - public static Sql AndSelect(this Sql sql, params Expression>[] fields) - { - return sql.AndSelect(GetFieldNames(SqlSyntaxContext.SqlSyntaxProvider, fields)); - } - + /// /// Adds another set of field to select. This method must be used with "Select" when fecthing fields from different tables. /// diff --git a/src/Umbraco.Tests/Persistence/Querying/PetaPocoSqlTests.cs b/src/Umbraco.Tests/Persistence/Querying/PetaPocoSqlTests.cs index 25f0946257..50b78fa078 100644 --- a/src/Umbraco.Tests/Persistence/Querying/PetaPocoSqlTests.cs +++ b/src/Umbraco.Tests/Persistence/Querying/PetaPocoSqlTests.cs @@ -1,28 +1,27 @@ -using System; +using NUnit.Framework; using System.Diagnostics; using System.Linq; -using NUnit.Framework; -using Umbraco.Core.Models; -using Umbraco.Core.Models.Membership; using Umbraco.Core.Models.Rdbms; using Umbraco.Core.Persistence; -using Umbraco.Core.Persistence.Repositories; -using Umbraco.Tests.TestHelpers; using Umbraco.Core.Persistence.Querying; using Umbraco.Core.Persistence.SqlSyntax; +using Umbraco.Tests.TestHelpers; namespace Umbraco.Tests.Persistence.Querying { [TestFixture] public class PetaPocoSqlTests : BaseUsingSqlCeSyntax { + private readonly ISqlSyntaxProvider sqlSyntax = new SqlCeSyntaxProvider(); //x => [Test] public void Where_Clause_With_Starts_With_Additional_Parameters() { var content = new NodeDto() { NodeId = 123, Path = "-1,123" }; - var sql = new Sql("SELECT *").From().Where(x => x.Path.SqlStartsWith(content.Path, TextColumnType.NVarchar)); + var sql = new Sql("SELECT *") + .From(sqlSyntax) + .Where(x => x.Path.SqlStartsWith(content.Path, TextColumnType.NVarchar), sqlSyntax); Assert.AreEqual("SELECT * FROM [umbracoNode] WHERE (upper([umbracoNode].[path]) LIKE upper(@0))", sql.SQL.Replace("\n", " ")); Assert.AreEqual(1, sql.Arguments.Length); @@ -33,7 +32,9 @@ namespace Umbraco.Tests.Persistence.Querying public void Where_Clause_With_Starts_With_By_Variable() { var content = new NodeDto() { NodeId = 123, Path = "-1,123" }; - var sql = new Sql("SELECT *").From().Where(x => x.Path.StartsWith(content.Path) && x.NodeId != content.NodeId); + var sql = new Sql("SELECT *") + .From(sqlSyntax) + .Where(x => x.Path.StartsWith(content.Path) && x.NodeId != content.NodeId, sqlSyntax); Assert.AreEqual("SELECT * FROM [umbracoNode] WHERE ((upper([umbracoNode].[path]) LIKE upper(@0) AND ([umbracoNode].[id] <> @1)))", sql.SQL.Replace("\n", " ")); Assert.AreEqual(2, sql.Arguments.Length); @@ -45,7 +46,9 @@ namespace Umbraco.Tests.Persistence.Querying public void Where_Clause_With_Not_Starts_With() { var level = 1; - var sql = new Sql("SELECT *").From().Where(x => x.Level == level && !x.Path.StartsWith("-20")); + var sql = new Sql("SELECT *") + .From(sqlSyntax) + .Where(x => x.Level == level && !x.Path.StartsWith("-20"), sqlSyntax); Assert.AreEqual("SELECT * FROM [umbracoNode] WHERE ((([umbracoNode].[level] = @0) AND NOT (upper([umbracoNode].[path]) LIKE upper(@1))))", sql.SQL.Replace("\n", " ")); Assert.AreEqual(2, sql.Arguments.Length); @@ -57,7 +60,9 @@ namespace Umbraco.Tests.Persistence.Querying public void Where_Clause_With_EqualsFalse_Starts_With() { var level = 1; - var sql = new Sql("SELECT *").From().Where(x => x.Level == level && x.Path.StartsWith("-20") == false); + var sql = new Sql("SELECT *") + .From(sqlSyntax) + .Where(x => x.Level == level && x.Path.StartsWith("-20") == false, sqlSyntax); Assert.AreEqual("SELECT * FROM [umbracoNode] WHERE ((([umbracoNode].[level] = @0) AND NOT (upper([umbracoNode].[path]) LIKE upper(@1))))", sql.SQL.Replace("\n", " ")); Assert.AreEqual(2, sql.Arguments.Length); @@ -68,7 +73,9 @@ namespace Umbraco.Tests.Persistence.Querying [Test] public void Where_Clause_With_Equals_Clause() { - var sql = new Sql("SELECT *").From().Where(x => x.Text.Equals("Hello@world.com")); + var sql = new Sql("SELECT *") + .From(sqlSyntax) + .Where(x => x.Text.Equals("Hello@world.com"), sqlSyntax); Assert.AreEqual("SELECT * FROM [umbracoNode] WHERE (upper([umbracoNode].[text]) = upper(@0))", sql.SQL.Replace("\n", " ")); Assert.AreEqual(1, sql.Arguments.Length); @@ -78,7 +85,9 @@ namespace Umbraco.Tests.Persistence.Querying [Test] public void Where_Clause_With_False_Boolean() { - var sql = new Sql("SELECT *").From().Where(x => !x.Trashed); + var sql = new Sql("SELECT *") + .From(sqlSyntax) + .Where(x => !x.Trashed, sqlSyntax); Assert.AreEqual("SELECT * FROM [umbracoNode] WHERE (NOT ([umbracoNode].[trashed] = @0))", sql.SQL.Replace("\n", " ")); Assert.AreEqual(1, sql.Arguments.Length); @@ -88,7 +97,9 @@ namespace Umbraco.Tests.Persistence.Querying [Test] public void Where_Clause_With_EqualsFalse_Boolean() { - var sql = new Sql("SELECT *").From().Where(x => x.Trashed == false); + var sql = new Sql("SELECT *") + .From(sqlSyntax) + .Where(x => x.Trashed == false, sqlSyntax); Assert.AreEqual("SELECT * FROM [umbracoNode] WHERE (NOT ([umbracoNode].[trashed] = @0))", sql.SQL.Replace("\n", " ")); Assert.AreEqual(1, sql.Arguments.Length); @@ -98,7 +109,9 @@ namespace Umbraco.Tests.Persistence.Querying [Test] public void Where_Clause_With_Boolean() { - var sql = new Sql("SELECT *").From().Where(x => x.Trashed); + var sql = new Sql("SELECT *") + .From(sqlSyntax) + .Where(x => x.Trashed, sqlSyntax); Assert.AreEqual("SELECT * FROM [umbracoNode] WHERE ([umbracoNode].[trashed] = @0)", sql.SQL.Replace("\n", " ")); Assert.AreEqual(1, sql.Arguments.Length); @@ -108,7 +121,9 @@ namespace Umbraco.Tests.Persistence.Querying [Test] public void Where_Clause_With_ToUpper() { - var sql = new Sql("SELECT *").From().Where(x => x.Text.ToUpper() == "hello".ToUpper()); + var sql = new Sql("SELECT *") + .From(sqlSyntax) + .Where(x => x.Text.ToUpper() == "hello".ToUpper(), sqlSyntax); Assert.AreEqual("SELECT * FROM [umbracoNode] WHERE ((upper([umbracoNode].[text]) = upper(@0)))", sql.SQL.Replace("\n", " ")); Assert.AreEqual(1, sql.Arguments.Length); @@ -118,7 +133,9 @@ namespace Umbraco.Tests.Persistence.Querying [Test] public void Where_Clause_With_ToString() { - var sql = new Sql("SELECT *").From().Where(x => x.Text == 1.ToString()); + var sql = new Sql("SELECT *") + .From(sqlSyntax) + .Where(x => x.Text == 1.ToString(), sqlSyntax); Assert.AreEqual("SELECT * FROM [umbracoNode] WHERE (([umbracoNode].[text] = @0))", sql.SQL.Replace("\n", " ")); Assert.AreEqual(1, sql.Arguments.Length); @@ -128,7 +145,9 @@ namespace Umbraco.Tests.Persistence.Querying [Test] public void Where_Clause_With_Wildcard() { - var sql = new Sql("SELECT *").From().Where(x => x.Text.StartsWith("D")); + var sql = new Sql("SELECT *") + .From(sqlSyntax) + .Where(x => x.Text.StartsWith("D"), sqlSyntax); Assert.AreEqual("SELECT * FROM [umbracoNode] WHERE (upper([umbracoNode].[text]) LIKE upper(@0))", sql.SQL.Replace("\n", " ")); Assert.AreEqual(1, sql.Arguments.Length); @@ -138,7 +157,9 @@ namespace Umbraco.Tests.Persistence.Querying [Test] public void Where_Clause_Single_Constant() { - var sql = new Sql("SELECT *").From().Where(x => x.NodeId == 2); + var sql = new Sql("SELECT *") + .From(sqlSyntax) + .Where(x => x.NodeId == 2, sqlSyntax); Assert.AreEqual("SELECT * FROM [umbracoNode] WHERE (([umbracoNode].[id] = @0))", sql.SQL.Replace("\n", " ")); Assert.AreEqual(1, sql.Arguments.Length); @@ -148,7 +169,9 @@ namespace Umbraco.Tests.Persistence.Querying [Test] public void Where_Clause_And_Constant() { - var sql = new Sql("SELECT *").From().Where(x => x.NodeId != 2 && x.NodeId != 3); + var sql = new Sql("SELECT *") + .From(sqlSyntax) + .Where(x => x.NodeId != 2 && x.NodeId != 3, sqlSyntax); Assert.AreEqual("SELECT * FROM [umbracoNode] WHERE ((([umbracoNode].[id] <> @0) AND ([umbracoNode].[id] <> @1)))", sql.SQL.Replace("\n", " ")); Assert.AreEqual(2, sql.Arguments.Length); @@ -159,7 +182,9 @@ namespace Umbraco.Tests.Persistence.Querying [Test] public void Where_Clause_Or_Constant() { - var sql = new Sql("SELECT *").From().Where(x => x.Text == "hello" || x.NodeId == 3); + var sql = new Sql("SELECT *") + .From(sqlSyntax) + .Where(x => x.Text == "hello" || x.NodeId == 3, sqlSyntax); Assert.AreEqual("SELECT * FROM [umbracoNode] WHERE ((([umbracoNode].[text] = @0) OR ([umbracoNode].[id] = @1)))", sql.SQL.Replace("\n", " ")); Assert.AreEqual(2, sql.Arguments.Length); @@ -174,7 +199,7 @@ namespace Umbraco.Tests.Persistence.Querying expected.Select("*").From("[cmsContent]"); var sql = new Sql(); - sql.Select("*").From(); + sql.Select("*").From(sqlSyntax); Assert.That(sql.SQL, Is.EqualTo(expected.SQL)); @@ -191,9 +216,9 @@ namespace Umbraco.Tests.Persistence.Querying .On("[cmsDocument].[versionId] = [cmsContentVersion].[VersionId]"); var sql = new Sql(); - sql.Select("*").From() - .InnerJoin() - .On(left => left.VersionId, right => right.VersionId); + sql.Select("*").From(sqlSyntax) + .InnerJoin(sqlSyntax) + .On(sqlSyntax, left => left.VersionId, right => right.VersionId); Assert.That(sql.SQL, Is.EqualTo(expected.SQL)); @@ -207,7 +232,7 @@ namespace Umbraco.Tests.Persistence.Querying expected.Select("*").From("[cmsContent]").OrderBy("([cmsContent].[contentType])"); var sql = new Sql(); - sql.Select("*").From().OrderBy(x => x.ContentTypeId); + sql.Select("*").From(sqlSyntax).OrderBy(x => x.ContentTypeId, sqlSyntax); Assert.That(sql.SQL, Is.EqualTo(expected.SQL)); @@ -221,7 +246,7 @@ namespace Umbraco.Tests.Persistence.Querying expected.Select("*").From("[cmsContent]").GroupBy("[contentType]"); var sql = new Sql(); - sql.Select("*").From().GroupBy(x => x.ContentTypeId); + sql.Select("*").From(sqlSyntax).GroupBy(x => x.ContentTypeId, sqlSyntax); Assert.That(sql.SQL, Is.EqualTo(expected.SQL)); @@ -235,7 +260,7 @@ namespace Umbraco.Tests.Persistence.Querying expected.Select("*").From("[cmsContent]").Where("([cmsContent].[nodeId] = @0)", 1045); var sql = new Sql(); - sql.Select("*").From().Where(x => x.NodeId == 1045); + sql.Select("*").From(sqlSyntax).Where(x => x.NodeId == 1045, sqlSyntax); Assert.That(sql.SQL, Is.EqualTo(expected.SQL)); @@ -253,9 +278,9 @@ namespace Umbraco.Tests.Persistence.Querying var sql = new Sql(); sql.Select("*") - .From() - .Where(x => x.NodeId == 1045) - .Where(x => x.ContentTypeId == 1050); + .From(sqlSyntax) + .Where(x => x.NodeId == 1045, sqlSyntax) + .Where(x => x.ContentTypeId == 1050, sqlSyntax); Assert.That(sql.SQL, Is.EqualTo(expected.SQL)); @@ -270,8 +295,8 @@ namespace Umbraco.Tests.Persistence.Querying .From("[cmsContent]"); var sql = new Sql(); - sql.Select() - .From(); + sql.Select(sqlSyntax) + .From(sqlSyntax); Assert.That(sql.SQL, Is.EqualTo(expected.SQL)); @@ -286,8 +311,8 @@ namespace Umbraco.Tests.Persistence.Querying .From("[cmsContent]"); var sql = new Sql(); - sql.Select(c => c.NodeId) - .From(); + sql.Select(sqlSyntax, c => c.NodeId) + .From(sqlSyntax); Assert.That(sql.SQL, Is.EqualTo(expected.SQL)); @@ -302,8 +327,8 @@ namespace Umbraco.Tests.Persistence.Querying .From("[cmsContent]"); var sql = new Sql(); - sql.Select(c => c.NodeId, c => c.ContentTypeId, c => c.PrimaryKey) - .From(); + sql.Select(sqlSyntax, c => c.NodeId, c => c.ContentTypeId, c => c.PrimaryKey) + .From(sqlSyntax); Assert.That(sql.SQL, Is.EqualTo(expected.SQL)); @@ -320,11 +345,11 @@ namespace Umbraco.Tests.Persistence.Querying .On("[cmsDocument].[versionId] = [cmsContentVersion].[VersionId]"); var sql = new Sql(); - sql.Select(d => d.NodeId, d => d.Published) - .AndSelect(cv => cv.Id) - .From() - .InnerJoin() - .On(left => left.VersionId, right => right.VersionId); + sql.Select(sqlSyntax, d => d.NodeId, d => d.Published) + .AndSelect(sqlSyntax, cv => cv.Id) + .From(sqlSyntax) + .InnerJoin(sqlSyntax) + .On(sqlSyntax, left => left.VersionId, right => right.VersionId); Assert.That(sql.SQL, Is.EqualTo(expected.SQL)); diff --git a/src/Umbraco.Web.UI.Client/vwd.webinfo b/src/Umbraco.Web.UI.Client/vwd.webinfo deleted file mode 100644 index 44d783b1df..0000000000 --- a/src/Umbraco.Web.UI.Client/vwd.webinfo +++ /dev/null @@ -1,8 +0,0 @@ - - - - - - \ No newline at end of file From 1fc09cda0f6b3587c27d8c5ed903b48b26f8b7f3 Mon Sep 17 00:00:00 2001 From: Sebastiaan Janssen Date: Sun, 15 Jul 2018 08:01:16 +0200 Subject: [PATCH 41/47] Following small R# hints, removed commented out line --- .../Persistence/Querying/PetaPocoSqlTests.cs | 106 +++++++++--------- 1 file changed, 52 insertions(+), 54 deletions(-) diff --git a/src/Umbraco.Tests/Persistence/Querying/PetaPocoSqlTests.cs b/src/Umbraco.Tests/Persistence/Querying/PetaPocoSqlTests.cs index 50b78fa078..7bb2ee581d 100644 --- a/src/Umbraco.Tests/Persistence/Querying/PetaPocoSqlTests.cs +++ b/src/Umbraco.Tests/Persistence/Querying/PetaPocoSqlTests.cs @@ -1,6 +1,5 @@ using NUnit.Framework; using System.Diagnostics; -using System.Linq; using Umbraco.Core.Models.Rdbms; using Umbraco.Core.Persistence; using Umbraco.Core.Persistence.Querying; @@ -12,16 +11,15 @@ namespace Umbraco.Tests.Persistence.Querying [TestFixture] public class PetaPocoSqlTests : BaseUsingSqlCeSyntax { - private readonly ISqlSyntaxProvider sqlSyntax = new SqlCeSyntaxProvider(); - //x => + private readonly ISqlSyntaxProvider _sqlSyntax = new SqlCeSyntaxProvider(); [Test] public void Where_Clause_With_Starts_With_Additional_Parameters() { var content = new NodeDto() { NodeId = 123, Path = "-1,123" }; var sql = new Sql("SELECT *") - .From(sqlSyntax) - .Where(x => x.Path.SqlStartsWith(content.Path, TextColumnType.NVarchar), sqlSyntax); + .From(_sqlSyntax) + .Where(x => x.Path.SqlStartsWith(content.Path, TextColumnType.NVarchar), _sqlSyntax); Assert.AreEqual("SELECT * FROM [umbracoNode] WHERE (upper([umbracoNode].[path]) LIKE upper(@0))", sql.SQL.Replace("\n", " ")); Assert.AreEqual(1, sql.Arguments.Length); @@ -33,8 +31,8 @@ namespace Umbraco.Tests.Persistence.Querying { var content = new NodeDto() { NodeId = 123, Path = "-1,123" }; var sql = new Sql("SELECT *") - .From(sqlSyntax) - .Where(x => x.Path.StartsWith(content.Path) && x.NodeId != content.NodeId, sqlSyntax); + .From(_sqlSyntax) + .Where(x => x.Path.StartsWith(content.Path) && x.NodeId != content.NodeId, _sqlSyntax); Assert.AreEqual("SELECT * FROM [umbracoNode] WHERE ((upper([umbracoNode].[path]) LIKE upper(@0) AND ([umbracoNode].[id] <> @1)))", sql.SQL.Replace("\n", " ")); Assert.AreEqual(2, sql.Arguments.Length); @@ -45,10 +43,10 @@ namespace Umbraco.Tests.Persistence.Querying [Test] public void Where_Clause_With_Not_Starts_With() { - var level = 1; + const int level = 1; var sql = new Sql("SELECT *") - .From(sqlSyntax) - .Where(x => x.Level == level && !x.Path.StartsWith("-20"), sqlSyntax); + .From(_sqlSyntax) + .Where(x => x.Level == level && x.Path.StartsWith("-20") == false, _sqlSyntax); Assert.AreEqual("SELECT * FROM [umbracoNode] WHERE ((([umbracoNode].[level] = @0) AND NOT (upper([umbracoNode].[path]) LIKE upper(@1))))", sql.SQL.Replace("\n", " ")); Assert.AreEqual(2, sql.Arguments.Length); @@ -59,10 +57,10 @@ namespace Umbraco.Tests.Persistence.Querying [Test] public void Where_Clause_With_EqualsFalse_Starts_With() { - var level = 1; + const int level = 1; var sql = new Sql("SELECT *") - .From(sqlSyntax) - .Where(x => x.Level == level && x.Path.StartsWith("-20") == false, sqlSyntax); + .From(_sqlSyntax) + .Where(x => x.Level == level && x.Path.StartsWith("-20") == false, _sqlSyntax); Assert.AreEqual("SELECT * FROM [umbracoNode] WHERE ((([umbracoNode].[level] = @0) AND NOT (upper([umbracoNode].[path]) LIKE upper(@1))))", sql.SQL.Replace("\n", " ")); Assert.AreEqual(2, sql.Arguments.Length); @@ -74,8 +72,8 @@ namespace Umbraco.Tests.Persistence.Querying public void Where_Clause_With_Equals_Clause() { var sql = new Sql("SELECT *") - .From(sqlSyntax) - .Where(x => x.Text.Equals("Hello@world.com"), sqlSyntax); + .From(_sqlSyntax) + .Where(x => x.Text.Equals("Hello@world.com"), _sqlSyntax); Assert.AreEqual("SELECT * FROM [umbracoNode] WHERE (upper([umbracoNode].[text]) = upper(@0))", sql.SQL.Replace("\n", " ")); Assert.AreEqual(1, sql.Arguments.Length); @@ -86,8 +84,8 @@ namespace Umbraco.Tests.Persistence.Querying public void Where_Clause_With_False_Boolean() { var sql = new Sql("SELECT *") - .From(sqlSyntax) - .Where(x => !x.Trashed, sqlSyntax); + .From(_sqlSyntax) + .Where(x => x.Trashed == false, _sqlSyntax); Assert.AreEqual("SELECT * FROM [umbracoNode] WHERE (NOT ([umbracoNode].[trashed] = @0))", sql.SQL.Replace("\n", " ")); Assert.AreEqual(1, sql.Arguments.Length); @@ -98,8 +96,8 @@ namespace Umbraco.Tests.Persistence.Querying public void Where_Clause_With_EqualsFalse_Boolean() { var sql = new Sql("SELECT *") - .From(sqlSyntax) - .Where(x => x.Trashed == false, sqlSyntax); + .From(_sqlSyntax) + .Where(x => x.Trashed == false, _sqlSyntax); Assert.AreEqual("SELECT * FROM [umbracoNode] WHERE (NOT ([umbracoNode].[trashed] = @0))", sql.SQL.Replace("\n", " ")); Assert.AreEqual(1, sql.Arguments.Length); @@ -110,8 +108,8 @@ namespace Umbraco.Tests.Persistence.Querying public void Where_Clause_With_Boolean() { var sql = new Sql("SELECT *") - .From(sqlSyntax) - .Where(x => x.Trashed, sqlSyntax); + .From(_sqlSyntax) + .Where(x => x.Trashed, _sqlSyntax); Assert.AreEqual("SELECT * FROM [umbracoNode] WHERE ([umbracoNode].[trashed] = @0)", sql.SQL.Replace("\n", " ")); Assert.AreEqual(1, sql.Arguments.Length); @@ -122,8 +120,8 @@ namespace Umbraco.Tests.Persistence.Querying public void Where_Clause_With_ToUpper() { var sql = new Sql("SELECT *") - .From(sqlSyntax) - .Where(x => x.Text.ToUpper() == "hello".ToUpper(), sqlSyntax); + .From(_sqlSyntax) + .Where(x => x.Text.ToUpper() == "hello".ToUpper(), _sqlSyntax); Assert.AreEqual("SELECT * FROM [umbracoNode] WHERE ((upper([umbracoNode].[text]) = upper(@0)))", sql.SQL.Replace("\n", " ")); Assert.AreEqual(1, sql.Arguments.Length); @@ -134,8 +132,8 @@ namespace Umbraco.Tests.Persistence.Querying public void Where_Clause_With_ToString() { var sql = new Sql("SELECT *") - .From(sqlSyntax) - .Where(x => x.Text == 1.ToString(), sqlSyntax); + .From(_sqlSyntax) + .Where(x => x.Text == 1.ToString(), _sqlSyntax); Assert.AreEqual("SELECT * FROM [umbracoNode] WHERE (([umbracoNode].[text] = @0))", sql.SQL.Replace("\n", " ")); Assert.AreEqual(1, sql.Arguments.Length); @@ -146,8 +144,8 @@ namespace Umbraco.Tests.Persistence.Querying public void Where_Clause_With_Wildcard() { var sql = new Sql("SELECT *") - .From(sqlSyntax) - .Where(x => x.Text.StartsWith("D"), sqlSyntax); + .From(_sqlSyntax) + .Where(x => x.Text.StartsWith("D"), _sqlSyntax); Assert.AreEqual("SELECT * FROM [umbracoNode] WHERE (upper([umbracoNode].[text]) LIKE upper(@0))", sql.SQL.Replace("\n", " ")); Assert.AreEqual(1, sql.Arguments.Length); @@ -158,8 +156,8 @@ namespace Umbraco.Tests.Persistence.Querying public void Where_Clause_Single_Constant() { var sql = new Sql("SELECT *") - .From(sqlSyntax) - .Where(x => x.NodeId == 2, sqlSyntax); + .From(_sqlSyntax) + .Where(x => x.NodeId == 2, _sqlSyntax); Assert.AreEqual("SELECT * FROM [umbracoNode] WHERE (([umbracoNode].[id] = @0))", sql.SQL.Replace("\n", " ")); Assert.AreEqual(1, sql.Arguments.Length); @@ -170,8 +168,8 @@ namespace Umbraco.Tests.Persistence.Querying public void Where_Clause_And_Constant() { var sql = new Sql("SELECT *") - .From(sqlSyntax) - .Where(x => x.NodeId != 2 && x.NodeId != 3, sqlSyntax); + .From(_sqlSyntax) + .Where(x => x.NodeId != 2 && x.NodeId != 3, _sqlSyntax); Assert.AreEqual("SELECT * FROM [umbracoNode] WHERE ((([umbracoNode].[id] <> @0) AND ([umbracoNode].[id] <> @1)))", sql.SQL.Replace("\n", " ")); Assert.AreEqual(2, sql.Arguments.Length); @@ -183,8 +181,8 @@ namespace Umbraco.Tests.Persistence.Querying public void Where_Clause_Or_Constant() { var sql = new Sql("SELECT *") - .From(sqlSyntax) - .Where(x => x.Text == "hello" || x.NodeId == 3, sqlSyntax); + .From(_sqlSyntax) + .Where(x => x.Text == "hello" || x.NodeId == 3, _sqlSyntax); Assert.AreEqual("SELECT * FROM [umbracoNode] WHERE ((([umbracoNode].[text] = @0) OR ([umbracoNode].[id] = @1)))", sql.SQL.Replace("\n", " ")); Assert.AreEqual(2, sql.Arguments.Length); @@ -199,7 +197,7 @@ namespace Umbraco.Tests.Persistence.Querying expected.Select("*").From("[cmsContent]"); var sql = new Sql(); - sql.Select("*").From(sqlSyntax); + sql.Select("*").From(_sqlSyntax); Assert.That(sql.SQL, Is.EqualTo(expected.SQL)); @@ -216,9 +214,9 @@ namespace Umbraco.Tests.Persistence.Querying .On("[cmsDocument].[versionId] = [cmsContentVersion].[VersionId]"); var sql = new Sql(); - sql.Select("*").From(sqlSyntax) - .InnerJoin(sqlSyntax) - .On(sqlSyntax, left => left.VersionId, right => right.VersionId); + sql.Select("*").From(_sqlSyntax) + .InnerJoin(_sqlSyntax) + .On(_sqlSyntax, left => left.VersionId, right => right.VersionId); Assert.That(sql.SQL, Is.EqualTo(expected.SQL)); @@ -232,7 +230,7 @@ namespace Umbraco.Tests.Persistence.Querying expected.Select("*").From("[cmsContent]").OrderBy("([cmsContent].[contentType])"); var sql = new Sql(); - sql.Select("*").From(sqlSyntax).OrderBy(x => x.ContentTypeId, sqlSyntax); + sql.Select("*").From(_sqlSyntax).OrderBy(x => x.ContentTypeId, _sqlSyntax); Assert.That(sql.SQL, Is.EqualTo(expected.SQL)); @@ -246,7 +244,7 @@ namespace Umbraco.Tests.Persistence.Querying expected.Select("*").From("[cmsContent]").GroupBy("[contentType]"); var sql = new Sql(); - sql.Select("*").From(sqlSyntax).GroupBy(x => x.ContentTypeId, sqlSyntax); + sql.Select("*").From(_sqlSyntax).GroupBy(x => x.ContentTypeId, _sqlSyntax); Assert.That(sql.SQL, Is.EqualTo(expected.SQL)); @@ -260,7 +258,7 @@ namespace Umbraco.Tests.Persistence.Querying expected.Select("*").From("[cmsContent]").Where("([cmsContent].[nodeId] = @0)", 1045); var sql = new Sql(); - sql.Select("*").From(sqlSyntax).Where(x => x.NodeId == 1045, sqlSyntax); + sql.Select("*").From(_sqlSyntax).Where(x => x.NodeId == 1045, _sqlSyntax); Assert.That(sql.SQL, Is.EqualTo(expected.SQL)); @@ -278,9 +276,9 @@ namespace Umbraco.Tests.Persistence.Querying var sql = new Sql(); sql.Select("*") - .From(sqlSyntax) - .Where(x => x.NodeId == 1045, sqlSyntax) - .Where(x => x.ContentTypeId == 1050, sqlSyntax); + .From(_sqlSyntax) + .Where(x => x.NodeId == 1045, _sqlSyntax) + .Where(x => x.ContentTypeId == 1050, _sqlSyntax); Assert.That(sql.SQL, Is.EqualTo(expected.SQL)); @@ -295,8 +293,8 @@ namespace Umbraco.Tests.Persistence.Querying .From("[cmsContent]"); var sql = new Sql(); - sql.Select(sqlSyntax) - .From(sqlSyntax); + sql.Select(_sqlSyntax) + .From(_sqlSyntax); Assert.That(sql.SQL, Is.EqualTo(expected.SQL)); @@ -311,8 +309,8 @@ namespace Umbraco.Tests.Persistence.Querying .From("[cmsContent]"); var sql = new Sql(); - sql.Select(sqlSyntax, c => c.NodeId) - .From(sqlSyntax); + sql.Select(_sqlSyntax, c => c.NodeId) + .From(_sqlSyntax); Assert.That(sql.SQL, Is.EqualTo(expected.SQL)); @@ -327,8 +325,8 @@ namespace Umbraco.Tests.Persistence.Querying .From("[cmsContent]"); var sql = new Sql(); - sql.Select(sqlSyntax, c => c.NodeId, c => c.ContentTypeId, c => c.PrimaryKey) - .From(sqlSyntax); + sql.Select(_sqlSyntax, c => c.NodeId, c => c.ContentTypeId, c => c.PrimaryKey) + .From(_sqlSyntax); Assert.That(sql.SQL, Is.EqualTo(expected.SQL)); @@ -345,11 +343,11 @@ namespace Umbraco.Tests.Persistence.Querying .On("[cmsDocument].[versionId] = [cmsContentVersion].[VersionId]"); var sql = new Sql(); - sql.Select(sqlSyntax, d => d.NodeId, d => d.Published) - .AndSelect(sqlSyntax, cv => cv.Id) - .From(sqlSyntax) - .InnerJoin(sqlSyntax) - .On(sqlSyntax, left => left.VersionId, right => right.VersionId); + sql.Select(_sqlSyntax, d => d.NodeId, d => d.Published) + .AndSelect(_sqlSyntax, cv => cv.Id) + .From(_sqlSyntax) + .InnerJoin(_sqlSyntax) + .On(_sqlSyntax, left => left.VersionId, right => right.VersionId); Assert.That(sql.SQL, Is.EqualTo(expected.SQL)); From 24e9136f578335f5f98d73d05915528d86a0c84b Mon Sep 17 00:00:00 2001 From: Sebastiaan Janssen Date: Sun, 15 Jul 2018 08:36:37 +0200 Subject: [PATCH 42/47] Fixes U4-11517 - E-mail property editor triggers validation when validation has not been enabled in the settings --- .../src/views/propertyeditors/email/email.html | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/Umbraco.Web.UI.Client/src/views/propertyeditors/email/email.html b/src/Umbraco.Web.UI.Client/src/views/propertyeditors/email/email.html index 315a7f15f3..8d695966e7 100644 --- a/src/Umbraco.Web.UI.Client/src/views/propertyeditors/email/email.html +++ b/src/Umbraco.Web.UI.Client/src/views/propertyeditors/email/email.html @@ -4,7 +4,7 @@ id="{{model.alias}}" class="umb-editor umb-textstring textstring" val-email - ng-required="model.config.IsRequired || model.validation.mandatory" + ng-required="model.config.IsRequired != 0 || model.validation.mandatory" val-server="value" /> Required From 12dbb0ec20bc729b5d117823500471484de35447 Mon Sep 17 00:00:00 2001 From: Jan Skovgaard Date: Sun, 15 Jul 2018 12:26:35 +0200 Subject: [PATCH 43/47] Add a border and other adjustments specifically for the "validation" textarea when setting up a datatype on a document type --- .../src/less/components/umb-group-builder.less | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/src/Umbraco.Web.UI.Client/src/less/components/umb-group-builder.less b/src/Umbraco.Web.UI.Client/src/less/components/umb-group-builder.less index a341c69129..50eb3c622b 100644 --- a/src/Umbraco.Web.UI.Client/src/less/components/umb-group-builder.less +++ b/src/Umbraco.Web.UI.Client/src/less/components/umb-group-builder.less @@ -530,6 +530,13 @@ input.umb-group-builder__group-title-input { overflow: hidden; } + .editor-validation-pattern{ + border: 1px solid @gray-7; + margin: 10px 0 0; + padding: 6px; + max-height: 32px; + } + .umb-dropdown { width: 100%; } From e7932b6ae4e66761e237507e0f3116283a170fa8 Mon Sep 17 00:00:00 2001 From: Jan Skovgaard Date: Sun, 15 Jul 2018 18:00:20 +0200 Subject: [PATCH 44/47] Replace the checkboxes with the umb-toggle directive using a toggleHandler in the controller --- .../views/content/content.copy.controller.js | 27 ++++++++++++++++--- .../src/views/content/copy.html | 8 +++--- 2 files changed, 28 insertions(+), 7 deletions(-) diff --git a/src/Umbraco.Web.UI.Client/src/views/content/content.copy.controller.js b/src/Umbraco.Web.UI.Client/src/views/content/content.copy.controller.js index 562a2291b1..0cd217c3a8 100644 --- a/src/Umbraco.Web.UI.Client/src/views/content/content.copy.controller.js +++ b/src/Umbraco.Web.UI.Client/src/views/content/content.copy.controller.js @@ -20,9 +20,10 @@ angular.module("umbraco").controller("Umbraco.Editors.Content.CopyController", } $scope.treeModel = { hideHeader: false - } + } + $scope.toggle = toggleHandler; userService.getCurrentUser().then(function (userData) { - $scope.treeModel.hideHeader = userData.startContentIds.length > 0 && userData.startContentIds.indexOf(-1) == -1; + $scope.treeModel.hideHeader = userData.startContentIds.length > 0 && userData.startContentIds.indexOf(-1) == -1; }); var node = dialogOptions.currentNode; @@ -57,7 +58,27 @@ angular.module("umbraco").controller("Umbraco.Editors.Content.CopyController", if (args.node.metaData.isContainer) { openMiniListView(args.node); } - } + } + + function toggleHandler(type){ + // If the relateToOriginal toggle is clicked + if(type === "relate"){ + if($scope.relateToOriginal){ + $scope.relateToOriginal = false; + return; + } + $scope.relateToOriginal = true; + } + + // If the recurvise toggle is clicked + if(type === "recursive"){ + if($scope.recursive){ + $scope.recursive = false; + return; + } + $scope.recursive = true; + } + } $scope.hideSearch = function () { $scope.searchInfo.showSearch = false; diff --git a/src/Umbraco.Web.UI.Client/src/views/content/copy.html b/src/Umbraco.Web.UI.Client/src/views/content/copy.html index ca7714c284..e02855ea8f 100644 --- a/src/Umbraco.Web.UI.Client/src/views/content/copy.html +++ b/src/Umbraco.Web.UI.Client/src/views/content/copy.html @@ -68,13 +68,13 @@ - + - - + + @@ -90,4 +90,4 @@ Copy - \ No newline at end of file + From 615d57a78bc84816196ddfe8dec415703ad9ed79 Mon Sep 17 00:00:00 2001 From: Gareth Evans Date: Mon, 16 Jul 2018 16:43:20 +1200 Subject: [PATCH 45/47] U4-10898 U4-9309 When node is copied, the copy is logged as being completed by the original node writer, not the person performing the copy --- src/Umbraco.Core/Services/ContentService.cs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/Umbraco.Core/Services/ContentService.cs b/src/Umbraco.Core/Services/ContentService.cs index 33131bbb52..2f8b9c8340 100644 --- a/src/Umbraco.Core/Services/ContentService.cs +++ b/src/Umbraco.Core/Services/ContentService.cs @@ -1872,7 +1872,7 @@ namespace Umbraco.Core.Services } copyEventArgs.CanCancel = false; uow.Events.Dispatch(Copied, this, copyEventArgs); - Audit(uow, AuditType.Copy, "Copy Content performed by user", content.WriterId, content.Id); + Audit(uow, AuditType.Copy, "Copy Content performed by user", userId, content.Id); uow.Commit(); } From abf61417ad201cb91a62197361c3621417c96e14 Mon Sep 17 00:00:00 2001 From: Lee Kelleher Date: Mon, 16 Jul 2018 09:44:54 +0100 Subject: [PATCH 46/47] U4-10561 - Performance improvement for IsAncestor/IsDescendant extension methods (#2768) --- src/Umbraco.Web/PublishedContentExtensions.cs | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/src/Umbraco.Web/PublishedContentExtensions.cs b/src/Umbraco.Web/PublishedContentExtensions.cs index 45afecccd9..a7dcdccdf0 100644 --- a/src/Umbraco.Web/PublishedContentExtensions.cs +++ b/src/Umbraco.Web/PublishedContentExtensions.cs @@ -860,7 +860,7 @@ namespace Umbraco.Web public static bool IsDescendant(this IPublishedContent content, IPublishedContent other) { - return content.Ancestors().Any(x => x.Id == other.Id); + return other.Level < content.Level && content.Path.InvariantStartsWith(other.Path); } public static HtmlString IsDescendant(this IPublishedContent content, IPublishedContent other, string valueIfTrue) @@ -875,7 +875,7 @@ namespace Umbraco.Web public static bool IsDescendantOrSelf(this IPublishedContent content, IPublishedContent other) { - return content.AncestorsOrSelf().Any(x => x.Id == other.Id); + return content.Path.InvariantStartsWith(other.Path); } public static HtmlString IsDescendantOrSelf(this IPublishedContent content, IPublishedContent other, string valueIfTrue) @@ -890,8 +890,8 @@ namespace Umbraco.Web public static bool IsAncestor(this IPublishedContent content, IPublishedContent other) { - // avoid using Descendants(), that's expensive - return other.Ancestors().Any(x => x.Id == content.Id); + // avoid using Descendants(), or Ancestors(), they're expensive + return content.Level < other.Level && other.Path.InvariantStartsWith(content.Path); } public static HtmlString IsAncestor(this IPublishedContent content, IPublishedContent other, string valueIfTrue) @@ -906,8 +906,8 @@ namespace Umbraco.Web public static bool IsAncestorOrSelf(this IPublishedContent content, IPublishedContent other) { - // avoid using DescendantsOrSelf(), that's expensive - return other.AncestorsOrSelf().Any(x => x.Id == content.Id); + // avoid using DescendantsOrSelf() or AncestorsOrSelf(), they're expensive + return other.Path.InvariantStartsWith(content.Path); } public static HtmlString IsAncestorOrSelf(this IPublishedContent content, IPublishedContent other, string valueIfTrue) From 596157f90bcf1cce4e71676d236f4f9c15b175f3 Mon Sep 17 00:00:00 2001 From: Sebastiaan Janssen Date: Mon, 16 Jul 2018 15:59:36 +0200 Subject: [PATCH 47/47] Default to section route if not specified --- src/Umbraco.Web.UI.Client/src/routes.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/Umbraco.Web.UI.Client/src/routes.js b/src/Umbraco.Web.UI.Client/src/routes.js index 314b3b0c53..2f1a7583b4 100644 --- a/src/Umbraco.Web.UI.Client/src/routes.js +++ b/src/Umbraco.Web.UI.Client/src/routes.js @@ -146,7 +146,7 @@ app.config(function ($routeProvider) { //This controller will execute for this route, then we replace the template dynamnically based on the current tree. controller: function ($scope, $route, $routeParams, treeService) { - if (!$routeParams.tree || !$routeParams.method) { + if (!$routeParams.method) { $scope.templateUrl = "views/common/dashboard.html"; }