[morph-developer] [ morph-Bugs-1373284 ] Incorrect caching implementation

[SourceForge.net <no...@so...>]

Bugs item #1373284, was opened at 2005-12-05 00:28
Message generated for change (Comment added) made by sgarlatm
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=700930&aid=1373284&group_id=124910

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
>Resolution: Fixed
Priority: 5
Submitted By: Eduard Letifov (techtime)
Assigned to: Nobody/Anonymous (nobody)
Summary: Incorrect caching implementation

Initial Comment:
Hello,

 I am using Morph 0.9.1

DelegatingTransformer caching of visited sources seems 
to be completely wrong as it doesn't take into account 
destination. If one tries to convert the same source 
object into 2 different destination objects in a row - 
the first destination object (cached) is always 
returned.

In my view the caching should take into account 
destination class.

Attached is a test case and a rudely patched version 
of DelegatingTransformer, that takes care of this bug, 
bug 1344515 and suspicios caching in copy procedure 
(checking if source is in cache/visited but never 
checking if a copy is being attempted into the same 
instance of an object)


----------------------------------------------------------------------

>Comment By: Matt Sgarlata (sgarlatm)
Date: 2005-12-06 11:29

Message:
Logged In: YES 
user_id=1161666

Fixed in Morph 0.9.2 (release is already prepared, and will
be made available soon)

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=700930&aid=1373284&group_id=124910