The assets for elfinder.min.js are giving 404 when using a different asset_path

[ghost]

Describe the bug
I have my asset_path configured to a different folder now with the recent changes: #493 i breaks it probably was already broke before this fix. Some more context: https://github.com/helios-ag/FMElfinderBundle/pull/493/files#r1162540304

To Reproduce
Have a asset_path defined to different folder then the default and check you get 404 when loading the assets.

[ghost]

Also can confirm that /img/volume_icon_ftp.png is also not loading for me

[enboig]

I am affected by this bug, when will it be merged and released?

[ghost]

The fix is in #509 but there is not much activity there atm.

For me i just locked my composer.json to older version for temporary fix to: "helios-ag/fm-elfinder-bundle": "12.2.1",

[enboig]

I was thinking of using https://github.com/symplify/vendor-patches

[helios-ag]

I can merge it guys, but didn't test for back compatiblity, if someone confirms, ok, will proceed

[JohJohan]

I can confirm it works for me, but thats expected since i supplied the fix. When will we get a release?

@enboig maybe you can confirm it works for you?

[jontjs]

This update appears to have broken elfinder for us (default assets path, and also still using Symfony 6.4 in case that's relevent). With version 12.5 of this bundle, the elfinder assets are loaded from bundles/fmelfinder/ (which matches where other bundles but their assets) but they are now loaded from assets/bundles/fmelfinder/.

I've got a feeling that the problem is that the default value of assets_path in the bundle config is "assets", is different to the default value of the docroot argument to the install command (which is blank).

As a workaround, you can either set the assets_path to an empty string in the bundle config, or run elfinder:install --docroot=public/assets to install the assets.

[JohJohan]

Ah sorry @jontjs it broke for you and probably others, i'm thinking changing the default assets_path to empty is also not an option as that could possible break other things i think.

[helios-ag]

@JohJohan @jontjs if you confirm we can make value empty by default