From 52d9e7001d232e2976e1dc80e8228a2f3507708b Mon Sep 17 00:00:00 2001 From: Sebastian Stehle Date: Tue, 6 Feb 2018 21:15:43 +0100 Subject: [PATCH] Various style fixes. --- .../Rules/Triggers/ContentChangedTrigger.cs | 2 + .../Triggers/ContentChangedTriggerHandler.cs | 5 +++ .../Converters/RuleTriggerDtoFactory.cs | 7 ++-- .../Triggers/ContentChangedTriggerDto.cs | 12 ++++-- .../content/content-field.component.html | 20 ++++----- .../pages/contents/search-form.component.html | 5 ++- .../asset-changed-trigger.component.html | 8 ++-- .../content-changed-trigger.component.html | 9 +++- .../content-changed-trigger.component.ts | 9 +++- .../schemas/pages/schema/field.component.html | 25 ++++++----- .../schema/schema-edit-form.component.html | 12 +++--- .../pages/schema/schema-page.component.html | 5 ++- .../types/assets-validation.component.html | 37 ++++++++-------- .../types/assets-validation.component.scss | 4 -- .../schema/types/boolean-ui.component.html | 6 +-- .../types/boolean-validation.component.html | 15 ++++--- .../types/boolean-validation.component.scss | 6 +-- .../schema/types/date-time-ui.component.html | 6 +-- .../types/date-time-validation.component.html | 13 +++--- .../types/date-time-validation.component.scss | 6 +-- .../types/geolocation-ui.component.html | 2 +- .../geolocation-validation.component.html | 11 +++-- .../geolocation-validation.component.scss | 4 -- .../types/json-validation.component.html | 11 +++-- .../types/json-validation.component.scss | 4 -- .../schema/types/number-ui.component.html | 8 ++-- .../types/number-validation.component.html | 19 +++++---- .../types/number-validation.component.scss | 4 -- .../references-validation.component.html | 25 ++++------- .../references-validation.component.scss | 4 -- .../schema/types/string-ui.component.html | 8 ++-- .../types/string-validation.component.html | 27 ++++++------ .../types/string-validation.component.scss | 4 -- .../types/tags-validation.component.html | 15 ++++--- .../types/tags-validation.component.scss | 4 -- .../pages/schemas/schema-form.component.html | 4 +- .../pages/languages/language.component.html | 42 ++++++++++++------- .../shared/components/app-form.component.html | 4 +- .../shared/components/asset.component.html | 4 +- .../geolocation-editor.component.html | 4 +- src/Squidex/app/theme/_bootstrap-vars.scss | 2 - src/Squidex/app/theme/_bootstrap.scss | 1 + src/Squidex/wwwroot/theme.html | 2 +- .../Triggers/ContentChangedTriggerTests.cs | 10 +++++ 44 files changed, 231 insertions(+), 204 deletions(-) diff --git a/src/Squidex.Domain.Apps.Core.Model/Rules/Triggers/ContentChangedTrigger.cs b/src/Squidex.Domain.Apps.Core.Model/Rules/Triggers/ContentChangedTrigger.cs index 147cbbee4..e62138920 100644 --- a/src/Squidex.Domain.Apps.Core.Model/Rules/Triggers/ContentChangedTrigger.cs +++ b/src/Squidex.Domain.Apps.Core.Model/Rules/Triggers/ContentChangedTrigger.cs @@ -15,6 +15,8 @@ namespace Squidex.Domain.Apps.Core.Rules.Triggers { public ImmutableList Schemas { get; set; } + public bool HandleAll { get; set; } + public override T Accept(IRuleTriggerVisitor visitor) { return visitor.Visit(this); diff --git a/src/Squidex.Domain.Apps.Core.Operations/HandleRules/Triggers/ContentChangedTriggerHandler.cs b/src/Squidex.Domain.Apps.Core.Operations/HandleRules/Triggers/ContentChangedTriggerHandler.cs index 2194abae6..13ca5f459 100644 --- a/src/Squidex.Domain.Apps.Core.Operations/HandleRules/Triggers/ContentChangedTriggerHandler.cs +++ b/src/Squidex.Domain.Apps.Core.Operations/HandleRules/Triggers/ContentChangedTriggerHandler.cs @@ -17,6 +17,11 @@ namespace Squidex.Domain.Apps.Core.HandleRules.Triggers { protected override bool Triggers(Envelope @event, ContentChangedTrigger trigger) { + if (trigger.HandleAll) + { + return true; + } + if (trigger.Schemas != null && @event.Payload is SchemaEvent schemaEvent) { foreach (var schema in trigger.Schemas) diff --git a/src/Squidex/Areas/Api/Controllers/Rules/Models/Converters/RuleTriggerDtoFactory.cs b/src/Squidex/Areas/Api/Controllers/Rules/Models/Converters/RuleTriggerDtoFactory.cs index 00ada885a..0c58dc1e9 100644 --- a/src/Squidex/Areas/Api/Controllers/Rules/Models/Converters/RuleTriggerDtoFactory.cs +++ b/src/Squidex/Areas/Api/Controllers/Rules/Models/Converters/RuleTriggerDtoFactory.cs @@ -33,10 +33,9 @@ namespace Squidex.Areas.Api.Controllers.Rules.Models.Converters public RuleTriggerDto Visit(ContentChangedTrigger trigger) { - return new ContentChangedTriggerDto - { - Schemas = trigger.Schemas.Select(x => SimpleMapper.Map(x, new ContentChangedTriggerSchemaDto())).ToList() - }; + var schemas = trigger.Schemas.Select(x => SimpleMapper.Map(x, new ContentChangedTriggerSchemaDto())).ToList(); + + return new ContentChangedTriggerDto { Schemas = schemas, HandleAll = trigger.HandleAll }; } } } diff --git a/src/Squidex/Areas/Api/Controllers/Rules/Models/Triggers/ContentChangedTriggerDto.cs b/src/Squidex/Areas/Api/Controllers/Rules/Models/Triggers/ContentChangedTriggerDto.cs index 1a37562df..aa2b70386 100644 --- a/src/Squidex/Areas/Api/Controllers/Rules/Models/Triggers/ContentChangedTriggerDto.cs +++ b/src/Squidex/Areas/Api/Controllers/Rules/Models/Triggers/ContentChangedTriggerDto.cs @@ -25,12 +25,16 @@ namespace Squidex.Areas.Api.Controllers.Rules.Models.Triggers [Required] public List Schemas { get; set; } + /// + /// Determines whether the trigger should handle all content changes events. + /// + public bool HandleAll { get; set; } + public override RuleTrigger ToTrigger() { - return new ContentChangedTrigger - { - Schemas = Schemas.Select(x => SimpleMapper.Map(x, new ContentChangedTriggerSchema())).ToImmutableList() - }; + var schemas = Schemas.Select(x => SimpleMapper.Map(x, new ContentChangedTriggerSchema())).ToImmutableList(); + + return new ContentChangedTrigger { HandleAll = HandleAll, Schemas = schemas }; } } } diff --git a/src/Squidex/app/features/content/pages/content/content-field.component.html b/src/Squidex/app/features/content/pages/content/content-field.component.html index b8f447f0f..24a5f9ff2 100644 --- a/src/Squidex/app/features/content/pages/content/content-field.component.html +++ b/src/Squidex/app/features/content/pages/content/content-field.component.html @@ -1,6 +1,6 @@
Disabled @@ -24,7 +24,7 @@
- +
@@ -37,8 +37,9 @@
+
@@ -47,10 +48,10 @@
- +
- +
@@ -69,8 +70,9 @@
+
@@ -82,10 +84,8 @@
-
- +
+
diff --git a/src/Squidex/app/features/content/pages/contents/search-form.component.html b/src/Squidex/app/features/content/pages/contents/search-form.component.html index ee3ad5c71..19fbb6b0c 100644 --- a/src/Squidex/app/features/content/pages/contents/search-form.component.html +++ b/src/Squidex/app/features/content/pages/contents/search-form.component.html @@ -26,8 +26,9 @@
-
diff --git a/src/Squidex/app/features/rules/pages/rules/triggers/asset-changed-trigger.component.html b/src/Squidex/app/features/rules/pages/rules/triggers/asset-changed-trigger.component.html index 54ea6f1a0..91bb083d2 100644 --- a/src/Squidex/app/features/rules/pages/rules/triggers/asset-changed-trigger.component.html +++ b/src/Squidex/app/features/rules/pages/rules/triggers/asset-changed-trigger.component.html @@ -4,7 +4,7 @@
- + @@ -12,7 +12,7 @@
- + @@ -20,7 +20,7 @@
- + @@ -28,7 +28,7 @@
- + diff --git a/src/Squidex/app/features/rules/pages/rules/triggers/content-changed-trigger.component.html b/src/Squidex/app/features/rules/pages/rules/triggers/content-changed-trigger.component.html index e2dec973b..c0da6129e 100644 --- a/src/Squidex/app/features/rules/pages/rules/triggers/content-changed-trigger.component.html +++ b/src/Squidex/app/features/rules/pages/rules/triggers/content-changed-trigger.component.html @@ -1,6 +1,6 @@

Trigger rule when an events for a schemas happens

- +
@@ -70,4 +70,11 @@ + + +
+ +
\ No newline at end of file diff --git a/src/Squidex/app/features/rules/pages/rules/triggers/content-changed-trigger.component.ts b/src/Squidex/app/features/rules/pages/rules/triggers/content-changed-trigger.component.ts index 0039cce44..faf6b0439 100644 --- a/src/Squidex/app/features/rules/pages/rules/triggers/content-changed-trigger.component.ts +++ b/src/Squidex/app/features/rules/pages/rules/triggers/content-changed-trigger.component.ts @@ -9,7 +9,8 @@ import { Component, EventEmitter, Input, OnInit, Output } from '@angular/core'; import { ImmutableArray, - SchemaDto + SchemaDto, + Types } from 'shared'; export interface TriggerSchemaForm { @@ -36,6 +37,8 @@ export class ContentChangedTriggerComponent implements OnInit { @Output() public triggerChanged = new EventEmitter(); + public handleAll = false; + public triggerSchemas: ImmutableArray; public schemaToAdd: SchemaDto; @@ -48,6 +51,8 @@ export class ContentChangedTriggerComponent implements OnInit { public ngOnInit() { const triggerSchemas: any[] = (this.trigger.schemas = this.trigger.schemas || []); + this.handleAll = Types.isBoolean(this.trigger.handleAll) ? this.trigger.handleAll : false; + this.triggerSchemas = ImmutableArray.of( triggerSchemas.map(triggerSchema => { @@ -88,7 +93,7 @@ export class ContentChangedTriggerComponent implements OnInit { }; }); - this.triggerChanged.emit({ schemas }); + this.triggerChanged.emit({ schemas, handleAll: this.handleAll }); } public removeSchema(schemaForm: TriggerSchemaForm) { diff --git a/src/Squidex/app/features/schemas/pages/schema/field.component.html b/src/Squidex/app/features/schemas/pages/schema/field.component.html index b0df523e0..e48fa79be 100644 --- a/src/Squidex/app/features/schemas/pages/schema/field.component.html +++ b/src/Squidex/app/features/schemas/pages/schema/field.component.html @@ -81,10 +81,10 @@
- +
- + The name of the field in the API response. @@ -93,12 +93,12 @@
- +
- + Define the display name for the field for documentation and user interfaces. @@ -107,12 +107,12 @@
- +
- + Define some hints for the user and editor for the field for documentation and user interfaces. @@ -121,13 +121,16 @@
-
- +
+
+ + +
- List fields are shown as a column in the content list. If no list field is defined, the first field is shown. + List fields are shown as a column in the content list. If no list field is defined, the first field is shown by default.
diff --git a/src/Squidex/app/features/schemas/pages/schema/schema-edit-form.component.html b/src/Squidex/app/features/schemas/pages/schema/schema-edit-form.component.html index da9813b46..04663aa5a 100644 --- a/src/Squidex/app/features/schemas/pages/schema/schema-edit-form.component.html +++ b/src/Squidex/app/features/schemas/pages/schema/schema-edit-form.component.html @@ -1,24 +1,24 @@
- + - +
- + - +
- + - +
diff --git a/src/Squidex/app/features/schemas/pages/schema/schema-page.component.html b/src/Squidex/app/features/schemas/pages/schema/schema-page.component.html index 538327a0e..3fc4d2c90 100644 --- a/src/Squidex/app/features/schemas/pages/schema/schema-page.component.html +++ b/src/Squidex/app/features/schemas/pages/schema/schema-page.component.html @@ -93,8 +93,9 @@
-
diff --git a/src/Squidex/app/features/schemas/pages/schema/types/assets-validation.component.html b/src/Squidex/app/features/schemas/pages/schema/types/assets-validation.component.html index 0dea43cb6..c9d4e39d5 100644 --- a/src/Squidex/app/features/schemas/pages/schema/types/assets-validation.component.html +++ b/src/Squidex/app/features/schemas/pages/schema/types/assets-validation.component.html @@ -1,9 +1,12 @@
- - -
- +
+
+ + +
@@ -11,12 +14,12 @@
- +
- +
@@ -24,12 +27,12 @@
- +
- +
@@ -37,10 +40,10 @@
- +
- +
@@ -48,12 +51,12 @@
- +
- +
@@ -64,12 +67,12 @@
- +
- +
@@ -80,12 +83,12 @@
- +
- +
@@ -93,7 +96,7 @@
-