Fixed Error about push notifications

[akhan]

Affected version

2.1.0

Error about push notifications
appears in the admin panel error:

ErrorException: [E_USER_WARNING] [WebPush] mbstring extension is not loaded but is required for sending push notifications with payload or for VAPID authentication. You can fix this in your php.ini.
src/vendor/minishlink/web-push/src/WebPush.php:72

#0 [internal function]: XF::handlePhpError(512, '[E_USER_WARNING...', '/home/mazdagara...', 72, Array)
#1 src/vendor/minishlink/web-push/src/WebPush.php(72): trigger_error('[WebPush] mbstr...', 512)
#2 src/XF/Service/PushNotification.php(254): Minishlink\WebPush\WebPush->__construct(Array, Array, 10, Array)
#3 src/XF/Service/PushNotification.php(127): XF\Service\PushNotification->getWebPushObject()
#4 src/XF/Service/PusherTrait.php(64): XF\Service\PushNotification->sendNotifications()
#5 src/XF/Repository/UserAlert.php(147): XF\Service\Alert\Pusher->push()
#6 src/XF/Repository/UserAlert.php(111): XF\Repository\UserAlert->insertAlert(1159, 3, 'Oğuzhan', 'post', 260937, 'mention', Array)
#7 src/XF/Notifier/AbstractNotifier.php(55): XF\Repository\UserAlert->alert(Object(XFRM\XF\Entity\User), 3, 'Oğuzhan', 'post', 260937, 'mention', Array)
#8 src/XF/Notifier/Post/Mention.php(29): XF\Notifier\AbstractNotifier->basicAlert(Object(XFRM\XF\Entity\User), 3, 'Oğuzhan', 'post', 260937, 'mention')
#9 src/XF/Service/AbstractNotifier.php(86): XF\Notifier\Post\Mention->sendAlert(Object(XFRM\XF\Entity\User))
#10 src/XF/Service/AbstractNotifier.php(44): XF\Service\AbstractNotifier->notifyType(Object(XF\Notifier\Post\Mention), Array, G)
#11 src/XF/Service/AbstractNotifier.php(56): XF\Service\AbstractNotifier->notify(3)
#12 src/XF/Service/Thread/Replier.php(205): XF\Service\AbstractNotifier->notifyAndEnqueue(3)
#13 src/XF/Pub/Controller/Thread.php(316): XF\Service\Thread\Replier->sendNotifications()
#14 src/XF/Pub/Controller/Thread.php(449): XF\Pub\Controller\Thread->finalizeThreadReply(Object(XF\Service\Thread\Replier))
#15 src/XF/Mvc/Dispatcher.php(321): XF\Pub\Controller\Thread->actionAddReply(Object(XF\Mvc\ParameterBag))
#16 src/XF/Mvc/Dispatcher.php(248): XF\Mvc\Dispatcher->dispatchClass('XF:Thread', 'AddReply', Object(XF\Mvc\RouteMatch), Object(XFMG\XF\Pub\Controller\Thread), NULL)
#17 src/XF/Mvc/Dispatcher.php(100): XF\Mvc\Dispatcher->dispatchFromMatch(Object(XF\Mvc\RouteMatch), Object(XFMG\XF\Pub\Controller\Thread), NULL)
#18 src/XF/Mvc/Dispatcher.php(50): XF\Mvc\Dispatcher->dispatchLoop(Object(XF\Mvc\RouteMatch))
#19 src/XF/App.php(2155): XF\Mvc\Dispatcher->run()
#20 src/XF.php(389): XF\App->run()
#21 index.php(20): XF::runApp('XF\\Pub\\App')
#22 {main}

 

[Chris D]

We do need to make a change here, but ultimately you will need to ensure the mbstring extension is enabled to use push notifications. Your "Server environment report" on your Admin home page probably lists mbstring support as "No".

 

[akhan]

We do need to make a change here, but ultimately you will need to ensure the mbstring extension is enabled to use push notifications. Your "Server environment report" on your Admin home page probably lists mbstring support as "No".

Fixed the problem when activated. Thank you.

 

[XF Bug Bot]

Thank you for reporting this issue. The issue is now resolved and we are aiming to include that in a future XF release (2.1.0 Beta 3).

Change log:

Improve detection for PHP extensions required for enabling push notifications.

Any changes made as a result of this issue being resolved may not be rolled out here until later.