meeting-room-booking-system / mrbs-code

MRBS application code
Other
117 stars 57 forks source link

from mysql to mysqli #2208

Open jberanek opened 6 years ago

jberanek commented 6 years ago

I have updated my php from 5.4 to 7.0 and as result of this upgradation mysql is no longer valid. I tried updating config.inc.php by chnaging the $dbsys = mysql to mysqli but still getting errors . Apart from that I have also the mysql .inc to mqsli.inc(updating the functions as well) but still no result.

Reported by: *anonymous

Original Ticket: mrbs/support-requests/1499

jberanek commented 6 years ago

Which version of MRBS are you running? I suspect you need to upgrade to the latest version of MRBS.

Original comment by: campbell-m

jberanek commented 6 years ago

1.5.0

Original comment by: *anonymous

jberanek commented 6 years ago

I upgraded the version it now shows this error -- Fatal error: unfortunately the database is not available at the moment. !

Original comment by: *anonymous

jberanek commented 6 years ago

Below is the log file for the abaove error.

PHP Warning:  You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'AND end_time > 
        ORDER BY start_time' at line 9 in E:\inetpub\websites\RoomReservation\functions_table.inc on line 696
[19-Apr-2018 15:49:15 America/Chicago] MRBS: Fatal error: unfortunately the database is not available at the moment.
[19-Apr-2018 15:49:15 America/Chicago] MRBS GET: Array
(
    [year] => 2018
    [month] => 04
    [day] => 19
    [area] => 2
    [room] => 11
)

[19-Apr-2018 15:49:15 America/Chicago] MRBS SESSION: Array

Original comment by: *anonymous

jberanek commented 6 years ago

Are you sure that you have uploaded all the MRBS 1.7.1 files? Line 696 in functions_table.inc in MRBS 1.7.1 doesn't look like that.

Original comment by: campbell-m

jberanek commented 6 years ago

I was running 1.6.0 . I have upgraded it to 1.7.0 but still shows the same error and also this error PHP Warning: PHP Startup: Unable to load dynamic library 'C:\php\php_new\php-7.0.29-nts-Win32-VC14-x86\ext\php_mysql.dll' - The specified module could not be found. in Unknown on line 0

I know php 7.7 doesn't support mysql I did the chnages in the files to change "mysql" to "mysqli"

Original comment by: *anonymous

jberanek commented 6 years ago

*php 7.0

Original comment by: *anonymous

jberanek commented 6 years ago

That line doesn't appear in 1.6.0 either, so I'm still confused. It still sounds like not all the files have been uploaded properly. The second error may be due to you not having the correct extensions enabled in your PHP ini file.

By the way, is there any reason why you upgraded to 1.7.0 and not 1.7.1?

Original comment by: campbell-m

jberanek commented 6 years ago

No, I was just following the documentation from the website so upgraded till 1.7.0 only .

Correct me if I am wrong but to upgrade to 1.7.1 i will just have to change the version in the file "version.inc" or are there any other changes required as well.

Original comment by: *anonymous

jberanek commented 6 years ago

The other thing worth mentioning is that with current MRBS, people who install MRBS needn't worry about "mysql" vs. "mysqli". Specifying the DB type as "mysql" does not imply a particular PHP extension. In fact if you specify "mysql" or "pgsql" MRBS uses PDO.

Original comment by: jberanek

jberanek commented 6 years ago

You have to download 1.7.1, unpack the zip and put the files from there on your web server.

Original comment by: jberanek

jberanek commented 6 years ago

Remember to take a backup of your old files (especially config.inc.php) and a backup of your database too.

Original comment by: jberanek

jberanek commented 6 years ago

okay will do that

Original comment by: *anonymous