Christian Frick

Hi

Please checkout https://de.www.ads-software.com/plugins/folders-for-maxgalleria/ and let me know if this would solve your request. The plugin is not yet officially released but I would like to hear your feedback.

Best regards
Christian

Hi Alan

I also realized that thumbnails are generated when showing the gallery, e.g. 180×180 when having a squared 6 column image tiles. I currently disable this by e.g. using the maxgalleria_thumb_size_square_six_column filter but don’t feel very comfortable with it because I fear some performance issues when applying all those filters.

Can you please also consider this issue when developing your new approach? IMO, it would be totally okay to use the default WP thumbnail by default and may introduce 1 optional MaxGalleria specific thumbnail.

Best regards
Christian

Those approaches are all quite brute force and inefficient. The one I choose is quite simple:
When an attachment is added, the path of the attachment is figured out, it is checked if the folder is already indexed, if not, it is recrusively added.

Your implementation has an issue at this specific part anyway: you assume that all attachments are always added to the default folder resp. the current month. This is afaik not the case: When adding an attachment to an older post resp. gallery, it is added to the months folder of the post, which could be an older one. The approach above does fix this issue.

Cheers, Christian

Hi Alan

IMO is having to check for new folders not really a proper solution. That’s why I spent some time and adapted your plugin a bit. I will send you the adapted file by email.

Here’s what I changed:
* When uploading an attachment to a folder which is not yet indexed by your plugin, it automatically adds the related folder recursively in an efficient way
* When performing a sync and finding an attachment which is already indexed by the Media Library but not your plugin, it only indexes it by your plugin but don’t generate a duplicate
* When moving an attachment on a Windows server, the guid is properly updated

Please let me know if you understand my adaptions and if they are fine for you. I would love to see them in a future release.

Best regards
Christian

Hi Alan
Sorry for my late answer. It’s currently almost too nice weather in Switzerland to spend time working on websites during leisure time ?? But let’s focus on the idea I had.

Afaik images are uploaded to default folders if you add non-existing images to a gallery. It’s possible to add images from another folder to a gallery via the “Media Library Folders” (MLF) plugin.

Well what I thought is: Let’s have the option to assign a folder which is managed via the MLF plugin to a gallery. If a non-existing image is added to the gallery via the gallery edit form, it is saved in this specific folder instead of the default folder of WP Media Library.

It’s not about adding all images which are present in this folder automatically to the gallery.

What is your opinion? Would this make sense and is it something what you could implement in near future?

Best regards
Christian

• This reply was modified 6 years, 8 months ago by Christian Frick.

Hi Alan

Thanks for your elaborated answer. It seems like the attachment post in the database was corrupt (I actually played around with it before I experienced the described problems).

It all works well now. Sorry, my bad.

Can you please also address the idea about the individually assigned folder to which newly added images would be added to?

Best regards
Christian

Hi Alan

Thanks a lot for your response. I solved the issue by adding corresponding functionality as add-on.

Best regards
Christian

Hi Alan

Thanks a lot for your response. Yes, I totally agree that the architecture of MaxGalleria is very nice. I actually really like to work with it.

I solved the issue by adding corresponding functionality as add-on.

Best regards
Christian

phoenixobia,

a first version of my plugin is available but not published. Let me know if you like to test it ([email protected]).

As far as I know is maxfoundry currently developing their own solution which might be inconsistent with mine or even make mine obsolete. That’s why I currently don’t push the plugin.

Cheers, Christian

1. Create two galleries
2. Add same image to both galleries
3. Remove both images

There are two equal posts in DB, both with post_parent = 0. Media Library pop-up filters duplicate. But Media Library admin page (Menu->Media->Library) shows same image twice.

4. Delete permanently first one

=> Image gets deleted (disappears from uploads folder)
=> but the duplicate is still in the Media Library and missing its file

Christian

I’m almost ready with it but the weather is currently to nice to work on it ?? And there is still some testing going on.

What do you prefer? Should I just add the code in here (it’s gonna be several functions of around 250 lines in total) or do you wan’t me to implement it as an add-on published as WP plugin. Just want you to know that I’m not experienced in writing a WP plugin – however, there’s allways a first time…

Maxfoundry, I don’t mind giving you the full rights of the code, assumed you are happy with it. I rather prefer it than maintaining it by myself.

Christian

Hi Alan,

Yes, this would probably solve the problem. However, I just wanted to report that little inconsistency. I just got to it while testing the “addon” regarding https://www.ads-software.com/support/topic/keep-photos-is-separate-locationsfolders.

Btw, do you wan’t me to report further bugs/requests on an other address?

Christian

Thank you Alan

I finally used maxgalleria_after_add_images_to_gallery, used the get_children function and ran it for all children.

Cheers, Christian

Hi phoenixobia

I’m currently working on solving this problem with a simple hook. Gonna post it as soon it is working.

Christian

I just had the same problem and could resolve it by updating the .htaccess file. Just browse to your WP settings => permalinks and click on the button “save changes”.