We have Drupal.behaviors.diffRevisions but no once() to guard multiple executions, i guess we need it?

CommentFileSizeAuthor
#4 use-once-2822523-4.patch857 bytespivica
#2 use-once-2822523-1.patch869 bytespivica
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pivica created an issue. See original summary.

pivica’s picture

pivica CreditAttribution: pivica at MD Systems GmbH commented
FileSize
use-once-2822523-1.patch869 bytes

Here is a patch.

miro_dietiker’s picture

miro_dietiker
Location Switzerland
CreditAttribution: miro_dietiker at MD Systems GmbH commented
Status: Active » Needs work

Needs a reroll after #2822007: Improve frontend code

pivica’s picture

pivica CreditAttribution: pivica at MD Systems GmbH commented
FileSize
use-once-2822523-4.patch857 bytes

Here it is.

miro_dietiker’s picture

miro_dietiker
Location Switzerland
CreditAttribution: miro_dietiker at MD Systems GmbH commented
Status: Needs work » Needs review

miro_dietiker’s picture

miro_dietiker
Location Switzerland
CreditAttribution: miro_dietiker at MD Systems GmbH commented
Status: Needs review » Fixed

Yeah, thx for finding this.

Status: Fixed » Needs work

The last submitted patch, 4: use-once-2822523-4.patch, failed testing.

miro_dietiker’s picture

miro_dietiker
Location Switzerland
CreditAttribution: miro_dietiker at MD Systems GmbH commented
Status: Needs work » Fixed

Unrelated fails.

The last submitted patch, 2: use-once-2822523-1.patch, failed testing.

Status: Fixed » Closed (fixed)

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