Automatically load the sql_comments Database extension when loading t…

…he sql_comments model plugin

Previously, users had to do this manually, but this makes it easier
for users, and there a number of other plugins that take this
approach.

CHANGELOG

@@ -1,5 +1,7 @@
 === master
 
+* Automatically load the sql_comments Database extension when loading the sql_comments model plugin (jeremyevans) (#2276)
+
 * Support :pg_auto_parameterize_min_array_size Database option in pg_auto_parameterize_in_array extension to control minimum array size to handle (jeremyevans)
 
 * Add pg_eager_any_typed_array plugin, automatically transform eager loads to use = ANY(array_expr::type[]) instead of IN (value_list) (jeremyevans)

lib/sequel/plugins/sql_comments.rb

@@ -40,7 +40,8 @@ module Plugins
     #   Album.sql_comments_dataset_methods :to_csv # csv_serializer plugin
     #
     # In order for the sql_comments plugin to work, the sql_comments
-    # Database extension must be loaded into the model's database.
+    # Database extension must be loaded into the model's database, so
+    # loading the plugin does this automatically.
     #
     # Note that in order to make sure SQL comments are included, some
     # optimizations are disabled if this plugin is loaded.
@@ -67,6 +68,10 @@ def self.def_sql_commend_method(mod, model, method_type, meth)
         # :nocov:
       end
 
+      def self.apply(model)
+        model.db.extension(:sql_comments)
+      end
+
       def self.configure(model)
         model.send(:reset_fast_pk_lookup_sql)
       end

spec/extensions/sql_comments_plugin_spec.rb

@@ -2,7 +2,7 @@
 
 describe "sql_comments plugin " do
   before do
-    @db = Sequel.mock(:fetch=>{:id=>1, :name=>'a'}).extension(:sql_comments)
+    @db = Sequel.mock(:fetch=>{:id=>1, :name=>'a'})
     @c = Class.new(Sequel::Model(@db[:t]))
     @c.columns :id, :name
     def @c.to_s; 'C' end