You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@pivot.apache.org by "Greg Brown (JIRA)" <ji...@apache.org> on 2010/04/17 20:52:24 UTC

[jira] Created: (PIVOT-464) Add a BeanMap (or BeanAdapter) class

Add a BeanMap (or BeanAdapter) class
------------------------------------

                 Key: PIVOT-464
                 URL: https://issues.apache.org/jira/browse/PIVOT-464
             Project: Pivot
          Issue Type: Improvement
          Components: core-beans
            Reporter: Greg Brown
             Fix For: 1.5


This class will extend (or replace) BeanDictionary. It may be called BeanAdapter. It will be used wherever BeanDictionary is currently used, and will also be used in JSONSerializer#writeObject() to allow a caller to write arbitrary bean graphs to JSON. 

The existing BeanDictionaryListener interface will be eliminated as part of this change, and the functionality for listening to native events will be moved elsewhere (e.g. ComponentPropertyInspectorSkin). While valuable, the current implementation isn't consistent with how other similar adapter classes work, since change events are fired even when bean properties change "behind the scenes" - not just when they are modified via the adapter interface. The potential for memory leaks due to the current implementation is also undesirable. 


-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Updated: (PIVOT-464) Add a BeanMap (or BeanAdapter) class

Posted by "Greg Brown (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/PIVOT-464?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Greg Brown updated PIVOT-464:
-----------------------------


As a first iteration, BeanAdapter could extend BeanDictionary, and the platform code could be updated to use BeanAdapter instead. BeanDictionary would then be deprecated and dropped in a future release.


> Add a BeanMap (or BeanAdapter) class
> ------------------------------------
>
>                 Key: PIVOT-464
>                 URL: https://issues.apache.org/jira/browse/PIVOT-464
>             Project: Pivot
>          Issue Type: Improvement
>          Components: core-beans
>            Reporter: Greg Brown
>             Fix For: 1.5
>
>
> This class will extend (or replace) BeanDictionary. It may be called BeanAdapter. It will be used wherever BeanDictionary is currently used, and will also be used in JSONSerializer#writeObject() to allow a caller to write arbitrary bean graphs to JSON. 
> The existing BeanDictionaryListener interface will be eliminated as part of this change, and the functionality for listening to native events will be moved elsewhere (e.g. ComponentPropertyInspectorSkin). While valuable, the current implementation isn't consistent with how other similar adapter classes work, since change events are fired even when bean properties change "behind the scenes" - not just when they are modified via the adapter interface. The potential for memory leaks due to the current implementation is also undesirable. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Resolved: (PIVOT-464) Add a BeanMap (or BeanAdapter) class

Posted by "Greg Brown (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/PIVOT-464?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Greg Brown resolved PIVOT-464.
------------------------------

    Resolution: Fixed

BeanDictionary has been replaced by BeanAdapter. Bean monitoring code has been moved to org.apache.pivot.tools.wtk.BeanMonitor.


> Add a BeanMap (or BeanAdapter) class
> ------------------------------------
>
>                 Key: PIVOT-464
>                 URL: https://issues.apache.org/jira/browse/PIVOT-464
>             Project: Pivot
>          Issue Type: Improvement
>          Components: core-beans
>            Reporter: Greg Brown
>             Fix For: 1.5
>
>
> This class will extend (or replace) BeanDictionary. It may be called BeanAdapter. It will be used wherever BeanDictionary is currently used, and will also be used in JSONSerializer#writeObject() to allow a caller to write arbitrary bean graphs to JSON. 
> The existing BeanDictionaryListener interface will be eliminated as part of this change, and the functionality for listening to native events will be moved elsewhere (e.g. ComponentPropertyInspectorSkin). While valuable, the current implementation isn't consistent with how other similar adapter classes work, since change events are fired even when bean properties change "behind the scenes" - not just when they are modified via the adapter interface. The potential for memory leaks due to the current implementation is also undesirable. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.