Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Perl-Critic CPAN distribution.

Report information
The Basics
Id: 18698
Status: resolved
Priority: 0/
Queue: Perl-Critic
People
Owner: Nobody in particular
Requestors: adamk [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Wishlist
Broken in: (no value)
Fixed in: 0.15_01

History Show all quoted text
  Thu Apr 13 08:56:40 2006 Guest - Ticket created
Subject: Policy idea ProhibitUniversalFunctions
Since you asked for more ideas... Using UNIVERSAL::isa or UNIVERSAL::can directly is considered bad. As is C< use UNIVERSAL 'isa' or use UNIVERSAL qw{ can } > Actually, I think that second case is being deprecated. In short, they break OO and encapsulation. As alternatives, call as a method once you know they are objects, or for the lazy, see Params::Util::_INSTANCE for the normal object->isa case. (I don't have a good can replacement yet)
  Thu Apr 13 10:21:59 2006 Guest - Correspondence added
From: cdolan [...] cpan.org
I'll take this one. -- Chris On Thu Apr 13 08:56:40 2006, guest wrote: Show quoted text
> Since you asked for more ideas... > > Using UNIVERSAL::isa or UNIVERSAL::can directly is considered bad. > > As is C< use UNIVERSAL 'isa' or use UNIVERSAL qw{ can } > > > Actually, I think that second case is being deprecated. > > In short, they break OO and encapsulation. > > As alternatives, call as a method once you know they are objects, or for > the lazy, see Params::Util::_INSTANCE for the normal object->isa case. > > (I don't have a good can replacement yet)
  Thu Apr 13 10:22:00 2006 The RT System itself - Status changed from 'new' to 'open'
  Thu Apr 13 12:36:47 2006 Guest - Correspondence added
From: cdolan [...] cpan.org
Resolved via SVN revision #361, viewable at the URL below. This fix will be present in the forthcoming v0.15_01. If you have any recommendations for improvements to the documentation of those policies, please pass them along. http://perlcritic.tigris.org/servlets/BrowseList?listName=commits -- Chris
  Thu Apr 20 09:43:25 2006 thaljef [...] cpan.org - Status changed from 'open' to 'resolved'
  Thu Apr 20 09:43:26 2006 thaljef [...] cpan.org - Fixed in 0.15_01 added