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

Port Virtual URI mappings to a Registry

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Neutral
    • Resolution: Done
    • Affects Version/s: 4.3.8
    • Fix Version/s: 5.5.6, 5.6
    • Component/s: Virtual URI mappings
    • Labels:
      None
    • Sprint:
      Saigon 87, Saigon 88, Saigon 89, Saigon 90, Saigon 91, Saigon 92, Saigon 93
    • Story Points:
      8
    • Magnolia Release:
      5.5.6, 5.6

      Description

      In order to support folder-hierarchy within Virtual URI mappings config, VirtualURIManager should be ported to an AbstractRegistry.

      • The registry is bound to configuration-sources upon module startup
        • allowing virtual URI mappings by file at the same time
      • Existing VirtualURIManager goes to deprecation
      • Beware: registries are currently populated upon startup of the "owning" module. This is different to when virtual URI mappings & commands are loaded now (after all modules are started).
        • we could also consider generally deferring population of registries in similar fashion, while we're at it.

      Side notes:

      • Virtual URI mappings could have their own app (outside of About app)
      • App could become a subset of Definitions app?
      • Can registry offer specific API to know which mapping fired? Would it be the right place?
        • just mentioning here but might as well be considered in a superseding ticket.

      Original title: Allow subfolders/nodes in the observed 'virtualURIMapping' module nodes

      When having a lot of virtualURIMappings registered in a module, it can be quite hard to keep the overview.
      Being able to sort them in folders/nodes would improve that a lot (as in paragraphs/dialogs already possible).

      I added a patch of this class, which does that. Code should be checked for core quality, was implemented very quickly.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                oanh.thai Oanh Thai Hoang
                Reporter:
                cringele Christian Ringele
              • Votes:
                2 Vote for this issue
                Watchers:
                10 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Date of First Response:

                  Time Tracking

                  Estimated:
                  Original Estimate - 5d Original Estimate - 5d
                  5d
                  Remaining:
                  Remaining Estimate - 0d
                  0d
                  Logged:
                  Time Spent - 6d 6.5h
                  6d 6.5h