Broken getHeaders #7

New Issue

admin · 2026-01-24T11:37:53+01:00

admin commented

2026-01-24 11:37:53 +01:00

Originally created by @marapper on GitHub (Sep 11, 2012).

After new commits overloaded ViewHandler was broken.

Notice: Trying to get property of non-object in /home/sergey/web/vr-as/vendor/bundles/FOS/RestBundle/View/View.php line 338

    #services.yml
    ironsoft_vras.jsonp_handler:
            class: %ironsoft_vras.jsonp_handler.class%
            arguments:
                container: "@serializer"
                logger: "@?logger"


    ironsoft_vras.vras_view_handler:
        parent: fos_rest.view_handler.default
        calls:
            - ['registerHandler', [ 'js', [@ironsoft_vras.jsonp_handler, 'createResponse'] ] ]

     #Controller.php
     $this->get('fos_rest.view_handler')->handle($view)

  #JsonpViewHandler
  class JsonpViewHandler {
     ...
    public function createResponse(ViewHandler $handler, View $view, Request $request, $format)
    {
       $content = $this->jsonEncode($view->getData());
       $code = $view->getStatusCode();
        ...
        return new Response($content, $code, $view->getHeaders());
       }

Originally created by @marapper on GitHub (Sep 11, 2012). After new commits overloaded ViewHandler was broken. Notice: Trying to get property of non-object in /home/sergey/web/vr-as/vendor/bundles/FOS/RestBundle/View/View.php line 338 ``` yaml #services.yml ironsoft_vras.jsonp_handler: class: %ironsoft_vras.jsonp_handler.class% arguments: container: "@serializer" logger: "@?logger" ironsoft_vras.vras_view_handler: parent: fos_rest.view_handler.default calls: - ['registerHandler', [ 'js', [@ironsoft_vras.jsonp_handler, 'createResponse'] ] ] ``` ``` php #Controller.php $this->get('fos_rest.view_handler')->handle($view) ``` ``` php #JsonpViewHandler class JsonpViewHandler { ... public function createResponse(ViewHandler $handler, View $view, Request $request, $format) { $content = $this->jsonEncode($view->getData()); $code = $view->getStatusCode(); ... return new Response($content, $code, $view->getHeaders()); } ```

admin closed this issue

2026-01-24 11:37:53 +01:00

admin commented

2026-01-24 11:37:53 +01:00

@lsmith77 commented on GitHub (Dec 19, 2012):

sorry ... i didnt see this ticket until now. is everything ok now?

@lsmith77 commented on GitHub (Dec 19, 2012): sorry ... i didnt see this ticket until now. is everything ok now?

admin commented

2026-01-24 11:37:53 +01:00

@marapper commented on GitHub (Jan 24, 2013):

Yes.

@marapper commented on GitHub (Jan 24, 2013): Yes.