Skip Menu |
 

This queue is for tickets about the JSON-Path CPAN distribution.

Report information
The Basics
Id: 122490
Status: resolved
Priority: 0/
Queue: JSON-Path

People
Owner: popefelix [...] gmail.com
Requestors: baldmantom [...] gmail.com
Cc:
AdminCc:

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



Subject: When passing a hashref to get(), it is modified if no match is found
Date: Fri, 14 Jul 2017 17:22:56 -0400
To: bug-JSON-Path [...] rt.cpan.org
From: Bald Man Tom <baldmantom [...] gmail.com>
Download (untitled) / with headers
text/plain 193b
Not sure where along the line the behavior changed. but as of v0.205, the reference was not modified. I'm attaching a script to show the problem. The first test fails, the other two pass.
Download json_path_bug.pl
text/x-perl 262b

Message body is not shown because sender requested not to inline it.

Download (untitled) / with headers
text/plain 256b
On Fri Jul 14 17:23:27 2017, baldmantom wrote: Show quoted text
> Not sure where along the line the behavior changed. but as of v0.205, > the reference was not modified. > > I'm attaching a script to show the problem. The first test fails, the > other two pass. > >
Download (untitled) / with headers
text/plain 379b
On Fri Jul 14 17:23:27 2017, baldmantom wrote: Show quoted text
> Not sure where along the line the behavior changed. but as of v0.205, > the reference was not modified. > > I'm attaching a script to show the problem. The first test fails, the > other two pass. > >
I know exactly why it changed. Now to figure out how to change it back and still keep the functionality that I added... :)
Download (untitled) / with headers
text/plain 472b
On Fri Jul 14 19:31:41 2017, popefelix wrote: Show quoted text
> On Fri Jul 14 17:23:27 2017, baldmantom wrote:
> > Not sure where along the line the behavior changed. but as of v0.205, > > the reference was not modified. > > > > I'm attaching a script to show the problem. The first test fails, the > > other two pass. > > > >
> > I know exactly why it changed. Now to figure out how to change it back > and still keep the functionality that I added... :)
Will be fixed in next release
Download (untitled) / with headers
text/plain 548b
On Mon Jul 17 12:35:15 2017, popefelix wrote: Show quoted text
> On Fri Jul 14 19:31:41 2017, popefelix wrote:
> > On Fri Jul 14 17:23:27 2017, baldmantom wrote:
> > > Not sure where along the line the behavior changed. but as of v0.205, > > > the reference was not modified. > > > > > > I'm attaching a script to show the problem. The first test fails, the > > > other two pass. > > > > > >
> > > > I know exactly why it changed. Now to figure out how to change it back > > and still keep the functionality that I added... :)
> > Will be fixed in next release


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.