Skip Menu |

This queue is for tickets about the Config-General CPAN distribution.

Report information
The Basics
Id: 18425
Status: resolved
Priority: 0/
Queue: Config-General

People
Owner: tlinden [...] cpan.org
Requestors: justice8 [...] wanadoo.fr
Cc:
AdminCc:

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



Subject: Add an option to have only blocks' names case insensitive
Hi, I would like to have less contrainst on block names, so I had a little patch to convert block name to lowercase. I hope it will be helpfull to smoeone else. Regards.
Subject: General.pm.diff
--- General.pm 2006-03-29 10:43:12.002995824 +0200 +++ General_new.pm 2006-03-24 14:00:15.441276504 +0100 @@ -50,7 +50,7 @@ MergeDuplicateBlocks => 0, LowerCaseNames => 0, - + LowerCaseBlockNames => 0, UseApacheInclude => 0, IncludeRelative => 0, IncludeDirectories => 0, @@ -675,7 +675,7 @@ $blockname = $this->_interpolate("<$blockname>", $blockname); } } - $block = lc($block) if $this->{LowerCaseNames}; # only for blocks lc(), if configured via new() + $block = lc($block) if ($this->{LowerCaseNames} or $this->{LowerCaseBlockNames}); # only for blocks lc(), if configured via new() $this->{level} += 1; undef @newcontent; next; @@ -1269,6 +1269,13 @@ +=item B<-LowerCaseBlockNames> + +If set to a true value, then all block names found in the config will be converted +to lowercase. This allows you to provide less strict configs. + + + =item B<-UseApacheInclude> If set to a true value, the parser will consider "include ..." as valid include
Sorry for the late reply - RT didn't notify me of new tickets. I consider this a bug. I'll not apply the patch but instead fix the behavior for blocks using the existing option LowerCaseNames.
Ahm - sorry this is wrong. If I turn on -LowerCaseNames blocks will also be converted lowercase. There is no need for an extra option.