Uploaded image for project: 'eZ Publish / Platform'
  1. eZ Publish / Platform
  2. EZP-27083

Errors in "Extending PlatformUI with new navigation" tutorial

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Medium Medium
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: n/a
    • Component/s: Documentation
    • Labels:
      None

      Description

      There are few errors in the fourth tutorial in the documentation (https://doc.ez.no/display/DEVELOPER/Extending+PlatformUI+with+new+navigation).

      Tutorial 4, step 3:
      Missing the line:

      service: Y.eZ.DashboardBlocksViewService,
      

      in the last code fragment, inside app.route. Without it the dashboard elements are not loading correctly and console is throwing an error when accessing the dashboard from the /ezconf/list URL.

      Tutorial 4, step 6.2:
      There is no information that configuration in file yui.yml should be changed to:

      ezconf-listviewservice:
          requires: ['ez-serversideviewservice']
          path: %extending_platformui.public_dir%/js/views/services/ezconf-listviewservice.js
      

      which means changing the "requires" parameter. Without that change lists is not showing correctly and the console is throwing an error.

      Tutorial 4, step 6.2, fixing the link issue:
      Code fragment where we are throwing an event should look like this:

      this.fire('navigateTo', {
          route: {
              name: link.getData('route-name'),
              params: {
                  id: link.getData('route-id'),
                  languageCode: link.getData('route-languagecode'),
              }
          }
      });
      

      This way we don't need to subscribe this event in the "ezconf-listviewservice.js" (we could just skip this part of the tutorial) and it still will be working as expected.
      Without that change after clicking any link from the list we don't move to any location and the console is throwing an error.

      Tutorial 4, step 7:
      Analogical to previous fix. Correct code to throw an event:

      this.fire('navigateTo', {
          route: {
              name: 'eZConfListOffset',
              params: {
                  offset: offset,
              }
          }
      });
      

      tutorial 4, step 8:
      Analogical to previous fix. Correct code to throw an event:

      this.fire('navigateTo', {
          route: {
              name: 'eZConfListOffsetTypeIdentifier',
              params: {
                  offset: "0", // offset: 0 does not work, because of a bug in PlatformUI
                  typeIdentifier: select.get('value'),
              }
          }
      });
      

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Confirmed Confirmed
        67d 19h 27m 1 Dominika Kurek 15/May/17 8:20 AM
        Confirmed Confirmed Backlog Backlog
        5s 1 Dominika Kurek 15/May/17 8:20 AM
        Backlog Backlog Documentation Documentation
        4s 1 Dominika Kurek 15/May/17 8:20 AM
        Documentation Documentation Documentation Review Documentation Review
        1d 1h 34m 1 Dominika Kurek 16/May/17 9:54 AM
        Documentation Review Documentation Review Documentation Review done Documentation Review done
        17d 2h 6m 1 michal.kusztelak@ez.no 02/Jun/17 12:01 PM
        Documentation Review done Documentation Review done QA QA
        3d 4h 41m 1 rui.silva@ez.no 05/Jun/17 4:42 PM
        QA QA Development Review done Development Review done
        132d 16h 57m 1 Paulo Nunes 16/Oct/17 9:40 AM
        Development Review done Development Review done Documentation Review done Documentation Review done
        10s 1 Paulo Nunes 16/Oct/17 9:40 AM
        Documentation Review done Documentation Review done Closed Closed
        157d 6h 34m 1 Dominika Kurek 22/Mar/18 3:15 PM

          People

          • Assignee:
            Unassigned
            Reporter:
            Jacek Foremski (Inactive)
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: