placeholder is undefined on some ajax loaded modals. This patch just tests for existence before trying. Should probably be added to those pages properly, rather than this method, but this makes it not break things.

CommentFileSizeAuthor
#6 placeholder-1968930-6.patch651 bytessquidhaven
#1 placeholder-1968930#1.patch650 bytesdanlinn
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

danlinn’s picture

danlinn CreditAttribution: danlinn commented
FileSize
placeholder-1968930#1.patch650 bytes

and the patch...

danlinn’s picture

danlinn CreditAttribution: danlinn commented
Status: Active » Needs review
damiankloip’s picture

damiankloip CreditAttribution: damiankloip commented

That patch doesn't work, I think the # in the name is breaking it. What error are you getting?

SocialNicheGuru’s picture

SocialNicheGuru CreditAttribution: SocialNicheGuru commented

there is no patch attached

damiankloip’s picture

damiankloip CreditAttribution: damiankloip commented
Status: Needs review » Postponed (maintainer needs more info)
squidhaven’s picture

squidhaven CreditAttribution: squidhaven commented
FileSize
placeholder-1968930-6.patch651 bytes

There was a ")" missing in the if statement in the patch in #1, use this one instead.

damiankloip’s picture

damiankloip CreditAttribution: damiankloip commented
Status: Postponed (maintainer needs more info) » Needs review
bleen’s picture

bleen CreditAttribution: bleen commented
Issue summary: View changes
Status: Needs review » Fixed

  • bleen18 committed b440539 on 7.x-1.x authored by danlinn 
    Issue #1968930 by squidhaven, danlinn: Fixed Error in some ctools modals...

Status: Fixed » Closed (fixed)

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