Uploaded image for project: 'Magnolia UI'
  1. Magnolia UI
  2. MGNLUI-1590

Replace FieldValidatorRegistry with a implementationClass property in FieldValidatorDefinition

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Neutral
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 5.0
    • Component/s: framework
    • Labels:
    • Sprint:
      RC 1

      Description

      • remove info.magnolia.ui.api.builder.MappingFactoryBase
      • remove info.magnolia.ui.form.validator.registry.FieldValidatorFactory
      • inline info.magnolia.ui.api.builder.FactoryBase into info.magnolia.ui.form.field.builder.FieldFactory
        • see info.magnolia.ui.api.action.AbstractActionExecutor#createAction
      • add implementationClass in FieldValidatorDefinition
      • remove bootstrap files for fieldValidatorRegistry
        • /mgnl-bootstrap/ui-admincentral/config.modules.ui-admincentral.fieldValidatorRegistry.xml
        • and the ones in security app

      The linked tickets are obsoleted by these changes.

        Checklists

        Acceptance criteria

          Attachments

            Issue Links

            supersedes

            Bug - A problem which impairs or prevents the functions of the product. MGNLUI-581 Lookup of an action implementation in BaseFactory based on the definition type is completely random

            • Major - Major loss of function, or significant new/changed feature
            • Closed

            Task - A task that needs to be done. MGNLUI-877 Factories should not copy contents of registries into state of their own

            • Major - Major loss of function, or significant new/changed feature
            • Closed

            Improvement - An improvement or enhancement to an existing feature or task. MGNLUI-1542 For configuring definition to implementation mappings the name definitionToImplementationMappings should be just mappings

            • Neutral -
            • Closed

              Activity

                People

                Assignee:
                tmattsson Tobias Mattsson
                Reporter:
                tmattsson Tobias Mattsson
                Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                  Dates

                  Created:
                  Updated:
                  Resolved:

                    Checklists

                    DoD