adam/Piwigo
1
0 Fork 0
mirror of https://github.com/Piwigo/Piwigo.git synced 2026-10-30 21:52:00 +01:00
Code Issues 657 Packages Projects Releases Wiki Activity

[14.x] keep compatibility with PHP 7.0 #1795

New Issue
Closed
opened 2026-10-05 00:56:55 +02:00 by adam · 1 comment
adam commented 2026-10-05 00:56:55 +02:00
Owner
Copy link
Copy source

Updating Smarty to version 4.3, see #1845 has brought compatibility with PHP 8.2 but lost compatibility with PHP 7.0. Unfortunately we still see many hosting with PHP 7.0 and I wondered if keeping compatibility with PHP 7.0 was still possible.

After some quick investigations it seems that the only problem is the use of nullable types like ?int $var or ?string $var. There are only a tiny use of them in Smarty. Removing them makes Smarty 4.3 compatible with PHP 7.0 (as fas as my tests go)

Updating Smarty to version 4.3, see #1845 has brought compatibility with PHP 8.2 but lost compatibility with PHP 7.0. Unfortunately we still see many hosting with PHP 7.0 and I wondered if keeping compatibility with PHP 7.0 was still possible. After some quick investigations it seems that the only problem is the use of `nullable types` like `?int $var` or `?string $var`. There are only a tiny use of them in Smarty. Removing them makes Smarty 4.3 compatible with PHP 7.0 (as fas as my tests go) <!-- github-sync-id:2039774989 -->
adam closed this issue 2026-10-05 00:56:55 +02:00
adam commented 2026-10-05 00:56:56 +02:00
Author
Owner
Copy link
Copy source

I also want to point out that it's perfectly possible we increase the PHP requirement to PHP 8 for Piwigo 15, I just want to keep Piwigo 14 compatible with PHP 7.0.

I also want to point out that it's perfectly possible we increase the PHP requirement to PHP 8 for Piwigo 15, I just want to keep Piwigo 14 compatible with PHP 7.0. <!-- github-comment-id:1853971999 -->
Sign in to join this conversation.
No Branch/Tag Specified
Labels
No items
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
adam (Adam Wojcieszonek)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: adam/Piwigo#1795