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

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

Bugs item #1373284, was opened at 2005-12-05 08:28
Message generated for change (Tracker Item Submitted) made by Item Submitter
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: None
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)


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

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