Skip to content

[Forwardport] CSS load order incorrect using default_head_blocks.xml #1821 #17053

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

Closed
wants to merge 2 commits into from
Closed

[Forwardport] CSS load order incorrect using default_head_blocks.xml #1821 #17053

wants to merge 2 commits into from

Conversation

torhoehn
Copy link
Contributor

Original Pull Request

#14290
Added new attribute 'order' for set loading order .
Those attribute resolve issue about render files for some order.

Fixed Issues (if relevant)

  1. CSS load order incorrect using default_head_blocks.xml #1821: CSS load order incorrect using default_head_blocks.xml CSS load order incorrect using default_head_blocks.xml #1821

Added new attribute 'order' for set loading order .
Those attribute resolve issue about render files for some order.
@magento-engcom-team
Copy link
Contributor

Hi @torhoehn. Thank you for your contribution
Here is 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-engcom-team give me test instance - deploy test instance based on PR changes
  • @magento-engcom-team give me {$VERSION} instance - deploy vanilla Magento instance

For more details, please, review the Magento Contributor Assistant documentation

@sidolov
Copy link
Contributor

sidolov commented Jul 24, 2018

Hi @torhoehn , please, take a look at failed test

@sidolov
Copy link
Contributor

sidolov commented Aug 9, 2018

Hi @torhoehn , tests still failing, will continue progress with this PR?

@torhoehn
Copy link
Contributor Author

torhoehn commented Aug 9, 2018

@sidolov Sorry, I don't know exactly what to fix here.

PHP Code Mess has found error(s):
/home/travis/build/magento/magento2/lib/internal/Magento/Framework/View/Page/Config/Reader/Head.php:69	The method interpret() has a Cyclomatic Complexity of 12. The configured cyclomatic complexity threshold is 10.
/home/travis/build/magento/magento2/lib/internal/Magento/Framework/View/Test/Unit/Page/Config/Generator/HeadTest.php:63	The method testProcess() has 114 lines of code. Current threshold is set to 100. Avoid really long methods.

I think complexity and lines of code in a method has been the same in 2.2. Proof me wrong.

@sidolov
Copy link
Contributor

sidolov commented Aug 10, 2018

@torhoehn you can suppress warning regarding excessive method length in the test but it would be better refactor it.
Cyclomatic complexity should be refactored in the current class by extracting logic to separate methods.

@sidolov
Copy link
Contributor

sidolov commented Aug 28, 2018

Hi @torhoehn , I am closing this PR now due to inactivity.
Please reopen and update if you wish to continue.
Thank you for the collaboration!

@sidolov sidolov closed this Aug 28, 2018
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.

4 participants