Skip to content
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

PHP8.0 Fixes passing null as 1st parameter to mktime() #2938

Merged
merged 1 commit into from
Jan 15, 2023
Merged

PHP8.0 Fixes passing null as 1st parameter to mktime() #2938

merged 1 commit into from
Jan 15, 2023

Conversation

sreichel
Copy link
Contributor

@sreichel sreichel commented Jan 14, 2023

Fixed Issues (if relevant)

Deprecated functionality: mktime(): Passing null to parameter #1 ($hour) of type int is deprecated

Manual testing scenarios (*)

  1. go to Admin config -> General -> Reports
  2. ...

Questions or comments

Contribution checklist (*)

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All automated tests passed successfully (all builds are green)
  • Add yourself to contributors list

@sreichel sreichel added the PHP 8.1 Related to PHP 8.1 label Jan 14, 2023
@github-actions github-actions bot added the Component: Adminhtml Relates to Mage_Adminhtml label Jan 14, 2023
@sreichel sreichel added PHP 8 Related to PHP8 and removed PHP 8.1 Related to PHP 8.1 labels Jan 14, 2023
@sreichel sreichel changed the title PHP8.1 Fixes passing null as 1st parameter to mktime() PHP8.0 Fixes passing null as 1st parameter to mktime() Jan 14, 2023
Copy link
Contributor

@luigifab luigifab left a comment

Choose a reason for hiding this comment

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

With phpx.x -a and echo mktime(null, null, null, 1),' ',mktime(0, null, null, 1);
I have:

  • php7.2: 1673737200 1673737200
  • php7.3: 1673737200 1673737200
  • php7.4: 1673737200 1673737200
  • php8.0: 1673739363 1673739363
  • php8.1: 1673739364 1673739364
  • php8.2: 1673742965 1673742965

@elidrissidev elidrissidev merged commit 1762d83 into OpenMage:1.9.4.x Jan 15, 2023
@fballiano
Copy link
Contributor

cherrypicked to v20 with no conflict

@sreichel sreichel deleted the fix/php8.1/mktime-null branch January 16, 2023 09:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Adminhtml Relates to Mage_Adminhtml PHP 8 Related to PHP8
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants