[ZCM] [ZC] 1463/ 9 Resolve "FSDump mishandles FSZSQL Method dump"

[Collector: Zope Bugs, Features, and Patches ...]

Issue #1463 Update (Resolve) "FSDump mishandles FSZSQL Method dump"
 Status Resolved, Zope/bug medium
To followup, visit:
  http://zope.org/Collectors/Zope/1463

==============================================================
= Resolve - Entry #9 by tseaver on Oct 13, 2004 2:25 pm

 Status: Accepted => Resolved

(Fixed) patch incorporated in the just-released FSDump-0.8.
________________________________________
= Comment - Entry #8 by cab on Oct 5, 2004 5:54 pm

Took my a while to find out why zope just didn't start after I applied this patch...

The 'arguments' line is missing a closing bracket.
________________________________________
= Comment - Entry #7 by tseaver on Sep 5, 2004 2:33 pm

AFAIK, this issue is only about the non-core FSDump product, and hence doesn't affect any Zope release.
________________________________________
= Comment - Entry #6 by ajung on Sep 5, 2004 12:07 am

@Tres: Should this included for 2.7.3?
________________________________________
= Comment - Entry #5 by elzubeir on Aug 13, 2004 3:51 am


Uploaded:  "final-dumper-patch"
 - http://zope.org/Collectors/Zope/1463/final-dumper-patch/view
Okay, I think this should deal with the different types of newlines.

________________________________________
= Accept - Entry #4 by tseaver on Aug 12, 2004 7:31 pm

 Status: Pending => Accepted

 Supporters added: tseaver

Thanks, I will look for the updated patch.
________________________________________
= Comment - Entry #3 by elzubeir on Aug 12, 2004 6:26 pm

One more thing I noticed about this. The arguments (obj.arguments_src) dumps it with \r\n so that won't show unless they are replaced with whitespace.

Doing a simple obj.arguments_src.replace('\r\n', ' ') did the trick for me, but I'm not sure how other browsers deal with newlines (firefox here).

Will do a little more investigation and update patch accordingly.
________________________________________
= Comment - Entry #2 by elzubeir on Aug 12, 2004 5:32 pm


Uploaded:  "Dumper.patch.gz"
 - http://zope.org/Collectors/Zope/1463/Dumper.patch.gz/view
Oops.. looks like I forgot to remove the _'s from some of the lines. Here is the 'right' patch.
________________________________________
= Request - Entry #1 by elzubeir on Aug 12, 2004 2:37 pm


Uploaded:  "Dumper.py.patch.gz"
 - http://zope.org/Collectors/Zope/1463/Dumper.py.patch.gz/view
FSDump incorrectly dumps ZSQL methods.

1. The extension is wrong (should be .zsql)
2. There are no .metadata files for zsql methods, they are kept in the same file under a dtml-comment.

The attached patch should solve this problem.
==============================================================