Bug 257260

Summary: Finance::Quote price detection does not work because financequote.pl is being run with diagnostics on
Product: [Applications] kmymoney Reporter: Cristian Oneț <onet.cristian>
Component: generalAssignee: KMyMoney Devel Mailing List <kmymoney-devel>
Status: RESOLVED FIXED    
Severity: normal    
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Unlisted Binaries   
OS: Linux   
Latest Commit: Version Fixed In:

Description Cristian Oneț 2010-11-18 18:19:20 UTC
Version:           unspecified
OS:                Linux

Printing diagnostic messages on the output messes up price detection.

See:

Executing /usr/share/kde4/apps/kmymoney/misc/financequote.pl za SLM...
Symbol found: SLM
Price found: 201011.17 (201,011)
Price for SLM updated (id E000008)

When this message is printed:
/usr/lib64/perl5/vendor_perl/5.12.2/Finance/Quote/ZA.pm line 148 (#1)
    (W uninitialized) An undefined value was used as if it were already
    defined.  It was interpreted as a "" or a 0, but maybe it was a mistake.
    To suppress this warning assign a defined value to your variables.
    
    To help you figure out what was undefined, perl will try to tell you the
    name of the variable (if any) that was undefined. In some cases it cannot
    do this, so it also tells you what operation you used the undefined value
    in.  Note, however, that perl optimizes your program and the operation
    displayed in the warning may not necessarily appear literally in your
    program.  For example, "that $foo" is usually optimized into "that "
    . $foo, and the warning will refer to the concatenation (.) operator,
    even though there is no . in your program.


Reproducible: Always
Comment 1 Cristian Oneț 2010-11-18 18:21:14 UTC
SVN commit 1198499 by conet:

BUG: 257260
Don't show diagnostics from financequote.pl.

 M  +1 -1      financequote.pl  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1198499
Comment 2 Cristian Oneț 2010-11-18 18:22:20 UTC
SVN commit 1198500 by conet:

BUG: 257260
Don't show diagnostics from financequote.pl.
Merged into the stable branch.

 M  +1 -1      financequote.pl  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1198500