-
-
Notifications
You must be signed in to change notification settings - Fork 450
Replace php date functios with carbon
#5131
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
sreichel
wants to merge
24
commits into
OpenMage:main
Choose a base branch
from
sreichel:rector/-carbon
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
carboncarbon
|
Contributor
There was a problem hiding this 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 102 out of 102 changed files in this pull request and generated 11 comments.
Comments suppressed due to low confidence (1)
app/code/core/Mage/Index/Model/Observer.php:154
- Using
Carbon::now()->sub()mutates the Carbon instance. This could cause issues if the$nowvariable is reused later. Consider usingCarbon::now()->subSeconds(self::OLD_INDEX_EVENT_THRESHOLD_SECONDS)or callingcopy()first to avoid mutation:$now->copy()->sub($dateInterval).
app/code/core/Mage/ImportExport/Model/Import/Entity/Abstract.php
Outdated
Show resolved
Hide resolved
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
chore
Component: Adminhtml
Relates to Mage_Adminhtml
Component: AdminNotification
Relates to Mage_AdminNotification
Component: Api
PageRelates to Mage_Api
Component: Authorizenet
Relates to Mage_Authorizenet
Component: Captcha
Relates to Mage_Captcha
Component: Catalog
Relates to Mage_Catalog
Component: CatalogIndex
Relates to Mage_CatalogIndex
Component: CatalogRule
Relates to Mage_CatalogRule
Component: Checkout
Relates to Mage_Checkout
Component: Cms
Relates to Mage_Cms
Component: Core
Relates to Mage_Core
Component: Cron
Relates to Mage_Cron
Component: Customer
Relates to Mage_Customer
Component: Dataflow
Relates to Mage_Dataflow
Component: Eav
Relates to Mage_Eav
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: Oauth
Relates to Mage_Oauth
Component: Payment
Relates to Mage_Payment
Component: PayPal
Relates to Mage_Paypal
Component: Reports
Relates to Mage_Reports
Component: Rss
Relates to Mage_Rss
Component: Rule
Relates to Mage_Rule
Component: Sales
Relates to Mage_Sales
Component: Shipping
Relates to Mage_Shipping
Component: Tag
Relates to Mage_Tag
Component: Usa
Relates to Mage_Usa
Component: Widget
Relates to Mage_Widget
Mage.php
Relates to app/Mage.php
phpunit
rector
Template : admin
Relates to admin template
Template : base
Relates to base template
Template : install
Relates to install template
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.

Description (*)
Use rector to replace native date functions with
carbon,(prepare to replace
Zend_Date)Related Pull Requests
Varien_Datewith Carbon date library #4463