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>
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.

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

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. > >
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... :)
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
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