Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

check rename error (DocumentManager::move results in an error) #52

Closed
lsmith77 opened this issue Sep 4, 2013 · 1 comment
Closed

check rename error (DocumentManager::move results in an error) #52

lsmith77 opened this issue Sep 4, 2013 · 1 comment
Assignees
Milestone

Comments

@lsmith77
Copy link
Member

lsmith77 commented Sep 4, 2013

followup for #46

@rmsint
Copy link
Contributor

rmsint commented Sep 27, 2013

Tested this a bit more and the issue is not related to Doctrine\ODM\PHPCR\DocumentManager::move but to the FM\ElFinderPHP\Driver\ElFinderVolumeDriver::rename.

In the last lines of the method it performs the Symfony\Cmf\Bundle\MediaBundle\Adapter\ElFinder\PhpcrDriver::_move and it expects back a string with the new path, we now return a boolean true. However the error is from theSymfony\Cmf\Bundle\MediaBundle\Adapter\ElFinder\PhpcrDriver::_stat being called afterwards. It tries to find for an unknown document, the document does not exist and then it tries to create a Directory for '/cms/media', the root, that already exists and thus throws an error. So this is related to issue #53.

While solving this we should also check that moving a document still works afterwards.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants