[#30569] - 3.9.21 and PHP 7.4.x compatibility issues with built in LESS compiler
- Closed
- 19 Nov 2020
- Low
- Build: master
- # 30569
Steps to reproduce the issue
- Install 3.9.21
- Install PHP 7.4.10
- Install Kunena Forum 5.1.19
- Install "Blue Eagle 5" Kunena template and make it default
- Enable PHP error reporting
- Browse to Forum's index page
Expected result
Forum displayed without problems
Actual result
Got following errors:
Deprecated: Array and string offset access syntax with curly braces is deprecated in <site_path>\libraries\vendor\leafo\lessphp\lessc.inc.php on line 749
Deprecated: Array and string offset access syntax with curly braces is deprecated in <site_path>\libraries\vendor\leafo\lessphp\lessc.inc.php on line 1860
Deprecated: Array and string offset access syntax with curly braces is deprecated in <site_path>\libraries\vendor\leafo\lessphp\lessc.inc.php on line 2519
Deprecated: Array and string offset access syntax with curly braces is deprecated in <site_path>\libraries\vendor\leafo\lessphp\lessc.inc.php on line 2573
System information (as much as possible)
Additional comments
Basically the problem is with "lessc.inc.php" that uses old/deprectaed syntax like "$subProp[1]{0}" to address arry elements instead of "$subProp[1][0]".
Not the major issue by far, but since this LESS compiler is distributed with Joomla and Joomla recommends to upgrade to 7.4 ... looks bad :)
Votes
joomla-cms-bot
- 
Attached the version of the file that fixes the issue.
Very simple , I just do not like doing core "hacking" , if this distributed with Joomla - everyone should get it as part of the install.
lessc.inc.php.txt
(just remove the .txt from extension)
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/30569.
The issue has been "discussed" already here: #30452
The only reliable solution (because it's a vendor library) would be to use another, still maintained less-php repo or provide a maintained one as replacement for the current one. BTW: As far as I know less has been removed from Joomla 4 (?).
(I used my custom system plugin in the past to load my fixed less library very early instead of the joomla less library. But that was a "very individual solution", not appropriate for Joomla core.)
See upstream PR leafo/lessphp#647
What's the point on submitting a pr on a upstream dead project ??
surely we should consider to fork this , if we really think we need to really support j.3 for 2 years after the release of j4 stable
An topic for this issue (backporting combat fixes for dead projects) has been added to the production department meeting agenda to be held on 08.09.2020
| Status | New | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2020-09-04 19:01:29 |
| Closed_By | ⇒ | alikon |
well then we can simply close this
Why should we close this here? Has there been an decision made? It was just me that added that to the document.
| Status | Closed | ⇒ | New |
| Closed_Date | 2020-09-04 19:01:29 | ⇒ | |
| Closed_By | alikon | ⇒ |
No,
sorry for being in hurry then let this be reopened and waiting for Godot
This errors fatally on PHP 8.
To foreshadow the meeting report to be published this is an acknowledged issue where it seems we got an agreement on how we want to backport the changes. Details will follow once that thing is finalized.
As there hasnt been a published report from the department for 60 days is there any chance of an update here
| Labels |
Added:
?
|
||
We will fork the repository and fix it for j3 packages. The meeting reports will come soon because of personal things they got delayed...
Thanks for the update
This is the repo where we will do the fixes: https://github.com/joomla-projects/lessphp
Help is appreciated.
@HLeithner OK to close per PR #31428?
Yes
| Status | New | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2020-11-19 17:44:10 |
| Closed_By | ⇒ | Quy |
fyi wikimedia maintains a fork at https://github.com/wikimedia/less.php#transitioning-from-leafolessphp
Attached the version of the file that fixes the issue.
Very simple , I just do not like doing core "hacking" , if this distributed with Joomla - everyone should get it as part of the install.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/30569.