rroonga 1.0.3 → 1.0.4

data/NEWS.ja.rdoc

@@ -1,6 +1,6 @@
 = お知らせ
 
-== 1.0.3: 2010-11-29
+== 1.0.4: 2010-11-29
 
 === 改良
 
@@ -31,6 +31,15 @@
 * Groonga::Database#touchを追加。
 * Groonga::Table#exist?を追加。
 * Groonga::Record#valid?を追加。
+* Groonga::Column#vector?を追加。
+* Groonga::Column#scalar?を追加。
+* Groonga::Record#vector_column?を追加。
+* Groonga::Record#scalar_column?を追加。
+* レコードIDが必要なところにrecord_raw_idを持つオブジェクトなら
+  Groonga::Record以外も渡せるようにした。
+* Groonga::Record#record_raw_idを追加。
+* ベクタの参照カラムの値にArray以外でもto_aryを持つオブジェクトを
+  設定できるようにした。
 
 == 変更
 
@@ -41,6 +50,8 @@
   場合はShortTextを使用するようにした。
 * Groonga::Schema#loadをGroonga::Schema#restoreに改名。
 * pkg-config 1.0.7対応。
+* Groonga::Column#index?を追加。Groonga::Column#index_column?は非推奨。
+* Groonga::Column#reference?を追加。Groonga::Column#reference_column?は非推奨。
 
 === 修正
 

data/NEWS.rdoc

@@ -31,6 +31,14 @@
 * Added Groonga::Database#touch.
 * Added Groonga::Table#exist?.
 * Added Groonga::Record#valid?.
+* Added Groonga::Column#vector?.
+* Added Groonga::Column#scalar?.
+* Added Groonga::Record#vector_column?.
+* Added Groonga::Record#scalar_column?.
+* Accepted any object that has record_raw_id method for record ID required
+  location. Groonga::Record isn't required.
+* Added Groonga::Record#record_raw_id.
+* Accepted any object that as to_ary method for reference vector column value.
 
 == Changes
 
@@ -41,6 +49,9 @@
   Groonga::Hash#create and Groonga::PatriciaTrie#create.
 * Renamed Groonga::Schema#load to Groonga::Schema#restore.
 * Supported pkg-confg 1.0.7.
+* Added Groonga::Column#index? and deprecated Groonga::Column#index_column?.
+* Added Groonga::Column#reference? and deprecated
+  Groonga::Column#reference_column?.
 
 === Fixes
 

data/ext/groonga/rb-grn-column.c

@@ -553,16 +553,18 @@ rb_grn_column_is_locked (int argc, VALUE *argv, VALUE self)
 }
 
 /*
- * Document-method: reference_column?
+ * Document-method: reference?
  *
  * call-seq:
- *   column.reference_column? -> true/false
+ *   column.reference? -> true/false
  *
  * _column_の値がテーブルのレコードとなる場合は+true+を返し、
  * そうでない場合は+false+を返す。
+ *
+ * @since: 1.0.5
  */
 static VALUE
-rb_grn_column_reference_column_p (VALUE self)
+rb_grn_column_reference_p (VALUE self)
 {
     grn_ctx *context;
     grn_obj *column;
@@ -589,16 +591,18 @@ rb_grn_column_reference_column_p (VALUE self)
 }
 
 /*
- * Document-method: index_column?
+ * Document-method: index?
  *
  * call-seq:
- *   column.index_column? -> true/false
+ *   column.index? -> true/false
  *
  * _column_がGroonga::IndexColumnの場合は+true+を返し、
  * そうでない場合は+false+を返す。
+ *
+ * @since: 1.0.5
  */
 static VALUE
-rb_grn_column_index_column_p (VALUE self)
+rb_grn_column_index_p (VALUE self)
 {
     grn_ctx *context;
     grn_obj *column;
@@ -614,6 +618,72 @@ rb_grn_column_index_column_p (VALUE self)
     }
 }
 
+/*
+ * Document-method: vector?
+ *
+ * call-seq:
+ *   column.vector? -> true/false
+ *
+ * _column_がベクターカラムの場合は+true+を返し、
+ * そうでない場合は+false+を返す。
+ *
+ * @since: 1.0.5
+ */
+static VALUE
+rb_grn_column_vector_p (VALUE self)
+{
+    grn_ctx *context;
+    grn_obj *column;
+
+    rb_grn_column_deconstruct(SELF(self), &column, &context,
+			     NULL, NULL,
+			     NULL, NULL, NULL);
+
+    if (column->header.type == GRN_COLUMN_VAR_SIZE &&
+	((column->header.flags & GRN_OBJ_COLUMN_TYPE_MASK) ==
+	 GRN_OBJ_COLUMN_VECTOR)) {
+	return Qtrue;
+    } else {
+	return Qfalse;
+    }
+}
+
+/*
+ * Document-method: scalar?
+ *
+ * call-seq:
+ *   column.scalar? -> true/false
+ *
+ * _column_がスカラーカラムの場合は+true+を返し、
+ * そうでない場合は+false+を返す。
+ *
+ * @since: 1.0.5
+ */
+static VALUE
+rb_grn_column_scalar_p (VALUE self)
+{
+    grn_ctx *context;
+    grn_obj *column;
+
+    rb_grn_column_deconstruct(SELF(self), &column, &context,
+			     NULL, NULL,
+			     NULL, NULL, NULL);
+
+    switch (column->header.type) {
+      case GRN_COLUMN_FIX_SIZE:
+	return Qtrue;
+      case GRN_COLUMN_VAR_SIZE:
+	if ((column->header.flags & GRN_OBJ_COLUMN_TYPE_MASK) ==
+	    GRN_OBJ_COLUMN_SCALAR) {
+	    return Qtrue;
+	} else {
+	    return Qfalse;
+	}
+      default:
+	return Qfalse;
+    }
+}
+
 void
 rb_grn_init_column (VALUE mGrn)
 {
@@ -628,10 +698,14 @@ rb_grn_init_column (VALUE mGrn)
     rb_define_method(rb_cGrnColumn, "unlock", rb_grn_column_unlock, -1);
     rb_define_method(rb_cGrnColumn, "clear_lock", rb_grn_column_clear_lock, -1);
     rb_define_method(rb_cGrnColumn, "locked?", rb_grn_column_is_locked, -1);
-    rb_define_method(rb_cGrnColumn, "reference_column?",
-		     rb_grn_column_reference_column_p, 0);
-    rb_define_method(rb_cGrnColumn, "index_column?",
-		     rb_grn_column_index_column_p, 0);
+    rb_define_method(rb_cGrnColumn, "reference?", rb_grn_column_reference_p, 0);
+    /* deprecated: backward compatibility */
+    rb_define_alias(rb_cGrnColumn, "reference_column?", "reference?");
+    rb_define_method(rb_cGrnColumn, "index?", rb_grn_column_index_p, 0);
+    /* deprecated: backward compatibility */
+    rb_define_alias(rb_cGrnColumn, "index_column?", "index?");
+    rb_define_method(rb_cGrnColumn, "vector?", rb_grn_column_vector_p, 0);
+    rb_define_method(rb_cGrnColumn, "scalar?", rb_grn_column_scalar_p, 0);
 
     rb_grn_init_fix_size_column(mGrn);
     rb_grn_init_variable_size_column(mGrn);

data/ext/groonga/rb-grn-object.c

@@ -1199,7 +1199,7 @@ rb_uvector_value_p (RbGrnObject *rb_grn_object, VALUE rb_value)
       case GRN_TABLE_NO_KEY:
       case GRN_TABLE_VIEW:
 	first_element = rb_ary_entry(rb_value, 0);
-	if (RVAL2CBOOL(rb_obj_is_kind_of(first_element, rb_cGrnRecord))) {
+	if (rb_respond_to(first_element, rb_intern("record_raw_id"))) {
 	    return RB_GRN_TRUE;
 	}
 	break;
@@ -1217,25 +1217,26 @@ rb_grn_object_set_raw (RbGrnObject *rb_grn_object, grn_id id,
     grn_ctx *context;
     grn_obj value;
     grn_rc rc;
-    VALUE exception;
+    VALUE exception, rb_values;
 
     context = rb_grn_object->context;
-    if (RVAL2CBOOL(rb_obj_is_kind_of(rb_value, rb_cArray))) {
-	if (rb_uvector_value_p(rb_grn_object, rb_value)) {
-	    GRN_OBJ_INIT(&value, GRN_UVECTOR, 0,
-			 rb_grn_object->object->header.domain);
-	    RVAL2GRNUVECTOR(rb_value, context, &value, related_object);
-	} else {
-	    GRN_OBJ_INIT(&value, GRN_VECTOR, 0, GRN_ID_NIL);
-	    RVAL2GRNVECTOR(rb_value, context, &value);
-	}
-    } else {
+    rb_values = rb_check_array_type(rb_value);
+    if (NIL_P(rb_values)) {
 	if (NIL_P(rb_value)) {
 	    GRN_OBJ_INIT(&value, GRN_BULK, 0, GRN_ID_NIL);
 	} else {
 	    GRN_OBJ_INIT(&value, GRN_BULK, 0, GRN_ID_NIL);
 	    RVAL2GRNBULK(rb_value, context, &value);
 	}
+    } else {
+	if (rb_uvector_value_p(rb_grn_object, rb_values)) {
+	    GRN_OBJ_INIT(&value, GRN_UVECTOR, 0,
+			 rb_grn_object->object->header.domain);
+	    RVAL2GRNUVECTOR(rb_values, context, &value, related_object);
+	} else {
+	    GRN_OBJ_INIT(&value, GRN_VECTOR, 0, GRN_ID_NIL);
+	    RVAL2GRNVECTOR(rb_values, context, &value);
+	}
     }
     rc = grn_obj_set_value(context, rb_grn_object->object, id,
 			   &value, flags);

data/ext/groonga/rb-grn-utils.c

@@ -555,13 +555,14 @@ rb_grn_uvector_from_ruby_object (VALUE object, grn_ctx *context,
 	    id = NUM2UINT(value);
 	    break;
 	  default:
-	    if (RVAL2CBOOL(rb_obj_is_kind_of(value, rb_cGrnRecord))) {
-		id = NUM2UINT(rb_funcall(value, rb_intern("id"), 0));
+	    if (rb_respond_to(value, rb_intern("record_raw_id"))) {
+		id = NUM2UINT(rb_funcall(value, rb_intern("record_raw_id"), 0));
 	    } else {
 		grn_obj_unlink(context, uvector);
 		rb_raise(rb_eArgError,
 			 "uvector value should be one of "
-			 "[Fixnum, Groonga::Record]: %s (%s): %s",
+			 "[Fixnum or object that has #record_raw_id]: "
+			 "%s (%s): %s",
 			 rb_grn_inspect(value),
 			 rb_grn_inspect(object),
 			 rb_grn_inspect(related_object));

data/ext/groonga/rb-grn.h

@@ -69,7 +69,7 @@ RB_GRN_BEGIN_DECLS
 
 #define RB_GRN_MAJOR_VERSION 1
 #define RB_GRN_MINOR_VERSION 0
-#define RB_GRN_MICRO_VERSION 3
+#define RB_GRN_MICRO_VERSION 4
 
 typedef int rb_grn_boolean;
 #define RB_GRN_FALSE (0)

data/html/index.html

@@ -49,7 +49,7 @@
 
 	<h3>rroongaの最新リリース</h3>
 	<p>
-	  2010-11-29にリリースされた1.0.3が最新です。
+	  2010-11-29にリリースされた1.0.4が最新です。
         </p>
 
 	<h3 id="install-rroonga">rroongaのインストール</h3>

data/lib/groonga/record.rb

@@ -21,8 +21,6 @@ module Groonga
   class Record
     # レコードが所属するテーブル
     attr_reader :table
-    # レコードのID
-    attr_reader :id
     # _table_の_id_に対応するレコードを作成する。_values_には各
     # カラムに設定する値を以下のような形式で指定する。
     #
@@ -129,7 +127,7 @@ module Groonga
     #
     # 名前が_name_のカラムが参照カラムであるなら+true+を返す。
     def reference_column?(name)
-      column(name).reference_column?
+      column(name).reference?
     end
 
     # call-seq:
@@ -138,7 +136,27 @@ module Groonga
     # 名前が_name_のカラムが索引カラム
     # （Groonga::IndexColumn）であるなら+true+を返す。
     def index_column?(name)
-      column(name).index_column?
+      column(name).index?
+    end
+
+    # call-seq:
+    #   record.vector_column?(name) -> true/false
+    #
+    # 名前が_name_のカラムの値がベクターであるなら+true+を返す。
+    #
+    # @since: 1.0.5
+    def vector_column?(name)
+      column(name).vector?
+    end
+
+    # call-seq:
+    #   record.scalar_column?(name) -> true/false
+    #
+    # 名前が_name_のカラムの値がスカラーであるなら+true+を返す。
+    #
+    # @since: 1.0.5
+    def scalar_column?(name)
+      column(name).scalar?
     end
 
     # call-seq:
@@ -181,13 +199,22 @@ module Groonga
       end
     end
 
+    # call-seq:
+    #   record.record_raw_id -> ID
+    #
+    # レコードのIDを返す。
+    def record_raw_id
+      @id
+    end
+    alias_method :id, :record_raw_id
+
     # call-seq:
     #   record.score -> スコア値
     #
     # レコードのスコア値を返す。検索結果として生成されたテーブル
     # のみに定義される。
     def score
-      self["._score"]
+      self["_score"]
     end
 
     # call-seq:
@@ -199,7 +226,7 @@ module Groonga
     # Groonga::Record#support_sub_recordsでこの値を利用でき
     # るかがわかる。
     def n_sub_records
-      self["._nsubrecs"]
+      self["_nsubrecs"]
     end
 
     # call-seq:

data/lib/groonga/schema.rb

@@ -421,6 +421,7 @@ module Groonga
       def normalize_type(type) # :nodoc:
         return type if type.nil?
         return type if type.is_a?(Groonga::Object)
+        type = type.to_s if type.is_a?(Symbol)
         case type.to_s
         when "string"
           "ShortText"

data/test/test-fix-size-column.rb

@@ -32,8 +32,16 @@ class FixSizeColumnTest < Test::Unit::TestCase
                                        :path => @viewed_column_path.to_s)
   end
 
-  def test_index_column?
-    assert_not_predicate(@viewed, :index_column?)
+  def test_index?
+    assert_not_predicate(@viewed, :index?)
+  end
+
+  def test_vector?
+    assert_not_predicate(@viewed, :vector?)
+  end
+
+  def test_scalar?
+    assert_predicate(@viewed, :scalar?)
   end
 
   def test_inspect

data/test/test-index-column.rb

@@ -22,7 +22,7 @@ class IndexColumnTest < Test::Unit::TestCase
     setup_database
   end
 
-  def test_index_column?
+  def test_index?
     articles = Groonga::Array.create(:name => "Articles")
     articles.define_column("content", "Text")
 
@@ -30,7 +30,29 @@ class IndexColumnTest < Test::Unit::TestCase
                                  :default_tokenizer => "TokenBigram")
     content_index = terms.define_index_column("content", articles,
                                               :with_section => true)
-    assert_predicate(content_index, :index_column?)
+    assert_predicate(content_index, :index?)
+  end
+
+  def test_vector?
+    articles = Groonga::Array.create(:name => "Articles")
+    articles.define_column("content", "Text")
+
+    terms = Groonga::Hash.create(:name => "Terms",
+                                 :default_tokenizer => "TokenBigram")
+    content_index = terms.define_index_column("content", articles,
+                                              :with_section => true)
+    assert_not_predicate(content_index, :vector?)
+  end
+
+  def test_scalar?
+    articles = Groonga::Array.create(:name => "Articles")
+    articles.define_column("content", "Text")
+
+    terms = Groonga::Hash.create(:name => "Terms",
+                                 :default_tokenizer => "TokenBigram")
+    content_index = terms.define_index_column("content", articles,
+                                              :with_section => true)
+    assert_not_predicate(content_index, :scalar?)
   end
 
   def test_array_set_with_record

data/test/test-record.rb

@@ -171,6 +171,20 @@ class RecordTest < Test::Unit::TestCase
     assert_true(index.index_column?("content"))
   end
 
+  def test_vector_column?
+    bookmark = @bookmarks.add
+    assert_false(bookmark.vector_column?("uri"))
+    morita = @users.add("morita")
+    assert_true(morita.vector_column?("addresses"))
+  end
+
+  def test_scalar_column?
+    bookmark = @bookmarks.add
+    assert_true(bookmark.scalar_column?("uri"))
+    morita = @users.add("morita")
+    assert_false(morita.scalar_column?("addresses"))
+  end
+
   def test_score
     groonga = @bookmarks.add
     groonga["content"] = "full text search search search engine."
@@ -282,4 +296,33 @@ class RecordTest < Test::Unit::TestCase
     users = @users.select {|record| record.key == "morita"}
     assert_predicate(users.to_a[0], :support_sub_records?)
   end
+
+  def test_set_record_like_object
+    kou_at_clear_code = @addresses.add(:mail => "kou@clear-code.com")
+    record_like_class = Class.new do
+      attr_reader :record_raw_id
+      def initialize(record_raw_id)
+        @record_raw_id = record_raw_id
+      end
+    end
+    record_like_object = record_like_class.new(kou_at_clear_code.record_raw_id)
+    kou = @users.add("kou")
+    kou.addresses = [record_like_object]
+  end
+
+  def test_set_array_like_records
+    kou_at_clear_code = @addresses.add(:mail => "kou@clear-code.com")
+    array_like_class = Class.new do
+      def initialize(records)
+        @records = records
+      end
+
+      def to_ary
+        @records
+      end
+    end
+    array_like_object = array_like_class.new([kou_at_clear_code])
+    kou = @users.add("kou")
+    kou.addresses = array_like_object
+  end
 end

data/test/test-variable-size-column.rb

@@ -50,8 +50,16 @@ class VariableSizeColumnTest < Test::Unit::TestCase
     @yu = @users.add(:name => "Yutaro Shimamura")
   end
 
-  def test_index_column?
-    assert_not_predicate(@nick_names, :index_column?)
+  def test_index?
+    assert_not_predicate(@nick_names, :index?)
+  end
+
+  def test_vector?
+    assert_predicate(@nick_names, :vector?)
+  end
+
+  def test_scalar?
+    assert_not_predicate(@nick_names, :scalar?)
   end
 
   def test_inspect

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: rroonga
 version: !ruby/object:Gem::Version 
-  hash: 17
+  hash: 31
   prerelease: false
   segments: 
   - 1
   - 0
-  - 3
-  version: 1.0.3
+  - 4
+  version: 1.0.4
 platform: ruby
 authors: 
 - Kouhei Sutou