Consistently use Prototype.emptyFunction in unit tests
Reported by Michael M Slusarz | July 29th, 2008 @ 11:51 PM | in 1.6.1
Just a small patch which consistently uses Prototype.emptyFunction instead of "function() { }" in the events test file.
Comments and changes to this ticket
-
John-David Dalton July 30th, 2008 @ 06:14 AM
Can you post one where you just set
observer = Prototype.emptyFunction;I think using the variable name "observer" helps keep people reading the code in the right frame of mind.
-
John-David Dalton July 30th, 2008 @ 06:14 AM
- → State changed from new to enhancement
-

Michael M Slusarz July 31st, 2008 @ 04:55 AM
- → Title changed from Minor cleanups in event test file to Consistently use Prototype.emptyFunction in test files
Here's a cleaned up patch, which also uses emptyFunction in several other test files also.
-

Michael M Slusarz July 31st, 2008 @ 04:57 AM
- no changes were found...
-
John-David Dalton July 31st, 2008 @ 05:44 AM
- → Tag changed from test to cleanup
- → Milestone changed from to 1.6.1
- → Title changed from Consistently use Prototype.emptyFunction in test files to Consistently use Prototype.emptyFunction in unit tests
- → Assigned user changed from to Andrew Dupont
Thanks :)
Please Login or create a free account to add a new comment.
You can update this ticket by sending an email to from your email client. (help)
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile »
The Prototype JavaScript library.
