Skip Menu |
 

This queue is for tickets about the Safe-Isa CPAN distribution.

Report information
The Basics
Id: 91186
Status: rejected
Priority: 0/
Queue: Safe-Isa

People
Owner: Nobody in particular
Requestors: dagolden [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: should check defined Scalar::Util::blessed()
MIME-Version: 1.0
X-Mailer: MIME-tools 5.504 (Entity 5.504)
Content-Disposition: inline
X-RT-Interface: Web
Message-ID: <rt-4.0.18-10493-1386199989-987.0-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Encrypt: 0
X-RT-Sign: 0
Content-Length: 218
Download (untitled) / with headers
text/plain 218b
Because "0" is a possible class name, Safe::Isa should check for a defined value from blessed(). Or, if you don't want to support stupid class names like that, then you should at least document the limitation. David
From ether [...] cpan.org Wed Dec 4 18: 55:06 2013
X-Sa-Exim-Connect-Ip: 69.50.167.197
MIME-Version: 1.0
X-Spam-Status: No, score=-6.57 tagged_above=-99.9 required=10 tests=[AWL=0.330, BAYES_00=-1.9, RCVD_IN_DNSWL_HI=-5] autolearn=ham
In-Reply-To: <rt-4.0.18-10493-1386199989-1541.91186-4-0 [...] rt.cpan.org>
Content-Disposition: inline
X-Spam-Flag: NO
X-RT-Interface: API
References: <RT-Ticket-91186 [...] rt.cpan.org> <rt-4.0.18-10493-1386199989-1541.91186-4-0 [...] rt.cpan.org>
X-Acl-Warn: !authenticated = *
X-Virus-Scanned: Debian amavisd-new at bestpractical.com
Message-ID: <20131204235454.GC28631 [...] tequila>
content-type: text/plain; charset="utf-8"
X-Spam-Score-Int: 0
X-RT-Original-Encoding: utf-8
X-Spam-Score: -6.57
Received: from localhost (localhost [127.0.0.1]) by hipster.bestpractical.com (Postfix) with ESMTP id B8A3A2409B3 for <cpan-bug+Safe-Isa [...] hipster.bestpractical.com>; Wed, 4 Dec 2013 18:55:06 -0500 (EST)
Received: from hipster.bestpractical.com ([127.0.0.1]) by localhost (hipster.bestpractical.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 5e8ajZmrdtq7 for <cpan-bug+Safe-Isa [...] hipster.bestpractical.com>; Wed, 4 Dec 2013 18:55:05 -0500 (EST)
Received: from la.mx.develooper.com (x1.develooper.com [207.171.7.70]) by hipster.bestpractical.com (Postfix) with SMTP id 54660240379 for <bug-Safe-Isa [...] rt.cpan.org>; Wed, 4 Dec 2013 18:55:05 -0500 (EST)
Received: (qmail 23877 invoked by alias); 4 Dec 2013 23:55:04 -0000
Received: from ns2.lightspeed.ca (HELO ns2.lightspeed.ca) (206.12.82.4) by la.mx.develooper.com (qpsmtpd/0.28) with ESMTP; Wed, 04 Dec 2013 15:55:03 -0800
Received: from 69-50-167-197.westerncable.ca ([69.50.167.197] helo=etheridge.ca) by ns2.lightspeed.ca with esmtp (Exim 4.72) (envelope-from <ether [...] cpan.org>) id 1VoMHO-0002YX-T9 for bug-Safe-Isa [...] rt.cpan.org; Wed, 04 Dec 2013 15:54:55 -0800
Delivered-To: cpan-bug+Safe-Isa [...] hipster.bestpractical.com
Subject: Re: [rt.cpan.org #91186] should check defined Scalar::Util::blessed()
User-Agent: Mutt/1.5.21 (2010-09-15)
X-Sa-Exim-Mail-From: ether [...] cpan.org
Return-Path: <ether [...] cpan.org>
X-Spam-Check-BY: la.mx.develooper.com
X-Original-To: cpan-bug+Safe-Isa [...] hipster.bestpractical.com
X-RT-Mail-Extension: safe-isa
Date: Wed, 4 Dec 2013 15:54:54 -0800
X-Sa-Exim-Scanned: No (on ns2.lightspeed.ca); SAEximRunCond expanded to false
X-Spam-Level:
X-Spam-Bar: /
To: David Golden via RT <bug-Safe-Isa [...] rt.cpan.org>
From: Karen Etheridge <ether [...] cpan.org>
RT-Message-ID: <rt-4.0.18-10493-1386201307-1497.91186-0-0 [...] rt.cpan.org>
Content-Length: 302
Download (untitled) / with headers
text/plain 302b
On Wed, Dec 04, 2013 at 06:33:10PM -0500, David Golden via RT wrote: Show quoted text
> Because "0" is a possible class name, Safe::Isa should check for a defined value from blessed(). > Or, if you don't want to support stupid class names like that, then you should at least document the limitation.
Agreed. Patching.
MIME-Version: 1.0
In-Reply-To: <rt-4.0.18-10493-1386199989-987.0-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.504 (Entity 5.504)
Content-Disposition: inline
X-RT-Interface: Mobile
References: <rt-4.0.18-10493-1386199989-987.0-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="utf-8"
Message-ID: <rt-4.0.18-4887-1386207895-937.91186-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Encrypt: 0
X-RT-Sign: 0
Content-Length: 151
Download (untitled) / with headers
text/plain 151b
A number of modules, for example audrey's Scalar::Defer, rely on this. This is not a bug, it's an intentional feature. Safe::Isa will not be changed.
MIME-Version: 1.0
In-Reply-To: <rt-4.0.18-4887-1386207895-937.91186-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.504 (Entity 5.504)
Content-Disposition: inline
X-RT-Interface: Web
References: <rt-4.0.18-10493-1386199989-987.0-0-0 [...] rt.cpan.org> <rt-4.0.18-4887-1386207895-937.91186-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="utf-8"
Message-ID: <rt-4.0.18-3727-1386212374-545.91186-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Encrypt: 0
X-RT-Sign: 0
Content-Length: 221
Download (untitled) / with headers
text/plain 221b
On Wed Dec 04 20:44:55 2013, MSTROUT wrote: Show quoted text
> A number of modules, for example audrey's Scalar::Defer, rely on this. > > This is not a bug, it's an intentional feature.
Then please document it, as originally requested.


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.