postgresql-for-doctrine icon indicating copy to clipboard operation
postgresql-for-doctrine copied to clipboard

Allows :dql_parameters with ARRAY_* functions

Open allan-simon opened this issue 1 year ago • 1 comments

before it was only possible to do ARRAY_APPEND(e.myarray, 'a_literal') so it was impossible to have the value coming from php (without resorting to DQL injection)

We now allow also the following syntax ARRAY_APPEND(e.myarray, :foobar)

allan-simon avatar Oct 16 '24 15:10 allan-simon

compare to the other PR I've added additional test and used ArithmeticPrimary that allow both Literal and input parameters , as well as other methods returning string (which allow chaining calls )

allan-simon avatar Oct 16 '24 16:10 allan-simon

I think only CI failure remains to be addressed. Thank you 😊

martin-georgiev avatar Oct 20 '24 22:10 martin-georgiev

@martin-georgiev it was some coding style issue, should be good now :)

allan-simon avatar Oct 21 '24 14:10 allan-simon

@martin-georgiev sorry, i was a bit fast in the correction of the coding style, it's ok now, the scrutinizer error is a timeout so i think it's just need a relaunch ?

allan-simon avatar Oct 22 '24 09:10 allan-simon