[kmymoney4] [Bug 360129] CSV Importer doesn't recognize sell operation in Polish

[NSLW via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=360129

--- Comment #16 from NSLW <lukasz.wojnilowicz at gmail.com> ---
(In reply to allan from comment #15)
> > > I know no Polish financial terms.  If the Buy list contains a correct Polish
> > > term, then that I presume was added by the translator.  A possible
> > > explanation is that at line 161 (in my version) of investprocessing.cpp the
> > > following appears - "m_buyList += i18nc("verb", "buy");  //                 
> > > some basic entries in case rc file missing".  It may be that that was when
> > > the translation for Buy was added.  However, the next line is similar for
> > > the Sell operation, and that was not translated apparently.
> > 
> > All terms are and were translated.
> 
> In the rc file snippet you attached, only Buy and Reinvdiv have content.

That's right and that also shows that KMM has bug inside.
Please see translation progress of KMM to Polish to know that it is translated
fully 
http://l10n.kde.org/stats/gui/trunk-kf5/team/pl/extragear-office/

Please read also what I wrote you about the bug I'm seeing
> > After the patch my csvimporterrc is
> > filled out automatically with all the data whereas before the patch only
> > BuyParam and ReinvdivParam were filled out.

And finally please see into code of investprocessing.cpp

BuyParam and  ReinvdivParam are assigned like this
>>>>>>>>>>>>>>>>>>>>>>>>
    QStringList list = profilesGroup.readEntry("BuyParam", QStringList());
    if (!list.isEmpty()) {
      m_buyList = list;
    }
>>>>>>>>>>>>>>>>>>>>>>>>

whereas other parameters and it includes SellParam is assigned like this

>>>>>>>>>>>>>>>>>>>>>>>>
m_sellList = profilesGroup.readEntry("SellParam", QStringList());
>>>>>>>>>>>>>>>>>>>>>>>>

and that's the reason why you don't see SellParam in rc file and not the
missing translation.

-- 
You are receiving this mail because:
You are the assignee for the bug.