They are, this patch just provides the ability to save this data into the existing database using existing variables. This variables are the ones that are used in the print checks plugin.

Removig superfluous lines in kpayeesview.cpp

I will fix this

Corrected variable name numSplits.

Changed all sahres to value and updated currency to print the currency symbol.

Added the currency for the accout=nt/transaction

questions

This should now give the proper amount.

I have replaced shares with value in: checkHTML.replace( valueVariable, MyMoneyUtils::formatMoney((*it).transaction().splits()[i].value().abs(), currency));.
As has been pointed out if one or more of the splits are in a different currency shares will give the wrong amount for the split.

Thanks for the review, please look at my changes.

Thanks for the feedback, my C++ skills are way out of date. I have made two of the three changes that were suggested. I do not understand the one about the currency and you reference line 206. That line gives the value in text not numbers. Line 207 gives the value in numbers and should be modified by the currency. My line 214 should give the value in numbers and is identical to line 207. It should be modified by the currency. If I am wrong please let me know.

Please review