PHP 7.X.X issue

Resolved dasgeestig
(@dasgeestig)

9 years, 5 months ago

Seems that with php >7.0 it’s not possible to select multiple rows and change their booking-status. Only 1 row get’s changed. Also the “You have made the following changes” page doesn’t appear when changing the booking-status.

But, after downgrading to php 5.6.30 it all works fine. I checked also checked for differences between apache & nginx; no more bugs or differences here.

My guess it that public static function changeStatusShow (blokaroom-meetings.php) needs a (small) change, but wasn’t able to lay my finger on it.

Any idea, anybody? Thanks!

Viewing 4 replies - 1 through 4 (of 4 total)

Plugin Author Cleveland Heights-University Heights Public Library Webdeveloper
(@chuhpl)

9 years, 5 months ago

I’ve not started looking at moving over to PHP 7 yet. In the wild, PHP 5 is used on about 95% of all installations so I’m focusing on making sure it works with 5 for now. Since I’m a single developer with other projects, I have very limited time and resources, so I have to make sure I’m serving the largest number of users.

Thread Starter dasgeestig
(@dasgeestig)

9 years, 5 months ago

I understand. Although leaving out depreciated elements could save you a ton of work later on :-).

Anyhow, I found out that it was about preme( mysql_error() on line 359 of version 1.3; depreciated in PHP5.5 and removed in PHP7.X. Seems like you add this only in the latest release of your plugin; that explains also why I didn’t encounter this issue before.

For people who experience the same issue: removing / out commenting line 359 fixes it!
Plugin Author Cleveland Heights-University Heights Public Library Webdeveloper
(@chuhpl)

9 years, 5 months ago
Oh my. That should be removed or commented. preme is just a testing function I use. It’s basically print_r wrapped in a pre wrapper for viewing array results. You can comment that out! I did a few quick bug fixes and must have left some testing code in!
- This reply was modified 9 years, 5 months ago by Cleveland Heights-University Heights Public Library Webdeveloper.
Plugin Author Cleveland Heights-University Heights Public Library Webdeveloper
(@chuhpl)

9 years, 4 months ago

I put out a new version of the plugin (1.4) that fixes a warning message if you have no amenities. I also used this as a chance to pull out all of the preme() uses. Give it an update and let me know if it fixes things.

Viewing 4 replies - 1 through 4 (of 4 total)

The topic ‘PHP 7.X.X issue’ is closed to new replies.

Tags