[jifty-devel] don't set_<foo>() a collection (again)

Jesse Vincent jesse at bestpractical.com
Mon Apr 24 11:27:42 EDT 2006


Thanks. applied.


On Sun, Apr 23, 2006 at 10:48:38PM -0700, Eric Wilhelm wrote:
> Now that the autoload() is gone from JDBI::Record, my wee patch to get 
> $rec->set_thatcollection() to bail-out gracefully is orphaned.
> 
> This patch includes tests.
> 
>   http://scratchcomputing.com/tmp/noset_collection.patch
> 
> Maybe with JDBI::CollectionArray, set_thatcollection() could do 
> something useful, but for now it should at least remind me that this is 
> not allowed.
> 
> If you instead want to just leave it as a "Can't locate method..." 
> error, you'll still need the conditional plus the following just before 
> the assignment.
> 
>   $subref or return();
> 
> --Eric
> -- 
> The only thing that could save UNIX at this late date would be a new $30
> shareware version that runs on an unexpanded Commodore 64.
> --Don Lancaster (1991)
> ---------------------------------------------------
>     http://scratchcomputing.com
> ---------------------------------------------------
> _______________________________________________
> jifty-devel mailing list
> jifty-devel at lists.jifty.org
> http://lists.jifty.org/cgi-bin/mailman/listinfo/jifty-devel
> 

-- 


More information about the jifty-devel mailing list