question on finding account id from name

[Thomas Baumgart]

On Mittwoch, 22. Juni 2022 16:47:24 CEST Dawid Wrobel via KMyMoney-devel wrote:

> Jack,
> 
> I believe that originally the 'kmm-brokerage-account' property was supposed
> to be used to look-up the linked-up brokerage account id.
> This, however, does not seem to work, see:
> https://bugs.kde.org/show_bug.cgi?id=329701
> and https://bugs.kde.org/show_bug.cgi?id=350360

This may come from a very old version and is not used anymore. Nowadays,
the brokerage name is tied by the (localized) suffix " (Brokerage)" to
the investment account. May not be ideal either but that is the current 
way. That property is only used in the statement reader and that could
be for backward compatibility reasons.

Thomas

> 
> On Wed, Jun 22, 2022 at 4:13 PM Jack via KMyMoney-devel <
> kmymoney-devel at kde.org> wrote:
> 
> > On 6/22/22 02:18, Thomas Baumgart via KMyMoney-devel wrote:
> > > On Dienstag, 21. Juni 2022 22:46:35 CEST Jack via KMyMoney-devel wrote:
> > >> I'm trying to find an way to easily switch between an investment
> > >> account and it's brokerage account.  I have figured out how to test for
> > >> the two types of accounts (testing the .accountType() or looking for "
> > >> (Brokerage)" at the end of the name.)  However, I don't see any way to
> > >> look up an account by name.  Does this actually require looping through
> > >> all accounts and testing if the name matches?
> > > In master you can do this:
> > >
> > >    QString accountName;
> > >    auto account =
> > MyMoneyFile::instance()->accountsModel()->itemByName(accountName);
> > >
> > >    or if you want to stick with the model type access to the data:
> > >
> > >    auto idx =
> > MyMoneyFile::instance()->accountsModel()->indexByName(accountName);
> > >    auto name = idx.data(eMyMoney::Model::AccountNameRole).toString();
> > >
> > > which in fact does the looping for you in
> > AccountsModel::indexListByName().
> > >
> > > Hope that helps.
> >
> > Thanks.  Either of those is shorter than what I came up with, although
> > mine does seem to work (looking with gdb.)
> >
> > My current problem is that I'm trying to add a "go to brokerage" line
> > after the "go to account" and "go to payee" menu choices, but it is not
> > appearing in any of the menus (either main menu or context menu.)  I
> > can't find anywhere in the code where either of the old ones appears
> > where I haven't added the new one, so I don't know what I am missing.
> >
> > Jack
> >
> >
> 
> 

-- 

Regards

Thomas Baumgart

https://www.signal.org/       Signal, the better WhatsApp
-------------------------------------------------------------
More does not equal better. Better equals better.
-------------------------------------------------------------
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 868 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kmymoney-devel/attachments/20220622/ee29f27d/attachment.sig>