<div dir="ltr"><div><div><div><div>Hello, I have added members m_idNumber (QString) and m_idSelector (enum) to AlkOnlineQuoteSource::Private:<br><a href="https://invent.kde.org/surajsloth/alkimia/-/tree/gsoc21">https://invent.kde.org/surajsloth/alkimia/-/tree/gsoc21</a><br><br></div>While writing the getter and setter for m_idSelector which is of type enum and defined in alkonlinequotesource.cpp whereas the declarations of the getters and setters are in alkonlinequotesource.h therefore I can't declare the function of type idSelector (enum). What should I do for that case?<br><br></div>Also, I didn't get why the locations of sym() should be changed.<br><br></div>Regards <br></div>Suraj Kumar Mahto<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jul 5, 2021 at 11:28 AM Thomas Baumgart <<a href="mailto:thb@net-bembel.de">thb@net-bembel.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Sonntag, 4. Juli 2021 21:54:54 CEST Suraj Kumar Mahto wrote:<br>
<br>
> Hello,<br>
> While replacing the classes like WebPriceQuoteSources with<br>
> AlkOnlineQuoteSources and other such cases, I saw that the member(like<br>
> identifyBy) is not implemented in alkimia. Does that mean I have to add<br>
> those members and functions in libalkimia repo similarly to that done in<br>
> webpricequote?<br>
<br>
I would suggest to add two more members to AlkOnlineQuoteSource::Private:<br>
<br>
a) m_idNumber (QString)<br>
b) m_idSelector (enum)<br>
<br>
The default should be empty and the selector should point to symbol to<br>
maintain backward compatibility. Add getters and setters and update<br>
read() and write() to store them with the quote source. You may also<br>
have to adjust the locations where sym() is used today in<br>
AlkOnlineQuote::Private.<br>
<br>
> I have tried to implement AlkOnlineQuote::setDate() (similar to that of<br>
> webpricequote) in alkimia repo earlier but getting some reference error. I<br>
> skipped that thinking that will check that part once I replace all the<br>
> WebPriceQuote implementation in that file (kequitypriceupdatedlg.cpp) .<br>
<br>
m_fromDate and m_toDate in WebPriceQuote are only used in connection with<br>
CSV downloads and I think we skipped those completely for the moment, as<br>
they are not supported by Alkimia. So don't worry about them for the moment.<br>
<br>
<br>
-- <br>
<br>
Regards<br>
<br>
Thomas Baumgart<br>
<br>
<a href="https://www.signal.org/" rel="noreferrer" target="_blank">https://www.signal.org/</a>       Signal, the better WhatsApp<br>
-------------------------------------------------------------<br>
Computers let you make more mistakes faster than any other<br>
invention in human history, with the possible exception<br>
of handguns and tequila. --Mitch Radcliffe<br>
-------------------------------------------------------------<br>
</blockquote></div>