Uploaded image for project: 'Nested Pages Migrator Application'
  1. Nested Pages Migrator Application
  2. NPMIG-56

Incorrect message "We have detected some pages that will have too long path after the migration (limit is 255). You should rename them (or rename one of their parents) before computing a new plan."

    XMLWordPrintable

Details

    • Bug
    • Resolution: Fixed
    • Critical
    • 0.8, 0.7.1
    • 0.7
    • UI
    • None
    • Trivial

    Description

      The importer tells about a path limit of 255 characters while since 13.2 the limit has been increased to 768 characters (https://forum.xwiki.org/t/xwiki-13-2-release-candidate-1-released/8492)

       

      The fix is trivial: there are only two places in 

         application-nestedpagesmigrator-ui/src/main/resources/NestedPagesMigration/Code/MigrationProjectSheet.xml

      to be changed:

      diff --git a/application-nestedpagesmigrator-ui/src/main/resources/NestedPagesMigration/Code/MigrationProjectSheet.xml b/application-nestedpagesmigrator-ui/src/main/resources/NestedPagesMigration/Code/MigrationProjectSheet.xml
      index 62740ff..131af2e 100644
      — a/application-nestedpagesmigrator-ui/src/main/resources/NestedPagesMigration/Code/MigrationProjectSheet.xml
      +++ b/application-nestedpagesmigrator-ui/src/main/resources/NestedPagesMigration/Code/MigrationProjectSheet.xml
      @@ -170,7 +170,7 @@
      <p>If you are ok with it, just run the migrator and these documents will be overwritten.</p>
      </div>
      <div class="box errormessage" data-bind="visible: plan.tooLongs().length > 0">

      • <p>We have detected some pages that will have too long path after the migration (limit is 255). You should rename them (or rename one of their parents) before computing a new plan.</p>
        + <p>We have detected some pages that will have too long path after the migration (limit is 767). You should rename them (or rename one of their parents) before computing a new plan.</p>
        <p>Theses pages are:</p>
        <ul data-bind="foreach: {data: plan.tooLongs(), as: 'action'}">
        <li class="monospace"><a data-bind="attr: {href: action.getSourceLink()}" target="_blank"><span data-bind="text: action.serializedSourceDocument()"></span></a> -&gt; <span data-bind="text: action.serializedTargetDocument()"></span></li>
        @@ -1301,7 +1301,7 @@ define('MigrationAction', ['utils', 'knockout'], function (utils, ko) {
      • @return if the document path is too long (and then cannot be saved in the database)
        */
        self.isTooLong = function () { - return self.serializedTargetDocument().length > 255; + return self.serializedTargetDocument().length > 767; }

      /**

      Attachments

        Activity

          People

            acotiuga Alex Cotiugă
            wech Christoph Weber
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: