I recently made use of a button with #type image_button and the ajax module didn't recognize it as a submit button.

Attached is a patch to fix this.

CommentFileSizeAuthor
#5 ajax_type=image.patch425 bytesMaxkupr
#4 ajax_type=image.patch418 bytesMaxkupr
ajax_image_button_fix.patch559 bytesvoxpelli
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

iamjon’s picture

iamjon CreditAttribution: iamjon commented

Hi,
I tried your patch it didn't work.
Can you post your your form api that you used to add the image. Maybe I didn't override the button properly.

iamjon’s picture

iamjon CreditAttribution: iamjon commented

I lied it works.

voxpelli’s picture

voxpelli CreditAttribution: voxpelli commented
Status: Needs review » Reviewed & tested by the community

If it works - then we can perhaps set this as reviewed by the community? It's such a simple patch that it shouldn't need a massive review

Maxkupr’s picture

Maxkupr CreditAttribution: Maxkupr commented
Status: Reviewed & tested by the community » Needs review
FileSize
ajax_type=image.patch418 bytes

what about adding handler in ajax.js?

Maxkupr’s picture

Maxkupr CreditAttribution: Maxkupr commented
FileSize
ajax_type=image.patch425 bytes

sorry, just discard previous patch. here is correct one

brendoncrawford’s picture

brendoncrawford CreditAttribution: brendoncrawford commented
Status: Needs review » Closed (fixed)

Maximus, patch has been applied. Thanks for your help. Should be available in dev release.

brendoncrawford’s picture

brendoncrawford CreditAttribution: brendoncrawford commented
Status: Closed (fixed) » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.