Remove collection.IsEmpty

It adds no value in any location, and will always return false anyway.  Function likely originates from a time where schemaless collections were still being considered a possibility.

client/descriptions.go

@@ -42,23 +42,19 @@ func (col CollectionDescription) IDString() string {
 // GetFieldByID searches for a field with the given ID. If such a field is found it
 // will return it and true, if it is not found it will return false.
 func (col CollectionDescription) GetFieldByID(id FieldID) (FieldDescription, bool) {
-	if !col.Schema.IsEmpty() {
-		for _, field := range col.Schema.Fields {
-			if field.ID == id {
-				return field, true
-			}
+	for _, field := range col.Schema.Fields {
+		if field.ID == id {
+			return field, true
 		}
 	}
 	return FieldDescription{}, false
 }
 
 // GetRelation returns the field that supports the relation of the given name.
 func (col CollectionDescription) GetRelation(name string) (FieldDescription, bool) {
-	if !col.Schema.IsEmpty() {
-		for _, field := range col.Schema.Fields {
-			if field.RelationName == name {
-				return field, true
-			}
+	for _, field := range col.Schema.Fields {
+		if field.RelationName == name {
+			return field, true
 		}
 	}
 	return FieldDescription{}, false
@@ -91,11 +87,6 @@ type SchemaDescription struct {
 	Fields []FieldDescription
 }
 
-// IsEmpty returns true if the SchemaDescription is empty and uninitialized
-func (sd SchemaDescription) IsEmpty() bool {
-	return len(sd.Fields) == 0
-}
-
 // GetFieldKey returns the field ID for the given field name.
 func (sd SchemaDescription) GetFieldKey(fieldName string) uint32 {
 	for _, field := range sd.Fields {
@@ -108,11 +99,9 @@ func (sd SchemaDescription) GetFieldKey(fieldName string) uint32 {
 
 // GetField returns the field of the given name.
 func (sd SchemaDescription) GetField(name string) (FieldDescription, bool) {
-	if !sd.IsEmpty() {
-		for _, field := range sd.Fields {
-			if field.Name == name {
-				return field, true
-			}
+	for _, field := range sd.Fields {
+		if field.Name == name {
+			return field, true
 		}
 	}
 	return FieldDescription{}, false

db/base/collection_keys.go

@@ -42,19 +42,7 @@ func MakePrimaryIndexKeyForCRDT(
 	case client.COMPOSITE:
 		return MakeCollectionKey(c).WithInstanceInfo(key).WithFieldId(core.COMPOSITE_NAMESPACE), nil
 	case client.LWW_REGISTER:
-		fieldKey := getFieldKey(c, key, fieldName)
-		return MakeCollectionKey(c).WithInstanceInfo(fieldKey), nil
+		return MakeCollectionKey(c).WithInstanceInfo(key).WithFieldId(fmt.Sprint(c.Schema.GetFieldKey(fieldName))), nil
 	}
 	return core.DataStoreKey{}, ErrInvalidCrdtType
 }
-
-func getFieldKey(
-	c client.CollectionDescription,
-	key core.DataStoreKey,
-	fieldName string,
-) core.DataStoreKey {
-	if !c.Schema.IsEmpty() {
-		return key.WithFieldId(fmt.Sprint(c.Schema.GetFieldKey(fieldName)))
-	}
-	return key.WithFieldId(fieldName)
-}

db/fetcher/fetcher.go

@@ -141,9 +141,6 @@ func (df *DocumentFetcher) Init(
 	showDeleted bool,
 ) error {
 	df.txn = txn
-	if col.Schema.IsEmpty() {
-		return client.NewErrUninitializeProperty("DocumentFetcher", "Schema")
-	}
 
 	err := df.init(col, fields, filter, docmapper, reverse)
 	if err != nil {