Pantheon, WP Native Sessions and Bookly

Hi @virtueagency,

I don’t have any prior experience with Bookly but, based on a quick skim of the codebase, it seems like Bookly should be compatible with WP Native PHP Sessions.

Can you share more details about the incompatibility issues you’re experiencing and steps to reproduce?

It’s a membership site so to test, you will need user access. I can send that privately. Basically, the appointment booking process gets hung up after the first step. You can’t actually click “next” to select the date and time. How can I send you credentials privately to test?

And thank you for the quick follow up! ??

Can you create a multidev environment named pantheon-cse, and then email access credentials to [email protected] ? It’d be great to have access to the Pantheon dashboard too, so I can enable WP_DEBUG in the environment and look at any errors that might be happening.

Hi, all.

I’ve done some investigation on the incompatibilities. The biggest problem is that Bookly decides to wait until objects are destructed to persist anything to the session (See Bookly\Lib\UserBookingData and the destruct method). By that logic, and from what I have seen, the custom session handler is going to be already destroyed by the time the plugin tries to write data to the session.

As a test, I added $userData->destruct() to the Bookly\Frontend\Modules\Booking\Ajax class in the sessionSave method beneath the call to $userData->fillData( $parameters ); and the session values appear to save correctly in some instances.

@doublecompile Great debugging!

By that logic, and from what I have seen, the custom session handler is going to be already destroyed by the time the plugin tries to write data to the session.

As a matter of fact, we’ve seen this issue in another context too. I don’t have any great ideas on how to solve for it right now, but it’s something we can explore further.

Have you seen any PHP errors in your logs, by chance?

Also, I’ve created a GitHub issue for this, which might be a better place to continue the conversation: https://github.com/pantheon-systems/wp-native-php-sessions/issues/103