Teach ::Tree that ILIKE (PostgreSQL) and REGEXP (MySQL) are binary ops

Fix ORDER BY clause type in synopsis and where() header

The ORDER BY clause can be either a string, stringref, arrayref or
hashref, so don't imply it has to be an arrayref in the docs.

Fix order clauses with bind parameters in ->where

->where was ignoring the bind parameters returned by ->_order_by

Ignore editor droppings

Bump Moo dep and remove warning defatalisation hack

Moo 2 no longer pulls in strictures

Add _update_returning method for UPDATE … RETURNING

It just calls _returning, but allows subclasses to override UPDATE …
RETURNING independently from INSERT … RETURNING

Keep calling _insert_returning for INSERT … RETURNING

DBIC::SQLMaker::Oracle overrides this method, and its insert calls
next::method, so we need keep calling the old method for the override to
work.

Fixed missing '}' in POD examples and reformat for clarity

Reinstate _insert_returning method for DBIC compat

Add support for UPDATE ... RETURNING ...

Similar to INSERT ... RETURNING ...

Release 1.81

Fix overly-enthusiastic parenthesis unroller (RT#99503)

Re-indent and quote versions in Makefile

No functional changes

Release 1.80

Skip tests on smokers with a broken DBIC trial

Croak on invalid top-level special ops

When they were originally introduced it never occured neither to me, nor
whoever reviewed that a plain special-op will not work at top level.
Add an extra check at the fallback point (ugly, but still effective)

Saner handling of nested_lhs, we now set it in one spot only

spelling mistake (RT#99147)

Whatever I was thinking it wasn't making much sense

Good thing I started pondering how to port this to DQ, otherwise would not
have noticed I fucked up... for a while

Release 1.79

Even though MRO::C is a dep of Moo, specify it explicitly

Some folks do this http://cpanratings.perl.org/dist/SQL-Abstract#9620

Deprecate and properly handle empty lhs

More robust Moo import block (modelled on DBIC)

Updating links to repositories, added GitHub mirror

Modelled on the corresponding DBIC setup

Minor documentation fixes; typos, POD formatting code syntax, return value explanation consistency

Link fixes (no other changes)

Warn if _recurse_where is called in scalar context

DBIC used to do this, but doesn't any more. Still something else might...

Truly fix literal+bind examples (build upon a3e3e6a18)

Throw away the dummy text - it was always misleading and incorrect
Add some space to clarify which part is which

Fix typos in literal + bind examples

IRC meta

Well how about them apples?! Add temporary overload-madness escape hatch

Boolification-derived-stringification is a thing... just shoot me now

An arrayref makes sense for literals, but no sense for values

Change the API before it goes live (extra benefit - no mixing up of
the functions by accident)

More (still passing!!!) test cases after conversation with FC

Even more versatile is_plain_value testing - feeble attempts to trip up perl

This is an attempt to simulate the behavior of JSON::PP::Boolean, and to
replicate the error autarch was seeing - clearly there is more to it than
just simple value juggling. Oh well, commit already written tests anyway.

See further commits for the actual code changes

Make sure is_plain_value returns the actual object pre-stringify

This is important since DBIC is starting to heavily rely on the
contents of the returned []

Even more reuse of @_

{ -ident => undef } makes zero sense

Correct comments about overload methods vs. ->can()

Aha - this is why I didn't use ->can originally

Punt for the time being, will investigate why later. The error looks like:

rabbit@Ahasver:~/devel/dbic$ git rev-parse HEAD
32497625ce5699576c8a63edd16401d4f741c282

rabbit@Ahasver:~/devel/dbic$ PERL5LIB=/home/rabbit/devel/sqla/lib prove -lT t/100populate.t
t/100populate.t .. 1/? Stub found while resolving method "???" overloading """" in package "Path::Class::File" at /home/rabbit/devel/sqla/lib/SQL/Abstract.pm line 118, <> line 1.
t/100populate.t .. Dubious, test returned 255 (wstat 65280, 0xff00)
All 43 subtests passed

Test Summary Report
-------------------
t/100populate.t (Wstat: 65280 Tests: 43 Failed: 0)
  Non-zero exit status: 255
  Parse errors: No plan found in TAP output
Files=1, Tests=43,  1 wallclock secs ( 0.04 usr  0.00 sys +  0.90 cusr  0.03 csys =  0.97 CPU)
Result: FAIL

New exportable functions: is_literal_value($) and is_plain_value($)

Ported from DBIC with some corner case fixes added.

Fix -values not being handled properly with NULLs

Escape closing quote character in table and column names

After 5f3fa0ac script/ is no more

Fixup descriptions a bit

Add basic .dir-locals.el

release 1.78

Drop reqs - works on 5.6.1 just fine

Rewrite sql formatter script and shove it into examples until more tested

Combined with the opportunistic parser this little tool has the promise of
actually being useful outside of the echo chamber. Someone with good UX
skillz ought to take this to the next level... or something ;)

Swap CAG with Moo, lazify a lot of the profile selection

There should be zero functional changes

Port the \[] *_sql_bind unpacker from DBIC

Use variables to simplify grocking the parenthesis unroller rules

Zero (hopefully) functional changes

Fix the reassembler to treat RNO as the precious special snowflake it is

Yes, the parser is accumulating a lot of cruft this way with special casing
and whatnot, and yes it is necessary for the future when we write a real
parser which will *HAVE* to deal with the corner cases. The accumulated set
of tests is the goal here (with the side effect of better UX) -- ribasushi

Treat RNO as a unary op, and properly normalize it

Original (now passing) test for 08e163601

Fix the binop leg of the parser to correctly consider only a single LHS node

Work gracefully around malformed "no lhs" cases as well

release 1.77

Port scalarref in array of sources test from dq (7732c37a)

Fix regression in column level { not => undef } op

In 3cdadcbe we stopped considering a bare 'not' being an inequality op.
However there is code in the wild relying on this, so add an extra
compat shim (cheap)

Check for extra warnings, and diag input and output

We want to test the _lack_ of warnings as well, and having the input and
output is handy for figuring out what's wrong.

Diag input and output for unexpected success

Ensure that multi-nested parenthesis are never unrolled after IN

This lets us remove parenthesis_significant from many tests that do
not need it, including reversal of 1ba9d0f03

Parentheses are significant in IN tests

release 1.75

Move simple IN/BETWEEN tests to t/05in_between.t

No changes in the actual tests except splitting the empty array and
scalar tests for IN into separate cases.

Actually use the descriptions in the test cases

An old (now passing) test for regression introduced in 6f2a5b6

Port fce4b7a9 from the DQ branch (with warning expectation ammendments)

Reintroduce handling of IS( NOT) ops

This is undocumented syntax, but it is very likely someone is using it in
the wild, and the previous inequality op handler rewrite broke this

Extensive tests and deprecations of multivalue/no value inequality op calls

All sql-generation tests match pre <= 1.74 behavior (except for the newly
added rlike operator, which now warns while it simply did not work with
undefs before)

Move test to a better location (no changes)

Normalize handling of expected warnings/exceptions in tests

Remove many of the settled-by-time comments, modernize a bit

Centralize handling of in-test dumpering

Extra stopword (we need to find a way to introduce dialects...)

No tabs, no trailers

Make it clearer in examples that bool takes expressions

timbunce++

Test BETWEEN after the IN debacle (luckily it throws in all expected cases)

Generalize the emitted exception text

We already have Storable as a dep, this is pointless

And yet more improvements in the parsing engine

Streamline the construction of -MISC (gah I hate that name) nodes, and make
sure we do not treat placeholders as list element node terminator

Do not yell loudly on TODOified is_same_* tests

Clarify in => undef exception (based on 90c87778)

Backport undef-with-in tests from DQ, add "roadwork ahead" exceptions

This is an amalgamation of test changes from 5b67050, 39221d2 and 038b0a7
ilmari++

Extra test from the DQ branch (passes in 1.5)

Document and test non-reference { -in => ... }

Also test the documented but untested { -in => [] } optimisation.

Fix typos in POD and comments (RT#87776)

Kill the plan

Finish a RHS branch before applying ASC/DESC mods

release 1.74

No dynamic deps

Make tests pass without Storable

Added (now passing) test for @@ full text search

Stop unknown token processing early in case a potential unknown function lies ahead

Add (now passing) reassembly test

Change parser to not eagerly slurp RHS expressions it doesn't recognize

The ORDER BY parsing fix in 73835ff0 only worked by accident

Remove a number of workarounds introduced before the parser rewrite - none of
them help, and one of them was in fact the reason multi-member ORDER BY
did not function correctly

All roundtrip tests now look for the exact string

Merge test files while at it

Stop differentiating between ORDER BY foo and ORDER BY foo ASC by default

Drop the silly Dumpered bind comparison

Pave the road to more global state

(Yes, yes an object is saner, but it means a thought out interface and
all that, and then it means optimization of when and how do we do the
modification passes, and really I am not after this stuff at the moment)