Bug #17570

onContentLoad() is called twice on every page

Added by Tomer Brisker 11 months ago. Updated 10 months ago.

Status:Closed
Priority:Normal
Assigned To:Tomer Brisker
Category:Web Interface
Target version:Team Daniel - iteration 6
Difficulty: Bugzilla link:
Found in release: Pull request:https://github.com/theforeman/foreman/pull/4080
Story points-
Velocity based estimate-
Release1.14.1Release relationshipAuto

Description

From the jQuery docs:

when calling .trigger() with an event name matches the name of a property on the object, prefixed by on (e.g. triggering click on window that has a non null onclick method), jQuery will attempt to invoke that property as a method.

When the document loads we trigger a `ContentLoad` event, which causes jQuery to execute the function called `onContentLoad` by default. In addition, we attach a listener to the document that catches the `ContentLoad` event and executes the function as well.
We need to either rename the function or remove the extra listener.

Associated revisions

Revision 83afcbec
Added by Tomer Brisker 10 months ago

Fixes #17570 - Only execute `onContentLoad` once per page

When an event is triggered, jQuery attempts to execute the on<name>
method if it exists for event <name>. We were also manually triggering
the `onContentLoad` every time the `ContentLoad` event was triggered,
leading to the function executing twice on every page.
This commit removes the explicit event listener so that the function
will only execute once.

Revision 78ab8401
Added by Tomer Brisker 9 months ago

Fixes #17570 - Only execute `onContentLoad` once per page

When an event is triggered, jQuery attempts to execute the on<name>
method if it exists for event <name>. We were also manually triggering
the `onContentLoad` every time the `ContentLoad` event was triggered,
leading to the function executing twice on every page.
This commit removes the explicit event listener so that the function
will only execute once.

(cherry picked from commit 83afcbecbdaac702253f494ff07351a74308b6be)

History

#1 Updated by The Foreman Bot 11 months ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/4080 added

#2 Updated by Daniel Lobato Garcia 10 months ago

  • Target version set to Team Daniel - iteration 6

#3 Updated by Anonymous 10 months ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#4 Updated by Dominic Cleal 10 months ago

  • Release set to 1.14.1

Also available in: Atom PDF