Skip Menu |

This queue is for tickets about the HTML-Mason CPAN distribution.

Report information
The Basics
Id: 55159
Status: resolved
Priority: 0/
Queue: HTML-Mason
People
Owner: Nobody in particular
Requestors: pause [...] nodekit.org
Cc:
AdminCc:
Bug Information
Severity: Normal
Broken in: 1.44
Fixed in: (no value)

History Show all quoted text
  Tue Mar 02 19:32:39 2010 pause [...] nodekit.org - Ticket created
Subject: Undef warning w/anonymous components
Hi, The documentation for HTML::Mason::Component states that the value returned by "dir_path" is undefined for anonymous components. However, when using anonymous components, this can lead to an undef warning from HTML::Mason::Tools: Use of uninitialized value $dir_path in concatenation (.) or string at /usr/local/lib/perl5/site_perl/5.10.1/HTML/Mason/Tools.pm line 114. The line referenced above is a join() in the absolute_comp_path function. I suggest that Component.pm and Request.pm should check to see if the current component's comp_id is set, before calling absolute_comp_path. Thanks, --Alex
  Wed Mar 03 16:04:26 2010 pause [...] nodekit.org - Correspondence added
From: pause [...] nodekit.org
More info-- This issue occurs when I execute an anonymous component which calls other components (via <& &> or $m->comp inside the anon component). A simpler fix might be to have absolute_comp_path always stringify the second arg-- I did not observe any side effects or other weirdness by hacking absolute_comp_path to do ||= "" on $dir_path before trying to use it. Please don't hesitate to contact me if I can be of any help. --Alex
  Fri Feb 03 12:29:04 2012 DROLSKY [...] cpan.org - Correspondence added
On Wed Mar 03 16:04:26 2010, pause@nodekit.org wrote: Show quoted text
> More info-- This issue occurs when I execute an anonymous component > which calls other components (via <& &> or $m->comp inside the anon > component). > > A simpler fix might be to have absolute_comp_path always stringify the > second arg-- I did not observe any side effects or other weirdness by > hacking absolute_comp_path to do ||= "" on $dir_path before trying to > use it. > > Please don't hesitate to contact me if I can be of any help.
Sorry for the ridiculously slow reply. If you have code that can replicate this I can probably fix it.
  Fri Feb 03 12:29:05 2012 The RT System itself - Status changed from 'new' to 'open'
  Sat May 16 12:34:24 2020 DROLSKY [...] cpan.org - Correspondence added
Ticket migrated to GitHub as https://github.com/houseabsolute/HTML-Mason/issues/18
  Sat May 16 12:34:24 2020 DROLSKY [...] cpan.org - Status changed from 'open' to 'resolved'