Problems with "Lowest credit limit to trigger notification"

MAPIIIAJI · January 13, 2021, 4:55am

Dear Anton, Hi!
Thank you for making the product better.
I have not yet had time for beta tests, I do not have time. But I want to remind you of one “bug” that I saw not long ago.
What we have: A user who has a mobile number in the “User Preference” and in the “Mobile”, for example +1234567890.
We also have global settings and an alert trigger “Lowest credit limit to trigger notification”, where it is indicated to notify the user if the number of SMS is below 50 (the price of each SMS is 0.01 and that 0.5 is exactly 50 SMS)
But when the number of SMS is released below 50, the notification does not come that the warning limit has been reached.

Or does the notification system work differently?

Thanks for the earlier reply.

Yours faithfully,
Jamshid Tursunov

anton · January 13, 2021, 9:03am

Hi,

This has been fixed after 1.4.3:

github.com

antonraharja/playSMS/blob/master/storage/application/plugin/feature/credit/fn.php#L317-L358


		// if lowest credit limit is non-zero and balance is under/same as lowest credit limit, and never been notified then notify admins, parent_uid and uid
		$credit_lowest_limit = (float) $core_config['main']['credit_lowest_limit'];
		$reg = registry_search($uid, 'feature', 'credit', 'lowest_limit_notif');
		$notified = ($reg['feature']['credit']['lowest_limit_notif'] ? TRUE : FALSE);
		
		if (($credit_lowest_limit > 0) && ($balance <= $credit_lowest_limit) && !$notified) {
		
			// set notified
			registry_update($uid, 'feature', 'credit', array(
				'lowest_limit_notif' => TRUE
			));
		
			// notif admins
			$admins = user_getallwithstatus(2);
			foreach ($admins as $admin) {
				$credit_message_to_admins = sprintf(_('Username %s with account ID %d has reached lowest credit limit of %s'), $username, $uid, $credit_lowest_limit);
				recvsms_inbox_add(core_get_datetime(), _SYSTEM_SENDER_ID_, $admin['username'], $credit_message_to_admins);
			}
		
			// get parent

This file has been truncated. show original

Try to replace your credit/fn.php with this (and then restart playSMS daemon):
https://raw.githubusercontent.com/antonraharja/playSMS/master/storage/application/plugin/feature/credit/fn.php

anton

MAPIIIAJI · January 13, 2021, 12:24pm

Thank you for reply Anton.
Will the notification be sent to the number specified in the User Preference?

Yours faithfully,
Jamshid Tursunov

anton · January 13, 2021, 12:59pm

It will be sent to user’s Inbox. User can forward Inbox that to their email and mobile phone.

anton

MAPIIIAJI · January 13, 2021, 1:01pm

Thank you, got it.

Yours faithfully,
Jamshid Tursunov

system · March 14, 2021, 1:01pm

This topic was automatically closed 60 days after the last reply. New replies are no longer allowed.

Topic		Replies	Views
[solved] Low Credit Notification Issue Dev	9	488	December 4, 2020
How to send low credit notification to users Help	4	442	March 7, 2020
Users not receive low credit notification Help	1	423	August 1, 2018
Credit limit notification Help	1	542	June 9, 2015
Bug in "credits" Help	12	1269	August 30, 2018

Problems with "Lowest credit limit to trigger notification"

Related Topics