From
#2725247: Review gateway plugins →
, sort out what to do with \Drupal\sms\Plugin\SmsGatewayPluginInterface::balance()
. It is not used by SMS Framework nor does it have test coverage.
Change name from balance()
to getBalance()
(similar to getDeliveryReports()
) or getCreditsBalance()
(preferred, similar to new reports). The 'get' prefix implies we need an external request, and should be used sparingly.
Make signature return float|NULL
. NULL
if unknown.
Add an annotation to gateway which determines whether the gateway supports balance inquiries.
Where can we use the balance system in SMS framework?
Closed: outdated
1.0
Core Framework
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.