Exception are not translated to store scope #32953

rikwillems · 2021-05-05T19:15:57Z

#31351

Description (*)

Related Pull Requests

magento/inventory#3318

Fixed Issues (if relevant)

Fixes magento/magento2#<issue_number>

Manual testing scenarios (*)

...
...

Questions or comments

Contribution checklist (*)

Pull request has a meaningful description of its purpose
All commits are accompanied by meaningful commit messages
All new or changed code is covered with unit/integration tests (if applicable)
README.md files for modified modules are updated and included in the pull request if any README.md predefined sections require an update
All automated tests passed successfully (all builds are green)

magento#31351

m2-assistant · 2021-05-05T19:15:59Z

Hi @rikwillems. Thank you for your contribution
Here are some useful tips how you can test your changes using Magento test environment.
Add the comment under your pull request to deploy test or vanilla Magento instance:

@magento give me test instance - deploy test instance based on PR changes
@magento give me 2.4-develop instance - deploy vanilla Magento instance

❗ Automated tests can be triggered manually with an appropriate comment:

@magento run all tests - run or re-run all required tests against the PR changes
@magento run <test-build(s)> - run or re-run specific test build(s)
For example: @magento run Unit Tests

<test-build(s)> is a comma-separated list of build names. Allowed build names are:

Database Compare
Functional Tests CE
Functional Tests EE,
Functional Tests B2B
Integration Tests
Magento Health Index
Sample Data Tests CE
Sample Data Tests EE
Sample Data Tests B2B
Static Tests
Unit Tests
WebAPI Tests
Semantic Version Checker

You can find more information about the builds here

ℹ️ Please run only needed test builds instead of all when developing. Please run all test builds before sending your PR for review.

For more details, please, review the Magento Contributor Guide documentation.

⚠️ According to the Magento Contribution requirements, all Pull Requests must go through the Community Contributions Triage process. Community Contributions Triage is a public meeting.

🕙 You can find the schedule on the Magento Community Calendar page.

📞 The triage of Pull Requests happens in the queue order. If you want to speed up the delivery of your contribution, please join the Community Contributions Triage session to discuss the appropriate ticket.

🎥 You can find the recording of the previous Community Contributions Triage on the Magento Youtube Channel

✏️ Feel free to post questions/proposals/feedback related to the Community Contributions Triage process to the corresponding Slack Channel

Den4ik · 2021-05-05T21:26:28Z

@magento run all tests

magento-automated-testing · 2021-05-05T21:26:33Z

The requested builds are added to the queue. You should be able to see them here within a few minutes. Please re-request them if they don't show in a reasonable amount of time.

Den4ik · 2021-05-06T20:07:59Z

@magento run Functional Tests B2b, WebAPI Tests

magento-automated-testing · 2021-05-06T20:08:04Z

The requested builds are added to the queue. You should be able to see them here within a few minutes. Please re-request them if they don't show in a reasonable amount of time.

slavvka · 2021-05-07T00:57:56Z

@magento run Semantic Version Checker

magento-automated-testing · 2021-05-07T00:58:00Z

The requested builds are added to the queue. You should be able to see them here within a few minutes. Please re-request them if they don't show in a reasonable amount of time.

Den4ik

@rikwillems Thanks for your contribution.
Please look at my comments and update adding of new constructor params based on Backward Compatibility Policy

Den4ik · 2021-05-10T11:51:50Z

app/code/Magento/GraphQl/Controller/GraphQl.php

@@ -114,7 +121,8 @@ public function __construct(
        QueryFields $queryFields,
        JsonFactory $jsonFactory = null,
        HttpResponse $httpResponse = null,
-        ContextFactoryInterface $contextFactory = null
+        ContextFactoryInterface $contextFactory = null,
+        AreaList $areaList


Suggested change

AreaList $areaList

AreaList $areaList = null

Den4ik · 2021-05-10T11:52:31Z

app/code/Magento/GraphQl/Controller/GraphQl.php

@@ -127,6 +135,7 @@ public function __construct(
        $this->jsonFactory = $jsonFactory ?: ObjectManager::getInstance()->get(JsonFactory::class);
        $this->httpResponse = $httpResponse ?: ObjectManager::getInstance()->get(HttpResponse::class);
        $this->contextFactory = $contextFactory ?: ObjectManager::getInstance()->get(ContextFactoryInterface::class);
+        $this->areaList = $areaList;


Suggested change

$this->areaList = $areaList;

$this->areaList = $areaList ?: ObjectManager::getInstance()->get(AreaList::class);

Please see the updated PR.

Den4ik · 2021-05-10T20:08:17Z

@magento run all tests

magento-automated-testing · 2021-05-10T20:08:22Z

The requested builds are added to the queue. You should be able to see them here within a few minutes. Please re-request them if they don't show in a reasonable amount of time.

engcom-Alfa · 2021-12-24T10:56:57Z

There is a internal PR ticket raised on this same issue and that is being processed.
So, since this Pr ticket will become duplicate, closing this PR. In case of any changes, we will come back to this same ticket!
Thanks for everybody's collaboration and contribution on this!

hostep · 2022-02-04T08:33:51Z

Hi @engcom-Alfa or @cpartica: is there any news around this?

It's kind of annoying that this public issue got closed and replaced by a private issue that the community can not see into, it would be nice if you stopped doing this in the future, thanks!

cpartica · 2022-02-08T16:56:00Z

The effect is because of the process we have.
This PR will automatically be merged when 2.4.4 is released and merged back into 2.4-develop
We kept your contribution part of the actual fix.
If it works for you I can keep this open until the 2.4.4 event happens and you get in merged status with no action

Please don't update the branch or add any new commits or it will not automerge

m2-assistant · 2022-02-08T16:56:07Z

Hi @rikwillems. Thank you for your contribution
Here are some useful tips how you can test your changes using Magento test environment.
Add the comment under your pull request to deploy test or vanilla Magento instance:

@magento give me test instance - deploy test instance based on PR changes
@magento give me 2.4-develop instance - deploy vanilla Magento instance

❗ Automated tests can be triggered manually with an appropriate comment:

@magento run all tests - run or re-run all required tests against the PR changes
@magento run <test-build(s)> - run or re-run specific test build(s)
For example: @magento run Unit Tests

<test-build(s)> is a comma-separated list of build names. Allowed build names are:

Database Compare
Functional Tests CE
Functional Tests EE,
Functional Tests B2B
Integration Tests
Magento Health Index
Sample Data Tests CE
Sample Data Tests EE
Sample Data Tests B2B
Static Tests
Unit Tests
WebAPI Tests
Semantic Version Checker

You can find more information about the builds here

ℹ️ Run only required test builds during development. Run all test builds before sending your pull request for review.

For more details, review the Magento Contributor Guide documentation.

⚠️ According to the Magento Contribution requirements, all Pull Requests must go through the Community Contributions Triage process. Community Contributions Triage is a public meeting.

🕙 You can find the schedule on the Magento Community Calendar page.

📞 The triage of Pull Requests happens in the queue order. If you want to speed up the delivery of your contribution, join the Community Contributions Triage session to discuss the appropriate ticket.

✏️ Feel free to post questions/proposals/feedback related to the Community Contributions Triage process to the corresponding Slack Channel

Exception are not translated to store scope

1b4a3bc

magento#31351

m2-community-project bot added Priority: P1 Once P0 defects have been fixed, a defect having this priority is the next candidate for fixing. Severity: S1 Affects critical data or functionality and forces users to employ a workaround. labels May 5, 2021

magento-engcom-team added Component: GraphQL GraphQL Release Line: 2.4 labels May 5, 2021

m2-community-project bot added the Progress: pending review label May 5, 2021

Den4ik self-assigned this May 5, 2021

m2-community-project bot added Progress: review and removed Progress: pending review labels May 5, 2021

Den4ik requested changes May 10, 2021

View reviewed changes

m2-community-project bot added Progress: needs update and removed Progress: review labels May 10, 2021

Update per Backward Compatibility Policy

f096a71

m2-community-project bot added Progress: review and removed Progress: needs update labels May 10, 2021

m2-community-project bot added Progress: ready for testing and removed Progress: accept labels Dec 17, 2021

magento-engcom-team added Component: GraphQL GraphQL and removed Component: GraphQL GraphQL labels Dec 17, 2021

m2-community-project bot added Progress: accept and removed Progress: ready for testing labels Dec 17, 2021

magento-engcom-team added Component: GraphQL GraphQL and removed Component: GraphQL GraphQL labels Dec 17, 2021

m2-community-project bot added Progress: ready for testing and removed Progress: accept labels Dec 17, 2021

magento-engcom-team added Component: GraphQL GraphQL and removed Component: GraphQL GraphQL labels Dec 17, 2021

engcom-Alfa closed this Dec 24, 2021

m2-community-project bot removed the Progress: ready for testing label Dec 24, 2021

cpartica reopened this Feb 8, 2022

m2-community-project bot added the Progress: ready for testing label Feb 8, 2022

engcom-Alfa added the Progress: on hold label Feb 9, 2022

m2-community-project bot removed the Progress: ready for testing label Feb 9, 2022

magento-devops-reposync-svc merged commit 6bd2d4a into magento:2.4-develop Apr 12, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Exception are not translated to store scope #32953

Exception are not translated to store scope #32953

rikwillems commented May 5, 2021 •

edited by shikhamis11

Loading

m2-assistant bot commented May 5, 2021

Den4ik commented May 5, 2021

magento-automated-testing bot commented May 5, 2021

Den4ik commented May 6, 2021

magento-automated-testing bot commented May 6, 2021

slavvka commented May 7, 2021

magento-automated-testing bot commented May 7, 2021

Den4ik left a comment

Den4ik May 10, 2021

Den4ik May 10, 2021

rikwillems May 10, 2021

Den4ik commented May 10, 2021

magento-automated-testing bot commented May 10, 2021

engcom-Alfa commented Dec 24, 2021

hostep commented Feb 4, 2022

cpartica commented Feb 8, 2022 •

edited

Loading

m2-assistant bot commented Feb 8, 2022

	$this->areaList = $areaList;
	$this->areaList = $areaList ?: ObjectManager::getInstance()->get(AreaList::class);

Exception are not translated to store scope #32953

Exception are not translated to store scope #32953

Conversation

rikwillems commented May 5, 2021 • edited by shikhamis11 Loading

Description (*)

Related Pull Requests

Fixed Issues (if relevant)

Manual testing scenarios (*)

Questions or comments

Contribution checklist (*)

m2-assistant bot commented May 5, 2021

Den4ik commented May 5, 2021

magento-automated-testing bot commented May 5, 2021

Den4ik commented May 6, 2021

magento-automated-testing bot commented May 6, 2021

slavvka commented May 7, 2021

magento-automated-testing bot commented May 7, 2021

Den4ik left a comment

Choose a reason for hiding this comment

Den4ik May 10, 2021

Choose a reason for hiding this comment

Den4ik May 10, 2021

Choose a reason for hiding this comment

rikwillems May 10, 2021

Choose a reason for hiding this comment

Den4ik commented May 10, 2021

magento-automated-testing bot commented May 10, 2021

engcom-Alfa commented Dec 24, 2021

hostep commented Feb 4, 2022

cpartica commented Feb 8, 2022 • edited Loading

m2-assistant bot commented Feb 8, 2022

rikwillems commented May 5, 2021 •

edited by shikhamis11

Loading

cpartica commented Feb 8, 2022 •

edited

Loading