Skip Menu |

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

Report information
The Basics
Id: 123512
Status: open
Priority: 0/
Queue: Config-General
People
Owner: tlinden [...] cpan.org
Requestors: marcin [...] sacha.pl
Cc:
AdminCc:
Bug Information
Severity: Wishlist
Broken in: (no value)
Fixed in: 2.63

History Show all quoted text
  Sat Nov 04 12:23:35 2017 marcin [...] sacha.pl - Ticket created
Subject: bug: ApacheCompatible inconsistency
Date: Sat, 4 Nov 2017 16:54:11 +0100
To: <bug-Config-General [...] rt.cpan.org>
From: "Marcin Sacha" <marcin [...] sacha.pl>
Hello, I'm sending a bug report in the Config::General module, version 2.63, regarding the ApacheCompatible option. Turning this option on does two things: 1. Turns on 8 other options (works as global switch). 2. Turns off support for explicit empty blocks. Turning this option off does one thing: 1. Turns on support for explicit empty blocks. Another words - there is no possibility to turn off support for explicit empty blocks and keep mentioned 8 options turned off either. It is an obvious inconsistency and significantly reduces the functionality of the module. I think it should be fixed by adding an extra option regulating the support for explicit empty blocks. This option should be on by default for backward compatibility. When off, support for empty blocks would be off too, but without touching other options. And by the way - I suggest, to also add an option regulating the quotation. Currently, there's no possibility to turn it off, which is a bit annoying. Best regards, Marcin
  Sun Nov 05 11:52:20 2017 tlinden [...] cpan.org - Correspondence added
Show quoted text
> Turning this option on does two things: > 1. Turns on 8 other options (works as global switch). > 2. Turns off support for explicit empty blocks.
Indeed, as described in the documentation. Show quoted text
> Turning this option off does one thing: > 1. Turns on support for explicit empty blocks.
No, there is no "turn on explicit block" option. They are supported as long as ApacheCompatible is not enabled. Show quoted text
> Another words - there is no possibility to turn off support for explicit > empty blocks and keep mentioned 8 options turned off either.
That is correct. Show quoted text
> It is an obvious inconsistency
It's not and it's documented as such. Show quoted text
> I think it should be fixed by adding an extra option > regulating the support for explicit empty blocks. This option should be on > by default for backward compatibility. When off, support for empty blocks > would be off too, but without touching other options.
Would be an idea. Show quoted text
> And by the way - I suggest, to also add an option regulating the quotation. > Currently, there's no possibility to turn it off, which is a bit annoying.
Please open an extra case for this and provide more details and sample code. - Tom
  Sun Nov 05 11:52:21 2017 The RT System itself - Status changed from 'new' to 'open'
  Sun Nov 05 11:52:21 2017 tlinden [...] cpan.org - Taken
  Sun Nov 05 11:52:21 2017 tlinden [...] cpan.org - Fixed in 2.63 added
  Sun Nov 05 11:52:52 2017 tlinden [...] cpan.org - Severity Wishlist added