line |
stmt |
bran |
cond |
sub |
pod |
time |
code |
1
|
|
|
|
|
|
|
package Perl::Critic::Policy::BuiltinFunctions::ProhibitUniversalCan; |
2
|
|
|
|
|
|
|
|
3
|
40
|
|
|
40
|
|
26553
|
use 5.010001; |
|
40
|
|
|
|
|
210
|
|
4
|
40
|
|
|
40
|
|
290
|
use strict; |
|
40
|
|
|
|
|
129
|
|
|
40
|
|
|
|
|
898
|
|
5
|
40
|
|
|
40
|
|
259
|
use warnings; |
|
40
|
|
|
|
|
170
|
|
|
40
|
|
|
|
|
981
|
|
6
|
40
|
|
|
40
|
|
269
|
use Readonly; |
|
40
|
|
|
|
|
133
|
|
|
40
|
|
|
|
|
2228
|
|
7
|
|
|
|
|
|
|
|
8
|
40
|
|
|
40
|
|
325
|
use Perl::Critic::Utils qw{ :severities :classification hashify }; |
|
40
|
|
|
|
|
166
|
|
|
40
|
|
|
|
|
2121
|
|
9
|
40
|
|
|
40
|
|
14322
|
use parent 'Perl::Critic::Policy'; |
|
40
|
|
|
|
|
145
|
|
|
40
|
|
|
|
|
316
|
|
10
|
|
|
|
|
|
|
|
11
|
|
|
|
|
|
|
our $VERSION = '1.150'; |
12
|
|
|
|
|
|
|
|
13
|
|
|
|
|
|
|
#----------------------------------------------------------------------------- |
14
|
|
|
|
|
|
|
|
15
|
|
|
|
|
|
|
Readonly::Scalar my $DESC => q{UNIVERSAL::can should not be used as a function}; |
16
|
|
|
|
|
|
|
Readonly::Scalar my $EXPL => q{Use eval{$obj->can($pkg)} instead}; ## no critic (RequireInterp); |
17
|
|
|
|
|
|
|
|
18
|
|
|
|
|
|
|
#----------------------------------------------------------------------------- |
19
|
|
|
|
|
|
|
|
20
|
89
|
|
|
89
|
0
|
1617
|
sub supported_parameters { return () } |
21
|
74
|
|
|
74
|
1
|
310
|
sub default_severity { return $SEVERITY_MEDIUM } |
22
|
74
|
|
|
74
|
1
|
317
|
sub default_themes { return qw( core maintenance certrule ) } |
23
|
30
|
|
|
30
|
1
|
96
|
sub applies_to { return 'PPI::Token::Word' } |
24
|
|
|
|
|
|
|
|
25
|
|
|
|
|
|
|
#----------------------------------------------------------------------------- |
26
|
|
|
|
|
|
|
|
27
|
|
|
|
|
|
|
sub violates { |
28
|
329
|
|
|
329
|
1
|
519
|
my ( $self, $elem, undef ) = @_; |
29
|
|
|
|
|
|
|
|
30
|
329
|
|
|
|
|
419
|
state $isa_can = { hashify( qw( can UNIVERSAL::can ) ) }; |
31
|
329
|
50
|
|
|
|
563
|
return if !$isa_can->{$elem}; |
32
|
0
|
0
|
|
|
|
|
return if ! is_function_call($elem); # this also permits 'use UNIVERSAL::can;' |
33
|
|
|
|
|
|
|
|
34
|
0
|
|
|
|
|
|
return $self->violation( $DESC, $EXPL, $elem ); |
35
|
|
|
|
|
|
|
} |
36
|
|
|
|
|
|
|
|
37
|
|
|
|
|
|
|
|
38
|
|
|
|
|
|
|
1; |
39
|
|
|
|
|
|
|
|
40
|
|
|
|
|
|
|
__END__ |
41
|
|
|
|
|
|
|
|
42
|
|
|
|
|
|
|
#----------------------------------------------------------------------------- |
43
|
|
|
|
|
|
|
|
44
|
|
|
|
|
|
|
=pod |
45
|
|
|
|
|
|
|
|
46
|
|
|
|
|
|
|
=head1 NAME |
47
|
|
|
|
|
|
|
|
48
|
|
|
|
|
|
|
Perl::Critic::Policy::BuiltinFunctions::ProhibitUniversalCan - Write C<< eval { $foo->can($name) } >> instead of C<UNIVERSAL::can($foo, $name)>. |
49
|
|
|
|
|
|
|
|
50
|
|
|
|
|
|
|
|
51
|
|
|
|
|
|
|
=head1 AFFILIATION |
52
|
|
|
|
|
|
|
|
53
|
|
|
|
|
|
|
This Policy is part of the core L<Perl::Critic|Perl::Critic> |
54
|
|
|
|
|
|
|
distribution. |
55
|
|
|
|
|
|
|
|
56
|
|
|
|
|
|
|
|
57
|
|
|
|
|
|
|
=head1 DESCRIPTION |
58
|
|
|
|
|
|
|
|
59
|
|
|
|
|
|
|
print UNIVERSAL::can($obj, 'Foo::Bar') ? 'yes' : 'no'; #not ok |
60
|
|
|
|
|
|
|
print eval { $obj->can('Foo::Bar') } ? 'yes' : 'no'; #ok |
61
|
|
|
|
|
|
|
|
62
|
|
|
|
|
|
|
As of Perl 5.9.3, the use of UNIVERSAL::can as a function has been |
63
|
|
|
|
|
|
|
deprecated and the method form is preferred instead. Formerly, the |
64
|
|
|
|
|
|
|
functional form was recommended because it gave valid results even |
65
|
|
|
|
|
|
|
when the object was C<undef> or an unblessed scalar. However, the |
66
|
|
|
|
|
|
|
functional form makes it impossible for packages to override C<can()>, |
67
|
|
|
|
|
|
|
a technique which is crucial for implementing mock objects and some |
68
|
|
|
|
|
|
|
facades. |
69
|
|
|
|
|
|
|
|
70
|
|
|
|
|
|
|
See L<UNIVERSAL::can|UNIVERSAL::can> for a more thorough discussion of |
71
|
|
|
|
|
|
|
this topic. |
72
|
|
|
|
|
|
|
|
73
|
|
|
|
|
|
|
|
74
|
|
|
|
|
|
|
=head1 CONFIGURATION |
75
|
|
|
|
|
|
|
|
76
|
|
|
|
|
|
|
This Policy is not configurable except for the standard options. |
77
|
|
|
|
|
|
|
|
78
|
|
|
|
|
|
|
|
79
|
|
|
|
|
|
|
=head1 SEE ALSO |
80
|
|
|
|
|
|
|
|
81
|
|
|
|
|
|
|
L<Perl::Critic::Policy::BuiltinFunctions::ProhibitUniversalIsa|Perl::Critic::Policy::BuiltinFunctions::ProhibitUniversalIsa> |
82
|
|
|
|
|
|
|
|
83
|
|
|
|
|
|
|
|
84
|
|
|
|
|
|
|
=head1 AUTHOR |
85
|
|
|
|
|
|
|
|
86
|
|
|
|
|
|
|
Chris Dolan <cdolan@cpan.org> |
87
|
|
|
|
|
|
|
|
88
|
|
|
|
|
|
|
|
89
|
|
|
|
|
|
|
=head1 COPYRIGHT |
90
|
|
|
|
|
|
|
|
91
|
|
|
|
|
|
|
Copyright (c) 2006-2023 Chris Dolan. |
92
|
|
|
|
|
|
|
|
93
|
|
|
|
|
|
|
This program is free software; you can redistribute it and/or modify |
94
|
|
|
|
|
|
|
it under the same terms as Perl itself. The full text of this license |
95
|
|
|
|
|
|
|
can be found in the LICENSE file included with this module. |
96
|
|
|
|
|
|
|
|
97
|
|
|
|
|
|
|
=cut |
98
|
|
|
|
|
|
|
|
99
|
|
|
|
|
|
|
# Local Variables: |
100
|
|
|
|
|
|
|
# mode: cperl |
101
|
|
|
|
|
|
|
# cperl-indent-level: 4 |
102
|
|
|
|
|
|
|
# fill-column: 78 |
103
|
|
|
|
|
|
|
# indent-tabs-mode: nil |
104
|
|
|
|
|
|
|
# c-indentation-style: bsd |
105
|
|
|
|
|
|
|
# End: |
106
|
|
|
|
|
|
|
# ex: set ts=8 sts=4 sw=4 tw=78 ft=perl expandtab shiftround : |