5
100%
master: 100%

Ran 21 Dec 2016 09:24AM UTC

Files 1

Run time 0s

Badge

Embed ▾

Committed 21 Dec 2016 09:21AM UTC coverage: 100.0%. Remained the same

Job # 11.5

Build Type

Pull #3

travis-ci

Committed by

web-flow

Commit Message

move handler into middleware and save in a lexical

There was a change somewhere (probably in Catalyst) at some point that made $env in $c->req->env not be the same as the $env we have in the handler. I believe it gets copied or weakened somewhere, but I could not find it. If $c->req->env or $c->engine->env (which is just for back compatibility and there are comments saying it will go away) are used inside of process in a Catalyst::View:: class, the old approach using $env in the debug pannel works. But we cannot do that here, because it needs to be generic so it works with any View class, and Catalyst::View::process gets overwritten by subclasses, so we cannot add a modifier to it as it's never called.

This solution of adding the handler with a closure over a local variable inside of the actual middleware function is less performant as it gets called every time and not just when the class is loaded. But since we're in a development environment when this gets used that shouldn't be a big problem.

Fixes #1

Pull Request Pull Request #3: move handler into middleware and save in a lexical

Run Details

20 of 20 relevant lines covered (100.0%)

1.4 hits per line

markwellis / plack-middleware-debug-catalyststash / 11 / 5
100%
master: 100%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 11.5

markwellis / plack-middleware-debug-catalyststash / 11 / 5 100% master: 100%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 11.5

markwellis / plack-middleware-debug-catalyststash / 11 / 5
100%
master: 100%

README BADGES
x