Skip to content

Quick fix for pr from SWAT team #32

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

Merged
merged 5 commits into from
Jun 26, 2020
Merged

Quick fix for pr from SWAT team #32

merged 5 commits into from
Jun 26, 2020

Conversation

oshmyheliuk
Copy link
Contributor

@oshmyheliuk oshmyheliuk commented Jun 26, 2020

Description

Fix for the latest release.
https://jira.corp.magento.com/browse/MCLOUD-6156

Associated documentation updates

magento/devdocs#7462

Contribution checklist

  • Pull request has a meaningful description of its purpose
  • Pull request introduces user-facing changes and includes meaningful updates for any required release notes and documentation changes
  • All commits are accompanied by meaningful commit messages

@oshmyheliuk oshmyheliuk changed the base branch from develop to 1.0 June 26, 2020 16:46
@oshmyheliuk oshmyheliuk changed the base branch from 1.0 to develop June 26, 2020 16:46
@oshmyheliuk oshmyheliuk changed the base branch from develop to 1.0 June 26, 2020 16:47
@oshmyheliuk oshmyheliuk changed the base branch from 1.0 to develop June 26, 2020 16:50
shiftedreality
shiftedreality previously approved these changes Jun 26, 2020
@shiftedreality shiftedreality changed the base branch from develop to 1.0 June 26, 2020 16:52
@shiftedreality shiftedreality dismissed their stale review June 26, 2020 16:52

The base branch was changed.

bbatsche
bbatsche previously approved these changes Jun 26, 2020
@@ -20,7 +20,10 @@ class InvalidateLogger extends \Magento\Framework\Cache\InvalidateLogger
*/
public function execute($invalidateInfo)
{
$invalidateInfo['trace'] = debug_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS);
if (is_array($invalidateInfo)) {
$invalidateInfo['trace'] = debug_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS);
Copy link
Member

Choose a reason for hiding this comment

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

We're ok with not always having this info?

Copy link
Member

Choose a reason for hiding this comment

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

This should be is something to fix for SWAT team next time

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@Alkar778 are we ok with this for now?

Copy link
Contributor

Choose a reason for hiding this comment

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

This debug backtrace is the main goal of this improvement.
I will try to fix it in a couple of hours

shiftedreality
shiftedreality previously approved these changes Jun 26, 2020
Copy link
Member

@bbatsche bbatsche left a comment

Choose a reason for hiding this comment

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

There's a cleaner way to handle this I think, but it's not worth messing with at this time

@andriyShevtsov
Copy link
Contributor

QA approved

@andriyShevtsov andriyShevtsov added Progress: approved PR/issue status and removed Progress: testing in progress PR/issue status labels Jun 26, 2020
@shiftedreality shiftedreality merged commit e6d0917 into 1.0 Jun 26, 2020
@shiftedreality shiftedreality deleted the swat-pr-fix branch July 10, 2020 13:45
magento-devops-reposync-svc pushed a commit that referenced this pull request Apr 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants