Paul M. Jones comments

Results 59 comments of


                                            Paul M. Jones

INSERT | onDuplicateKeyUpdateCols doesn't add bindvalues to getBindValues()

@Altegras -- as @harikt notes, it would be good to see some code (or even better, a failing test) that illustrates the issue. Can you provide a PR or other...

Upsert

@cdekok Excellent -- can you post a comment on PR #115 to that effect? The contributor there might like to know.

Upsert

@pavarnos Do you want to add the appropriate code to the various driver-specific classes? (FWIW, I don't think the method would need the $sql argument; just add the appropriate string...

Go back to pseudo-sequential where() and having() placeholders?

It's a BC break against both 2.x and the current 3.x, in that it uses alternating "SQL" and "value" params via `...$cond`, rather than `$cond, $bind = array()`. The following...

Go back to pseudo-sequential where() and having() placeholders?

Let me see about 2.x breaks; 3.x has not had a release, so *technically* it's not a BC break -- but still, there might be a reasonable way to make...

Go back to pseudo-sequential where() and having() placeholders?

@harikt I don't recall. :-/

This is an excellent idea. It may mess up [Atlas.Orm](https://github.com/atlasphp/Atlas.Orm), which uses `with()` to eager-load relationships -- but since Atlas is alpha, it can change to use `eager()` or something...

Offer a "Teller" object

Thanks @UlrichEckhardt, though I should have specified I was rather looking for feedback from @sagikazarmark, @frederikbosch, or one of the other more senior folks on this project, as to whether...

Offer a "Teller" object

@sagikazarmark @frederikbosch You can see a PR with docs and tests at #630 -- let me know if you need anything, and thanks for considering it.