Page MenuHomePhabricator
Create Task
Maniphest T303505

[S] Remove Deprecated methods "serialize" and "unserialize" after php production upgrade to PHP 7.4
Closed, InvalidPublic
Actions

Description

Since PHP 7.4 classes that implement the "serializable interface" have had some changes in the way they are implemented.

The code has been updated in the repository on this task T301223: [M] Handle deprecation of Serializable interface in WikibaseMediaInfo, but the fallback methods have been left in place as they require the migration of the live PHP servers before they can be changed.

This ticket is a cleanup ticket required to remove the redundant code after the following two tickets have been completed:

Related Objects
Search...

StatusSubtypeAssignedTask
 InvalidNoneT303505 [S] Remove Deprecated methods "serialize" and "unserialize" after php production upgrade to PHP 7.4
 ResolvedJdforrester-WMFT261872 Drop PHP 7.2 & 7.3 support from MediaWiki master branch, once Wikimedia production is on 7.4
 ResolvedJoeT271736 Migrate WMF production from PHP 7.2 to PHP 7.4
 ResolvedDzahnT245757 Upgrade MediaWiki clusters to Debian Buster (debian 10)
 ResolvedhasharT252434 Test MW code in buster as well as stretch
 ResolvedJdforrester-WMFT250514 Create buster-based images for quibble
 ResolvedLadsgroupT279068 Wikibase data-client bridge selenium failure on buster but not stretch
 Resolved• MoritzMuehlenhoffT250515 Please provide our special component/php72 in buster-wikimedia
 ResolvedjijikiT264991 Upgrade the MediaWiki servers to ICU 63
 Resolved• MoritzMuehlenhoffT253377 WMF deployed EasyTimeline extension depends on Ploticus package which is not available in Debian Buster (but available again in Debian Bullseye)
 ResolvedTrizek-WMFT267145 CommRel support for ICU 63 upgrade
 ResolvedDzahnT267248 create mwdebug1003 - ganeti VM with buster and appserver role
 Resolved• GillesT268188 Release WikimediaDebug for mwdebug1003 addition
 ResolvedDzahnT267607 upgrade mwmaint servers to buster
ResolvedRequestPapaulT275928 decommission mwmaint2001.codfw.wmnet
 ResolvedjijikiT268524 Upgrade Parsoid servers to buster
DeclinedNoneT245888 Rename wtp* servers to parse* (Parsoid PHP servers)
 ResolvedDzahnT268248 upgrade scandium to buster
 ResolvedDzahnT265963 Replace production deployment servers and update them to Buster
Unknown Object (Task)
 ResolvedPapaulT266363 (Need By: TBD) rack/setup/install deploy2002
 Resolved• CmjohnsonT265653 (Need By: TBD) setup/install deploy1002
 ResolvedRequest• CmjohnsonT275831 decommission deploy1001.eqiad.wmnet
 ResolvedRequestPapaulT275832 decommission deploy2001.codfw.wmnet
 ResolvedAndrewT269004 Upgrade labweb servers to buster
 ResolvedArielGlennT269377 Upgrade snapshot hosts to Buster
 ResolvedDzahnT270517 Investigate opcache hit rate on Buster appserver
 ResolvedLegoktmT273312 Investigate possible performance degradation on mediawiki servers after Debian Buster upgrade
 ResolvedPapaulT273803 mw2220 - broken IPMI / mgmt
ResolvedDzahnT274023 Convert mwdebug VMs to debian buster
 Declined• GillesT274026 Add 3 new canaries to WikimediaDebug
 ResolvedVolansT274689 sre.hosts.decomission -> generate_dns_snippets - > Cumin execution failed
 ResolvedDzahnT274403 mw1379 - down after reboot attempt and DRAC can't powercycle
 ResolvedLegoktmT275752 Jobrunner timeouts on cross-DC file uploads because of HTTP/2
 ResolvedPleaseStandT291127 MediaWiki session management does not work on split PHP 7.4 and < PHP 7.4 cluster
 ResolvedJoeT293450 Allow coexisting php version in our puppet code
 ResolvedtstarlingT295578 Test running php7.2 and php7.4 in parallel on the beta cluster
 ResolvedArielGlennT295580 Test php7.4 for dumps generation
 ResolvedJoeT293451 Allow sending traffic to php 7.2 or 7.4 selectively in the apache configuration for MediaWiki
 ResolvedtstarlingT263437 Allow easier ICU transitions in MediaWiki (change how sortkey collation is managed in the categorylinks table)
 ResolvedJdforrester-WMFT293924 Also run PHP 7.4 jobs on wmf branch patches
 ResolvedJdforrester-WMFT293851 Re-build all CI images of PHP 7.4 from sury's package to Wikimedia's one, to assure us that it will work
 ResolvedLegoktmT293449 Package php 7.4 for wikimedia production
 ResolvedJdforrester-WMFT295285 Upgrade primary branch of all Wikimedia-deployed repos to a version of mediawiki-tools-phan including T270553
 ResolvedDaimonaT295286 Release mediawiki-tools-phan 0.11.1
 ResolvedDaimonaT270553 Should mediawiki/mediawiki-phan-config ship a target php version to support running with newer php on the developer side?
 ResolvedJdforrester-WMFT314896 WikiEditor edit sampling code triggers 'PHP Deprecated: Invalid characters passed for attempted conversion, these have been ignored' on PHP7.4+
ResolvedJoeT293996 Build MediaWiki images for kubernetes based on php 7.4
 ResolvedJMeybohmT295725 Stop loading wddx PHP extension
 ResolvedJoeT306042 Serve beta cluster via PHP 7.4 by default
 ResolvedJoeT311386 Install php 7.4 in production
 ResolvedJoeT312634 Properly support php7.4 across the observability stack
 ResolvedJoeT313505 Remove prometheus-phpfpm-exporter from physical hosts
 ResolvedKrinkleT311388 Allow assigning each user to a specific php engine by setting a PHP_ENGINE cookie
 ResolvedJoeT311479 Split edge caches between php versions
 ResolvedClement_GoubertT312638 Parsoid migration to php 7.4
 ResolvedClement_GoubertT307219 Put parse parse10[01-24] in production
 ResolvedClement_GoubertT318946 Parsoid migration: Cleanup
 ResolvedMainframe98T312653 Add "php 7.4" option to the Wikimedia Debug extension
 ResolvedJoeT318894 Remove php 7.2 from production
 ResolvedZabeT316518 Run phan jobs for MediaWiki core on master + WMF branches using PHP 7.4, not just 7.2

Event Timeline

SimoneThisDot created this task.Mar 10 2022, 10:21 AM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMar 10 2022, 10:21 AM
Reedy added a project: Technical-Debt.Mar 10 2022, 7:58 PM
Reedy updated the task description. (Show Details)
Reedy changed the task status from Open to Stalled.Mar 10 2022, 8:35 PM
Reedy added a subtask: T261872: Drop PHP 7.2 & 7.3 support from MediaWiki master branch, once Wikimedia production is on 7.4.
CBogen edited projects, added Structured-Data-Backlog (Current Work); removed Structured-Data-Backlog.Mar 14 2022, 4:35 PM
CBogen moved this task from Incoming to Ready for Estimation on the Structured-Data-Backlog (Current Work) board.
CBogen renamed this task from Remove Deprecated methods "serialize" and "unserialize" after php production upgrade to PHP 7.4 to [S] Remove Deprecated methods "serialize" and "unserialize" after php production upgrade to PHP 7.4.Mar 23 2022, 4:33 PM
CBogen moved this task from Ready for Estimation to Blocked on the Structured-Data-Backlog (Current Work) board.
matthiasmullie closed this task as Invalid.Jun 8 2022, 2:54 PM
matthiasmullie subscribed.

These methods should not be removed or things will break.

The use of the Serializable interface is not deprecated; it's only deprecated when not *also* implementing the new magic methods (i.e. having both is fine)
Both are currently required because both implementations serialize data into a different format, and both are needed to be able to unserialize data for as long as we might have some stored in the old format.
PHP figures out which to use based on the format of the serialized data.

The Serializable interface will not be removed or further deprecated before the next major PHP version (PHP 9), which will take at least another few years.

Jdforrester-WMF closed subtask T261872: Drop PHP 7.2 & 7.3 support from MediaWiki master branch, once Wikimedia production is on 7.4 as Resolved.Sep 29 2022, 2:19 PM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL · Credits