<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Nov 10, 2014 at 12:46 PM, Lennart Hengstmengel <span dir="ltr"><<a href="mailto:lennart@tevreden.nl" target="_blank">lennart@tevreden.nl</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Sawyer,</blockquote><div><br></div><div>Hey. :)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
<br>
On 08-11-14 12:15, Sawyer X wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Submit the PR and let's check it out.<br>
<br>
</blockquote>
<br></span>
Thanks for the useful feedback! I submitted a PR at <a href="https://github.com/PerlDancer/Dancer2/pull/760" target="_blank">https://github.com/PerlDancer/<u></u>Dancer2/pull/760</a></blockquote><div><br></div><div>Wonderful!</div><div><br></div><div>I've seen it but haven't had time to review it. I hope to do so today.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>There's one thing I encountered and I'm not sure whether I did this right. In the code there was: setting('charset'), which triggered an error (undefined function). So I changed it to $self->config->{'charset'} however $self->config appears to be an empty hashref. I expected it to contain the app config (with at least a key 'charset' as that is in the config.yml). Is this assumption correct? If not, what is the best way to get access to the app config / charset setting?<br></blockquote><div><br></div><div>Since the internals are... well, internals, they should not be calling the DSL. DSL is for end-users.</div><div><br></div><div>I'll take a look at why it's missing. Perhaps it's not available yet. I'll reply on the issue!</div><div><br></div><div>Thanks again for the pull request!</div><div><br></div><div>Sawyer.</div></div></div></div>