New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Locale::Maketext failure_handler_auto problem #7767
Comments
From root@srv03.sparkit.noCreated by maybe@yes.noThis patch fixes a problem with Locale::Maketext Inline Patch--- Maketext.pm.orig Tue Aug 3 12:13:02 2004
+++ Maketext.pm Wed Sep 22 18:59:06 2004
@@ -126,7 +126,7 @@
# Meant to be used like:
# $handle->fail_with('failure_handler_auto')
- my($handle, $phrase, @params) = @_;
+ my($handle, $phrase) = splice(@_,0,2);
$handle->{'failure_lex'} ||= {};
my $lex = $handle->{'failure_lex'};
@@ -134,6 +134,9 @@
$lex->{$phrase} ||= ($value = $handle->_compile($phrase));
# Dumbly copied from sub maketext:
+ return $$value if ref($value) eq 'SCALAR';
+ return $value unless ref($value) eq 'CODE';
+
{
local $SIG{'__DIE__'};
eval { $value = &$value($handle, @_) };
@@ -173,6 +176,7 @@
Carp::croak "maketext requires at least one parameter" unless @_ > 1;
my($handle, $phrase) = splice(@_,0,2);
+Carp::confess("No handle/phrase") unless (defined($handle) && defined($phrase));
# Look up the value: Perl Info
|
From @schwernCalling Sean Burke. Come in, Sean. |
The RT System itself - Status changed from 'new' to 'open' |
From sburke@cpan.orgMichael G Schwern via RT wrote:
|
From sburke@cpan.orgMichael G Schwern via RT wrote:
|
From sburke@cpan.orgMichael G Schwern via RT wrote:
|
From @petdanceThis is in 1.11_01 of Locale::Maketext. |
@petdance - Status changed from 'open' to 'resolved' |
Migrated from rt.perl.org#33938 (status was 'resolved')
Searchable as RT33938$
The text was updated successfully, but these errors were encountered: