Uploaded image for project: 'Magnolia'
  1. Magnolia
  2. MAGNOLIA-7483

Definitions in registry that is using nameAsId are not unregistered when definition name changes

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Critical Critical
    • 6.2.7
    • 6.0
    • None
    • Yes
    • Features 7, Features 8, Features 9, Features 10, HL & LD 23, HL & LD 24
    • 8

      When the "defaultGroup" is configured on the app launcher (as documented https://documentation.magnolia-cms.com/display/DOCS57/App+launcher+layout), then CT generated apps are displayed twice in the app launcher.

      Furthermore, after opening one of the tiles and closing the app again, the tile remains hilighted.

      I am not sure if this only happens with CT generated apps - or with any kind of app.

      The app shows up twice in the Definitions app, but if you click on the first one, the second one is selected. 

      This can happen very easily because when you first create the app file, even with no contents in the file - the first app is created, as a name for the app, the filename of the app yaml file is used. When you then enter a different name property, you already have two apps.

      To reproduce:

      • Create a CT and an app.
      • See the app in the app launcher.
      • Change the "name" property" in the app.
        • (If the app had no name property - and then you add a name propertly that is different then the filename - this also causes the problem.)
      • Logout and in again.
      • Note that the app shows up TWICE in the app launcher.
      • Only a restart appears to solve the problem.

       

       

      All registries that are using nameAsId strategy are affected by this issue.

      When definition name changes, old defition is not removed from the registry. Leaked definitions stay in the registry until restart.

        Acceptance criteria

              canh.nguyen Canh Nguyen
              czimmermann Christopher Zimmermann
              Votes:
              1 Vote for this issue
              Watchers:
              4 Start watching this issue

                Created:
                Updated:
                Resolved:

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0d
                  0d
                  Logged:
                  Time Spent - 3.5d
                  3.5d