search

Heyuri / kokonotsuba

Heyuri's BBS software
Other
19 stars 12 forks source link

Remove trailing comma which causes a syntax error #3

Closed RealAngeleno closed 1 year ago

RealAngeleno commented 1 year ago

Parse error: syntax error, unexpected ')' in [path-to-kokonotsuba-directory]/install.php on line 167

The syntax error is resolved by removing the trailing comma inside of the bind_param().

Tested in PHP 7.2 Apache/2.4.29 (Ubuntu) MySQL

kaguy4 commented 1 year ago

Kokonotsuba is designed and tested on debian 10, nginx, mariadb, and php7.3, and isn't guaranteed to work on any other stack. Try modifying yours to resemble this one and see if that fixes the issue

RealAngeleno commented 1 year ago

Yep. It works just fine with the stack you described. There are also no errors when display_errors was set to on either.

I will also try it on an apache/MySQL Ubuntu stack (similar to the one I was originally using) with php 7.3 instead and report back before closing this.

RealAngeleno commented 1 year ago

I tested with the Apache/MySQL/Ubuntu stack but with php 7.3 instead. It worked flawlessly as well with no modifications to the code. Although there were non-fatal errors when display_errors was set to on.