πŸ‡³πŸ‡±Netherlands @clivesj

Account created on 26 March 2007, over 17 years ago
#

Recent comments

πŸ‡³πŸ‡±Netherlands clivesj

Please re-open if and as needed

πŸ‡³πŸ‡±Netherlands clivesj

Please re-open when and as needed

πŸ‡³πŸ‡±Netherlands clivesj

I can't recall why the lines are commented out. MR accepted thanks!

πŸ‡³πŸ‡±Netherlands clivesj

clivesj β†’ made their first commit to this issue’s fork.

πŸ‡³πŸ‡±Netherlands clivesj

My preference would be to increase the size of the "path" column.
I thought 255 was already generous. Until now we didn't hit this limit. What size do you think is useful without increasing it too much?

For the paths that are too long (with respect to the new limit) I will display an error notifying that that file is not added to the directory listing. Is this workable?

I have changed the version to 3.1.x-dev since 3.0.1 is D9

πŸ‡³πŸ‡±Netherlands clivesj

Thanks Vlooi, fixed in dev

πŸ‡³πŸ‡±Netherlands clivesj

I get it now: This is due PHP >=8.2
I will come back with a patch shortly.

πŸ‡³πŸ‡±Netherlands clivesj

Can you give me the steps to reproduce this? I can not.
Thanks

πŸ‡³πŸ‡±Netherlands clivesj

Hi Tim,

I will have a stable release for 3.1.0 shortly. Isn't that a better solution for you?
I don't want to work the D10 issues on two different branches.

πŸ‡³πŸ‡±Netherlands clivesj

I am not able to replicate this.
Take a look at attached screenshot.
Drupal will handle this and I just checked it and it works file.
Can you have another look and provide more info?

πŸ‡³πŸ‡±Netherlands clivesj

The fields on the dir-listing nodes are not real fields. The are injected on the node-create or edit form to create a Filebrowser object. that was a design choice early on on the design of the module. That is why Populate module will not work.

It is definitely doable to make it work, but I will not be able to find time to work on it at this moment.

If anyone wants to work on it please feel free to reopen this.

πŸ‡³πŸ‡±Netherlands clivesj

Fixed in rc-2

πŸ‡³πŸ‡±Netherlands clivesj

OK, that's a compatibility issue with D10 that requires explicite access check. We fixed it several other places but missed this one.
I will fix this coming days.

πŸ‡³πŸ‡±Netherlands clivesj

Your request contains two parts:
1. Provide default configuration for folder path.
2. Accept token for the folder path.

These two will indeed facilitate programmatically creation of directory listings.
We can sure add a configuration for a default folder path.
Please refer to
https://www.drupal.org/project/filebrowser/issues/2941911 β†’
to read the discussion on using tokens in the folder path. We have to make sure that once created, the directory name does not change accidentally due to a unstable token.

I short time I have no time to work on this, but if you want to provide a patch, make sure to also include the update for the configuration in the install.yml file.

πŸ‡³πŸ‡±Netherlands clivesj

Please provide all work for D10 on Filebrowser 3.1.x

πŸ‡³πŸ‡±Netherlands clivesj

Hi,
3.1.0 dev should compatible with D10.
Did you ceck that out already?

πŸ‡³πŸ‡±Netherlands clivesj

Hello jmaerckaert,
The D10 development is by branch 3.1.0:
https://www.drupal.org/project/filebrowser/releases/3.1.x-dev β†’
Is this what you are looking for?

πŸ‡³πŸ‡±Netherlands clivesj

I will close this one, feel free to open it if and when needed.

πŸ‡³πŸ‡±Netherlands clivesj

If I remember well, that class was removed because it is not necessary.
But I will have to check further, I don't think that code was removed by accident.
On my system it working properly on a D10 install:

Production build 0.69.0 2024