500 on reorder route

[ElectricMaxxx]

Sometimes we get an 50 when moving a node. On the one hand there is no error handling (Wheel is spinning) and on the other this error should not happen.

[ElectricMaxxx]

successfull move are going after even i selected before a node.

We should fix both to have a fluffy feeling, when moving nodes.

[ElectricMaxxx]

{▼
  /home/maximilian/OpenSource/Cmf/cmf-sandbox/vendor/phpcr/phpcr-utils/src/PHPCR/Util/NodeHelper.php:362: {▼
    : if (false === $oldpos) {
    :     throw new ItemNotFoundException("$name is not a child of this node");
    : }
  }
  /home/maximilian/OpenSource/Cmf/cmf-sandbox/vendor/jackalope/jackalope/src/Jackalope/Node.php:461: {▼
    : 
    : $this->nodes = NodeHelper::orderBeforeArray($srcChildRelPath, $destChildRelPath, $this->nodes);
    : $this->setModified();
    arguments: {▼
      $name: null
      $destination: "company"
      $list: [ …3]
    }
  }
  /home/maximilian/OpenSource/Cmf/cmf-sandbox/vendor/sonata-project/doctrine-phpcr-admin-bundle/src/Controller/TreeController.php:125: {▼
    : }
    : $parentNode->orderBefore($targetName, PathHelper::getNodeName($dropedAtPath));
    : $this->session->save();
    arguments: {▼
      $srcChildRelPath: null
      $destChildRelPath: "company"
    }
  }
  Sonata\DoctrinePHPCRAdminBundle\Controller\TreeController->reorderAction(): {}
  /home/maximilian/OpenSource/Cmf/cmf-sandbox/vendor/symfony/symfony/src/Symfony/Component/HttpKernel/HttpKernel.php:153: {▶}
  /home/maximilian/OpenSource/Cmf/cmf-sandbox/vendor/symfony/symfony/src/Symfony/Component/HttpKernel/HttpKernel.php:68: {▶}
  /home/maximilian/OpenSource/Cmf/cmf-sandbox/vendor/symfony/symfony/src/Symfony/Component/HttpKernel/Kernel.php:171: {▶}
  /home/maximilian/OpenSource/Cmf/cmf-sandbox/web/app_dev.php:39: {▶}
  /home/maximilian/OpenSource/Cmf/cmf-sandbox/vendor/symfony/symfony/src/Symfony/Bundle/WebServerBundle/Resources/router.php:42: {▶}
}

Request Params

dropped | "/cms/routes/fr/company"
parent | "/cms/routes/fr/company"
position | "over"
target  | "/cms/routes/fr/company/contact"

When dropping contactroute over (not good before) company

So either, we shouldn't drop in, or we should drop in with a defined position. Or we should open a node before dropping in.

At least we should catch the error and inform the user.

[ElectricMaxxx]

Fixed throug #158