Bug #7795
NTP status widget: d.getSeconds is not a function
100%
Description
Already mentioned in #7784 but creating a separate bug for this.
When you select either the tunnels or the mobile tab on the IPsec widget, the browser console immediately gets spammed with Uncaught TypeError: d.getSeconds is not a function
- error logged every second. Traced it to the NTP widget [1], apparently there's some weird interaction between the two widgets.
P.S. Same problem with 2.3.4-p1.
Associated revisions
fixed #7795
By changing var name to no longer conflict with ipsec widget
(cherry picked from commit 3ed475b11dfcb2999a71195987495ccf05808127)
fixed #7795
By changing var name to no longer conflict with ipsec widget
(cherry picked from commit 3ed475b11dfcb2999a71195987495ccf05808127)
(cherry picked from commit bc2772c18ef2aa9a32b941fcf79c0c160fad7666)
fixed #7795
By changing var name to no longer conflict with ipsec widget
(cherry picked from commit 3ed475b11dfcb2999a71195987495ccf05808127)
(cherry picked from commit bc2772c18ef2aa9a32b941fcf79c0c160fad7666)
History
#1
Updated by Steve Beaver over 2 years ago
- Assignee set to Steve Beaver
Looks like both widgets have a global var called "d". I bet they are stepping on each other. Would have been a good idea to use a widget-name-based prefix with all globals, like ntp_d, ipsec_d etc.
Renaming the var in the NTP widget seems to have fixed the issue.
#2
Updated by Steve Beaver over 2 years ago
- Status changed from New to Feedback
- % Done changed from 0 to 100
Applied in changeset 3ed475b11dfcb2999a71195987495ccf05808127.
#3
Updated by Steve Beaver over 2 years ago
- % Done changed from 100 to 0
#4
Updated by Steve Beaver over 2 years ago
- % Done changed from 0 to 100
Applied in changeset da058fe4c156e607227f1bac03a3ade42c475115.
#5
Updated by Kill Bill over 2 years ago
Steve Beaver wrote:
Looks like both widgets have a global var called "d". I bet they are stepping on each other.
Good catch. The fix works, thanks.
#6
Updated by Steve Beaver over 2 years ago
- Status changed from Feedback to Resolved
#7
Updated by Jim Pingle about 2 years ago
- Target version changed from 2.4.1 to 2.4.0
fixed #7795
By changing var name to no longer conflict with ipsec widget