Version 1.0.1 of iubenda-cookie-class library causes problems on PHP 7.3

Created on 14 August 2020, almost 4 years ago
Updated 20 October 2023, 8 months ago

Problem/Motivation

Version 1.0.1 of the cookie class, from this URL: http://simon.s3.iubenda.com/iubenda-cookie-class causes errors with PHP 7.3, and greatly degrades site performance. Error:

Warning: preg_match(): Compilation failed: invalid range in character class at offset 4 in simple_html_dom->read_tag() (line 1375 of /code/sites/all/libraries/iubenda-cookie-class/simple_html_dom.php).

Steps to reproduce

Install using drush, or following the steps on the module home page. Enable the module on a live site with an iubenda profile.

Proposed resolution

Use the newest version of the module, from here: https://github.com/iubenda/iubenda-cookie-class

And also update code to change references to Page class (1.0.1) to iubendaParser (4.1.2) - see patch.

I'm not taking responsibility right now for updating the version installed by drush. Retroactively changing the class name (in this patch) is going to break sites who have the 1.0.1 version installed. That means we need an install hook to either replace the library or just deactivate the module until the user installs the correct one.

All in all I think it should be at the user's discretion to install the library, as automatic install doesn't work for all use cases (e.g. Pantheon) and causes legacy code problems like this one. (This is obviously a problem best solved by composer in Drupal 8. Composer Manager might be a possible direction?) At any rate, anyone else using this module on PHP 7.3 should be able to solve the problem using this patch and the newer library, and I wanted that to be known.

Does the maintainer have suggestions on how to integrate this change without breaking users of the 1.0.1 version?

πŸ› Bug report
Status

Fixed

Version

2.0

Component

Code

Created by

πŸ‡ΊπŸ‡ΈUnited States firestonej

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.69.0 2024