Uploaded image for project: 'Magnolia REST Framework'
  1. Magnolia REST Framework
  2. MGNLREST-160

Implement: Moving reference resolver to NodeWriter

    XMLWordPrintable

Details

    • Improvement
    • Resolution: Fixed
    • Neutral
    • 2.1
    • None
    • None
    • Saigon 126, Saigon 127, Saigon 128, Saigon 129, Saigon 130, Saigon 131, Saigon 132, Saigon 133, Saigon 134, Saigon 135, Saigon 136
    • 5

    Description

      After discussed, we agreed new approach as bellow:

      • Create simple expander by using Decorator/Wrapper to resolve item to canonical format: [provider]:[workspace]:[uuid] (see DEV-649 for more details). Then resolve that id in NodeWriter.
      • Register the ComponentProvider to RestDispatchServlet, then inject it by @Context in NodeWriter, then get resolver by provider in resolving canonical format.

      Once this is done, if reference expansion becomes automatic (for the new canonical format for example), then we will have to think about how to enable/disable reference expansion per endpoint. But we don't have to think about this for now, since all expansions have to be configured.

      Checklists

        Acceptance criteria

        Attachments

          Issue Links

            Activity

              People

                sang.ngo Sang Ngo Huu
                sang.ngo Sang Ngo Huu
                Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved:

                  Checklists

                    Task DoD

                    Time Tracking

                      Estimated:
                      Original Estimate - 3d Original Estimate - 3d
                      3d
                      Remaining:
                      Time Spent - 16.75d Remaining Estimate - 1h
                      1h
                      Logged:
                      Time Spent - 16.75d Remaining Estimate - 1h
                      16.75d