Hi,
InAppPurchases::Purchase::purchaseTime’s in juce_InAppPurchases_ios.cpp are being returned as “dd mmm yyyy time” format strings instead of ISO8601 format strings as stated in the docs.
E.g. Line 278-279:
auto purchaseTime = Time (1000 * (int64) transaction.transactionDate.timeIntervalSince1970)
.toString (true, true, true, true);
Should this not be .toISO8601(true) instead?
Kris.
I think it should simply be a juce::Time object instead, much more convenient. I have in fact made this modification in my own fork. There was a thread or feature request about this some time ago.
Agreed.
For myself, the ISO8601 string is only being used to re-construct Time objects later on for time comparisons. Just getting a Time object would be way more efficient. Curious as to what the reasoning was at the time for doing it this way. But, it’s been like this for so long that I don’t see there would be any chance of getting it changed now from a backwards compatibility perspective. Making sure purchaseTime is always ISO8601 would suffice for me.
