I have two pull-requests
Just a reminder, that I have two pull requests sitting: #200 changes the way cookies are handled by the developer, according to the wishes of dagolden in one of his documents. It looks pretty important to me to deal with as soon as possible, since it will break backwards compatibility and so it's best to affect as few people as possible. #211 allows you to set the "views" directory somewhere else, in accordance with D2 documentation. This is also important, as it will let the front-end developer upload his stuff without affecting your (the back-end developer's) stuff. I guess there could be other ways to achieve this same benefit, but this looks like the most straightforward and easy-to-figure one. In the last of bigpresh's session of looking-at-pull-requests two weeks ago, this pull request was accidentally overlooked.
Those are D2 pr's On 04/12/2013 12:05 PM, Punter wrote:
Just a reminder, that I have two pull requests sitting:
#200 changes the way cookies are handled by the developer, according to the wishes of dagolden in one of his documents. It looks pretty important to me to deal with as soon as possible, since it will break backwards compatibility and so it's best to affect as few people as possible.
#211 allows you to set the "views" directory somewhere else, in accordance with D2 documentation. This is also important, as it will let the front-end developer upload his stuff without affecting your (the back-end developer's) stuff. I guess there could be other ways to achieve this same benefit, but this looks like the most straightforward and easy-to-figure one. In the last of bigpresh's session of looking-at-pull-requests two weeks ago, this pull request was accidentally overlooked.
I believe the D2 maintainers should come clean and tell us whether in the future cookies will be handled like this: my %stuff = cookie "stuff" ...or like they are now, which is (if I recall correctly) similar to this: my $cookie = cookie "stuff"; my %stuff = $cookie ? $cookie->value : (); Because if I'm making a website that has to work on D2 now, and I want it to keep on working in the future, I have to keep in the back of my mind that one day I will have to change my cookie-related code to keep it compatible with D2. The maintainers shouldn't be letting us in the dark. Below is the link to a pull request that changes (modernizes, is my opinion) the way cookies are handled. Does this pull request interest you enough so that I write some tests for it? https://github.com/PerlDancer/Dancer2/pull/200 On 04/12/2013 12:05 PM, Punter wrote:
Just a reminder, that I have two pull requests sitting:
#200 changes the way cookies are handled by the developer, according to the wishes of dagolden in one of his documents. It looks pretty important to me to deal with as soon as possible, since it will break backwards compatibility and so it's best to affect as few people as possible.
#211 allows you to set the "views" directory somewhere else, in accordance with D2 documentation. This is also important, as it will let the front-end developer upload his stuff without affecting your (the back-end developer's) stuff. I guess there could be other ways to achieve this same benefit, but this looks like the most straightforward and easy-to-figure one. In the last of bigpresh's session of looking-at-pull-requests two weeks ago, this pull request was accidentally overlooked.
My inspiration for this pull request was this supposed documentation that describes how the "cookie" keyword works: https://github.com/PerlDancer/Dancer2/blob/ea7bc77993277f34fcd27f3eb7b013668... On 04/19/2013 07:36 PM, Punter wrote:
I believe the D2 maintainers should come clean and tell us whether in the future cookies will be handled like this:
my %stuff = cookie "stuff"
...or like they are now, which is (if I recall correctly) similar to this:
my $cookie = cookie "stuff"; my %stuff = $cookie ? $cookie->value : ();
Because if I'm making a website that has to work on D2 now, and I want it to keep on working in the future, I have to keep in the back of my mind that one day I will have to change my cookie-related code to keep it compatible with D2. The maintainers shouldn't be letting us in the dark.
Below is the link to a pull request that changes (modernizes, is my opinion) the way cookies are handled. Does this pull request interest you enough so that I write some tests for it?
https://github.com/PerlDancer/Dancer2/pull/200
On 04/12/2013 12:05 PM, Punter wrote:
Just a reminder, that I have two pull requests sitting:
#200 changes the way cookies are handled by the developer, according to the wishes of dagolden in one of his documents. It looks pretty important to me to deal with as soon as possible, since it will break backwards compatibility and so it's best to affect as few people as possible.
#211 allows you to set the "views" directory somewhere else, in accordance with D2 documentation. This is also important, as it will let the front-end developer upload his stuff without affecting your (the back-end developer's) stuff. I guess there could be other ways to achieve this same benefit, but this looks like the most straightforward and easy-to-figure one. In the last of bigpresh's session of looking-at-pull-requests two weeks ago, this pull request was accidentally overlooked.
On Fri, 19 Apr 2013 19:39:02 +0300 Punter <punter@punter.gr> wrote:
My inspiration for this pull request was this supposed documentation that describes how the "cookie" keyword works: https://github.com/PerlDancer/Dancer2/blob/ea7bc77993277f34fcd27f3eb7b013668...
Yes, the manual still needs some work. You're looking at unfinished work on my branch. That's why it's not shipped with Dancer2 yet. Looking at the Dancer2 code, you'll get Dancer2::Core::Cookie objects back. That's the design, and I'm not aware of any plans for it to change. You can fetch a cookie with: my $cookie = cookie 'foo'; my $value = $cookie->value; One change that I think would be valuable actually would be to have Dancer2::Core::Cookie use overloading, so that a Dancer2::Core::Cookie object stringifies to its value - a bit of DWIMmery. I'll look in to that and discuss it with the rest of the team. Cheers Dave P -- David Precious ("bigpresh") <davidp@preshweb.co.uk> http://www.preshweb.co.uk/ www.preshweb.co.uk/twitter www.preshweb.co.uk/linkedin www.preshweb.co.uk/facebook www.preshweb.co.uk/cpan www.preshweb.co.uk/github
participants (2)
-
David Precious -
Punter