Cross-posting:
[snip]
I'm currently investigating, if image.module is needed at all in future. Basically, Image module provides nothing more than a node with one image attached. In contrast, that's basically a custom content-type with just one imagefield.
So I've (dirty) hacked Img_assist a bit - just to see, if it could be an Imagefield_assist module, too. Guess what? It basically worked out.
[/snip]
see: http://drupal.org/node/119539#comment-305464
I wonder why there is no such issue in this (quite long) queue yet. ;) However, as contents become more and more CCK content-types, the chance of having all images in image.module based nodes decreases with each new feature in Imagefield (v5.x-2.x is right on the way with several great enhancements like ajax upload). Img_assist should therefore open the doors to re-use images of imagefields.
Comment | File | Size | Author |
---|---|---|---|
#8 | img_assist.imagefield.patch | 32.83 KB | mbehrman |
#6 | img_assist.imagefield.patch | 134.8 KB | mbehrman |
Comments
Comment #1
zoo33 CreditAttribution: zoo33 commentedI suppose all image related modules (or all modules that work with "image nodes") should have the ability to work with both Image nodes and node types that have an image field. This would apply to Image assist, Image gallery, Image attach etc. I'm imagining a setting where you can choose what node types you want to include in Image assist.
Keeping compatibility with both Image and Imagefield would be a big challenge though. It could easily turn into a real hack fest. :) I suppose we could maintain two separate versions of Image assist, of maybe have two different sub modules that take care of the integration with each image solution.
This is all very interesting, but I don't personally have any real nead for this. So I guess an effort like this would have to be done by someone other than me for the most part. I would be interested in seeing it happen though.
By the way, it's very cool that you already have a working example for Imagefield integration, although I haven't tested it myself.
Comment #2
dnewkerk CreditAttribution: dnewkerk commentedHey Daniel -
I just wanted to mention how great it is that you're tackling this issue... thanks a ton for this and all your other contributions to Drupal :)
Lately I've been growing fond of Image Assist as compared to the other methods of handling images in Drupal (seems the most "usable" for end-users), but wished it could do the same for all my CCK types with Imagefields + Image cache scaling/cropping... and lo and behold (haha) here it is already on the way :)
Again thanks, and I'll try to help in any way I can (testing for bugs once you release it, etc).
Comment #3
chromix CreditAttribution: chromix commentedThe patch doesn't work on the latest release of Image Assist (surprise surprise). Has anything else come of the patch? I'm actually a big fan of trying to spin off a separate module for Image Assist for Image Field Nodes.
Comment #4
zoo33 CreditAttribution: zoo33 commentedsun has some great ideas about what should be done with this module in terms of refactoring, splitting it's different functionalities into separate modules etc. Only question is, who will do it, and when?
Comment #5
chromix CreditAttribution: chromix commentedMe and another developer here are going to start working on generalizing Image Assist to support CCK nodes that have Image Fields. Hopefully this can be useful to everyone as Image Assist incrementally moves towards where sun envisions it going.
Comment #6
mbehrman CreditAttribution: mbehrman commentedSo I've got an update to the work sun did before to integrate imagefield support. The patch needs to be reviewed, and I'm sure there are issues I haven't thought of and we haven't tested yet. It will add the following features (all options in the 'other options' area of the image_assist settings):
- support for imagefield (enabled optionally in the 'other options' area)
- you can choose which content type to use that contains an imagefield, only one can be used
- you can choose another CCK field to get the description
- the add node functionality when inserting an image works - you'll have the option to add a node of the content type chosen
Some things this doesn't do...
- resize each image - you are stuck with the original size image, it is just displayed at the asked for size in the html
Where this might break:
- I don't know the complexities of CCK that well. Things get strange in the CCK tables when a field is used in multiple content types and I haven't tested this.
- What will happen to Image node support? I'm fairly sure everything will work fine and did a bit of testing. But it should be tested more since our main environment doesn't use the Image node.
Hope this is useful to people.
Comment #7
sunSorry, Michael, but 90% of your patch are completely unnecessary or unrelated (tab-) changes. No one is able to review what you've actually changed. Please re-roll.
[EDIT]
See:
- http://drupal.org/coding-standards
- http://drupal.org/patch/create
Comment #8
mbehrman CreditAttribution: mbehrman commentedSorry - I hadn't realized eclipse had decided to tabify every function I touched. This should be cut down to the actual changes and in space, not tab format. Also note that the patch is off the 5.x-1.x-dev tree, not the head (contrary to the version in this thread).
Comment #9
sunLooks better. However:
- Patch is reverse; you might want to create a patch directly against the DRUPAL-5 branch instead of your potentially outdated working copy.
- You're testing module_exists() subsequently in img_assist_admin_settings() whereas one test would be sufficient.
- There are some commented out debugging? lines
- Isn't LEFT OUTER JOIN just the same as plain LEFT JOIN?
- Please remove all "mbehrman [date]" comments.
- The changes in img_assist_display() look like Image module compatibility has gone.
...and more, but I have to break now.
Comment #10
mbehrman CreditAttribution: mbehrman commentedOkay, I'll aim to make those changes and get it working on the current DRUPAL-5 branch. I made these mods for work (see chromix above) and they're sufficient for that purpose. But if I've got some down time I'll try to polish it up a bit more. Thanks for looking it over.
Comment #11
twooten CreditAttribution: twooten commentedsubscribing..
Comment #12
sun@mbehrman: Any updates? I would really like to review this patch :)
Comment #13
kirilius CreditAttribution: kirilius commentedI just found this thread. I a very interested in having an image browser that works with ImageField.
Is there any recent status on this? Is the patch included in v6?
Comment #14
pkej CreditAttribution: pkej commentedOuch. I'm trying to do this against 6.x head, I notice that the sql doesn't take into account table prefixes.
EDIT: It actually does, but it does queries even when tables doesn't exist. It is taking it for granted that some kind of table and data will be returned.
I'll digg deeper.
Comment #15
pkej CreditAttribution: pkej commentedI don't think this patch can work together with the views based browser, as it is now. I tried to find out how img_assist_thumbs works in 6.x head, and there are no similarities to the info in the patch in this thread.
Furthermore, the view img_assist_browser stopped working after I patched all other patchable functions based on the patch in #8.
Comment #16
pkej CreditAttribution: pkej commentedimg_assist_nodeapi is way different from what #8 patched against.
Comment #17
jferjan CreditAttribution: jferjan commentedsubscribing
Comment #18
pancake-1 CreditAttribution: pancake-1 commentedsubscribing
Comment #19
kirilius CreditAttribution: kirilius commentedIs there a chance that this patch makes its way to a release version of Image Assist in the near future?
Comment #20
sgriffin CreditAttribution: sgriffin commentedBump.
Imagefield and other field related modules seem to be the way to go.
For instance for stock photography websites, the main node will be an ubercart node with imagefields.
Lack of Image_assist is the only downside from using imagefield over image.
Comment #21
Kripsy CreditAttribution: Kripsy commentedI also think a patch to support imagefields is going to grow in demand over time. There is a huge deficit right now for an easy way to insert imagefield uploaded images into a text field.
Comment #22
scroogie CreditAttribution: scroogie commentedWe certainly need a way to insert images to cck content types, but I'm not sure if mixing in support for imagefield is the right thing to do. AFAIK the plan is that on the long run, Image will internally use imagefield or filefield, so img_assist would not need to worry about that.
Comment #23
pkej CreditAttribution: pkej commentedIn the prosepoint install profile there is a modified tinymce w/filefield support. Unfortunately it is tightly woven into a couple of other custom modules.
But, at least it works for image fields
Comment #24
sunSomeone should inform prosepoint then that TinyMCE module has been officially abandoned in favor of Wysiwyg API.
Re-assigning proper version. Now that we are using Views, the whole issue boils down to making the row style plugin more flexible, allowing the properties page to work with a fid (replacing the nid), and making the macro tag aware of the new image source, using different parameters (while generating and parsing of macro tags is deferred to Inline API).
Comment #25
Kripsy CreditAttribution: Kripsy commentedSounds very promising! I'll take a look at version 3.x tonight and see if it is within my capabilities.
Comment #26
pkej CreditAttribution: pkej commentedI have mailed the developer; I think it is yet another TinyMCE (javascript) wrapper for Drupal, not a rework of the TinyMCE (module).
Comment #27
scroogie CreditAttribution: scroogie commentedsun: so you want to go the fid route? What do you plan to do with the title and description of the image? To be honest I don't like the possible implicitations of multiple value image fields.
Comment #28
zoo33 CreditAttribution: zoo33 commentedThere is some interesting discussions around these issues in #208136: Imagecache/Image* support
See for example this rant and the following comments. I think there is agreement that Image Assist should continue to work with nodes as opposed to just image files. At least until there is some other rich entity/thingy/object in Drupal that can be used to represent an image.
Comment #29
Flying Drupalist CreditAttribution: Flying Drupalist commentedAny news on this?
If image assist works only with nodes then multi-image nodes would not be usable.
Comment #30
thamasI think an ideal tool for insert images inline to text should be able to
Comment #31
scroogie CreditAttribution: scroogie commentedI understand this all a lot better now.
"the whole issue boils down to making the row style plugin more flexible"
Do you want to use files as the base table? Both imagefields and image.modules images are stored in the files table, so I don't know if we should use the node table. It could only be a bit problematic to leave out the entries of the derivatives.
Comment #32
kirilius CreditAttribution: kirilius commentedI think in order to make this module really flexible, it should be based on views only.
Comment #33
Flying Drupalist CreditAttribution: Flying Drupalist commented@kirilius, it already is. What am I missing?
Comment #34
kirilius CreditAttribution: kirilius commentedI already is but it still depends on the actual type of image node that's behind it - Image vs Imagefield
Comment #35
sgriffin CreditAttribution: sgriffin commentedWhat is the status of this?
Is this project helpful in accomplishing imagefield support?
http://drupal.org/project/filefield_sources
Comment #36
GuillaumeDuveau@sgriffin We first need a WYSIWYG image tool that works with ImageField, then that tool could support FileField sources as well.
Wiki page summarizing the different projects on ImageField and WYSIWYG editors : http://groups.drupal.org/node/21639
Comment #37
Gabriel R. CreditAttribution: Gabriel R. commentedIMHO, an even more basic starting point would be to be able to use images attached to the the current node via the Upload module.
Comment #38
kirilius CreditAttribution: kirilius commentedAny status?
Comment #39
trofimchouk CreditAttribution: trofimchouk commentedhttp://drupal.org/project/imagefield_assist ?
Comment #40
kirilius CreditAttribution: kirilius commentedLancelot, ImageFIeld assist is far for complete and it does not support views yet.
Comment #41
trofimchouk CreditAttribution: trofimchouk commentedThere is a patch to use views in imagefield_assist
Comment #42
asb CreditAttribution: asb commentedsub
Comment #43
EgbertB CreditAttribution: EgbertB commentedsubscribing
Comment #44
Anonymous (not verified) CreditAttribution: Anonymous commentedSubscribing. Getting this done would hopefully also make it very easy to create a D7 version of Image Asisst.
Comment #45
udvranto CreditAttribution: udvranto commentedI would love to see the converted image to imagefield works with img_assist.imagefield!
Reference:
http://drupal.org/node/201983
Comment #46
unc0nnected CreditAttribution: unc0nnected commentedLord I would give my first born to see this implemented, it's one of the last pieces of my CMS and it all hinges on this.. Any dev patches for 6.0 work that bring this functionality in? Even in a limited way?
EDIT: For the most part the dev version of imagefield_assist does the trick.. It isn't quite as functional, there's no way to split it off into my images and all images and it doesn't seem to be working with views yet but at least I can attach images from outside the traditional image nodes..
Comment #47
naught101 CreditAttribution: naught101 commentedsubscribe
Is this likely to bee a won'tfix, considering that media module is planning to fulfill this role in d7?
Comment #48
alexbk66- CreditAttribution: alexbk66- commented+1
Comment #49
kirilius CreditAttribution: kirilius commentedHi, is there any update on this issue?
Comment #50
chromix CreditAttribution: chromix commentedhttp://drupal.org/project/imagefield_assist
That's probably the closest thing to an "update" you're going to get. It's essentially a fork of ImageAssist that allows for imagefield images.
Comment #51
naught101 CreditAttribution: naught101 commentedThis issue is fairly closely linked with: #513096: The Future of Image in Drupal 7
image.module, if it upgrades to 7.x, will probably end up using imagefield, so this module will need to be able to too, if it is to be upgraded to 7.x
Comment #52
Encarte CreditAttribution: Encarte commentedSubscribing