0 Replies Latest reply on Jul 29, 2009 9:04 AM by Peter Farland

    [svn:fx-trunk] 8902: Ensuring that id information is set on components that have an appropriate property so that the information is available at runtime .

    Peter Farland Level 3

      Revision: 8902

      Author:   pfarland@adobe.com

      Date:     2009-07-29 09:04:36 -0700 (Wed, 29 Jul 2009)

       

      Log Message:

      ***********

      Ensuring that id information is set on components that have an appropriate property so that the information is available at runtime.

      See ARB issue:

      http://opensource.adobe.com/wiki/display/flexsdk/idpropertyinMXML2009

       

      QE notes: I confirmed that Advanced CSS tests still pass (to check that id selectors were still working). However, we should add mxmlunit tests that check that an id attribute on a component that has a writable, public, instance id property (and that isn't a UIComponent or GraphicElement, such as a TLF's component) is available at runtime. We should also test that when qualified with the language namespace that the id property is not set on the component.

       

      Doc notes: As part of our MXML 2009 documentation, we should mention that in order to get id information at runtime, a component should implement a public, writable, non-static property called "id".

       

      Bugs:

      SDK-21700 - MXML compiler isn't generating the right code to set the 'id' property on FlowElements.

       

      Reviewer: Paul

       

      Tests run: checkintest Pass, mustella AdvancedCSS tests pass.

       

      Is noteworthy for integration: Not really, this does help programmatic use of TLF nodes by id at runtime, but for Flex

      components they already had id information available at runtime.

       

      Ticket Links:

      ************

          http://bugs.adobe.com/jira/browse/SDK-21700

       

      Modified Paths:

      **************

          flex/sdk/trunk/modules/compiler/src/java/flex2/compiler/mxml/builder/ComponentBuilder.jav a