You seem to not copy the 'Changeset:' line into your mail. I was able to reconstruct it, but it is easier if it is there to click on. :)

I think you just want:

   if ( delegate instanceof LzDelegate ) {

not 'delegate.prototype'. Aren't you just testing that delegate is an LzDelegate?

Otherwise approved!

On 2007-05-30, at 14:42 EDT, Henry Minsky wrote:

Change 20070530-hqm-a by [EMAIL PROTECTED] on 2007-05-30 14:38:46 EDT
   in /cygdrive/c/users/hqm/openlaszlo/legals3
   for http://svn.openlaszlo.org/openlaszlo/branches/legals

Summary: fix for javarpc DHTML

New Features:

Bugs Fixed: LPP-4044

Technical Reviewer: ptw (pending)
QA Reviewer: (pending)
Doc Reviewer: (pending)

Documentation:

Release Notes:

Details:

needed to find an acceptable way to ask for instanceof, don't use __proto__
in IE7 (works in FF though!)



Tests:

example from docs
http://localhost:8080/lps4/docs/guide/programs/rpc-javarpc-$2.lzx? lzr=dhtml
checked in as test/rpc/javarpc/lpp-4044.lzx

Files:

M      lps/components/rpc/library/dhtml/javarpc.js
A      test/rpc/javarpc/lpp-4044.lzx

--
Henry Minsky
Software Architect
[EMAIL PROTECTED]

Reply via email to