* Removed temporary notes file (WHATSNEW.txt).

Merge 'trunk' into 'sqla_1.50_compat'

r5410@vger:  mendel | 2009-01-18 20:40:15 +0100
 r5392@vger (orig r5301):  norbi | 2009-01-12 11:55:54 +0100
  r5391@vger:  mendel | 2009-01-12 11:55:48 +0100
   * Reworded $rs->current_source_alias POD (thanks to castaway).

 r5398@vger (orig r5307):  caelum | 2009-01-14 02:25:28 +0100
 auto_savepoint support for Oracle and a note on txn_do for with "AutoCommit => 0"

 r5404@vger (orig r5313):  ribasushi | 2009-01-16 11:29:16 +0100
 Proper MSSQL last_insert_id() scoping patch by abraxxa
 No tests as it is very hard to simulate the problematic behavior - it would only manifest if:
   1) An INSERT statement is issued from connection A
   2) An INSERT statement is issued from connection B
   3) last_insert_id is called from connection A

 Without this patch the last id from the second INSERT will be returned in step 3
 Tested with both DBICTEST_MSSQL_ODBC_DSN and DBICTEST_MSSQL_DSN against a MSSQL 2005
 r5405@vger (orig r5314):  plu | 2009-01-16 19:25:57 +0100
 Fixed typo in link
 r5406@vger (orig r5315):  ribasushi | 2009-01-17 19:39:50 +0100
 Stop 99dbic_sqlt_parser.t from breaking every time a new table is added
 r5407@vger (orig r5316):  ribasushi | 2009-01-17 19:47:53 +0100
 Fix borked t/bindtype_columns.t - TODO: still only tests PG, need to test Oracle as well

Merge 'trunk' into 'sqla_1.50_compat'

r5390@vger:  mendel | 2009-01-12 11:30:01 +0100
 r5351@vger (orig r5266):  norbi | 2008-12-21 01:39:44 +0100
  r5350@vger:  mendel | 2008-12-21 01:39:37 +0100
   * Added myself to the contributors list.

 r5354@vger (orig r5269):  ribasushi | 2008-12-21 12:24:39 +0100
 delete() POD patch from Michael Hendricks (slightly reworded)
 r5365@vger (orig r5278):  nothingmuch | 2008-12-22 17:43:50 +0100
 map blob to PG_BYTEA on DBD::Pg
 r5366@vger (orig r5279):  matthewt | 2008-12-24 18:26:35 +0100
 make clear that search_like is only really there for Class::DBI users
 r5367@vger (orig r5280):  castaway | 2008-12-29 22:51:54 +0100
 Added standard arguments/return value to all docced methods.
 Upadted some method docs.

 r5368@vger (orig r5281):  castaway | 2008-12-29 23:17:37 +0100
 Re-ordered method docs to be in a more user-needs-it order.
 Fixed some obj/table mentions in docs to be consistently $source.

 r5369@vger (orig r5282):  castaway | 2008-12-30 11:58:19 +0100
 Add definitions of relationship and relationship bridge, link to them from the rel docs

 r5377@vger (orig r5290):  rjbs | 2009-01-08 21:49:29 +0100
  failing test for callback on_connect_do getting storage obejct
 r5378@vger (orig r5291):  rjbs | 2009-01-08 21:50:57 +0100
 fix the just-added test; pass the Storage object to pure-code on_connect callbacks
 r5379@vger (orig r5292):  rjbs | 2009-01-08 21:51:39 +0100
 credit where credit is due; namely: ME!
 r5381@vger (orig r5294):  wreis | 2009-01-08 23:45:35 +0100
 failing test for load_namespace
 r5382@vger (orig r5295):  wreis | 2009-01-08 23:57:47 +0100
 ->result_source_instance->source_name should return undef for any class
 r5383@vger (orig r5296):  wreis | 2009-01-09 00:11:50 +0100
 reverting changes for versioning tests
 r5384@vger (orig r5297):  rjbs | 2009-01-09 02:55:53 +0100
 by setting conn_?id first, we prevent $storage->dbh from recursing through on_connect_do actions
 r5385@vger (orig r5298):  matthewt | 2009-01-09 04:28:44 +0100
 fixup source registration bugs where schema-specific things were being set on class' result_source_instance objects
 r5386@vger (orig r5299):  wreis | 2009-01-09 13:14:49 +0100
 not necessary
 r5389@vger (orig r5300):  norbi | 2009-01-12 11:18:47 +0100
  r5388@vger:  mendel | 2009-01-12 11:18:22 +0100
   * Implemented $rs->current_source_alias.

Fix borked t/bindtype_columns.t - TODO: still only tests PG, need to test Oracle as well

Stop 99dbic_sqlt_parser.t from breaking every time a new table is added

Fixed typo in link

Proper MSSQL last_insert_id() scoping patch by abraxxa
No tests as it is very hard to simulate the problematic behavior - it would only manifest if:
  1) An INSERT statement is issued from connection A
  2) An INSERT statement is issued from connection B
  3) last_insert_id is called from connection A

Without this patch the last id from the second INSERT will be returned in step 3
Tested with both DBICTEST_MSSQL_ODBC_DSN and DBICTEST_MSSQL_DSN against a MSSQL 2005

auto_savepoint support for Oracle and a note on txn_do for with "AutoCommit => 0"

 * Reworded $rs->current_source_alias POD (thanks to castaway).

 * Implemented $rs->current_source_alias.

not necessary

fixup source registration bugs where schema-specific things were being set on class' result_source_instance objects

by setting conn_?id first, we prevent $storage->dbh from recursing through on_connect_do actions

reverting changes for versioning tests

>result_source_instance->source_name should return undef for any class

failing test for load_namespace

credit where credit is due; namely: ME!

fix the just-added test; pass the Storage object to pure-code on_connect callbacks

 failing test for callback on_connect_do getting storage obejct

Add definitions of relationship and relationship bridge, link to them from the rel docs

Re-ordered method docs to be in a more user-needs-it order.
Fixed some obj/table mentions in docs to be consistently $source.

Added standard arguments/return value to all docced methods.
Upadted some method docs.

make clear that search_like is only really there for Class::DBI users

map blob to PG_BYTEA on DBD::Pg

Remove some cruft

delete() POD patch from Michael Hendricks (slightly reworded)

whops

Fix ordering of 'use lib' and 'use' in tests

 * Added myself to the contributors list.

 * Removed an outdated comment.

 * Removed hiding of packages under t/.

 * Removed hiding of packages under t/.

 * Updated WHATSNEW.txt.

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)