Skip to content

[PHP83] rector: added Deprecated php attribute#5476

Draft
sreichel wants to merge 40 commits into
OpenMage:mainfrom
sreichel:upd-1
Draft

[PHP83] rector: added Deprecated php attribute#5476
sreichel wants to merge 40 commits into
OpenMage:mainfrom
sreichel:upd-1

Conversation

@sreichel
Copy link
Copy Markdown
Contributor

@sreichel sreichel commented Apr 16, 2026

Note:

With php84+ and dev-mode on using a deprecated method/class will throw an exception.

Copilot AI review requested due to automatic review settings April 16, 2026 19:01
@github-actions github-actions Bot added Component: PayPal Relates to Mage_Paypal Component: Core Relates to Mage_Core Component: Catalog Relates to Mage_Catalog Component: Cms Relates to Mage_Cms Component: Reports Relates to Mage_Reports Component: CatalogInventory Relates to Mage_CatalogInventory Component: Checkout Relates to Mage_Checkout Component: AdminNotification Relates to Mage_AdminNotification Component: lib/Varien Relates to lib/Varien Component: Sales Relates to Mage_Sales Component: Usa Relates to Mage_Usa Component: Customer Relates to Mage_Customer Component: Eav Relates to Mage_Eav Component: lib/Mage Relates to lib/Mage Component: Adminhtml Relates to Mage_Adminhtml Component: ProductAlert Relates to Mage_ProductAlert Mage.php Relates to app/Mage.php Component: Sitemap Relates to Mage_Sitemap Component: Page Relates to Mage_Page Component: Api PageRelates to Mage_Api Component: Captcha Relates to Mage_Captcha Component: Contacts Relates to Mage_Contacts labels Apr 16, 2026
# Conflicts:
#	.phpstan.dist.baseline.php
#	.rector.php
#	app/code/core/Mage/Adminhtml/Block/Sales/Order/Create/Customer/Grid.php
#	app/code/core/Mage/Adminhtml/controllers/Promo/CatalogController.php
#	app/code/core/Mage/Adminhtml/controllers/Report/SalesController.php
#	app/code/core/Mage/Api/Model/Server/Handler/Abstract.php
#	app/code/core/Mage/Captcha/Model/Zend.php
#	app/code/core/Mage/Catalog/Block/Product/View/Type/Configurable.php
#	app/code/core/Mage/Catalog/Model/Design.php
#	app/code/core/Mage/Catalog/Model/Resource/Abstract.php
#	app/code/core/Mage/Catalog/controllers/ProductController.php
#	app/code/core/Mage/CatalogIndex/Model/Indexer.php
#	app/code/core/Mage/CatalogIndex/Model/Resource/Indexer.php
#	app/code/core/Mage/CatalogRule/Model/Rule.php
#	app/code/core/Mage/CatalogSearch/Block/Layer.php
#	app/code/core/Mage/Cms/Model/Resource/Page/Collection.php
#	app/code/core/Mage/Index/Model/Indexer/Abstract.php
#	app/code/core/Mage/Index/Model/Resource/Abstract.php
#	app/code/core/Mage/Payment/Model/Method/Abstract.php
#	app/code/core/Mage/Rule/Model/Resource/Rule/Collection/Abstract.php
#	app/code/core/Mage/Sales/Model/Order/Creditmemo/Api/V2.php
#	app/code/core/Mage/Sales/controllers/OrderController.php
#	app/code/core/Mage/SalesRule/Model/Rule.php
#	app/code/core/Mage/Tag/controllers/CustomerController.php
#	app/code/core/Mage/Tax/Model/Sales/Total/Quote/Subtotal.php
#	composer.lock
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 204 out of 204 changed files in this pull request and generated 1 comment.

Comment thread app/Mage.php
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 204 out of 204 changed files in this pull request and generated 2 comments.

Comment thread app/OpenMageVersionInterface.php Outdated
Comment thread app/code/core/Mage/Core/Model/Translate/Inline.php Outdated
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Apr 26, 2026

Quality Gate Passed Quality Gate passed

Issues
0 New issues
3 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
1.6% Duplication on New Code

See analysis details on SonarQube Cloud

@sreichel
Copy link
Copy Markdown
Contributor Author

Wait for php83 and phpunit12.

sreichel added 2 commits May 26, 2026 06:09
# Conflicts:
#	.phpstan.dist.baselines/_loader.php
#	.phpstan.dist.baselines/method.deprecated.php
#	.phpstan.dist.baselines/staticMethod.deprecated.php
#	app/OpenMageVersionInterface.php
#	app/code/core/Mage/Usa/Model/Shipping/Carrier/Fedex.php
@sonarqubecloud
Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Component: Admin Relates to Mage_Admin Component: Adminhtml Relates to Mage_Adminhtml Component: AdminNotification Relates to Mage_AdminNotification Component: Api PageRelates to Mage_Api Component: Api2 Relates to Mage_Api2 Component: Bundle Relates to Mage_Bundle Component: Catalog Relates to Mage_Catalog Component: CatalogIndex Relates to Mage_CatalogIndex Component: CatalogInventory Relates to Mage_CatalogInventory Component: CatalogRule Relates to Mage_CatalogRule Component: CatalogSearch Relates to Mage_CatalogSearch Component: Centinel Relates to Mage_Centinel Component: Checkout Relates to Mage_Checkout Component: Cms Relates to Mage_Cms Component: Core Relates to Mage_Core Component: Customer Relates to Mage_Customer Component: Directory Relates to Mage_Directory Component: Downloadable Relates to Mage_Downloadable Component: Eav Relates to Mage_Eav Component: GoogleAnalytics Relates to Mage_GoogleAnalytics Component: ImportExport Relates to Mage_ImportExport Component: Index Relates to Mage_Index Component: Install Relates to Mage_Install Component: lib/Mage Relates to lib/Mage Component: lib/Varien Relates to lib/Varien Component: lib/* Relates to lib/* Component: Log Relates to Mage_Log Component: Newsletter Relates to Mage_Newsletter Component: Page Relates to Mage_Page Component: Payment Relates to Mage_Payment Component: PayPal Relates to Mage_Paypal Component: Reports Relates to Mage_Reports Component: Review Relates to Mage_Review Component: Rule Relates to Mage_Rule Component: Sales Relates to Mage_Sales Component: SalesRule Relates to Mage_SalesRule Component: Shipping Relates to Mage_Shipping Component: Tag Relates to Mage_Tag Component: Tax Relates to Mage_Tax Component: Usa Relates to Mage_Usa Component: Weee Relates to Mage_Weee Component: Widget Relates to Mage_Widget Component: Wishlist Relates to Mage_Wishlist enhancement hold Mage.php Relates to app/Mage.php PHP 8.4 Relatee to PHP 8.4 phpstan rector

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants