Changes between Initial Version and Version 1 of Ticket #2108


Ignore:
Timestamp:
Jun 24, 2019, 2:24:39 PM (6 years ago)
Author:
Tom Goddard
Comment:

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #2108

    • Property Component Unassigned → Sessions
    • Property Owner set to Greg Couch
    • Property Platform → all
    • Property Project → ChimeraX
    • Property Status new → assigned
    • Property Summary ChimeraX bug report submission → Session adds non-statemanager to list of state managers

  • Ticket #2108 – Description

    initial v1  
    66The following traceback was created by my adding code that raised an exception.  It is not an error.  But it reveals that Session.__setattr__() is adding ses._last_label_view which is a Place instance to the list of state managers.  "Place" is not a StateManager and also this attribute is private (leading underscore) so it should not be saved in sessions.
    77
    8 This is the cause of bug #2106 where Place.reset_state() is called on an immutable Place instance.
     8This is the cause of bug #2106 where Place.reset_state() is called on a Place instance.
    99
    1010Probably Session.__setattr__() should only recognize StateManager values that are not private.