Skip Menu |

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

Report information
The Basics
Id: 63079
Status: resolved
Priority: 0/
Queue: Config-IniFiles
People
Owner: Nobody in particular
Requestors: RHONDA [...] cpan.org
Cc: CARNIL [...] cpan.org
AdminCc:
Bug Information
Severity: Normal
Broken in: 2.39
Fixed in: (no value)

History Show all quoted text
  Wed Nov 17 06:41:15 2010 RHONDA [...] cpan.org - Ticket created
Hi! The module is able to read and write multiline values pretty well. Though, it seems to use hardwired the string EOT for the here documents it writes into the config file, and when a multiline string actually contains EOT on a single line the module breaks the file for itself to be able to read it back in. I would suggest rejecting to accept such a value. I see it that there is an explicit SetParameterEOT($section, $EOT) mentioned in the manpage to mitigate the effect of this, but the module really shouldn't break the file in the first place and rather reject such values. Also it might be helpful to set the marker not by section but actually even by section/value pair. That way one wouldn't have to scan through all the multiline values of a section to find a marker that wouldn't break the section. Thanks, Gerfried Fuchs
  Wed Nov 17 09:33:10 2010 CARNIL [...] cpan.org - Cc CARNIL added
  Fri Nov 19 10:07:22 2010 SHLOMIF [...] cpan.org - Correspondence added
I refactored the code into shape (with a little bug fixing). I'm not sure the bug exists in the current code on CPAN (not Config-IniFiles version 2.49 which is very old and buggier than the current release as it was submitted for), but now we can be better sure it's gone. Please test version 2.63 as it was submitted today and see if it still exists there.
  Fri Nov 19 10:07:23 2010 The RT System itself - Status changed from 'new' to 'open'
  Fri Nov 19 10:07:23 2010 SHLOMIF [...] cpan.org - Status changed from 'open' to 'resolved'