Skip Menu |

This queue is for tickets about the Data-NDS CPAN distribution.

Report information
The Basics
Id: 34759
Status: resolved
Priority: 0/
Queue: Data-NDS
People
Owner: Nobody in particular
Requestors: ahindmar [...] office.caiw.nl
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Tue Apr 08 05:53:14 2008 ahindmar [...] office.caiw.nl - Ticket created
Subject: merge() or merge_path() don't accept $new parameter
Date: Tue, 08 Apr 2008 10:52:44 +0100
To: bug-Data-NDS [...] rt.cpan.org
From: Ashley Hindmarsh <ahindmar [...] office.caiw.nl>
Useful-looking module, but the docs could do with a few more worked examples. I had a problem calling merge() with $new=1. Ash. --- NDS.pm Wed Feb 27 21:39:32 2008 +++ NDS2.pm Tue Apr 8 11:43:14 2008 @@ -392,7 +392,7 @@ } elsif ($#args == 1) { $ruleset = $args[0]; - $ruleset = $args[1]; + $new = $args[1]; } else { die "[merge] Unknown argument list"; @@ -1406,7 +1406,7 @@ } elsif ($#args == 1) { $ruleset = $args[0]; - $ruleset = $args[1]; + $new = $args[1]; } else {
  Tue Apr 08 07:23:31 2008 sbeck [...] cpan.org - Correspondence added
Thanks for the bug report. I've applied your patch and it'll be in the next release. I'll get some examples in the docs too... I agree that the docs are a little sparse, so you can expect them to grow significantly with future releases.
  Tue Apr 08 07:23:34 2008 The RT System itself - Status changed from 'new' to 'open'
  Tue Apr 08 07:23:36 2008 sbeck [...] cpan.org - Status changed from 'open' to 'resolved'