magento / magento2

Prior to making any Submission(s), you must sign an Adobe Contributor License Agreement, available here at: https://opensource.adobe.com/cla.html. All Submissions you make to Adobe Inc. and its affiliates, assigns and subsidiaries (collectively “Adobe”) are subject to the terms of the Adobe Contributor License Agreement.
http://www.magento.com
Open Software License 3.0
11.57k stars 9.32k forks source link

[Developer Experience] PHPDoc and Typo #39376

Open wubinworks opened 1 week ago

wubinworks commented 1 week ago

Summary

Confirmed on Magento 2.4.x

Issue 1:

PHPDoc

Issue 2:

Typo

See Examples.

Examples

Links are for Branch: 2.4.8-beta1

Issue 1:

\Magento\Framework\App\ScopeResolver#L13

/**
 * @var \Magento\Store\Model\StoreManagerInterface
 */
protected $objectManager;

should be

/**
 * @var ObjectManagerInterface
 */
protected $objectManager;

Issue 2:

\Magento\LoginAsCustomer\Model\GetLoggedAsCustomerAdminId#L38

return (int)$this->session->getLoggedAsCustomerAdmindId();
                                                    ^ extra d

\Magento\LoginAsCustomer\Model\SetLoggedAsCustomerAdminId#L38

$this->session->setLoggedAsCustomerAdmindId($adminId);
                                        ^ extra d

Proposed solution

See above.

Release note

No response

Triage and priority

m2-assistant[bot] commented 1 week ago

Hi @wubinworks. Thank you for your report. To speed up processing of this issue, make sure that the issue is reproducible on the vanilla Magento instance following Steps to reproduce.

m2-assistant[bot] commented 1 week ago

Hi @engcom-Hotel. Thank you for working on this issue. In order to make sure that issue has enough information and ready for development, please read and check the following instruction: :point_down:

engcom-Hotel commented 1 week ago

Hello @wubinworks,

Thanks for the report and collaboration!

We have reviewed the mentioned code in the main description, and the issue is reproducible for us.

Hence confirming this issue for further processing.

Thanks

github-jira-sync-bot commented 1 week ago

:white_check_mark: Jira issue https://jira.corp.adobe.com/browse/AC-13365 is successfully created for this GitHub issue.

m2-assistant[bot] commented 1 week ago

:white_check_mark: Confirmed by @engcom-Hotel. Thank you for verifying the issue.
Issue Available: @engcom-Hotel, You will be automatically unassigned. Contributors/Maintainers can claim this issue to continue. To reclaim and continue work, reassign the ticket to yourself.

srsathish92 commented 1 week ago

@magento I am working on this