Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the DateTime CPAN distribution.

Report information
The Basics
Id: 117799
Status: resolved
Priority: 0/
Queue: DateTime
People
Owner: Nobody in particular
Requestors: nebulous [...] crashed.net
Cc:
AdminCc:
Bug Information
Severity: Wishlist
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Mon Sep 12 15:16:00 2016 nebulous [...] crashed.net - Ticket created
Subject: TO_JSON
Often I will subclass (or yes, sometimes monkeypatch like an idiot), DateTime in order to add a TO_JSON method: sub TO_JSON { shift->iso8601.'Z' } Would it be possible/advisable to add said method to DateTime itself? If so I'll be happy to submit a PR with tests.
  Mon Sep 12 15:18:26 2016 DROLSKY [...] cpan.org - Correspondence added
On 2016-09-12 14:16:00, NEBULOUS wrote: Show quoted text
> Often I will subclass (or yes, sometimes monkeypatch like an idiot), > DateTime in order to add a TO_JSON method: > > sub TO_JSON { shift->iso8601.'Z' } > > Would it be possible/advisable to add said method to DateTime itself? > If so I'll be happy to submit a PR with tests.
Yes, see https://github.com/houseabsolute/DateTime.pm/pull/12 for some context.
  Mon Sep 12 15:18:26 2016 The RT System itself - Status changed from 'new' to 'open'
  Sat Nov 12 12:53:06 2016 DROLSKY [...] cpan.org - Correspondence added
Ticket migrated to GitHub as https://github.com/houseabsolute/DateTime.pm/issues/35
  Sat Nov 12 12:53:12 2016 DROLSKY [...] cpan.org - Status changed from 'open' to 'resolved'