Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
It would be great if those fields could also work with imagefield_crop. Iam not 100% sure, but this implementation should be rather trivial or?
Thanks!
Comment | File | Size | Author |
---|---|---|---|
#12 | 749272-insert-support-for-imagefield-crop-v2.patch | 732 bytes | davidwhthomas |
#10 | 749272-insert-support-for-imagefield-crop.patch | 424 bytes | davidwhthomas |
Comments
Comment #1
Alan D. CreditAttribution: Alan D. commentedAt the first glance, the IF Crop element is fairly customized. Moving to that queue to see if it is possible.
The IF Extended module simply works by adding more 'data' sub-elements that get handled by FileField. I think that if IF Crop themed it's custom elements first using drupal_render(), it could then add drupal_render($e['data']) to render the other values?
Comment #2
EugenMayer CreditAttribution: EugenMayer commentedI will look into it Alan. I fairly new to the CCK widget API ... but iam worked in in the FAPI topic not too bad.
I will help and contribute if possible somehow.
Comment #3
ikeizer CreditAttribution: ikeizer commentedWould be interested in this as well...any clue if it can be accomplished?
Comment #4
Nick Lewis CreditAttribution: Nick Lewis commentedThe imagefield_extended_module needs to declare the process callback against imagefield_crop_widget, not just imagefield_widget etc...
working patch here: http://drupal.org/node/792904
Comment #5
waono CreditAttribution: waono commentedHi -
Can this module be implemented and used for unique user accounts? We're building a site where users are suppose to be able to crop their profile photos during and after registration. However, I heard from developers that this module can only be implemented to crop admin generated content.
If this can be verified, we would greatly appreciate it.
Thank you,
Wayne
Comment #6
Alan D. CreditAttribution: Alan D. commentedImagefield Extended is not supported on Drupal 7. (Albeit we need this for a D7 project and neither File entities nor Field Collections can be used...)
Leaving open in case you are referring to another integration issue.
Comment #7
davidwhthomas CreditAttribution: davidwhthomas commentedSubscribing.
Any idea how this can be done for the D7 version?
As imagefield_extended is D6 only, does some functionality from that module need porting across too in order to modify the element correctly in D7?
DT
Comment #8
Alan D. CreditAttribution: Alan D. commented"does some functionality from that module need porting across too in order to modify the element correctly"
Not sure what this refers to, but I am currently working on File Attributes module after ruling out File Entities and Field Collections on a project we are working on required additional fields on images. No upgrade path yet, but the D6 to D7 upgrade path does not delete the column that ImageField saved it's data too (well thats what I read - yet to investigate this)
ETA of around a week before a semi-stable alpha release. Massive task, it effectively wraps the webform functionality onto the file or image field and based on the concept of assigning profiles that are selectable from the front end, so not only can you add a group of components, an ajaxed field allows you to choice what profile is relevant.
But off topic: If ImageField Crop does not work, try Manual Crop (http://drupal.org/project/manual-crop). Both have different UIs / workflows to resolve the same issue. Maybe both authors should cross-link to each others projects to assist users finding the best solution for their particular site.
[edit] We use both modules, currently favoring MC on complex sites, but IFC for smaller sites
Comment #9
davidwhthomas CreditAttribution: davidwhthomas commentedNice to see an update to this.
I've actually just got insert working with imagefield_crop using a standard D7 install and small patch to implement the insert_widgets hook via the insert module.
It looks like the integration can probably be done from the Insert module side rather than here, though it might be useful to be able to adjust imagefield_crop's themed markup to better work with insert, if needed.
My current patch needs some final tuning to insert.js to work with imagefield_crop's current markup, if possible as is - mainly the expected html wrapper and child elements. I'll will post an update when complete, possibly tomorrow.
cheers,
DT
P.S Filefield Attributes looks great!
Comment #10
davidwhthomas CreditAttribution: davidwhthomas commentedOK, here's the patch required to imagefield_crop to get it to work with Insert.
It's really trivial patch, the insert module needs everything to be inside .imagefield-crop-widget wrapper on the widget form.
The patch simply moves
inside
inside the theme function
theme_imagefield_crop_widget
It can also be done by overriding the theme function, but would be good if that wasn't necessary.
thanks for the useful module!
DT
Comment #11
davidwhthomas CreditAttribution: davidwhthomas commentedSorry, this still needs work, moving the element children inside the imagefield-crop-widget div has affected saving the crop settings.
Will post an update soon.
Comment #12
davidwhthomas CreditAttribution: davidwhthomas commentedOK, here's the adjusted version that works.
It only renders the insert elements inside the wrapper, if they're present.
It would be nice if could render all the element_children inside the div.imagefield-crop-widget wrapper but for some reason that approach causes the stored cropbox to not be applied.
cheers,
DT
Comment #13
davidwhthomas CreditAttribution: davidwhthomas commentedComment #14
Alan D. CreditAttribution: Alan D. commentedThis issue appears to be highjacked, changing title to reflect changes from #9.
Original ticket is a duplicate of #792904: Integrate with imagefield crop module :)