Merge 'trunk' into 'sqla_1.50_compat'

r5340@vger:  mendel | 2008-12-20 22:01:53 +0100
 r5134@vger (orig r5117):  ribasushi | 2008-11-13 08:34:26 +0100
 result_class goodies by Caelum:
 - result_class can now be specified as a search attribute, attaching it to the returned resultset
 - the specified class is now autoloaded via ensure_loaded()

 r5136@vger (orig r5119):  ribasushi | 2008-11-13 19:34:29 +0100
 resultset attribute from can now take a scalarref and use it verbatim
 r5145@vger (orig r5128):  ribasushi | 2008-11-14 10:46:00 +0100
 Document the new from usage
 r5152@vger (orig r5134):  ribasushi | 2008-11-14 18:44:25 +0100
 we use register_extra_source() now
 r5162@vger (orig r5140):  castaway | 2008-11-15 22:08:30 +0100
 Added failing create-multi test

 r5163@vger (orig r5141):  ribasushi | 2008-11-16 10:53:07 +0100
 whops
 r5166@vger (orig r5142):  norbi | 2008-11-16 20:12:42 +0100
  r5165@vger:  mendel | 2008-11-16 20:11:42 +0100
   * Merged in changes from 'resultsetcolumn_custom_columns' branch,

 r5179@vger (orig r5149):  ribasushi | 2008-11-16 23:14:13 +0100
 Revert castaway's test - mildly bogus
 r5180@vger (orig r5150):  ribasushi | 2008-11-16 23:16:31 +0100
 Add new test relationship - a track can have a relates single_cd
 r5181@vger (orig r5151):  ribasushi | 2008-11-16 23:23:38 +0100
 might_have test for castaway to break
 r5182@vger (orig r5152):  ribasushi | 2008-11-17 01:33:51 +0100
 rip away a horribly wrong create_via_update test (will pass when multicreate is merged)
 r5183@vger (orig r5153):  ribasushi | 2008-11-17 02:00:28 +0100
 One more (passing) multicreate test and a bit of cleanup
 r5184@vger (orig r5154):  ribasushi | 2008-11-17 02:59:53 +0100
 Two more sets of might_have - has_many relationships for extra tests
 r5185@vger (orig r5155):  ribasushi | 2008-11-17 03:09:18 +0100
 Two failing multicreate tests (the root cause of castaway's problem)
 r5186@vger (orig r5156):  ribasushi | 2008-11-17 03:48:57 +0100
 Silence cdbi tests like everything else
 r5187@vger (orig r5157):  ribasushi | 2008-11-17 04:01:39 +0100
 Failing tests by zby, showing that recursing insert() gets in the way of some exotic insert scenarious - waiting for mst to decide course of action
 r5188@vger (orig r5158):  ribasushi | 2008-11-17 12:04:54 +0100
 Extend might_have test with ideas from zby
 r5192@vger (orig r5161):  ribasushi | 2008-11-20 11:25:32 +0100
 Trunk passes tests again - todoify everything multicreate related to branch it out, as the task turned out to be more complex (no indentation to aid future merging)
 r5195@vger (orig r5164):  ribasushi | 2008-11-20 12:01:14 +0100
 CDBI-compat tests by Dave Horwoth:

     OK. I've attached a patch to t/cdbi-t/15-accessor.t that does several
     things:

     (1) Fixes the way the arguments are built so the sheep argument is
     correct in the tests.

     (2) Adds explicit tests for the number of sheep so we can be sure which
     database record is actually returned.

     (3) Adds tests for find_or_create() with modified accessor names.

     (4) Fixes the test for search() to report all errors

     I still don't fully understand what's going on in the output below but I
     think they are genuine test failures. I'd appreciate it if somebody else
     could run the tests to make sure the results are not some artefact of my
     test environment. I also attached a copy of the modified test script so
     you don't even have to apply the patch first :)

     I don't know what the DestroyWarning at the end is all about.

 r5196@vger (orig r5165):  ribasushi | 2008-11-20 12:09:11 +0100
 Minor doc patch by Caelum
 r5199@vger (orig r5168):  castaway | 2008-11-20 14:49:39 +0100
 Fix pod errors so that some storage subclasses show up in cpan properly

 r5291@vger (orig r5222):  ash | 2008-12-03 18:23:00 +0100
 Make the many-to-many warning use warnings::register;
 r5317@vger (orig r5239):  jnapiorkowski | 2008-12-15 23:02:19 +0100
 changes to replication so that if a replicant is offline when we do the initial connection (or if we need to globally reconnect later) the connection does not die.  This is different from when a replicant is available just not listening, or if the replicant is too laggy.  Thanks David Steinbrunner for the patch suggestions.
 r5323@vger (orig r5245):  groditi | 2008-12-16 23:56:07 +0100
  r24740@martha (orig r5230):  groditi | 2008-12-09 20:28:10 -0500
  fix for bug. all tests seem to pass, we still need a new test and more research

 r5324@vger (orig r5246):  groditi | 2008-12-16 23:56:29 +0100
  r24754@martha (orig r5241):  groditi | 2008-12-16 16:27:06 -0500
  Introduce 'any_null_means_no_value' option to eliminate wasteful queries. The option is off by default and must be explicitly turned on. Tests, + docs included

 r5325@vger (orig r5247):  groditi | 2008-12-16 23:57:11 +0100
  r24756@martha (orig r5243):  groditi | 2008-12-16 17:42:10 -0500
  rename option to undef_on_null_fk and make it default for belongs_to

 r5326@vger (orig r5248):  groditi | 2008-12-16 23:57:34 +0100
  r24757@martha (orig r5244):  groditi | 2008-12-16 17:52:12 -0500
  minor typo and style change

 r5328@vger (orig r5250):  groditi | 2008-12-17 00:27:57 +0100
 somehow i messed up the merge. this fixes it
 r5329@vger (orig r5251):  ash | 2008-12-17 00:45:51 +0100
 Try to fix test on 5.10
 r5332@vger (orig r5254):  ribasushi | 2008-12-18 12:28:38 +0100
 Some cleanups to the m2m warnings test
 r5336@vger (orig r5258):  jnapiorkowski | 2008-12-19 20:52:44 +0100
 more noise debugging messages if debug is on, minor doc tweaks, changes so that the fake sqlite tests will work and laid the groundwork for replication without dbatabase native replication support.
 r5337@vger (orig r5259):  jnapiorkowski | 2008-12-19 21:48:35 +0100
 altered schema->populate so that it is a very thin wrapper on top of resultset->populate and changed resultset_populate so that is accepted both the arrayref of hashes and the arrayref of arrayref style of args.  Documented this, updated the tests a bit to make sure it is all good.
 r5338@vger (orig r5260):  ribasushi | 2008-12-19 22:09:17 +0100
 Todoify cdbi failing tests, waiting for schwern

Todoify cdbi failing tests, waiting for schwern

altered schema->populate so that it is a very thin wrapper on top of resultset->populate and changed resultset_populate so that is accepted both the arrayref of hashes and the arrayref of arrayref style of args.  Documented this, updated the tests a bit to make sure it is all good.

more noise debugging messages if debug is on, minor doc tweaks, changes so that the fake sqlite tests will work and laid the groundwork for replication without dbatabase native replication support.

Some cleanups to the m2m warnings test

Try to fix test on 5.10

somehow i messed up the merge. this fixes it

Merge 'belongs_to_null_col_fix' into 'trunk'

Merge 'belongs_to_null_col_fix' into 'trunk'

r24740@martha (orig r5230):  groditi | 2008-12-09 20:28:10 -0500
fix for bug. all tests seem to pass, we still need a new test and more research

minor typo and style change

rename option to undef_on_null_fk and make it default for belongs_to

Introduce 'any_null_means_no_value' option to eliminate wasteful queries. The option is off by default and must be explicitly turned on. Tests, + docs included

Merge 'trunk' into 'belongs_to_null_col_fix'

r24752@martha (orig r5239):  jnapiorkowski | 2008-12-15 17:02:19 -0500
changes to replication so that if a replicant is offline when we do the initial connection (or if we need to globally reconnect later) the connection does not die.  This is different from when a replicant is available just not listening, or if the replicant is too laggy.  Thanks David Steinbrunner for the patch suggestions.

changes to replication so that if a replicant is offline when we do the initial connection (or if we need to globally reconnect later) the connection does not die.  This is different from when a replicant is available just not listening, or if the replicant is too laggy.  Thanks David Steinbrunner for the patch suggestions.

fix for bug. all tests seem to pass, we still need a new test and more research

Make the many-to-many warning use warnings::register;

 * Replaced eq_bind() implementation to use Test::Deep::eq_deeply().

 * Replaced eq_bind() implementation with the current copy from SQL::Abstract::Test.

 * Removed extra parens from the ON expression of JOIN (SQLA::Test now handles it properly - in SQLA branch '1.50_RC-extraparens' branch). That means that all the tests pass with SQLA 1.24 and SQLA 1.50RC.

 * Fixed the order of args of skip().

 * Undid bumping of SQL::Abstract version dependency.

 * Renamed DBICTest::DBICDebugObj to DBIC::DebugObj.

 * Fixed the number of tests skipped in a SKIP block.

 * Wrapped SQL::Abstract::Test functionality in a new module (DBIC::SqlMakerTest).
 * Made a test SKIP that does not work with SQL::Abstract < 1.49 (used to be a TODO test).

 * Documented using PostgreSQL arrays in Cookbook.

 * Fixed the test plan in 72pg.t.
 * Fixed SQLA version check in SKIP tests (it's 1.49 in current 1.50RC branch).

 * Added a test for arrayref bind values in search to match PostgreSQL array type values.

 * Made some more tests SKIP if SQLA version < 1.50.

 * Disabled stringification of arrayref bind values (used to pass values for PostgreSQL arrays).

 * Made PostgreSQL array tests SKIP if SQLA version < 1.50.

 * Added tests for passing arrayrefs as bind values for PostgreSQL array values.

 * Updated svn:ignore.

 * Added a WHATSNEW.txt file that summarises the changes on this branch.

 * Documented order_by => [ { -desc => 'col' } ] syntax.

 * Test cases for every supported order_by syntax.

Fix pod errors so that some storage subclasses show up in cpan properly

Minor doc patch by Caelum

CDBI-compat tests by Dave Horwoth:

    OK. I've attached a patch to t/cdbi-t/15-accessor.t that does several
    things:

    (1) Fixes the way the arguments are built so the sheep argument is
    correct in the tests.

    (2) Adds explicit tests for the number of sheep so we can be sure which
    database record is actually returned.

    (3) Adds tests for find_or_create() with modified accessor names.

    (4) Fixes the test for search() to report all errors

    I still don't fully understand what's going on in the output below but I
    think they are genuine test failures. I'd appreciate it if somebody else
    could run the tests to make sure the results are not some artefact of my
    test environment. I also attached a copy of the modified test script so
    you don't even have to apply the patch first :)

    I don't know what the DestroyWarning at the end is all about.

Trunk passes tests again - todoify everything multicreate related to branch it out, as the task turned out to be more complex (no indentation to aid future merging)

Extend might_have test with ideas from zby

Failing tests by zby, showing that recursing insert() gets in the way of some exotic insert scenarious - waiting for mst to decide course of action

Silence cdbi tests like everything else

Two failing multicreate tests (the root cause of castaway's problem)

Two more sets of might_have - has_many relationships for extra tests

One more (passing) multicreate test and a bit of cleanup

rip away a horribly wrong create_via_update test (will pass when multicreate is merged)

might_have test for castaway to break

Add new test relationship - a track can have a relates single_cd

Revert castaway's test - mildly bogus

 * Added test cases to test if arrayref bind values in insert/update are passed through sql_maker intact.
 * Added 'array_datatypes' parameter to the sql_maker constructor.

Merge 'resultsetcolumn_custom_columns' into 'trunk'

r5165@vger:  mendel | 2008-11-16 20:11:42 +0100
 * Merged in changes from 'resultsetcolumn_custom_columns' branch,

whops

Added failing create-multi test

 * Rewrote fetching the 'select' and 'as' values from the parent resultset to something that's easier to read for everyone.

 * Fixed a bug: did not create the underlying resultset if neither ->next or ->reset is called before ->first was called.

 * Added ->first and ->reset implementation to ResultSetColumn.
 * A bit of refactoring/cleanup of ResultSetColumn.

we use register_extra_source() now

Document the new from usage

resultset attribute from can now take a scalarref and use it verbatim

result_class goodies by Caelum:
- result_class can now be specified as a search attribute, attaching it to the returned resultset
- the specified class is now autoloaded via ensure_loaded()

 * Bumped SQL::Abstract version dependency.

 * Made DBIC::SQL::Abstract pass on order_by hashref ({-desc => 'colname'}) to SQL::Abstract.

Merge 'trunk' into 'sqla_1.50_compat'

r25478@agaton (orig r5092):  ribasushi | 2008-11-10 22:46:45 +0000
Switch out one more reliance on connect_info
r25483@agaton (orig r5097):  ribasushi | 2008-11-10 23:09:23 +0000
belongs_to relationships are explicitly flagged by default, to aid the DBIC SQLT parser (by abraxxa)
r25484@agaton (orig r5098):  ribasushi | 2008-11-10 23:18:01 +0000
Clarify attributes argument of find()
r25499@agaton (orig r5113):  plu | 2008-11-12 16:42:44 +0000
fixed 73oracle.t, added charfield

fixed 73oracle.t, added charfield

 * Fixed test cases of ResultSetColumn vs. +select/+as (they used to unconditionally pass).
 * Added a test case of ResultSetColumn for a non-existent column (note: a valid SQL fragment still can be passed instead of a column).
 * Added a test case of ResultSetColumn that tests overriding a column using select/as.
 * ResultSetColumn now respects select/as/+select/+as.
 * Added missing throw_exception() to ResultSetColumn.

Clarify attributes argument of find()

belongs_to relationships are explicitly flagged by default, to aid the DBIC SQLT parser (by abraxxa)

 * Removed dead code (sub _RowNumberOver).

 * Removed a wrong TODO test: "order_by with quoting needs fixing (ash/castaway)" (asked by mst).
 * Made a TODO test non-TODO: "select attr with star needs fixing (mst/nate)" (works with SQLA 1.50).

 * Converted some of the test cases to use SQL::Abstract::Test.
 * Added the parens for the join condition (SQL::Abstract::Test can't cope with it, and, besides, they help readability there; plus this gives the same output that older SQLA generated).

Switch out one more reliance on connect_info

changed the way we get the DSN for debugging in replication

Switch schema management version calls from VERSION() to schema_version()

Don't rely on connect_info(), instead fetch the dsn from _dbi_connect_info (which is stable no matter which connect_info() flavor is used)
todo: ping jnpiorkowski about Replicated::Pool and Replicated::Replicant - it blindly assumes that connect_info->[0] contains the dsn, which might not be the case. Even when using _dbi_connect_info, the first element might be a coderef

Fix detection of multiple rows being returned for single(): calling fetchrow_array in scalar context leads to indefined results (search.cpan.org/~timb/DBI/DBI.pm#fetchrow_array)

Some testdb changes and alignment, preparing for test refactor branch

SQLT behaves sanely in scalar context, no need for stupid version detections (ribasushi--)

More TODO items

One extra multicreate test and some cleanup

Fix loss of objects on multicreate over several 1:M relations

Final insanification of the multicreate test

unhide SQL::Translator::Parser::DBIx::Class from pause indexer

pod documentation for SQL::Translator::Parser::DBIx::Class

A little indentation clean up

Test the result of an eval directly.

discard the changes to $sandl to quiet an "object destroyed with changes"
warning.

mk_group_accessor() is sometimes called internally when inflating a column
and with an array ref field spec, but the accessor and column names are
the same.  Special case that to go through the usual accessor_name_for stuff.

[rt.cpan.org 36863]
Fix mk_group_accessors() to handle [$field, $name].  It ignores
accessor/mutator_name_for() because if you pass in your own accessor
name you probably mean it.

Normalize indentation

And now really untabify

normalize the indentation and detabify

fixed 73oracle.t

Test prompted by discussion about tasty stuff

Better linking in Relatiomship POD

More svn:ignore

Clarify Relationship POD

refactor of t/96multi_create.t:
- simplify test case submitted by wreis
- incorporate it into the large multicreate at the end
- eval{} independent sections of the test so more output can be gathered

Augment commit r5037 - the column_info feature on SQLite is not buggy but partially unimplemented

rolling back my last change, waiting until new sqlt is on cpan

fixed next warnings

updated Changes

split sql statements for deploy only if SQLT::Producer returned a scalar