Skip to content
This repository has been archived by the owner on Apr 29, 2019. It is now read-only.

Eliminate usage of Zend_Filter from Magento 2 Open Source #81

Open
vkublytskyi opened this issue Feb 17, 2018 · 0 comments
Open

Eliminate usage of Zend_Filter from Magento 2 Open Source #81

vkublytskyi opened this issue Feb 17, 2018 · 0 comments
Labels
help wanted zf1-elimination Elimination of ZF1 usages

Comments

@vkublytskyi
Copy link

References count: 151
Used in EE/B2B: yes

Acceptance Criteria:

  1. Usage of Zend_Filter removed from Magento 2 Open Source
    • all references to Zend_Filter are removed from Magento 2 Open Source
    • added static a test that verifies that Zend_Filter is not used
    • Travis builds are green on PHP 7.1 environment
    • all backward incompatible changes are documented (if any)
    • if possible ZF1 component replaced by embedded PHP functionality or PSR interface otherwise migrated to ZF2
    • if ZF1 component replaced by ZF2 component:
      • there is an explanation why it is required to use external dependency
      • if usage of old ZF1 component was exposed in M2 interface/class signatures it is removed for new code
      • there is a list of PHP 7.2 compatibility issues of new ZF2 component
        OR
  2. There is an explanation why it is impossible to delete this dependency now:
    • described scope of work that is necessary to totally remove usage of Zend_Filter
    • described what backward incompatible changes will be caused by dependency removal
    • listed what other ZF1 components are required for Zend_Filter to support M2
      (if only limited functionality of Zend_Filter is used by M2
      then not all dependencies may be required and some part of Zend_Filter
      still may be removed if has incompatibility with PHP 7.2)
    • listed PHP 7.2 compatibility issues for Zend_Filter
@buskamuza buskamuza added help wanted zf1-elimination Elimination of ZF1 usages labels Mar 22, 2018
magento-team pushed a commit that referenced this issue Aug 26, 2018
MAGETWO-62891: New address is not marked as "Default Billing"
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
help wanted zf1-elimination Elimination of ZF1 usages
Projects
None yet
Development

No branches or pull requests

2 participants