Import images into corresponding sub-gallery, basing on folders tree

What an absolutetly splendid solution to all the various suggestions for tree structures!

promising idea! Just what I need.

Yes - last time I tested 6.x things I didn't get anywhere, though. I'll give things another shot.

Lovely, works a treat!
What it does, though, is add image galleries which correspond to the import directories.
It doesn't replicate the directory structure in the /images/ folder on the hard drive ... bummer.

Yes please!!! Keep an eye here by the way:

#103793: Add token support for image file directory and image names

If ever you could commit this to a 5.x version as well, that would be Sooooooo great!

Kind regards!

nice idea. Not ready for production yet.

?? The patch does exactly what it advertises: creates image galleries based on the file structure of the import folder.

How is this "not ready for production yet"?

Thanks ...

Yeah, what's the actual objection?
Some code formatting like a half dozen whitespaces is about the only issue I can see from looking at it (although I've not actually run the code ;-)

Try using coder.module.
That'll tell you that "Drupal" expects

if ($form_state['values']['tree'] ) {
...
for ($i=0; $i<sizeof($requested_subgalleries); $i++) {

not

if( $form_state['values']['tree'] ) {
...
for($i=0; $i<sizeof($requested_subgalleries); $i++) {

:-)
Totally trivial, but there you go.
That's about it I think (wherever you see it)
;-)

OK, I can confirm that this patch still works a treat, no bugs, with importing images and displaying the resulting galleries.

(Image import says "# Created new gallery a-b." and "# Created new gallery c-e.", but it will import the images into existing galleries a-b and c-e without problems.)

My apologies, it did indeed create the folders it said it did - I'd forgotten one part of my gallery structure, on making folders in the import folder.

Tried again with the currently valid gallery structure on hard drive: everything works as advertised, including the correct "created gallery" message only on creating a new gallery.

It really is a lovely patch.

Hello, is it working with drupal 5.7?

I've been using this patch to import images in D6, works great! Thanks!

I don't think this is necessary for you to make a 5.x version of your patch in order for your patch to be commited...

... but it might definately be interesting for many existing drupal sites already running under the 5.x release.

looks pretty good (i'd suggest proper sentence capitalization and punctuation on the comments but i won't hold up a patch on that count), the only thing that needs work is that the hook_uninstall() implementation should delete the variables.

whoops, my bad mis-read that...

is this in the alpha release?

i applied the patch to the 6.x-1.x-dev 2008-Apr-18 release and it worked great for the small test I did. I mean really beautiful, I had jpgs in folders, in folders, etc and it created galleries and subgalleries corresponding to my directories. Just perfect! I really hope this gets in a release. It is a huge usability / newbie friendly feature!

Same comments, really good patch/addon !!

Thanks a lot

Will this work with this http://drupal.org/node/103793 (naming of files/storing in the directory structure)?

http://drupal.org/node/253631 seems to be related (the "hierarchical taxonomy mode, additionally sets the terms hierarchical relations to resemble the path structure" part).

i can't get to the patch. i'd like to see some other people testing out this patch.

"file not found" in French on the patch site ... sto: you can't upload a file with the same name as one that already exists. I suggest you upload to drupal as image_import_keep_tree-225014_39.patch (where "39" is the number of the comment in the thread ...)

Does the patch working fine with the version "6.x-1.x-dev 2009-Mar-23"?

Or what version should i use to apply the patch?

i am going to develop a photo site with several thousands images, it's painful if all the images are store into one folder..

Any help would be appreaciated!

The latest uploaded patch is not available any more. Could someone please upload the latest patch again?

Any chance this will be updated David? This is a crucial patch for those of use dealing with complicated gallery structures. Thanks for your work and I hope you see this through until it is committed. (How do we help you get this committed by the way?)

I'd like to have this patch updated too. Have to import 2700 Images at once. The Images are well structured in a folder hierarchy. But uploading every folder by itself would be horrible. I've searched about hours but found no solution for image_import 6.x-1.0-alpha5. And I would really like to use image module because everything else is working like a charm with this module.
Thanks in advance for every helping hint.

> (How do we help you get this committed by the way?)

Image 1.0 is getting released at DrupalCon, whatever state things are in. For this patch to go in, it needs to be rerolled against CVS HEAD and tested by at least one other person.
Anyone can reroll the patch against the current CVS HEAD. (well anyone with basic coding skills ;)

Cool!
I'll keep this thread updated: http://paris2009.drupalcon.org/forum/image-10

Works perfectly!
Committed.

Thanks for your perseverance with this :)

Couple of minor things to note for future patches -- please roll them against the main module root (ie the image folder), and also avoid tweaks like the fixes to '!edit-link' on line 77. The latter are good, but should be in a separate issue so the patch is easier to read and understand. (Read webchick's post

I didn't find anyone else for sprinting yesterday and was hampered by lack of wifi.
The plan is to sprint saturday if there are people around -- Sat morning probably, watch the forum post and IRC.

Cool.
I'm joachim on IRC. Currently in the aegir session if you want to come say hello today.
Also -- sun, quicksketch, and I will be figuring stuff out after the imagefield session tomorrow.

Hi,

this addition is a very nice add-on to the image_import sub-module; I tried it on a quite fresh D6 site and imported a bunch (~ 1000) of images. After clicking through a sequence of import screens for about an hour, all the image nodes were created properly - very nicely done; there are not many working alternatives for bulk uploading images to a Drupal site at the moment, so this is a huge improvement!

But (sorry ;) - I couldn't access any of the newly created galleries or sub-galleries: Access denied. You have no access permission for this page. I'm User #1 and already tried to rebuild the Content access permissions at ./admin/content/node-settings without success. Basically nobody seems to be able to access the newly created galleries/taxonomy terms.

Does anyone else experience this error with the recent image_module, or is this just another issue personally crafted fo me?

Thanks for this well done feature & greetings,
-asb

> Are you using other modules, such as Taxonomy Access Control ?

Nope. But after some digging I think your patch is "not guilty", all older galleries seem to have become inaccessible, too (taxonomy term pages are displayed, image gallery pages based on taxonomy terms are inaccessible). Needs further investigation and requires probably a separate issue...

Edit: #571504: Galleries not working anymore

Greetings, -asb