Comments on: prototype 1.5.0 patch for Safari http://rajeev.name/2007/03/18/prototype-150-patch-for-safari/ technologist defying the peter principle Fri, 25 Jun 2010 12:53:52 +0000 hourly 1 http://wordpress.org/?v=3.0 By: Joannou Ng http://rajeev.name/2007/03/18/prototype-150-patch-for-safari/comment-page-1/#comment-820 Joannou Ng Sat, 16 Jun 2007 03:04:25 +0000 http://rajeev.name/2007/03/18/prototype-150-patch-for-safari/#comment-820 For clarity, the for...in statement enumerates the properties (including methods) of the object. However, core methods of core objects (like Array) are not enumerated. This means that enumerating an unextended core object is fine, but enumerating an extended core object requires extra checks to ignore methods that extend the core object. In this particular case, I ran into this issue because I was using the json.js provided by json.org, which ended up extending the request headers. Not using the redundant json.js solved the issue for me. Cheers, Joannou. For clarity, the for…in statement enumerates the properties (including methods) of the object. However, core methods of core objects (like Array) are not enumerated. This means that enumerating an unextended core object is fine, but enumerating an extended core object requires extra checks to ignore methods that extend the core object.

In this particular case, I ran into this issue because I was using the json.js provided by json.org, which ended up extending the request headers. Not using the redundant json.js solved the issue for me.

Cheers, Joannou.

]]> By: rajeev karamchedu http://rajeev.name/2007/03/18/prototype-150-patch-for-safari/comment-page-1/#comment-808 rajeev karamchedu Fri, 15 Jun 2007 20:49:29 +0000 http://rajeev.name/2007/03/18/prototype-150-patch-for-safari/#comment-808 Joannou The patch was not mine to submit but I did alert the guys over there. Not sure if they have accepted it or not. Since then, I saw much traffic on that issue and resolution and trust that it has been taken care of. Joannou

The patch was not mine to submit but I did alert the guys over there. Not sure if they have accepted it or not. Since then, I saw much traffic on that issue and resolution and trust that it has been taken care of.

]]> By: Joannou Ng http://rajeev.name/2007/03/18/prototype-150-patch-for-safari/comment-page-1/#comment-807 Joannou Ng Fri, 15 Jun 2007 18:25:17 +0000 http://rajeev.name/2007/03/18/prototype-150-patch-for-safari/#comment-807 Did you submit the patch? Did you submit the patch?

]]>