_enabled is declared on basecomponent - but it's doing some fancy footwork:

        <!--- used by the component to determine whether
              to display its disabled state.  A component is enabled if its
              "parent component" is enabled and its 'enabled' attribute
              evaluates to 'true' (its "parent component" is the first
              component above it in the view hierarchy)
              @keywords protected -->
        <attribute name="_enabled"
value="${this.enabled &amp;&amp; (this._parentcomponent ?
                            this._parentcomponent._enabled : true)}"
                   setter="this._setEnabled(_enabled)"/>


Henry Minsky wrote:
This change fixes the redeclaration of instance vars, and some other stuff, but there seems to be one bug, when I run smokecheck, I get a warning at runtime that _enabled is not
defined in

ERROR @base/basebutton.lzx#212: reference to undefined variable '_enabled'
ERROR @base/multistatebutton.lzx#32: reference to undefined variable '_enabled'


If you have any idea where that is coming from please tell me ... I'm looking into it


Change 20080329-hqm-o by [EMAIL PROTECTED] on 2008-03-29 14:33:23 EDT
    in /Users/hqm/openlaszlo/trunk4
    for http://svn.openlaszlo.org/openlaszlo/trunk

Summary: change to class compiler, declarations of some methods

New Features:

Bugs Fixed:

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

Documentation:

Release Notes:

Details:

+ add check to ClassCompiler so it doesn't redeclare vars that are declared in a superclass

+ make signatures match on all destroy() methods in component library

+ make some other methods signature match for checkbox


Tests:

smoke
grid example

test/swf9/checkbox, check that it no longer warns about "frame" instance var redeclaration in swf9

Files:
A      test/swf9/checkbox.lzx
M      WEB-INF/lps/schema/lfc.lzx
M      WEB-INF/lps/lfc/core/LzNode.lzs
M      WEB-INF/lps/lfc/views/LzText.lzs
M      WEB-INF/lps/lfc/controllers/LaszloLayout.lzs
M      WEB-INF/lps/server/src/org/openlaszlo/compiler/ClassCompiler.java
M      lps/components/lz/list.lzx
M      lps/components/utils/layouts/simplelayout.lzx
M      lps/components/base/basebutton.lzx
M      lps/components/base/basetabelement.lzx
M      lps/components/base/basescrollbar.lzx
M      lps/components/base/multistatebutton.lzx
M      lps/components/base/basetree.lzx
M      lps/components/base/basecomponent.lzx
M      lps/components/base/basewindow.lzx
M      lps/components/base/submit.lzx
M      lps/components/base/basetrackgroup.lzx
M      lps/components/base/baseformitem.lzx
M      lps/components/base/basetabs.lzx
M      lps/components/base/basedatacombobox.lzx
M      examples/components/grid_example.lzx

Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20080329-hqm-o.tar

--
Regards,
Max Carlson
OpenLaszlo.org

Reply via email to