Importing the node type using Content Copy as documented in README.txt can be much confusing.
It would be great to implement this CCK-based nodetype programatically by default.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | imagefield_assist.patch | 5.71 KB | karens |
Importing the node type using Content Copy as documented in README.txt can be much confusing.
It would be great to implement this CCK-based nodetype programatically by default.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | imagefield_assist.patch | 5.71 KB | karens |
Comments
Comment #1
karens commentedHere's a patch to do this.
Comment #2
karens commentedOops, meant to mark it 'Needs Review', but it should work fine.
Comment #3
karens commentedAdd one more line to the bottom of the install to be sure that the content and node type information is all flushed out so it will pick up this new type immediately:
Comment #4
lourenzo commentedAwesome!
Thank you
I have one question: fivestar node settings are there by accident?
Comment #5
karens commentedThey are there because it was in the code I exported from. They will be ignored if the five star module is not enabled. You can also remove them if you want.
I also added a max_resolution to the image field to shrink huge files down to some maximum size. Administrators can alter that value in the field settings but I think it is better to have some sort of default than to accept the huge image files many end users will inadvertently add from their photo galleries.
Comment #6
franzCommited on dev. =)
Comment #7
lourenzo commentedSweet.
That's good news!
Comment #9
Scott J commentedI just installed 6.x-1.x-dev but it seems that I still need to do this manually.
Comment #10
pkej commentedI really don't like it when modules create their own nodetypes, especially for a module like this which isn't an image module, but an image inserter. It should be an additional sub-module which can be enabled by the user if the user wants this functionality.
Comment #11
lourenzo commentedHi pkej,
We're aiming to offer a works-out-of-the-box module, and for now we'd need to create a node type that would be the images' container. Although, we could detect if any node types have imagefields, and offer options on install so the user could set an existing node type as default repository for uploads.
Comment #12
pkej commentedExactly my thoughts as well In case I was unclear.
A simple method of creating such a node type would be to have a sub module imagefield_assist_node_type inside the imagefield_assist module. The imagefield_assist_node_type module would depend on imagefield_assist module being installed and enabled.
Both modules would be in the same download, so it would work out of the box for both use-cases.
Comment #13
Scott J commentedOf course what we really should have is version 6.3.x of the Image module (http://drupal.org/project/image), updated to use imagecache, then we wouldn't have this dilemma; all other modules such as this one, could just depend on the new image module.
Much like img_assist, the current maintainers of Image don't seem interested before Drupal 7, so either we ask Image module about maintaining a new version, or as pkej suggests, start a new image node module. I wouldn't make it dependant on imagefield_assist though, so that it can be used as a new image standard upon which to update gallery module etc.
ps.
I just found another module with the same issue at #232373: Content type import doesn't work and other installation issues.
Comment #14
pkej commentedDrupal 7 is months and months away, I believe, some of the problems with several modules, like img_assist and image, has been there since the dawn of the 6.x series.
In 5.x I used Image and img_assist modules, but this module is the coolest since sliced bread. I'm just going to ask for some features :P
Who is responsible for marking as duplicate?
Comment #15
jeffschulerI spawned #976952: Allow choice of upload content type from lourenzo's comment in #11, as it's a separate issue that deserves its own attention.
Comment #16
Scott J commentedIn case anyone is still looking at this, there is now a Debut Image module that just creates a new content type with an imagefield already set up ready to go.