Well I was surprised to find the module actually works perfectly and as described. I was quickly able to launch a bunch of nodes from dragging/dropping images into the form. I like this a lot better than the older flash based modules.

Ideas on additions....

  • Show Image on node/edit screens, helps context when entering description & name etc.
  • Use some relationship module to relate the created Image nodes to Gallery nodes or other node-types that have galleries. Possibly http://drupal.org/project/nodereference_url
      • Thanks for starting this module! :)

Comments

pyrollo’s picture

pyrollo commented

Thanks for opening that issue.

Show Image on node/edit screens : Image should be displayed instead of the file field. I'll check and if it works on my side, I will ask you for precisions.

Use some relationship module : That's also one of my needs. I started to work on the possibility of adding node fields to the upload screen which values will be copied in each created nodes. This would do the trick for relationships and more.

Macronomicus’s picture

Macronomicus commented

It seems the image is working when viewing the full page view somehow but in node/edit the file is named as such p16a6qjkfeelfatq1npv1tm7kda4.tmp and therefore cannot show as an image.

Here is the output of of that section on the node/edit page.

<div class="form-item form-type-item" id="edit-field-image">
  <label for="edit-field-image">Image </label>
 <img alt="" src="http://mesite.dev/sites/mesite.dev/files/styles/medium/public/image-galleries/p16a6qqg6e1vev1pq0cbg1jvc1i6p3.tmp">
</div>

As to the http://drupal.org/project/nodereference_url this is how many folks did it in Drupal 6 as it makes a links to [Add Images] or whatever sub content type your adding to your Container-Node that launch node/add/image for example.... and it fills the reference-field with the Container-Node id in all the created sub-nodes - when you save it returns you to the Container-Node being reference. With this you can use views to show the images in the Container-Nodes - Galleries etc.

I have the above working fine in d7 the only issue is this only allows one image-node added at a time.
In drupal6 something like your module 'fupload' enabled bulk image upload based on one image per/node that created the nodes... I like yours better though.

Whats mainly needed is the Container-Node (ie Gallery) needs to be in the url when creating the Image-nodes, then the node-reference would be sorted by default to the container-node being viewed. I hope that makes sense... ive been reading too much d7, views, references stuff today... lol

That sounds pretty cool about adding node fields to the upload screen, Im interested to see how you set things up.

Cheers!

Macronomicus’s picture

Macronomicus commented

I could file a separate bug report for the .tmp image name thing if you want me to? I've been tinkering around with it but no luck so far, it must be imagecache thats setting the .tmp name for the files?

I looked and it seems in the full node-view the image shows but the img tag is still using the temp filename .tmp, I have no idea why that image is viable in the full node view.

pyrollo’s picture

pyrollo commented

Yes you may open a separate bug report for the image display because it does not seem obvious to solve...

I had a try here. I also have a .tmp file but it displays well. It may be a navigator or a mime type issue.

I'm using Firefox under Linux and I'm getting an http content type of image/jpeg for the .tmp file.

pyrollo’s picture

pyrollo commented

I've just commited a new version with that field management.
It allows to add fields to the upload form and should work also with nodes references as soon as they are considered as fields.
I recommand to uninstall previous version before installing this one !

Macronomicus’s picture

Macronomicus commented

Sweet, i will test it out right away, thanks!
I admit I tried to tweak/fix your first upload of the module, but failed miserably .... lol, I'm finally going study php on a much deeper level this winter.
:/

BarisW’s picture

BarisW commented

See #1289812: Images are saving as .tmp files instead of proper name

Patch fixes the .tmp naming bug

pyrollo’s picture

pyrollo commented

Hello,

I've committed a new version that adds "normal" entry points as dynamic menus.
I've also removed specific the image preview, the default one is now used.

Tell me if you have other ideas of entry points for that module.