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!

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Alan D.’s picture

Title: Support for imagefiled_crop would be awesome! » Imagefield Extended / ImageField Crop integration
Project: ImageField Extended » Imagefield Crop
Version: 6.x-4.x-dev » 6.x-1.x-dev

At 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?

EugenMayer’s picture

I 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.

ikeizer’s picture

Would be interested in this as well...any clue if it can be accomplished?

Nick Lewis’s picture

Status: Active » Closed (duplicate)

The 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

waono’s picture

Version: 6.x-1.x-dev » 7.x-2.x-dev
Category: feature » support
Status: Closed (duplicate) » Active

Hi -

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

Alan D.’s picture

Title: Imagefield Extended / ImageField Crop integration » ImageField Crop integration

Imagefield 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.

davidwhthomas’s picture

Subscribing.

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

Alan D.’s picture

"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

davidwhthomas’s picture

Nice 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!

davidwhthomas’s picture

Status: Active » Needs review
FileSize
424 bytes

OK, 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

$output .= drupal_render_children($element);

inside

$output .= '<div class="imagefield-crop-widget form-managed-file clearfix">';

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

davidwhthomas’s picture

Status: Needs review » Needs work

Sorry, 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.

davidwhthomas’s picture

OK, 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

davidwhthomas’s picture

Status: Needs work » Needs review
Alan D.’s picture

Title: ImageField Crop integration » ImageField Crop / Insert integration

This issue appears to be highjacked, changing title to reflect changes from #9.

Original ticket is a duplicate of #792904: Integrate with imagefield crop module :)