Code review comment for lp:~therp-nl/account-financial-tools/6.1-account-purchase-default

Revision history for this message
Ronald Portier (Therp) (rportier1962) wrote :

I kept the browse instead of the read. According to the famous Technical Mememto from Stephane Wirtel, read() should be used from webservices, but internally browse() should always be used.

I could not find any real performance comparison between the two (which nmight be interesting to do). For the moment I left the browse calls.

« Back to merge proposal