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

Fix for admin shipment, creditmemo, invoice pdf emulation #4451

Conversation

ksz2013
Copy link

@ksz2013 ksz2013 commented May 10, 2016

No description provided.

@NadiyaS NadiyaS added the CS label May 11, 2016
@NadiyaS NadiyaS self-assigned this May 11, 2016
@NadiyaS
Copy link
Contributor

NadiyaS commented May 11, 2016

Hi @ksz2013 ,
Please follow Magento coding Standarts in your code:

  • property names SHOULD NOT be prefixed with a single underscore to indicate protected or private visibility.
  • All non-public properties and methods SHOULD be private
  • backward incompatible changes are not allowed:
    If you need to change constructor signature to inject new dependency within existing class, please make it in next way:
    • Introduce a private getter for the new dependency (getNewDependency() in the example below).
    • Within getNewDependency() method, use Magento\Framework\App\ObjectionManager::getInstance() to obtain the new dependency and initialize the member variable accordingly.
    • Use $this->getNewDependency() where ever that dependency is needed within the class
    • In unit tests use Reflection to make the member variable accessible and inject a mocked object
    • In the unit test class, as part of setUp(), inject the appropriate mock object for the new dependency ($newDependencyMock in the example below).
    • To discourage the use of the getter method else where outside the capacity of this specific fix, please mark it as @deprecated
    • E.g.

@aakimov
Copy link
Contributor

aakimov commented Aug 31, 2016

Hi @ksz2013,

Unfortunately we cannot accept your pull request as is.
It will be closed.

Thank you for your understanding and interest in Magento.

@aakimov aakimov closed this Aug 31, 2016
@gabrieldagama gabrieldagama added the Fixed in 2.4.x The issue has been fixed in 2.4-develop branch label Jan 23, 2021
@gabrieldagama gabrieldagama removed the Fixed in 2.4.x The issue has been fixed in 2.4-develop branch label Mar 25, 2021
@gabrieldagama
Copy link
Contributor

@gabrieldagama gabrieldagama added the Fixed in 2.4.x The issue has been fixed in 2.4-develop branch label Apr 11, 2021
@gabrieldagama
Copy link
Contributor

Hi @ksz2013. Thank you for your report.
The issue has been fixed in #31809 by @vlmed in 2.4-develop branch
Related commit(s):

The fix will be available with the upcoming 2.4.3 release.

@gabrieldagama
Copy link
Contributor

Hi @ksz2013. Thank you for your report.
The issue has been fixed in #31969 by @vlmed in 2.4-develop branch
Related commit(s):

The fix will be available with the upcoming 2.4.3 release.

@sidolov
Copy link
Contributor

sidolov commented Sep 23, 2021

Hi @ksz2013. Thank you for your report.
The issue has been fixed in #33124 by @vlmed in 2.4-develop branch
Related commit(s):

The fix will be available with the upcoming 2.4.3 release.

@sidolov
Copy link
Contributor

sidolov commented Oct 24, 2021

Hi @ksz2013. Thank you for your report.
The issue has been fixed in #31894 by @vlmed in 2.4-develop branch
Related commit(s):

The fix will be available with the upcoming 2.4.3 release.

@sidolov
Copy link
Contributor

sidolov commented Oct 24, 2021

Hi @ksz2013. Thank you for your report.
The issue has been fixed in #31774 by @vlmed in 2.4-develop branch
Related commit(s):

The fix will be available with the upcoming 2.4.3 release.

@sidolov
Copy link
Contributor

sidolov commented Oct 24, 2021

Hi @ksz2013. Thank you for your report.
The issue has been fixed in #31785 by @vlmed in 2.4-develop branch
Related commit(s):

The fix will be available with the upcoming 2.4.3 release.

@sidolov
Copy link
Contributor

sidolov commented Oct 24, 2021

Hi @ksz2013. Thank you for your report.
The issue has been fixed in #34171 by @ravi-chandra3197 in 2.4-develop branch
Related commit(s):

The fix will be available with the upcoming 2.4.3 release.

@sidolov
Copy link
Contributor

sidolov commented Oct 24, 2021

Hi @ksz2013. Thank you for your report.
The issue has been fixed in #31877 by @vlmed in 2.4-develop branch
Related commit(s):

The fix will be available with the upcoming 2.4.3 release.

@sidolov
Copy link
Contributor

sidolov commented Feb 3, 2022

Hi @ksz2013. Thank you for your report.
The issue has been fixed in #34620 by @ravi-chandra3197 in 2.4-develop branch
Related commit(s):

The fix will be available with the upcoming release.

This was referenced May 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix Component: Sales Fixed in 2.4.x The issue has been fixed in 2.4-develop branch Progress: reject
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants