Clarify that sql_maker is just a getter at this point
Peter Rabbitson [Thu, 10 Apr 2014 22:38:10 +0000 (00:38 +0200)]
lib/DBIx/Class/Storage/DBI.pm

index 9b5e3b0..5245bea 100644 (file)
@@ -991,8 +991,13 @@ sub _get_dbh {
   return $self->_dbh;
 }
 
+# *DELIBERATELY* not a setter (for the time being)
+# Too intertwined with everything else for any kind of sanity
 sub sql_maker {
-  my ($self) = @_;
+  my $self = shift;
+
+  $self->throw_exception('sql_maker() is not a setter method') if @_;
+
   unless ($self->_sql_maker) {
     my $sql_maker_class = $self->sql_maker_class;