sprintf function error - Githubissues

lirantal / daloradius

daloRADIUS is an advanced RADIUS web management application for managing hotspots and general-purpose ISP deployments. It features user management, graphical reporting, accounting, a billing engine, and integrates with OpenStreetMap for geolocation. The system is based on FreeRADIUS with which it shares access to the backend database.

http://www.daloradius.com

GNU General Public License v2.0

680 stars 340 forks source link

sprintf function error #551

Open lianhe opened 3 months ago

lianhe commented 3 months ago

Many PHP files have an extra comma in their sprintf function in 3.2.5.such as config-operators-list.php line159 , mng-list-all.php line311...

filippolauria commented 3 months ago

Thank you for your feedback. If you want to push your fixes to this repository, it is actually quite simple. Have a look at the first contributions how to :smile:

leewp14-inf commented 1 month ago

@lianhe What's your php version? Initially I was using PHP7.2 due to environment restriction, after upgraded to PHP7.4 there's no error anymore, no need to fix anything. Also if you happen to use PHP7.2, you will get plenty of syntax error, which is resolved by simply upgrade to PHP7.4.