Skip Menu |

This queue is for tickets about the Fey CPAN distribution.

Report information
The Basics
Id: 49697
Status: rejected
Priority: 0/
Queue: Fey
People
Owner: Nobody in particular
Requestors: ARISTOTLE [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Wishlist
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Mon Sep 14 14:58:42 2009 ARISTOTLE [...] cpan.org - Ticket created
Subject: Support for auto_placeholders in Fey::Literal::Function
Given something like ->where( $t_acct->column('passhash'), '=', Fey::Literal::Function->new( SHA1 => Fey::Literal::Function->new( CONCAT => $t_acct->column('salt'), $password ) ) ) it would be nice if the $password didn't turn up as a literal value in the rendered SQL, but became a placeholder instead.
  Mon Sep 10 21:48:54 2018 ARISTOTLE [...] cpan.org - Correspondence added
No, that’s silly. It’s constructing a literal, so literal values turn up as literals. #49698 is the right way to address this need – a smarter Fey::Placeholder than can hold a bind value, and then it would be incumbent on the user to say that they mean to wrap $password as a value passed through a placeholder. It’s a bit more verbose to have to write it that way but concision is not the point of Fey, flexibility is, and turning strings into placeholders automatically would reduce flexibility.
  Mon Sep 10 21:48:55 2018 ARISTOTLE [...] cpan.org - Status changed from 'new' to 'rejected'