Trimming the first character in the directory name when integrating into CKEditor5

[besnovatyj]

Integration into CKEditor5 using https://github.com/Studio-42/elFinder/wiki/Integration-with-CKEditor-5.

The hash is generated using #1564 (comment)

If you pass not only the volumeId, but also an additional path to the directory, then the first character in the path to the directory is cut off in line 3334.

elFinder/php/elFinderVolumeDriver.class.php

Line 3334 in 66f4975

$path = str_replace($this->separator, '/', substr($this->decode($hash), strlen(rtrim($this->root, '/' . $this->separator)) + 1));

---

Initial data:

$uploadTargetPath = 'documents_module';
$volumeId = 'fls1_';
$fileName = 'yadisk-3.jpg';
$hash = 'fls1_ZG9jdW1lbnRzX21vZHVsZS95YWRpc2stMy5qcGc';

Expected value: documents_module/yadisk-3.jpg
The resulting value: ocuments_module/yadisk-3.jpg

---

As a result, the script returns an incorrect URL to the locally downloaded file.

 fm.exec('upload', {files: [file], target: uploadTargetHash}, void(0), uploadTargetHash)
                                .done(data => {
                                    if (data.added && data.added.length) {

In the code above, data.added contains an invalid URL to the file.

[github-actions]

This issue is stale because it has been open for 50 days with no activity.

[parpar78]

Glad i am not the only one with this problem.
Is this package still maintained? Last update was 2023...

[nao-pon]

Will this be fixed by changing L.3334 to the following?

$path = str_replace($this->separator, '/', substr($this->decode($hash), strlen(trim($this->root, '/' . $this->separator))));

[besnovatyj]

Will this be fixed by changing L.3334 to the following?

$path = str_replace($this->separator, '/', substr($this->decode($hash), strlen(trim($this->root, '/' . $this->separator))));

Yes, it works for me! :)

[github-actions]

This issue is stale because it has been open for 50 days with no activity.