Code updates to accomodate MySQL5.5

OpenEMR Development
1

fndtn357 wrote on Friday, May 25, 2012:

I modified the all the table.sql files in interface/forms and contrib/forms, as well as, formmaker and xmlformgenerator scripts. Changed TYPE= to ENGINE=
https://github.com/fndtn357/openemr/tree/medbase=dev

2

bradymiller wrote on Saturday, May 26, 2012:

Hi James,
Thanks for doing this. Looks good and committed your code to sourceforge.
-brady
OpenEMR

3

avantsys wrote on Friday, June 15, 2012:

Does this also address the incompatibility issues of the CAMOS module with recent versions of MySQL?

4

fndtn357 wrote on Friday, June 15, 2012:

yes

5

avantsys wrote on Friday, June 15, 2012:

I’m going to elaborate a bit more on this here. Before doing anything on our commercial server and the demo version of our implementation, we test everything locally. Our commercial server has MySQL 5.1.61, runs Debian Linux 6 and the version of Apache is 2.2.16. Locally, we’re using a Windows box, MySQL 5.5 and Apache 2.2.21.

While installing CAMOS locally,  we came across this error when we pressed “Install DB”:

ERROR: query failed: CREATE TABLE IF NOT EXISTS `form_CAMOS` ( id
bigint(20) NOT NULL auto_increment, date datetime default NULL, pid
bigint(20) default NULL, user varchar(255) default NULL, groupname
varchar(255) default NULL, authorized tinyint(4) default NULL,
activity tinyint(4) default NULL, category TEXT, subcategory TEXT,
item TEXT, content TEXT, PRIMARY KEY (id) ) TYPE=MyISAM
Error: 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 ‘TYPE=MyISAM’ at line 15

After some search we found out that there is an issue with the ‘TYPE=MyISAM’ command and MySQL 5.5. We installed the latest patch, 4.1.0 Patch (5/26/2012) but the error persists. Any thoughts?

6

yehster wrote on Friday, June 15, 2012:

The Engine vs. Type changes are not in any patch yet.
They are in the development tip:
https://github.com/openemr/openemr/commit/3fd0f5a386c6aa3b8cb87090741dc79b0676bbf5

7

fndtn357 wrote on Saturday, June 16, 2012:

the last remaining instances of TYPE=MyISAM are listed below:

Documentation.html
line 3573
line 3585
line 3594

Documentation.txt
line 2641
line 2651
line 2660

create_tables.sql
line 47
line 66
line 83
line 97
line 114
line 129
line 142
line 158

athletic_team_layout.sql
line 55

openemr-ea-mixed-complete.sql
line 12

I don’t know if these upgrade files are necessary to change or not
11 instances in 2_6_0-to-2_6_1_upgrade.sql
2 instances in 2_7_0-to-2_7_2_upgrade.sql
2 instances in 2_8_1-to-2_8_2_upgrade.sql
4 instances in 2_8_3-to-2_9_0_upgrade.sql

8

bradymiller wrote on Saturday, June 16, 2012:

Hi,
I wouldn’t touch the phpmyadmin stuff (will be better to upgrade to the phpmyadmin version that supports php/mysql 5 in the future). Otherwise, would convert the rest (including the upgrade scripts).
-brady
OpenEMR

9

deschel wrote on Saturday, June 16, 2012:

Hi,

Version of phpMyAdmin embedded in OpenEMR is 2.11.10.

Current version of phpMyAdmin is 3.5.1.

I don’t think many modification were made to phpMyAdmin to embed it.  Wouldn’t it be easy to upgrade the version of phpMyAdmin in OpenEMR by just switching the code?  Why hasn’t this been done?  Or, when will it be done?

David Eschelbacher, MD

10

bradymiller wrote on Saturday, June 16, 2012:

Hi,

See here for details:
http://www.open-emr.org/wiki/index.php/PhpMyAdmin
(note it was updated about 2 years ago with a version towards php4)

Agree that it’s time to update it again. Guessing would not be too tough if just do what was explained in the above wiki page. Anybody can feel free to take this on.

-brady
OpenEMR