On 6 Dec 2006, at 15:11, Bernhard Graf wrote:

Matt S Trout wrote:

So please tell me - what is the status of these patches?

The join one I'll probably apply next time I do a pass across that
are of the code.

Good.

Anybody else with a commit bit feel free to apply that one if you've got a spare 30 seconds.

ISTR the inflate_column one renamed a method rather than creating an
inflate_columns method that calls inflate_column as requested; did
you post a version that implemented it the other way? If so, a link
to the web archive pointing at the mail where you did so would be
appreciated since evidently I've missed it.

inflate_columns is downwards compatible to the previous inflate_column.
After patching inflate_column becomes an alias to inflate_columns.
Which problems do you see with this?

What problems did you see with the way I originally proposed implementing it that caused you to do it differently?

--
Matt S Trout, Technical Director, Shadowcat Systems Ltd.
Offering custom development, consultancy and support contracts for Catalyst, DBIx::Class and BAST. Contact mst (at) shadowcatsystems.co.uk for details. + Help us build a better perl ORM: http://dbix- class.shadowcatsystems.co.uk/ +



_______________________________________________
List: http://lists.rawmode.org/cgi-bin/mailman/listinfo/dbix-class
Wiki: http://dbix-class.shadowcatsystems.co.uk/
IRC: irc.perl.org#dbix-class
SVN: http://dev.catalyst.perl.org/repos/bast/trunk/DBIx-Class/
Searchable Archive: http://www.mail-archive.com/[email protected]/

Reply via email to