fix: use data.id for ContentPageletEntryPoint id (#151, #152)

intershop · Mar 16, 2020 · 0529af5 · 0529af5
1 parent 274ca49
commit 0529af5
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/e2e/cypress/integration/specs/cms/server-html.mock.e2e-spec.ts b/e2e/cypress/integration/specs/cms/server-html.mock.e2e-spec.ts
@@ -22,6 +22,7 @@ describe('Server Html', () => {
       response: {
         definitionQualifiedName: 'app_sf_responsive_cm:pwa.include.homepage.pagelet2-Include',
         displayName: 'PWA Homepage Content',
+        id: 'pwa.include.homepage.pagelet2-Include',
         link: {
           title: 'pwa.include.homepage.pagelet2-Include',
           type: 'Link',

diff --git a/src/app/core/models/content-pagelet-entry-point/content-pagelet-entry-point.mapper.spec.ts b/src/app/core/models/content-pagelet-entry-point/content-pagelet-entry-point.mapper.spec.ts
@@ -21,7 +21,7 @@ describe('Content Pagelet Entry Point Mapper', () => {
   it('should convert a complex example to complex type', () => {
     const input: ContentPageletEntryPointData = {
       link: {
-        title: 'include-id',
+        title: 'Readable Title',
         uri: 'uri://test',
         type: 'ContentInclude',
       },

diff --git a/src/app/core/models/content-pagelet-entry-point/content-pagelet-entry-point.mapper.ts b/src/app/core/models/content-pagelet-entry-point/content-pagelet-entry-point.mapper.ts
@@ -36,7 +36,7 @@ export class ContentPageletEntryPointMapper {
     const configurationParameters = this.contentConfigurationParameterMapper.fromData(data.configurationParameters);
 
     const pageletEntryPoint: ContentPageletEntryPoint = {
-      id: data.link.title,
+      id: data.id,
       definitionQualifiedName: data.definitionQualifiedName,
       displayName: data.displayName,
       domain: data.domain,