Ticket #2532329 (closed defect)

Reporter


reese.kensington
Opened: 05/17/12
Last modified: 08/1/12
Status: closed
Type: defect
Resolution: fixed

Owner


Luke Smith
Target Release: 3.6.0
Priority: P3 (normal)
Summary: Focus event throws JavaScript error
Description:

I have a TinyMCE instance in a panel. When I click in the edit area I get the following alert:



Webpage error details

User Agent: Mozilla/4.0 (compatible; MSIE 7.0; Windows NT 6.1; WOW64; Trident/5.0; SLCC2; .NET CLR 2.0.50727; .NET CLR 3.5.30729; .NET CLR 3.0.30729; Media Center PC 6.0; .NET4.0C; .NET4.0E;
InfoPath.3; BOIE9;ENUS)
Timestamp: Thu, 17 May 2012 12:47:22 UTC

Message: Unable to get value of the property 'ms__id47520': object is null or undefined
Line: 119
Char: 21
Code: 0
URI: http://localhost/glw/common/js/yui3/event-focus/event-focus-debug.js


Looking at the code, it seems to recognize on line 96 that the variable notifierData may be undefined, but there is no test for the same on line 119. Not sure what all this does, and my code is
too complex to post.

Type: defect Observed in Version: 3.5.1
Component: Event Severity: S3 (normal)
Assigned To: Luke Smith Target Release: 3.6.0
Location: Library Code Priority: P3 (normal)
Tags: Relates To:
Browsers: IE 9.x
URL:
Test Information:

Change History

Luke Smith

YUI Contributor

Posted: 05/24/12
  • location changed to Library Code
  • milestone changed to 3.6.0
  • priority changed to P3 (normal)
  • status changed from new to checkedin

Thanks for catching this

Luke Smith

YUI Contributor

Posted: 05/24/12
  • completed changed from 0 to .1
  • estimated changed from 0 to 0.1
  • sprint changed to backlog

Luke Smith

YUI Contributor

Posted: 05/24/12
  • resolution changed to fixed

Fix NPE in focus handler. Fixes #2532329

notifierData was being checked in one place, but not another.
View Commit: 9a83a84b7c5baff17c718744babf829448f8f39b

Jenny Donnelly

YUI Developer

Posted: 08/1/12
  • status changed from checkedin to closed

Shipped in 3.6.0. Marking closed/fixed.