I'm overwriting the service commerce_order.availability_checker because I want to assign licenses based on a factor other than if a user is the one who purchased it (we have "Household" licenses for example). This is working well, but I am not overwriting the checkPurchasable or getRenewalStartTimeMessage methods. But since they're private, I have to copy/paste them into my extended class but leave them as-is.
With the first method, I worry that in the future these will change and we won't know to update our versions of the code. I could, alternatively, do something like this: https://stackoverflow.com/a/61921662, but that feels a bit clever/confusing for future maintenance as well.
Is there some fundamental reason to keep those two methods "private", or could they be "protected"?.
Determine an approach and implement
None
None
Fixed
3.0
Code
Seems reasonable. Thanks for the patch, @mariacha1 !
Automatically closed - issue fixed for 2 weeks with no activity.