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

MAGE_PROFILER = 'firebug' error #7068

Closed
webmak opened this issue Oct 18, 2016 · 6 comments
Closed

MAGE_PROFILER = 'firebug' error #7068

webmak opened this issue Oct 18, 2016 · 6 comments
Labels
bug report Issue: Format is valid Gate 1 Passed. Automatic verification of issue format passed Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development

Comments

@webmak
Copy link

webmak commented Oct 18, 2016

Preconditions

  1. php 7.0.9, php-fpm, apache
  2. Magento CE 2.1.2 the same was in Magento CE2.1.1 not tested before

Steps to reproduce

  1. turn on profile env MAGE_PROFILER = 'firebug';
  2. reload php-fpm
  3. reload site page

Expected result

Profiling info

Actual result

Uncaught TypeError: Argument 1 passed to Zend_Controller_Plugin_Abstract::setResponse() must be an instance of Zend_Controller_Response_Abstract, instance of Magento\Framework\HTTP\PhpEnvironment\Response given, called in /home/user/public_html/some.site/www/vendor/magento/framework/Profiler/Driver/Standard/Output/Firebug.php on line 61 and defined in /home/user/public_html/some.site/www/vendor/magento/zendframework1/library/Zend/Controller/Plugin/Abstract.php:70 Stack trace: #0 /home/user/public_html/some.site/www/vendor/magento/framework/Profiler/Driver/Standard/Output/Firebug.php(61): Zend_Controller_Plugin_Abstract->setResponse(Object(Magento\Framework\HTTP\PhpEnvironment\Response)) #1 /home/user/public_html/some.site/www/vendor/magento/framework/Profiler/Driver/Standard.php(195): Magento\Framework\Profiler\Driver\Standard\Output\Firebug->display(Object(Magento\Framework\Profiler\Driver\Standard\Stat)) #2 [internal function]: Magento\Framework\Profiler\Driver\Standard->display() #3 {main} thrown in /home/user/public_html/some.site/www/vendor/magento/zendframework1/library/Zend/Controller/Plugin/Abstract.php on line 70

After checking

Was detected that passing object has this parents types

array(4) { ["Zend\Http\PhpEnvironment\Response"]=> string(33) "Zend\Http\PhpEnvironment\Response" ["Zend\Http\Response"]=> string(18) "Zend\Http\Response" ["Zend\Http\AbstractMessage"]=> string(25) "Zend\Http\AbstractMessage" ["Zend\Stdlib\Message"]=> string(19) "Zend\Stdlib\Message" }

But method aspect other type
setResponse(Zend_Controller_Response_Abstract $response) { $this->_response = $response; return $this; }

@veloraven
Copy link
Contributor

@webmak thank you for your feedback.
Please, format this issue according to the Issue reporting guidelines: with steps to reproduce, actual result and expected result.

@webmak
Copy link
Author

webmak commented Oct 19, 2016

Have update issue

@sevos1984
Copy link
Contributor

Thanks for reporting, internal ticket created MAGETWO-60429

@sevos1984 sevos1984 added the Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development label Nov 2, 2016
@markoweb
Copy link

The same problem

sergiirepin added a commit to sergiirepin/magento2 that referenced this issue Apr 2, 2017
@orlangur
Copy link
Contributor

Due to Zend_Wildfire deprecation this issue was fixed in develop via corresponding profiler mode removal in #9622. It no more works in latest Firefox thus I don't think it's worth fixing in 2.1.x.

Ping @veloraven @sevos1984

@magento-engcom-team magento-engcom-team added 2.1.x Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development bug report Issue: Format is not valid Gate 1 Failed. Automatic verification of issue format is failed labels Sep 11, 2017
@magento-engcom-team
Copy link
Contributor

@webmak Closing this issue as this deprecated feature is removed since 2.2 (see PR #9622 for more information)

@magento-engcom-team magento-engcom-team added Issue: Format is valid Gate 1 Passed. Automatic verification of issue format passed and removed Issue: Format is not valid Gate 1 Failed. Automatic verification of issue format is failed labels Sep 21, 2017
magento-engcom-team pushed a commit that referenced this issue Sep 4, 2021
…date

[Lynx] Move static tests to magento-coding-standard
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug report Issue: Format is valid Gate 1 Passed. Automatic verification of issue format passed Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development
Projects
None yet
Development

No branches or pull requests

6 participants