Skip Menu |

This queue is for tickets about the Dancer-Plugin-Database-Core CPAN distribution.

Report information
The Basics
Id: 104620
Status: resolved
Priority: 0/
Queue: Dancer-Plugin-Database-Core
People
Owner: Nobody in particular
Requestors: nickm [...] positive-internet.com
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Fri May 22 06:35:16 2015 nickm [...] positive-internet.com - Ticket created
Subject: BUG in Dancer::Plugin::Database::Core::Handle quick_update jumbles params
Date: Fri, 22 May 2015 11:34:58 +0100
To: bug-Dancer-Plugin-Database-Core [...] rt.cpan.org, davidp [...] preshweb.co.uk
From: Nick Mailer <nickm [...] positive-internet.com>
Hi, Just to let you know that the quick_update convenience method in Dancer::Plugin::Database::Core::Handle randomly jumbles the params and the associated values, so that query does not properly associate the paramters and database values with the associative hash key and value: eg: my %person=(firstname=>'John', lastname=>'Smith', age=>23); Database->quick_update('person', {id => 'johnsmith'}, \%person ); In this example code, your module does not properly associate placeholders with the proper hash value. It places random hash values in the resulting "SET" clauses, so that sometimes the firstname gets set to '23', sometimes the lastname gets set to 'John' etc. It does it different every time. Thanks, Nick -- All postal correspondence to: The Positive Internet Company, 24 Ganton Street, London. W1F 7QY *Follow us on Twitter* @posipeople The Positive Internet Company Limited is registered in England and Wales. Registered company number: 3673639. VAT no: 726 7072 28. Registered office: Northside House, Mount Pleasant, Barnet, Herts, EN4 9EE.
  Fri May 22 06:38:41 2015 davidp [...] preshweb.co.uk - Correspondence added
Subject: Re: [rt.cpan.org #104620] BUG in Dancer::Plugin::Database::Core::Handle quick_update jumbles params
Date: Fri, 22 May 2015 11:38:21 +0100
To: bug-Dancer-Plugin-Database-Core [...] rt.cpan.org
From: David Precious <davidp [...] preshweb.co.uk>
Set-Status: resolved Hi Nick, Thanks for the report; this unfortunate bit of stupidity on my part has already been fixed in the latest version on CPAN, so please ensure you have the latest Dancer::Plugin::Database and Dancer::Plugin::Database::Core installed and you should be safe. Apologies for any problems this silly bug caused you. Cheers Dave P
  Fri May 22 06:38:41 2015 The RT System itself - Status changed from 'new' to 'open'
  Fri May 22 06:40:15 2015 davidp [...] preshweb.co.uk - Status changed from 'open' to 'resolved'
  Fri May 22 06:40:35 2015 nickm [...] positive-internet.com - Correspondence added
Subject: Re: [rt.cpan.org #104620] BUG in Dancer::Plugin::Database::Core::Handle quick_update jumbles params
Date: Fri, 22 May 2015 11:40:17 +0100
To: bug-Dancer-Plugin-Database-Core [...] rt.cpan.org
From: Nick Mailer <nickm [...] positive-internet.com>
Hi David, Don't worry - we all make these mistakes :-) And thanks for the fix. I love your convenience methods. They're very elegant in their own way - just the right balance of abstraction without the OTT dbix nonsense! :-) On 22/05/15 11:38, David Precious via RT wrote: Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=104620 > > > Set-Status: resolved > > Hi Nick, > > Thanks for the report; this unfortunate bit of stupidity on my part has > already been fixed in the latest version on CPAN, so please ensure you > have the latest Dancer::Plugin::Database and > Dancer::Plugin::Database::Core installed and you should be safe. > > Apologies for any problems this silly bug caused you. > > Cheers > > Dave P > >
-- All postal correspondence to: The Positive Internet Company, 24 Ganton Street, London. W1F 7QY *Follow us on Twitter* @posipeople The Positive Internet Company Limited is registered in England and Wales. Registered company number: 3673639. VAT no: 726 7072 28. Registered office: Northside House, Mount Pleasant, Barnet, Herts, EN4 9EE.