With the latest version of jQuery allowed by jquery_update module (1.10), the version of jcrop in this module throws 'Uncaught type' error on $.browser.msie check, because $browser was removed from jQuery in v.1.9. Updating jcrop to the latest version, v0.9.12 resolves the issue for me.

Suggestions:

  • Make jquery_update a dependency for the module?
  • Take jcrop out of the module altogether and require users to install it on their own, e.g. in sites/all/libraries?
CommentFileSizeAuthor
#3 imagefield_crop-2694499-1.patch111.05 KBprat
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

rohnjeynolds created an issue. See original summary.

ram4nd’s picture

ram4nd CreditAttribution: ram4nd as a volunteer commented
Issue tags: -jQuery Version

Before adding tags read the issue tag guidelines. Do NOT use tags for adding random keywords or duplicating any other fields. Separate terms with a comma, not a space.

prat’s picture

prat CreditAttribution: prat commented
FileSize
imagefield_crop-2694499-1.patch111.05 KB

I would like to propose a patch against 7.x-1.1 (which seems to be the only completely working solution) except JCrop issue

CatherineOmega’s picture

CatherineOmega CreditAttribution: CatherineOmega commented

Seconding this. Updating it worked for me.

ram4nd’s picture

ram4nd CreditAttribution: ram4nd as a volunteer commented

Needs to be refactored against dev branch. Since there is no "7.x-1.1" branch, it's a tag. If there are any issues they should be patched.

ram4nd’s picture

ram4nd CreditAttribution: ram4nd as a volunteer commented
Version: 7.x-1.1 » 7.x-1.x-dev
Status: Active » Needs work
PimD1988’s picture

PimD1988 CreditAttribution: PimD1988 commented

Another small fix is recommended:

In imagefield_crop.module there is still a reference to the old jquery.Jcrop.js file which is deleted on applying this patch.
It should be changed to 

  $element['#attached']['js'][] = "$path/Jcrop/js/jquery.Jcrop.min.js";

see line 201.

Hope this helps!